Merge pull request #20301 from eileenmcnaughton/mem_move

[REF] Remove unused parameters

Merge pull request #20344 from civicrm/5.38

5.38

Merge pull request #20274 from agileware/CIVICRM-1737

Create User Record now hides the Password field and uses the check isUserRegistrationPermitted - which prevents Administrators from setting passwords for new Users

Merge pull request #20307 from colemanw/searchTaskHook

SearchKit - Rename crmSearchActions module to crmSearchTasks & add hook

Merge pull request #20343 from seamuslee001/nfc_tiered_test_timecop

[NFC] Remove timecop function references from Cache Tiered Test

CIVICRM-1737 Create User Record now hides the Password field and uses the check isUserRegistrationPermitted - which prevents Administrators from setting passwords for new Users

[NFC] Remove timecop function references from Cache Tiered Test

Merge pull request #20341 from colemanw/greenwichLinkColor

dev/core#2403 - Change Greenwich link color from gray to blue

Merge pull request #20239 from eileenmcnaughton/act

[REF] Cleanup BAO_ActionSchedule::getlist() signature

dev/core#2403 - Change Greenwich link color from gray to blue

Fixes dev/core#2403

Merge pull request #20336 from colemanw/searchKitUseDefaultPagerSize

SearchKit - Use default_pager_size setting

Merge pull request #20337 from totten/master-field-dao

(REF) Regen CRM/Core/DAO/CustomField.php

(REF) Regen CRM/Core/DAO/CustomField.php

Before: The committed DAO doesn't match the generated DAO.

After: The commited DAO matches the generated DAO.

Technical Details: This DAO file grew slightly stale following 0388de0d6435161344c9483066765e7a9ca65dfe.

Merge pull request #20333 from agileware/CIVICRM-1732

Fix case sensitive parameter for call to getCustomFieldTokens

SearchKit - Use default_pager_size setting

Merge pull request #20332 from colemanw/links_callback

Core_DAO - fire links_callback for all entities

Merge pull request #20285 from colemanw/cke5Fix

Fix CKEditor5 link editing in modal dialog

Merge pull request #20210 from eileenmcnaughton/cancel

Fix old-ish regression on paypal standard self-service cancel links

Merge pull request #20243 from eileenmcnaughton/group2

[Ref] remove redundant call to clearGroupContactCache

Merge pull request #20326 from mattwire/payjunctiondopayment

Convert payJunction to doPayment

Merge pull request #20325 from mattwire/firstdatadopayment

Convert FirstData to doPayment

Merge pull request #20324 from mattwire/elavondopayment

Convert elavon to doPayment

Merge pull request #20327 from mattwire/realexdopayment

Convert Realex to doPayment

Merge pull request #20328 from mattwire/payflowprodopayment

Convert payflowpro to doPayment

Core_DAO - fire links_callback for all entities even if they don't explicitly declare links

CRM_Core_DAO has a fallback function for entities with no foreign key fields.
Previously, that function did not call createReferenceColumns or fire links_callback

Merge pull request #20323 from mattwire/ewaydopayment

Convert eway to doPayment

Merge pull request #20329 from highfalutin/patch-6

Correct copy-paste error in CustomField schema

Merge pull request #20322 from mattwire/authnetdopayment

Convert AuthorizeNet to doPayment()

Fix test to support invalidate change

[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

Correct copy-paste error in CustomField schema

Convert payflowpro to doPayment

Convert Realex to doPayment

Convert payJunction to doPayment

Convert FirstData to doPayment

Convert elavon to doPayment

Convert eway to doPayment

Convert authorizenet to doPayment

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

Fix self-service recurring link on dashboard

This picks up on two of the issues that were blocking on
https://github.com/civicrm/civicrm-core/pull/18787 and
https://github.com/civicrm/civicrm-core/pull/18196
(but does not solve all of them). It

1) adds the basis for unit test cover and
2) separates the function for retrieving links
for self-service from back office user.

I think there is still some jumbling together of these concepts but
as long as we are careful to extend the tests as we extend the logic...

This fixes what I suspect is an oldish regression whereby the cancel link
for paypal std users was wrong

Merge pull request #20214 from ixiam/dev#issue_2584

Separates the default language for contacts from the site language

Improve unit test on rendered urls

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.

SearchKit - Rename crmSearchActions module to crmSearchTasks & add hook_civicrm_searchKitTasks().

Merge pull request #20216 from colemanw/afformEntityRef

Afform Gui - Add support for entityRef fields

[REF] Remove unused parameters

This takes out some parameters that are never passed in & stops returning things that are unused

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