Extract getContributionObject

Merge pull request #20318 from eileenmcnaughton/ppp

[REF] Add getter for contributionID

Merge pull request #20345 from eileenmcnaughton/mem_move

[REF] Make previous unshared function protected & Remove hard-coded params

Merge pull request #20330 from mattwire/fixpaymentformstest

Fix form params for testEventPaymentForms

Merge pull request #20352 from seamuslee001/test_credit_card_validation

[NFC] Add in unit tests of credit card validation rules

Merge pull request #20351 from seamuslee001/update_pear_finance_creditcard

[REF] Update Pear Validate Finance CreditCard to use latest tagged re…

Merge pull request #20340 from eileenmcnaughton/schema2

Fix extraneous white space in generated sql

Merge pull request #20278 from aydun/name_custom_css

Give the custom css file a 'name' to make it easier to manipulate in hook_civicrm_alterBundle()

Merge pull request #20346 from eileenmcnaughton/confirm

[REF] Make previously unshared function protected

[NFC] Add in unit tests of credit card validation rules

[REF] Update Pear Validate Finance CreditCard to use latest tagged release that now includes the improved mastercard 2bin regex

Merge pull request #20319 from eileenmcnaughton/pp

Remove extraneous debug

Merge pull request #20347 from demeritcowboy/pass-compare

dev/core#2613 - Replace deprecated function in useradd task

[REF] Make previously unshared function protected

This clarifies that it is not used outside core - which we
know as it was only just moved here 5187a13c4d0e4278bb9b18f9fb327b2407766dff

switch from deprecated function

[REF] Remove hard-coded params

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

Fix extraneous white space in generated sql

- trailing space when comment is not declared for a db table
- spaces before arguments that don't exist
- extra lines

Schema.tpl whitespace overhaul

Note I tried regenerating civicrm.mysql with this & it looked good - jenkins can
pass judgement too...

I also tried regenerating the sql in search_kit but I hit what seems to me to be
an unreleated regression - ie search_kit generates
 ROW_FORMAT=DYNAMIC

With my version (lastest from master) of civix I get ENGINE = INNODB but
not the ROW_FORMAT

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

Fix form params for testEventPaymentForms

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

[REF] Add getter for contributionID

Remove extraneous debug

We added this a while back when we were getting reports of an error that
we couldn't track down. We never got feedback of these lines appearing in the
logs and the error appears to be otherwise solved so
I think the debug adds more confusion & has not proven helpful

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.