[Ref] remove redundant call to clearGroupContactCache

This code does a clear on the cache and then calls the api which will eventually
call load(). Load ALSO clears the cache so this seems redundant.

However, there are a couple of places where add() is called from where it might
serve to force the rebuild due to changes to group make up. In general we have switched
to invalidating and letting the rebuild-on-demand do it's thing to allow
things like crons or lack of need for the group to reduce to browser demand.
So, I think that by switching to invalidating rather than rebuilding
we can remove this potentially locky line from GroupContactCache::add()

Merge pull request #20309 from jmcclelland/profile-county

get county fields populated via api call

Merge pull request #20303 from totten/master-extver-xml

dev/release#17 - Synchronize version#s for core and core-extensions

Merge pull request #20269 from colemanw/bulkSave

APIv4 - Save custom fields in bulk

Merge pull request #20273 from seamuslee001/report_default_pager

[REF] Make use of recently added default pager size setting in Reports

Merge pull request #20313 from eileenmcnaughton/mem_move2

[REF] Duplicate & deprecate processMembership

Merge pull request #20320 from eileenmcnaughton/managed

Add unit test for #20144

Merge pull request #19840 from MikeyMJCO/patch-7

Explicitly throw an error/status bounce on Edit/New if no types.

[REF] Make use of recently added default pager size setting in Reports

Update code as per feedback from Dave D

Add unit test for #20144

Merge pull request #20316 from eileenmcnaughton/ppp

[REF] Switch to using getContributionRecurID function, remove now unused param

Merge pull request #20281 from colemanw/afformDragDrop

Afform drag n drop fixes

Merge pull request #20317 from eileenmcnaughton/pp

[REF] Fully remove ids variable from paypal std

Merge pull request #20302 from totten/master-extver-ui

(dev/release#17) Extension UI - Show developmental icon for alpha/beta-stage extensions

Merge pull request #20310 from eileenmcnaughton/group_lock

Cleanup tracking on group.load

Merge pull request #20081 from totten/master-authx-defaults

authx - Accept API keys by default

Merge pull request #20229 from pradpnayak/usercreate

Activate user when no activation is required

Merge pull request #20214 from ixiam/dev#issue_2584

Separates the default language for contacts from the site language

APIv4 - Use bulk method for saving custom fields

Renames the prototype bulkSave => writeRecords to follow the convention now in APIv4 and CRM_Core_DAO

APIv4 - Default to saving and deleting DAO records in bulk

This adds new writeRecords() and deleteRecords() methods
to CRM_Core_DAO which can be overridden by DAOs who wish to optimize.

[REF] Fully remove ids variable from paypal std

The ids variable is no longer used - bye

Extension page markup tweaks

Duplicate & deprecate processMembership

Function only called from one place + tests. Duplicate to where it is
called & deprecated as it seems civimobile may be using this
(in an unsupported way but we shouldn't actively break it since we are aware of that)

[REF] Switch to using getContributionRecurID function, remove now unused param

This is a follow up to https://github.com/civicrm/civicrm-core/pull/20294
- getContributionRecurID now always has the id so we don't need to pass it around
in the ids array

Cleanup tracking on group.load

1) deprecate force parameter
2) ditch the static & let the db lookups speak for themselves

The force paramter seems to mostly be about the static & probably only exists to support tests.
The static doesn't make sense now because either they group needs loading or it doesn't.
The fact the process might have tried and succeeded or failed before doesn't need
to be recorded in the static

Cleanup up progress tracking in Group::load

Fix test to do things in a logical order so contacts exist when the group is created

Merge pull request #20314 from mattwire/contactdetailacl

Switch to non deprecated buildPermissionClause() for contact detail report

Merge pull request #20209 from eileenmcnaughton/qfail

Do not error on view contribution recur if not displaying contributions

Merge pull request #20300 from eileenmcnaughton/pp

dev/core#2593 stop calling loadRelatedObjects / fix for bug

Merge pull request #20294 from eileenmcnaughton/ppp

[REF] Provide standardised methods to get recurring contribution id & object

Merge pull request #20211 from eileenmcnaughton/reorg

[REF] Reverse if statement

Merge pull request #20205 from eileenmcnaughton/dep_pend

Remove deprecation of order-must-be-pending

Switch to non deprecated buildPermissionClause() for contact detail report

Separates the default language for contacts from the site language

Merge pull request #20312 from seamuslee001/improve_sample_data_trxn_id_payment_procesor_link

Improve Sample data by ensuring all Credit Card Contributions have a …

Merge pull request #20304 from eileenmcnaughton/mem_move2

Remove never-passed parameter from now unshared function

Improve Sample data by ensuring all Credit Card Contributions have a transaction id and tha they are shown to be linked to the test payment processor

Merge pull request #20311 from demeritcowboy/dummy

dev/core#2614 - Sample payment processor is the wrong id in the generated sample data for demos

the id keeps changing so look it up by name

Merge pull request #20082 from mattwire/paypalwarning

Add warning when url_site is not specified for paypal (it won't work if not set)

Merge pull request #20139 from eileenmcnaughton/home_url

Remove defunct default return property

get county fields populated via api call

Ensure that county fields are properly populated when
calling the Profile.getfields api.

Merge pull request #20216 from colemanw/afformEntityRef

Afform Gui - Add support for entityRef fields

Remove never-passed parameter from now unshared parameter

dev/core#2593 stop calling loadRelatedObjects

We are no longer using ids after this function so we don't need to
call that function to load them.

In addition this extacts getContribution & reflects the fact the contact_id
is only being used to warn if the contribution is not a match (in
the debug log)

Merge pull request #20299 from eileenmcnaughton/pp

Remove now-unused ids parameter from signature

(dev/release#17) Extension UI - Show developmental icon for alpha/beta-stage extensions

Extensions are published with two fields, `<version>` (e.g.  `1.1`) and `<develStage>` (e.g.  `alpha`).
Confusingly, the two fields sometimes convey redundant information - but not always. A few cases:

* For example, sometimes a `<version>` as `1.0-alpha1`, `1.2.beta2`, or `2.3dev`. This would make sense if the
developer does formal tagging/releasing for developmental versions. (In this case, you don't really need the
icon - because the version-number tells you.)

* Other times, you might have a `<version>` which simply says `1.0` or `2.0` -- and then supplemental
information where the `<develStage>` says `alpha` or `beta`.  This could make sense if the developmental
version is being continuously updated without formal tags/releases.

* Under dev/release#17, we have another case -- where the core-extensions have version#s which match the
core version# (because they are released together), but the devel-stage of the extension is only
alpha/beta (because the extension is still evolving/optional/not-fully-supported).

Before: The "Development Stage" is entirely obscure.

After: There's an icon in the table to signal the development stage.

(REF) Extensions - Remove dead code

To see that this code is unused, simply grep the entire source-tree for `upgradable`
and `upgradeVersion`. There is nothing that would plausible set these fields.

I think this is left-over from an older UX. When there's an upgrade available in this UX, it shows
an alert box under the "Description".

(dev/release#17) ext/*/info.xml - Update versions to match core version

(NFC) ext/*/info.xml - Normalize whitespace

(dev/release#17) For new versions, auto-update ext/{$CORE_EXT}/info.xml

(REF, dev/release#17) Move list of core-exts to separate file

[REF] Add getters for contribution id & contact id

[REF] Provide standardised methods to get recurring contribution id & object

This adds methods to get the recurring contribution id & object that work from the paypal
express and from the paypal pro flows. In the paypal express flow it is
retrieved from a known value (and this flow is only ever hit for recurrings
as evidenced by the existing getsingle) and then set on the class. For paypal pro
it is optional and will be in the cheap-to-access POST array if relevant

Remove now-unused ids parameter from signature

Merge pull request #20293 from eileenmcnaughton/pp

[REF] Use function to getContributionRecurID

Merge pull request #20295 from eileenmcnaughton/mem_move

[REF] Duplicate processMembership preliminary to cleanup

[REF] Use function to getContributionRecurID

We are simply fetching it from the POST so it's cheap to fetch & we can stop passing it around
once we are using a function, not a variable. This also changes the debug logging slightly
to pass the values more correctly to log->debug().

[REF] Duplicate processMembership preliminary to cleanup

This used to be shared by 3 places. We've copied & cleaned up one. This duplicates to the
second once I've done some cleaning up I'll deprecate this & also duplicate to the 3rd

Merge pull request #20287 from mattwire/contactreportaclduplicates

Switch to non deprecated buildPermissionClause() for contact summary report

Merge pull request #20288 from demeritcowboy/fintypeaccount

[NFC/TEST] - Add unit test enforcing automatic chaining for financial type/account creation via api

Merge pull request #20290 from civicrm/5.38

5.38

Merge pull request #20246 from eileenmcnaughton/ft

dev/core#2593 [REF] Fix function signature to actually-used variables

Switch to non deprecated buildPermissionClause() for contact summary report

Merge pull request #20289 from seamuslee001/5.38

5.37.{1|2} release notes

Add release-notes/5.37.2.md

fixup release-notes/5.37.1.md

Add release-notes/5.37.1.md

Merge pull request #20286 from agh1/5.38.0-releasenotes-initial

5.38.0 release notes initial run

add test enforcing automatic chaining for financial type/account create

5.38.0 release notes: added boilerplate

5.38.0 release notes: raw from script

Afform GUI - Scroll fieldset into view when selecting entity

Afform - improve drag-n-drop UI

Fixes a number of issues with drag-n-drop on the Afform GUI palette & canvas.
Forces panels to take up full vertical space so palette never scrolls offscreen.
Compresses tabs above palette to save space.

Merge pull request #20077 from eileenmcnaughton/mem_order

Use order api when creating a recurring membership from the Membershi…

Merge pull request #20259 from agileware/CIVICRM-1734

Change the help text for Message Templates and Scheduled Reminders to include link to CiviCRM Token Documentation page

Afform Gui - Add support for entityRef fields

Adds a widget for EntityRef and allows it to be changed to Number (for entering ID)
or Select (for choosing another entity on a form).

Merge pull request #20284 from civicrm/5.38

5.38

Merge pull request #20264 from colemanw/afformJoinSaveFix

Afform - Fix saving joined entities (email, address, phone, etc)

Merge pull request #20283 from colemanw/afformEnabledEntities

Afform - hide disabled contact types & entities from disabled components/extensions

Afform - hide disabled contact types & entities from disabled components/extensions

Merge pull request #20280 from civicrm/5.38

5.38

Afform - Fix saving joined entities (email, address, phone, etc)

Merge pull request #20276 from demeritcowboy/help-pre

Typo when retrieving help_pre textarea attributes for price field

Merge pull request #20208 from seamuslee001/move_acl_test_trait

[REF] Move ACLPermission Trait into Civi Folder so that it can be acc…

Merge pull request #20277 from eileenmcnaughton/538

Fix money formatting opps

Merge pull request #20275 from demeritcowboy/big-c

dev/core#2606 - Typo in rowCount variable in pager

Fix money formatting opps

As Matt points out we regressed this when adding 2 decimal places

https://github.com/civicrm/civicrm-core/commit/78b338e07464becd1555a2c7d2d04ac0e4e9180c#diff-3d2a990f0b1960f0b3e6671c645e18033883e6d1eaf535b8d19d085b67dde00bR1171

letter needs to be bigger

ancient latin

Merge pull request #20270 from demeritcowboy/relative

dev/core#2590 - Remove unused function CRM_Utils_File::relativeDirectory

Merge pull request #20263 from colemanw/afformDragFix

Afform - Drag & drop fixes

Merge pull request #20271 from totten/5.38ormaster-perm-cleanup

(REF) dev/core#2601 - Cleanup stale parameter

Merge pull request #20258 from eileenmcnaughton/tear

[NFC] Test - Improve tearDown cleanup of domain contacts

Improve tearDown cleanup of domain contacts

    Fixes to ensure the domain contacts from addresses are re-instated

Merge pull request #18969 from seamuslee001/dev_core_2187

dev/core#2187 Allow sites to customise the default number of rows ret…

(REF) dev/core#2601 - Cleanup stale parameter

Overview
--------

This cleans up a parameter that became unused/unnecessary in #20256.

History
-------

This revision is a little bouncy. Instead of before/after, here's a history:

* `<= v5.36`: `getAllModulePermissions($descriptions = FALSE)`
* `=  v5.37.0`: `getAllModulePermissions($descriptions = FALSE, &$permissions)`
* `>= v.37.1`: `getAllModulePermissions($descriptions = FALSE)`

I grepped `universe` to confirm that no other repos were using the intermediate signature.

dev/core#2187 Allow sites to customsie the default number of rows returned from QuickForm Searches

unused function

Merge pull request #20204 from eileenmcnaughton/freeze

Freeze readonly fields

Merge pull request #20260 from agileware/CIVICRM-1733

CiviCRM Scheduled Reminders, Subject field missing Token selector - users must copy Token from Body field to Subject