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

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] 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

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

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...

Remove deprecated ids param

dev/core#1921 Further removal of iso date handling

Same deal as https://github.com/civicrm/civicrm-core/pull/18359

Simply loading of related objects in transition components

This alters transitionComponents to  bypass validateData to load the related objects

Stepping through validateData it's clear that validate data
1) loads the contribution - this is now in transitionComponents
2) loads the contact - that is not specifically necessary as it is not used in transitionComponents
3) does some validation that relates to the IPNyness of BaseIPN but not to an internally called function
4) calls 'loadObjects'

LoadObjects does some error handling, sets the processorID if passed in, which it isn't  and then calls
contribution->loadRelatedObjects( )

- we now do that last call directly in-function

OrderTest::testCancelWithParticipant passes directly through this code

Fix Invoice class to not call validateData

This avoids calling the BaseIPN validateData class from Invoice.php and calls the narrower
loadRelatedObjects to avoid having to change the code around it

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

Tidy up testSubmitContributionPageWithPriceSetQuantity

Remove inaccessible call to baseIPN failed

As is now clear the updateContributionStatus function is only accessible to complete
a contribution. This removes the call to baseIPN->failed() as it is unreachable,
along with the hacky handling to support it

[REF] Ensure that all bundle container services are public for Symfony v4

Merge pull request #18357 from eileenmcnaughton/fail

[REF] Simplify logic on calling self::updateContributionStatus

Remove unnecessary call to 'validateData' from pdf generator

the call to validateData fills the objects array - but the only object being used is 'contribution'
so there is an easier way to load it....

The UI way to pass through this code is to send a pdf receipt from contribution search

          'title' => ts('Receipts - print or email'),

[NFC] Enable APIv4 Testing on the statusPrefence API Tests

Merge pull request #18340 from pradpnayak/lineItemFix

Fixed pre hook params for LineItem

Merge pull request #18363 from colemanw/5.30

Forward-merge 5.29 -> 5.30

Change eWAY processor to use guzzle, fix style issues and update info.xml and add in test of eWAY single Currency payment processor

Update README

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

Merge pull request #18350 from eileenmcnaughton/payguzz

dev/financial#143 Convert Paypal Pro to use Guzzle

Merge pull request #18358 from eileenmcnaughton/event

Use eventID rather than the object in completeTransaction

Merge pull request #18354 from totten/master-bs3b

dev/user-interface#27 - Define a "bootstrap3" bundle (skeleton)

Merge pull request #18361 from seamuslee001/update_event_checksum

[NFC] Update Event DAO Checksum

Merge pull request #18359 from eileenmcnaughton/load

dev/core#1921 Remove some places where the ghost of 2014 is getting special love

Fix customField legacy handling in APIv3

dev/core#1998 Address dedupe clash on dashboard contact

Merge pull request #18246 from sunilpawar/report_45

report#45 Ability to Search Smart or Normal Group using additional filter on Manage Group page

Fixed pre hook params for LineItem

[NFC] Update Event DAO Checksum

Remove some places where the ghost of 2014 is getting special love

per https://lab.civicrm.org/dev/core/-/issues/1921 we fixed the root cause of this 6 years ago....

Use eventID rather than the object in completeTransaction

This allows us to stop passing it in as an object as a follow up. It potentially does an extra query but this is
mitigated by
1) it only does a look up if the contribution does not already have a source (rare)
2) it uses a pseudoconstant so would only look up a given event once

- we can later remove the whole loading of event and just get the id from the participant record

Merge pull request #18147 from eileenmcnaughton/ex_type

Improve consistency of metadata type declarations

[REF] Simplify logic on calling self::updateContributionStatus

self::updateContributionStatus is only called when contributionStatusId has been set - there is only
one place that is set so move the code up there rather than set a param to trigger action further down

Move tax handling from line item api to BAO to make it available from apiv4

This implementation has some limitations. I only address one in this PR - removing the rounding - as
the focus of the PR is the move.

The rounding from all save layers was previously removed but it was reverted when it was eroneously believed
to have caused a bug. The bug turned out to be https://github.com/civicrm/civicrm-core/pull/18297

Merge pull request #18348 from eileenmcnaughton/kg

Fix test to use validateAllContributions

Merge pull request #18331 from eileenmcnaughton/refs

Stop passing / using object when all we need is the id

Merge pull request #18343 from eileenmcnaughton/fail

Minor code cleanup - this is only ever called from one place so component is always event

Fix test to use validateAllContributions

Bootstrap3 - Move warning from Api4Explorer to bundle default

The Api4Explorer page has a neat trick - if (by any means) you manage to
have Bootstrap3 active within the body of this Civi page, then it will
display a warning.

This patch moves the trick to be part of the `bootstrap3` bundle, which
means that it will be applied to any page where `bootstrap3` is activated.

Api4Explorer - Activate bootstrap3

CRM_Core_Resources - Define 'bootstrap3' bundle

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

Add expected responses & enble the handler

Convert PaypalPro to use Guzzle - step 1 - prepare to capture

Merge pull request #18346 from eileenmcnaughton/linetes

Preliminary cleanup on test

Merge pull request #18247 from totten/master-res-coll

Add higher-level support for "bundles" and "collections" of resources

Preliminary cleanup on test

Just fixing unused variables, exception declarations etc

Update unit tests to match content

Merge pull request #18264 from demeritcowboy/mysql-ssl-guess

dev/core#1926 - Try to autodetect mysql ssl settings from the CMS config

CollectionAdderInterface - Support addMarkup($html)

(REF) CRM_Core_Resources::addBundle() - Improve readability. Add test.

(NFC) CRM_Core_Resources::addBundle() - More explanatory comments.

(NFC) Docblock tweaks

Forward port https://github.com/civicrm/civicrm-core/pull/18141

CRM_Core_Resources - Use splats, CollectionAdderTrait, CollectionAdderInterface

Before
------

`CRM_Core_Resources` directly implements a number of functions (`addFoo()`)

After
-----

`CRM_Core_Resources` re-uses the implementations from `CollectionAdderTrait`

Comment
-------

The method signatures in `CollectionAdderTrait` look different, but they are
compatible.  In particular, the use of splat (`...$options`) means that they
can accept either key-value options or positional options. The existing
test-cases in `CollectionTestTrait` assert as much.

CollectionAdderTrait - Use secondary ordering more akin to CRM_Core_Resources

This is a very subtle behavioral change.  To understand it, we must consider
the traditional ordering in `CRM_Core_Resources::addScriptUrl()` and
`CRM_Core_Region::add()`.  Compare these two examples:

```
Civi::resources()->addScriptUrl('http://example.com/foo.js', 100, 'page-footer');
Civi::resources()->addScriptUrl('http://example.com/bar.js', 100, 'page-footer');

CRM_Core_Region::instance('page-footer')->add([
  'scriptUrl' => 'http://example.com/foo.js',
  'weight' => 100,
]);
CRM_Core_Region::instance('page-footer')->add([
  'scriptUrl' => 'http://example.com/bar.js',
  'weight' => 100,
]);
```

You might expect this to output `<SCRIPT>`s in the same order.  After all,
the examples use the same URLs, the same weights, and the same
procedural/natural order.  Moreover, `Civi::resources()` is just a wrapper
for `CRM_Core_Region`.  Surely they were the same!

They were not. The output order would differ.

The reason is that the ordering had two keys (`weight,name`), and the
secondary-key (`name`) was hidden from us:

* In `CRM_Core_Resources::addScriptUrl()`, the default `name` was the URL.
  This was handy for de-duping resources.
* In `CRM_Core_Region::add()`, the default `name` was a numeric position.
  This made it behave like procedural/natural-ordering.

Of course, the goal in this branch is to make various collections support
the same methods and the same behaviors.  But they didn't agree before, so
something has to give.  Either:

1. Standardize on the behavior of `Resources::addScriptUrl()` and change `Region::add(scriptUrl)`.
2. Standardize on the behavior of `Region::add(scriptUrl)` and change `Resources::addScriptUrl()`.
3. Embrace a split. All `CRM_Core_Resources::add*()` behave one way, and all `CRM_Core_Region::add*()` behave another.
4. Embrace a split. All rich adders (`*::addScriptUrl()`) behave one way, and all generic adders (`*::add()`) behave another.

This developmental branch has been using `#3`. The patch changes it to `#4`.

Both splits achieve backward compatibility wrt `Resources::addScriptUrl()`
and `Region::add(scriptUrl)`.  The difference is that `#4` allows us to have
the same behavior in all collections (regardless of subtype), which means
that collections can be more interoperable and share more code.  From my
POV, it's still quirkier than `#1` or `#2`, but it's not as quirky as `#3`.

CRM_Core_Resources - Move addCoreResources to 'coreResources' bundle

CRM_Core_Resources - Move addCoreStyles to 'coreStyles' bundle

CRM_Core_Resources_Common - Add skeleton for bundles, 'coreStyles' and 'coreResources'