Merge pull request #21258 from civicrm/5.41

5.41

Changed support to dev/core from ext/oauth

Merge pull request #20935 from eileenmcnaughton/hhhh

dev/core#2717 Simplify batch membership renewal

Merge pull request #21106 from eileenmcnaughton/order

dev/core#2634 Add v4 Membership api, access it via order

Merge pull request #21230 from eileenmcnaughton/541-w-order

FIx order api regression

Merge pull request #21257 from eileenmcnaughton/cset

Template fixes - notices, syntax

Merge pull request #21255 from demeritcowboy/not-so-hidden

Fix invalid parameter giving E_WARNING

E notice fixes

fix invalid parameter

Settings.tpl - Fix syntax error

Add order  fix

Merge pull request #21252 from civicrm/JoeMurray-patch-1

Provided standard links in ext/oauth-client/info.xml, fixed typo

Merge pull request #21251 from eileenmcnaughton/email

[Ref] Copy emailcommon function back to email trait

Merge pull request #21101 from eileenmcnaughton/inv3

[REF] Update a few references to invoicing

Provided standard links, fixed typo

Merge pull request #21217 from colemanw/searchKitButtonTokens

SearchKit - Allow tokens in menu button text

Merge pull request #21134 from eileenmcnaughton/tok_nearly

 dev/core#2745 - Contribution Tokens - Support 'contributionId'

[Ref] Copy emailcommon function back to email trait

This is part of ending the email common trait. It is currently 'shared'
with a couple of other places - but most of the functionality is
not that shared & could be easily cleaned up when separated out.

I want to create a trait for PdfTrait like the email trait so
getting the email trait better separated for that

Merge pull request #21006 from deb1990/rename-pdf-file

dev/core#2121 Ability to change pdf filename before downloading

Merge pull request #21246 from eileenmcnaughton/master

5.41 to master

Merge pull request #21244 from demeritcowboy/angular-loader

[NFC] - Replace deprecated function in AngularLoaderTest

Merge pull request #21245 from agileware/CIVICRM-1816

CiviCRM Mailing, function unsub_from_mailing has spelling error, "experiement" impacts A/B Mailing unsubscribes

Revert "Re-fix test to work with new check"

This reverts commit 77136dc6d4216211998729cea305d81868309809.

Merge branch '5.41' of github.com:civicrm/civicrm-core

CIVICRM-1816 CiviCRM Mailing, function unsub_from_mailing has spelling error, "experiement" impacts A/B Mailing unsubscribes

Merge pull request #21169 from eileenmcnaughton/email

dev/core#2769 use php email validation not hacked & bad quickform function

replace deprecated function

Merge pull request #21212 from eileenmcnaughton/541-orig

dev/financial#180 Fix line item calculation regression on line items (incorrectly treating as exclusive)

Merge pull request #20898 from JMAConsulting/issue_2701

(Smart Group) is being constantly added while editing the smart group title from 'Manage Group' page

dev/core#2769 use php email validation not hacked qf

Per https://lab.civicrm.org/dev/core/-/issues/2769 we have had problems over the years with
quickform's email validation and we now have a hacked version that is
problematic from a maintenance pov & also doesn't work
with the string I have just encountered: name.-o-.i.10@example.com
(which I am told is valid and which passes the php filter).

We already have an email rule which calls a php native function
which is better maintained than our layers of hacks. This
PR registers our email rule - which overrides the quickform
one. If we merge this we can revert quickform back to
unhacked which will improve debugging
and maintenance (although it's actually bypassed
now with this change)

Merge pull request #21240 from civicrm/5.41

5.41 to master

Revert test change that 'supported' incorrect code change

https://github.com/civicrm/civicrm-core/commit/e967ce8fe2b58b94e2163dde395542e55599da13#diff-44a947115564a688b2d6e0772ad25039b8d244225cf250dbc27ed32a134467a3L395-R396

Merge pull request #21239 from kcristiano/dev-2784

dev/core#2784 Export PRIMARY fields Fails on CiviCRM 5.40.2 and MariaDB 10.3 with row size issue

Merge pull request #21105 from colemanw/afformSubmission

Afform - Store submissions in a new database table

Merge pull request #21231 from civicrm/5.41

5.41

Move the tested but crazy tax calc to the contribution api

Fix line item calculation from exclusive to inclusive

This was added https://github.com/civicrm/civicrm-core/commit/e967ce8fe2b58b94e2163dde395542e55599da13#diff-a16d4d7449cf5f3a0616d1d282a32f27ab6d3f7d2726d076c02ad1d4d655af41R512

but incorrectly assumes the totalAmount is exclusive at this point - but
it is inclusive

Merge pull request #21238 from eileenmcnaughton/541-lines

Fix test to actually pass valid data

Merge pull request #21237 from seamuslee001/5.41

Add release-notes/5.40.3.md

Export PRIMARY fields Fails on CiviCRM 5.40.2 and MariaDB 10.3

with row size issue  https://lab.civicrm.org/dev/core/-/issues/2784

Re-fix test to work with new check

A new check was enabled for this test recently and it passed because we
were doing recalculation - but that recalculation was not always right
and there is no expectation that passing in the wrong total_amount
is OK so this fixes total_amount to be equal to the
sum of the line items - which is what is checked

Add release-notes/5.40.3.md

Afform - Store submissions in a new database table

This adds the `civicrm_afform_submission` table and accompanying api entity,
and a new Afform setting "create_submission" which will store a
submission record whenever a form is submitted.

Merge pull request #21233 from eileenmcnaughton/xml

Add date metadata for email.on_hold, reset_date

Merge pull request #21228 from colemanw/afformFixTitle

Afform - fix contact source field & field defaults

Merge pull request #21218 from colemanw/afformLocationType

Afform - Rename blocks and joins for clarity

Merge pull request #21224 from ufundo/oauth-systoken-refresh-fix

dev/core#2779 - Specify row fields to fetch in Api4 OAuthSysToken.Refresh

Merge pull request #21208 from colemanw/refNotes

[REF] APIv4 Notes - Ensure child notes are deleted with parent, and hooks are called

Add date metadata for email.on_hold, reset_date

Merge pull request #21229 from eileenmcnaughton/cache

Test cleanup fix

Note API - Ensure child notes are deleted with parent, and hooks are called

Deprecates the CRM_Core_BAO_Note::del() function and refactors out all references to it.
Related to work on dev/core#2757

Merge pull request #21226 from colemanw/noteRef

[REF] Remove unused/unneeded variables from Note View page

Afform - Fix setting default value

String comparison was failing for integer keys.

Test cleanup fix

Afform - fix contact source field

Contact source is supposed to take its default from the afform title
This fixes it to initialize with the title and keep in sync.

Refactor out unused/unneeded variables

Merge pull request #21213 from colemanw/entityBatchRef

EntityBatch - Deprecate del() function (dev/core#2757)

(Smart Group) is being constantly added while editing the smart group title from 'Manage Group' page

Afform - Rename blocks and joins for clarity

Renames the ambiguously named "block" and "join" afform properties,
and changes the naming convention to "af{type}-title-munged".

implement-basicbatchaction-getselect

Merge pull request #21223 from jitendrapurohit/dedupeerror

Fixes dev/core#2778 - Fatal error on dedupe screen

Merge pull request #21222 from colemanw/delRecent

Improve test for CRM_Utils_Recent

Merge pull request #21216 from demeritcowboy/import-csv-blankline

Alternate to 20131 - Avoid crash during import for blank lines in a one-column csv file

Fix fatal error on dedupe screen

Test cleanup

Replace double quotes with single quote

Merge pull request #21204 from colemanw/delRecent

[REF] CRM_Utils_Recent - Use hook listener to delete items

[REF] - Clean up recentItems functions and add test

Use array_filter() instead of brittle for() loops.

Merge pull request #21219 from eileenmcnaughton/acl_cache

Extract ACL contact cache clearing part out

Merge pull request #21191 from JMAConsulting/dev_2774

dev/core#2774 : Sort by date column on multirecord field listing section on profile edit mode doesn't work

Extract ACL contact cache clearing part out

This allows us to be nuanced in the acl delete part & still clear the contact cache part

CRM_Utils_Recent - Use hook listener to delete items

dev/core#2757

Merge pull request #21201 from colemanw/hookDeleteAcl

[REF] dev/core#2757 Move acl delete logic to an event listener

Merge pull request #21215 from demeritcowboy/alterExtern

[NFC] CRM_Utils_SystemTest - Call to Uri->withPath() using deprecated format

Merge pull request #21214 from demeritcowboy/extupg

[NFC] CRM_Extension_Manager_ModuleUpgTest - use ?? instead of error-suppression operator

SearchKit - Allow tokens in menu button text

avoid crash with one column and blank lines

deprecated format

EntityBatch - Deprecate del() function

Merge pull request #21200 from colemanw/delFunctions

[REF] Deprecate unnecessary del() functions

Deprecate unnecessary del() functions

avoid error-supression operator

Merge pull request #21209 from colemanw/removeUnusedFunctions

Remove unused, duplicate functions getEntitiesByTag

Move acl delete logic to an event listener

Remove unused, duplicate functions getEntitiesByTag

According to https://issues.civicrm.org/jira/browse/CRM-13156
the functions were only ever used in the v2 API, which no longer exists.

Merge pull request #21206 from demeritcowboy/extmgrmodtest

[NFC] CRM_Extension_Manager_ModuleTest - use ?? instead of error-suppression operator

Merge pull request #21207 from demeritcowboy/queueservice

CRM_Queue_Service - Use ?? instead of error-supression operator

avoid error-supression operator

avoid error-suppression operator

Merge pull request #21203 from eileenmcnaughton/master

5.41 to master (conflicts resolved)

Merge branch '5.41' of github.com:civicrm/civicrm-core

Merge pull request #21184 from colemanw/pseudoconstantFixes

APIv4 pseudoconstant improvements

Merge pull request #21195 from colemanw/UnknownFieldError

APIv4 - Properly show deprecated joins in unit tests (fixes dev/core#2771)

Merge pull request #21193 from mattwire/settitle1

REF Switch to CRM_Core_Form::setTitle() instead of CRM_Utils_System::setTitle() part 1

dev/core#2774 : Sort by date column on multirecord field listing section on profile edit mode doesn't work

Merge pull request #21194 from kurund/grid-display-layout

add grid layout support for searchkit

Add tests for pdf filename assignation

APIv4 - Add 'suffixes' to getFields metadata

This breaks apart the concept of a field having 'options' vs
a field supporting suffixes like campaign_id:label.

It is now possible for a field to not have options but still support suffixes.

This also makes the available suffixes for each field discoverable,
e.g. fields like state_province_id support an :abbr suffix.

REF - Remove redundant `use` statements and add docblock

Pseudoconstant - always explicitly name abbrColumn

Stop the overly-clever guesswork of abbrColumn so that
it can always be know if a pseudoconstant has an abbreviation.