Remove deprecated ids param

Merge pull request #18357 from eileenmcnaughton/fail

[REF] Simplify logic on calling self::updateContributionStatus

Merge pull request #18340 from pradpnayak/lineItemFix

Fixed pre hook params for LineItem

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 #18359 from eileenmcnaughton/load

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

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

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

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

CRM_Core_Resources::addCoreResources - Option `$region` is not meaningfully used

If you grep universe for `addCoreResources` and `addCoreStyles`, all matches
fall into these buckets:

1. They use the implicit default region.
2. They explicitly use the `html-header` region.
3. (Singular case) addCoreResources($region) delegates to addCoreStyles($region);
   but given 1+2, this must still be `html-header`.

CollectionTrait - Add corresponding unit-test coverage (via RegionTest, BundleTest)

(NFC) CollectionTrait - Reconcile docblocks

CollectionTrait - Use "splats". Split out "adders". Define interfaces.

In the prior commits, the signatures for `addScriptFile()`,
`addScriptUrl()`, etc are not strictly interoperable between
`CRM_Core_Resources` and `CollectionTrait`.  This is because they use
key-value options instead of positional options.  This makes it easier
disregard positional options that don't make sense (e.g.  when calling
`CRM_Core_Region::addScriptFile()`, it's silly to reserve a positional
argument for the `$region` option).

The signatures *can* be unified by using "splats" (ie `...$options`) to
accept either key-value options or backward-compatible positional options.
The ultimate is hope is that:

* `CRM_Core_Resources`, `CRM_Core_Region`, and `CRM_Core_Bundle` all
  implement the `CollectionAdderInterface`.
* `CRM_Core_Resources`, `CRM_Core_Region`, and `CRM_Core_Bundle` all
  accept options in either format (key-value or positional).
* The positional format will fade-away.

The methods in CollectionTrait are newer terrain, so it's safer to change
those signatures, so we do that first.

Note that CollectionTrait formally builds on CollectionAdderTrait. This
ensures that IDE navigation to (eg) `CRM_Core_Region::add()` and
`CRM_Core_Resources_Bundle::add()` works as expected.

Add "Bundle" support - ie an unattached, mixable list of resources

CollectionTrait - Model scriptFile and styleFile as distinct types

CollectionTrait - Support merge()

CollectionTrait - Support addScriptFile(), addStyleFile()

CollectionTrait - Support addSetting(), addSettingsFactory(), ad nauseum

CollectionTrait - Support addScript(), addScriptUrl(), addStyle(), addStyleUrl()

CollectionTrait - Support getAll(), filter(), find(), and clear()

(REF) CRM_Core_Region - Extract $renderSnippet()

This makes it possible for renderSnippet to work recursively.

For example, the current `jquery` type can build on the `script` type, and
the hypothetical `scriptFile` type could build on the `scriptUrl` type.

CollectionTrait - Create baseline trait based on CRM_Core_Region

CRM_Core_Resources - Convert 'settings' to a type of snippet

Merge pull request #18344 from civicrm/5.30

5.30

(REF) CRM_Core_Resources_Strings - Move instance into container

Merge pull request #18339 from eileenmcnaughton/line

Move ACls on LineItem create to financialacls core extension

Merge pull request #18262 from colemanw/renderSettings

Take the guesswork out of rendering clientside CRM variables

Merge pull request #18321 from mattwire/completeorder_filter

REF Filter params in completetransaction

Merge pull request #18328 from eileenmcnaughton/partial

Remove unused deprecated handling for partial_amount_to_pay

Merge pull request #18342 from agh1/5.30.0-releasenotes-initial

5.30.0 release notes initial run

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

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

Rather than set id on the contribution object just to be able to access it via contribution->id
let's name the param we keep using & use that. Note I'm still getting contribution->id from contribution
here but I think this makes it clear that the object is mostly only used in addActivity now.

The sligtly larger change is in updateMembershipBasedOnCompletionOfContribution where there is
an instantiation of 'self()' since we no longer have the object

Move ACls on LineItem create to financialacls core extension

Merge pull request #18329 from pradpnayak/userRegistration

Do not block user incase 'Require approval' is checked

5.30.0 release notes: added boilerplate

5.30.0 release notes: raw from script

Merge pull request #18337 from eileenmcnaughton/ref

Remove pass-by-ref in PaypalProIPN::single

Merge pull request #18318 from eileenmcnaughton/cancel

[REF] Add test for existing Participant batch update cancel and fix to not call BaseIPN->cancelled

Set version to 5.31.alpha1

Set version to 5.30.beta1

Merge pull request #18335 from eileenmcnaughton/query

Add ability to enable query logging per process

Add ability to enable query logging per process

This allows us to run a command such as

env CIVICRM_DEBUG_LOG_QUERY=1 drush cvapi Contact.get

and the queries for that process (but not others) are logged.

This facilitates a cleaner record of queries as it is just one process and more flexibility
(and less risk of leaving it on by mistake)

Remove pass-by-ref in PaypalProIPN::single

This is called from 2 places. Neither use the values again & neither receive any pass-by-reference values
themselves

Merge pull request #18334 from civicrm/5.29

5.29

Merge pull request #18333 from totten/5.29-rn-catchup

(NFC) 5.29.0.md - Small updates

Merge pull request #18332 from totten/5.29-norm-basepage

dev/wordpress#73 - Be more forgiving about slash in wpBasePage

5.29.0.md - Small copy-edits

Merge pull request #17980 from eileenmcnaughton/dupe

[REF] Move function to delete merged contacts to the Merger class and include all instances of the pair

5.29.0.md - Last minute addition

dev/wordpress#73 - Be more forgiving about slash in wpBasePage

Overview
--------

The correct functioning of `wpBasePage` depends on whether the administrator
omitted or added a trailing slash.  This is apparent on URLs with a trailing slash, eg

```
http://wpmaster.bknix:8001/civicrm/contribute/transact/?reset=1&id=1
```

Make it less sensitive.

Before
------

URL is OK with default setting `wpBasePage=civicrm`

URL fails with setting `wpBasePage=civicrm/`

After
-----

URL is OK with either setting `wpBasePage=civicrm` or `wpBasePage=civicrm/`

Technical Details
-----------------

I believe the old symptom arose because of the formula which produces the WP
rewrite rules (`'^' .  $config->wpBasePage .  '/([^?]*)?'`) expects that
there is no trailing slash. You could theoretically patch there, but this
seems like it'll provide more thorough normalization.

When testing, the results affect the WP rewrite rules, so you may need to be particularly
aggressive about clearing caches whenever you make a change in code or settings, eg

```
wp cache flush ; cv flush;  wp rewrite flush
```

Merge pull request #18323 from demeritcowboy/reminder-groups-dropdown

dev/core#1956 - Typo in call to nestedGroup on scheduled reminders admin form

Do not block user incase 'Require approval' is checked

Merge pull request #18327 from civicrm/5.29

5.29

Merge pull request #18325 from aydun/spec_fix

dev/mail#24, item 1 - Fix spec labels MailingEventSubscribe.create

Remove deprecated handling for partial_amount_to_pay

We added deprecation notices back in Feb when we stopped using these params in core. They were never truly
supported for extensions but we deprecated in case someone was doing something odd. However, it's
7 months on so time to cleanup

Merge pull request #18326 from agh1/5.29.0-releasenotes

5.29.0 release notes final

Merge pull request #18320 from civicrm/5.29

5.29

5.29.0 release notes: get sections in right order

5.29.0 release notes: finished writing

sorting and annotating

commenting and sorting

5.29 Release Notes first pass

Fix spec labels - see dev/mail#24, item 1

typo in call to nestedGroup

Filter params in completetransaction

Merge pull request #18319 from seamuslee001/5.29

5.28.4 release notes

Merge pull request #18315 from eileenmcnaughton/ordergo

[REF] Remove most interaction with $objects in completeOrder

Merge pull request #18295 from eileenmcnaughton/linetests

Test & remove handling for max_related in renewal form

Fix anchors in release-notes/5.28.3.md

Add release-notes/5.28.4.md

[REF] Add test for existing Participant batch update cancel and fix to not call BaseIPN->cancelled

I dug into what is 'achieved' by calling BaseIPN->cancelled here and everything except the bit where the contribution
is updated to cancelled is actually bypassed so a simple api call suffices.

I also discovered the cancellation of the contribution is highly conditional and arguably illogical. I will separately
log a gitlab to discuss whether it still makes sense, but I wouldn't want that discussion to derail this
no-change cleanup

Merge pull request #18316 from civicrm/5.29

5.29 to master