Make period_type mandatory for MembershipType

We have a test ensuring period_type is mandatory at the api level. However, it isn't for api
v4 - the error is a different issue (handling of pseudoconstants when mapping v3 to v4). This
fixes that separate issue & declares required to re-fix for v4

Merge pull request #18418 from eileenmcnaughton/ref

[ref] Extract failContribution code

Merge pull request #18410 from agh1/buttonrama

Buttonrama

Merge pull request #16482 from eileenmcnaughton/super-perm

Implement more nuanced 'Administer CiviCRM permisions

Extract failContribution code

Basic extraction

Merge pull request #17641 from MegaphoneJon/core-1590

dev/core#1590: Don't send reminders on deleted events

Merge pull request #18232 from sunilpawar/dev_1942_new

dev/core#1942 handle multiple membership of same membership type to u…

Merge pull request #18413 from eileenmcnaughton/memtest

Membership form test cleanup, date cleanup on form

Merge pull request #18414 from seamuslee001/jaap_test

[REF] Refactor price field form to allow for unit testing of the form

[REF] Refactor price field form to allow for unit testing of the form

Set the assert equals to be an actual float number

Merge pull request #18415 from eileenmcnaughton/acl

[REF] Minor readability fix

Merge pull request #18076 from MegaphoneJon/better-on-behalf-of-2

[REF] change deprecated function to API4 call

Merge pull request #18297 from eileenmcnaughton/jaap

dev/core#2003 Civicrm_price_field_value.amount truncation when localisation in play

Membership form test cleanup, date cleanup on form

This converts the test set up to use the api and also does some mild date clean up on the form.

Supports https://github.com/civicrm/civicrm-core/pull/18395

[REF] Minor readability fix

None of the table names in this query are localisable so we don't need to use variables
to refer to them.

Note the legacy dao->query() would require it if they were localisable whereas CRM_Core_DAO::executeQuery()
does not

core#1590: Don't send reminders to add'l recipients on deleted/inactive/template events

support reminders with event types/templates and multiple event/type/template IDs

fix

Merge pull request #18317 from eileenmcnaughton/bug_test

Add postAssert to check payments and contributions are valid on all tests.

Merge pull request #18412 from civicrm/5.30

5.30

Merge pull request #18409 from eileenmcnaughton/j

Fix formatLocaleNumericRoundedByCurrency

Merge pull request #18402 from eileenmcnaughton/5.30

Fix search extension excluded from tarball

Merge pull request #18408 from alifrumin/5.30rn

[NFC] 5.30 release-notes: first stab at annotating and sorting release notes

dev/core#1603 fix tangental bug on form handling of long options

This fixes the bug that derailed the fix for https://lab.civicrm.org/dev/core/-/issues/1603

On further testing I agree with Jitendra that the price field form was mis-saving the longer values (I only
tested with Euro style decimal separators but maybe on both) and it was bad data caused by this
that made the merged fix look like a regression.

I also observed that the Money functions intended to round only appeared to since it was rounding
just fine to 2 places, as tested, but it was still rounding to 2 places when we wanted more.

This fixes both the New Price field screen (when options are entered) and the edit options screen

Remove obsolete OFC flash-to-image code

Fix formatLocaleNumericRoundedByCurrency

The function was rounding by a max of 2 regardless of the number of places passed - it
was only tested for 2 so seemed to work

More radical cleanup of button CSS

Fix CSS for new buttons

Fix JS for input buttons that are now button buttons

Remove deprecated submitOnce js function

No need for crm-button wrappers for real buttons

Give buttons a value when the submitted value is used to identify them

Put icons inside of button elements

Swap out button/submit inputs for button elements

Reset button height in the WP backend

Simplify button padding and set consistent line height

remove IE6-only CSS

Remove unused button-crm-i CSS

Remove weird styling for "Configure CKEditor" button

Standardize single space (not nbsp) between icon and text

Eliminate support for old jQuery UI based crm-button-icon icon buttons

No need for crm-i-button class

Icons on buttons positioned inline, not absolute

5.30 release-notes: first stab at release notes

Merge pull request #18365 from eileenmcnaughton/member

[REF] Code cleanup on membership renewal & test

Comment out implied permissions for administer CiviCRM system per discussion

Fix search extension excluded from tarball

Implement 'super permissions' as described by @totten

This picks up on an idea Tim has pushed several times - ie that instead of giving out 'Administer CiviCRM' willy nilly
we could deprioritise it in favour of 2 more granular permission bundles - ie Administer CiviCRM data & administe CiviCRM system.

This allows us to make some permissions more 'locked away' without endlessly adding new 'administer Payment Processors'
because we've realised not everyone who can create profiles needs to be able to see payment processor credentials.

It also allows us to make system checks less broadly visible where they are not appropriate.

Note that to proceed with this we would need to go through all places that check Administer CiviCRM & put in one
or both of the 2 new permissions. Having Administer CiviCRM implicitly includes anything granted to the existing
permissions so the implementation is smooth-ish there. However, I can imagine we would need a hook allowing people
to categorise themselves or we would find ourselves litigating all sorts

Merge pull request #18401 from seamuslee001/update_eway_upgrade

[REF] Improve the human readable name of the eway upgrade step to be …

[REF] Improve the human readable name of the eway upgrade step to be clearer about what it is doing and also fix issue with a hard fault if the eway single currency payment processor type has been already removed

Merge pull request #18378 from colemanw/noMoreStupidHTMLTypes

Remove redundant custom field types

Merge pull request #18399 from megaphonetech-dennis/contact-form-task_delete-php_spelling-fix

Contact form task delete php spelling fix

Merge pull request #18245 from eileenmcnaughton/loop

Fix for ongoing issues with static upsetting the apple cart

Merge pull request #18396 from swastikpareek/issue-1987-fix-template

dev/core#1987: Enable frontend and backend theme configuration in display preference page for Drupal CMS

fixed spelling of address on lines 122 and 247

fixed capitalization on line 247

fix capitalization fix on line 122

Merge pull request #18309 from demeritcowboy/link-cases-change-status

dev/core#1979 - Incorrect comparison of status_id when changing status of linked cases

Merge pull request #18381 from eileenmcnaughton/depcom

Finish deprecating BaseIPN->completeTransaction

Merge pull request #18372 from eileenmcnaughton/inv

Fix Invoice class to not call validateData

Merge pull request #18373 from eileenmcnaughton/trans

[REF] Simplify loading of related objects in transition components

Merge pull request #18385 from eileenmcnaughton/pay

[REF] simplify interaction with objects in complete order

Merge pull request #18367 from eileenmcnaughton/validate

Remove unnecessary call to 'validateData' from pdf generator

dev/core#1987 Enable frontend and backend theme configuration in display preferences

Merge pull request #18338 from agileware/CIVICRM-1555

Change wording on the Opt Out and Unsubscribe pages

Merge pull request #18387 from seamuslee001/format_amount_deprecated

[REF] Mark CRM_Contribute_BAO_Contribution_Utils::formatAmount deprec…

Merge pull request #18394 from seamuslee001/ref_memory_usage_create_single_value_alter

[NFC] Aim to reduce memory usage in create single value alter test by…

Merge pull request #18392 from demeritcowboy/test-names

[NFC/Test framework] Make class name match file name

Merge pull request #18391 from demeritcowboy/utils-array-18207-1

[REF] Swap out CRM_Utils_Array::value() - partial pull out from PR 18207

[NFC] Aim to reduce memory usage in create single value alter test by removing xdebug items from comparaison

swap out Array::value

Merge pull request #18389 from eileenmcnaughton/pp

[REF] Remove unused lines from loadObjects

Merge pull request #18369 from eileenmcnaughton/fail

Remove inaccessible call to baseIPN failed

[REF] change deprecated function to API4 call

[REF] Mark CRM_Contribute_BAO_Contribution_Utils::formatAmount deprecated and replace its only usage with CRM_Utils_Rule::cleanMoney

[REF] Remove unused lines from loadObjects

loadObjects is only called from one place. As we can see the contribution object is
always passed in so we don't need to handle the possibility of it being anything other than a
BAO (I tweaked the error handling around that line too to make it clearer where it was
coming from)

mismatched classnames

Merge pull request #18371 from eileenmcnaughton/key

Rename variable $key to $participantID to make it clear what it is

Merge pull request #18383 from eileenmcnaughton/iso

dev/core#1921 Fix legacy date handling

Merge pull request #18384 from eileenmcnaughton/validatepp

Add try catch to main loops on core ipn classes

Merge pull request #18386 from demeritcowboy/logging-customfield-notice

dev/core#1989 - E_WARNING when editing custom field with logging turned on

e_warning when editing custom field

Merge pull request #18352 from eileenmcnaughton/line_move

dev/core#1980 Move tax handling from line item api to BAO to make it available from apiv4

Remove redundant custom field types

The html_type 'Select Country' and 'Select State/Province' serve no purpose. They are the same as Select, just with a different data_type.

This adds an APIv3 shim to display the old types.

Add try catch to main loops on core ipn classes

[REF] simplify interaction with objects in complete order

Per https://github.com/civicrm/civicrm-core/pull/18382 this line seems to be heavily tested so
& per the test we can use the 'already calculated value' for payment_processor_id rather than look
it up in yet another place.

This adds a query but in most cases queries can be removed before calling this function as
the various objects are not really accessed anymore (apart from the few lines just above this)

Merge pull request #18379 from seamuslee001/report_45

report#45 Switch out function call CRM_Utils_Array::value and extend …

dev/core#1921 Fix legacy date handling

Per https://lab.civicrm.org/dev/core/-/issues/1921 this was required before dao->save()
right up until 2014

Finish deprecating BaseIPN->completeTransaction

This has been marked deprecated, but still called for a long timme.

Now that it is just a one-liner I think it's cleaner if the core processors that use it do the one line
directly

Add postAssert to check payments and contributions are valid on all tests.

report#45 Switch out function call CRM_Utils_Array::value and extend test coverage

Minor fix for MySQL 8

Merge pull request #18351 from eileenmcnaughton/line

Add test to cover existing v3 api setting of tax_amount on line items

Merge pull request #18377 from seamuslee001/eway_further

[REF] Hide eway extension in UI and only install it if the original e…

Merge pull request #18375 from eileenmcnaughton/dep

Remove deprecated ids param

Merge pull request #18376 from eileenmcnaughton/ev

Load event title from participantID

Merge pull request #18374 from eileenmcnaughton/iso

dev/core#1921 Further removal of iso date handling

[REF] Hide eway extension in UI and only install it if the original eway payment processor is still being used

Merge pull request #18370 from KarinG/tidyup_testSubmitContributionPageWithPriceSetQuantity

Tidy up testSubmitContributionPageWithPriceSetQuantity

Merge pull request #18368 from seamuslee001/bundle_public

[REF] Ensure that all bundle container services are public for Symfon…

Merge pull request #18364 from civicrm/5.30

5.30

Merge pull request #18349 from seamuslee001/eway_extension

Migrate Eway(Single Currency) Payment Processor Type out into its own…

Merge pull request #18366 from seamuslee001/nfc_enablev4_status_pref

[NFC] Enable APIv4 Testing on the statusPrefence API Tests

Load event title from participantID

This is a follow on to https://github.com/civicrm/civicrm-core/pull/18358 - after thinking
more I realised that I had already established this code is rarely reached and by
doing a very small query when it is we can avoid loading the whole event to pass in

Same test applies...

[REF] Code cleanup on membership renewal & test

This wound up being mainly a clean up of the test class & the addition of a test on changing
membership type.

However, there is a minor code consolidation which makes 3 actual changes
1) if we are doing a test renewal we shouldn't make the membership test
2) even when renewing a Pending membership we should save any submitted custom data
3) modified id is also set for pending (although future test work might indicate this can go