Merge pull request #19198 from seamuslee001/xdebug_singlevalue

NFC When printing out the result of the correctly update in single va…

NFC When printing out the result of the correctly update in single value alter ensure that xdebug isn't printed

Merge pull request #19190 from eileenmcnaughton/import22

Squash 2 if clauses into 1

Merge pull request #19197 from demeritcowboy/key-comments

[NFC] Update comments in CRM/Core/Key

update comments

Merge pull request #19187 from civicrm/5.33

5.33

Merge pull request #19145 from totten/master-qfkey

CRM_Core_Key - Provide more debugging hints about mismatched`qfKey`s

Merge pull request #19193 from eileenmcnaughton/ref

[REF] Extract determination of subscription status information

Merge pull request #19192 from totten/5.33-upgrade

dev/core#2232 - Upgrade UI contaminates cache via l10n-js. Consolidate isUpgradeMode().

Merge pull request #19160 from eileenmcnaughton/import

Clean up error handling in legacy functions in import parser

Merge pull request #19191 from eileenmcnaughton/deprecated

Remove some deprecated code chunks

Merge pull request #19195 from seamuslee001/5322_rn

Add release-notes/5.32.2.md

Add release-notes/5.32.2.md

[REF] Extract determination of subscription status information

This gets us away from it being buried in a switch

Remove some deprecated code chunks

Clean up error handling in legacy functions in import parser

This makes the handling of errors cleaner & makes it easier for us to unravel what is going on here.

Merge pull request #19189 from eileenmcnaughton/533q1

Fix failure to assign view tpl variables to view page if context=search is in the url

Partial revert "dev/core#2232 Permit hook_civicrm_container and some other prebootish hooks to run during upgrade and clear out the asset builder cache post upgrade"

This reverts commit 756d9e0dbbe4ff66d2568f02a8ee1f152bd9c5e5.

dev/core#2232 - Upgrade UI contaminates cache via l10n-js. Consolidate isUpgradeMode().

This patch fixes an upgrade bug where `CachedCiviContainer` has stale data after an upgrade.

To do this, it removes an edge-case where two overlapping functions disagree.

Steps to Reproduce
------------------

1. Create a new/empty WP build
2. Install CiviCRM 5.30.1 from zipball
3. Install Mosaico 2.5
4. Download+extract new 5.32.1 zipball
5. Navigate to the GUI upgrade screen. Execute upgrade
6. Click to go back to the CiviCRM dashboard.
7. Receive error about the service 'mosaico_graphics'

Discussion
----------

The problem is created while running the upgrade GUI (step 5).  The GUI sends several HTTP
requests (`civicrm/upgrade`, `civicrm/upgrade/queue/ajax/runNext`, etc).  At the end, there is
an HTTP request for `civicrm/ajax/l10n-js/en_US`.  The `l10n-js` request creates a flawed copy
of `CachedCiviContainer` which is responsible for subsequent errors.

This shouldn't happen - there are defensive mechanism which prevent it from happening (e.g.
during `civicrm/upgrade`).  What makes `civicrm/ajax/l10n-js/en_US` different?  It turns on
the implementation of `isUpgradeMode()` which activates the defensive mechanisms.

Before
------

There are two implementations of `isUpgradeMode()` (via `CRM_Core_Config` and
`CRM_Utils_System`).  They often agree, but not always.

Some defensive measures trigger on `CRM_Core_Config::isUpgradeMode()` and others trigger on
`CRM_Utils_System::isUpgradeMode()`.  They often trigger together, but not always.

Let's see how these can playout in a few HTTP requests:

1. `civicrm/dashboard`: The functions agree -- it's a regular page, not an upgrade.  Therefore:
     * (a) It allows extensions to run fully.
     * (b) It enables `CachedCiviContainer` (read or write automatically).
     * (a+b) It puts good/full information in the cache.
2. `civicrm/upgrade`: The functions agree -- it's an upgrade. Therefore:
     * (a) It runs in paranoid mode (suspended extensions).
     * (b) It disables `CachedCiviContainer`.
     * (a+b) It puts no information in the cache.
3. `civicrm/ajax/l10n-js/en_US`: The functions *disagree*. In this case:
     * (a) It runs in paranoid mode (suspended extensions).
     * (b) It enables `CachedCiviContainer` (read or write automatically).
     * (a+b) It puts bad/incomplete information in the cache.

After
-----

There is one implementation of `isUpgradeMode()`.  It may be called through
either class (`CRM_Core_Config` or `CRM_Utils_System`), but the results will
always agree.

This produces the same appropriate outcome for cases of agreement (1) (2), and it fixes the
wonky/mismatched behavior in (3).

Squash 2 if clauses into 1

The action is the same for both criteria so in_array rather than 2 * if =
makes it more readable

Merge pull request #19185 from eileenmcnaughton/533q

dev/core#1019 Fix currency formatting of Total Amount on Event and Contribution pages (with multi-currency form support)

Fix failure to assign view tpl variables to view page if context=search is in the url

Whack-a-mole round 10

Merge pull request #19183 from eileenmcnaughton/533e

dev/core#2248 Ensure variables are assigned to tpl for urls

dev/core#1019 Fix currency formatting of Total Amount on Event and Contribution pages (with multi-currency form support)

dev/core#2248 Ensure variables are assigned to tpl for urls

Merge pull request #19182 from eileenmcnaughton/else

Remove extraneous elses

Merge pull request #19085 from eileenmcnaughton/words

dev/financial#158  change UI parts of contribution soft schema to soft credit

CRM_Core_Key - Add some basic unit tests

CRM_Core_Key - Provide more debugging hints about `qfKey`s

Overview
--------

The `qfKey` parameter is a security mechanism (CSRF). The content of `qfKey` is also inscrutable - so when there's a problem
with `qfKey`, it can be difficult to determine the origin/nature of the problem.

Before
------

The `qfKey` provides a digital signature based on (1) session ID, (2) the
form being processed, (3) a random private-key (unique to the user/session).
In some cases, the `qfKey` also has a random nonce appended to distinguish
between concurrent tabs that work with the same form.

* Ex: `2abc4b7f23d9ae4dfcdbb0c692cc666e5f11256fe84ace7662b0e075834b81958` (w/o nonce)
* Ex: `2abc4b7f23d9ae4dfcdbb0c692cc666e5f11256fe84ace7662b0e075834b81958_9527` (w/nonce)

If there is a logic problem where the `qfKey` of form A gets mixed-up with
form B, then it's extremely hard to understand the mismatch All you see are
two long random codes.

* Inputted qfKey: `2abc4b7f23d9ae4dfcdbb0c692cc666e5f11256fe84ace7662b0e075834b81958`
* Expected fqKey: `89874b7a23192e4d6aab10c622ca369e5e11226f784a5c7652b95075536b81a5e`

After
-----

The `qfKey` has a prefix to indicate the token's intended usage (as well as a digital signature).

* Ex: `CRMContactControllerSearch154eitbf74v3ko0sw94k08gogo8448asdfw4goggkkwgkww08c` (w/o nonce)
* Ex: `CRMContactControllerSearch154eitbf74v3ko0sw94k08gogo8448asdfw4goggkkwgkww08c_9355` (w/ nonce)

If there is a logic problem where the `qfKey` of form A gets mixed-up with
form B, then the prefix can help you understand. Compare:

* Inputted qfKey: `CRMContactControllerSearch154eitbf74v3ko0sw94k08gogo8448asdfw4goggkkwgkww08c`
* Expected qfKey: `CRMContributeFormContribution5o2due205mp0384080sgc8omw8kwggoksw47sswchs80gw0kgs`

This tells you that there's logical mismatch - it's trying to render the contribution
screen using the key for the contact-search screen.

Comments
--------

* The identity of the `formname` is not sensitive information.
* The `qfKey` might look prettier with more delimeters (`{formname}_{signature}_{nonce}`). However, I belive there are random bits of existing code
  which use `explode('_')` to split apart the signature and the nonce. This formula seems to be drop-in/interoperable.
* I switched the encoding of the signature from hex (0-9a-f) to base-36 (0-9a-z) to make it a bit shorter.

Remove extraneous elses

dev/financial#158  change UI parts of contribution soft schema to soft credit

Merge pull request #19180 from civicrm/5.33

5.33

Merge pull request #19179 from demeritcowboy/depwarn

[NFC] Convert civi.tag-deprecated to use central function

Merge pull request #19176 from eileenmcnaughton/533m

dev/core#2246 Fix failure to filter exports

Merge pull request #19172 from eileenmcnaughton/dep_match

[REF] extract getIdsOfMatchingContact

Merge pull request #19167 from demeritcowboy/legacy-warnings

Unused writeLegacyWarnings() in PropertyBag

convert to central function

[REF] extract getIdsOfMatchingContact

It seems this really needs sorting to be shared between the parser classes
to fix up the error handling

Merge pull request #19171 from eileenmcnaughton/else

Remove extraneous elses

Merge pull request #19178 from civicrm/5.33

5.33 to master

Merge pull request #19173 from eileenmcnaughton/533

dev/core#2244 Simplify and consistently apply checking of whether financial acls are enabled

dev/core#2246 Fix failure to filter exports

This fixes regression caused by us more consistently setting componentTable.

The short version here is that the presence of a componentClause should take precedences
over the inner join intended as a blunt, and maybe unnecessary, measure to
limit the export to the entity in question

Merge pull request #19164 from eileenmcnaughton/member

Switch to non-static functions

dev/core#2244 Simplify and consistently apply checking of whether financial acls are enabled

Per https://lab.civicrm.org/dev/core/-/issues/2244 the move of some handling to this function
did not consistently include an early return when the setting is toggled off.

Eventually there will be no setting - just 'is this extension enabled or not' but
in the transition we need this. The new version is easier to copy and paste :-)
and uses a function in the extension. The function in the main code base is
mostly a lot of code to handle the way the setting used to be stored & it
could just check the setting too now

Merge pull request #19166 from eileenmcnaughton/cont_type

dev/financial#163 Rename / remove internal references to contributionTypeID

Merge pull request #19169 from eileenmcnaughton/cont_type2

Rename internal references to contributionTypeId

Remove extraneous elses

Further removal of elses directly after a return

Merge pull request #19168 from eileenmcnaughton/dup_error

Remove extraneous elses

Merge pull request #19017 from eileenmcnaughton/remove_recur

dev/financial#152 remove determination of source

no longer used

Merge pull request #18854 from eileenmcnaughton/memlabel

[REF] Extract function to retrieve the membership labels.

Merge pull request #19162 from civicrm/5.33

5.33

Merge pull request #18905 from eileenmcnaughton/mail

dev/core#2159 Handle exceptions in Mail:send class

Rename internal references to contributionTypeId

Remove extraneous elses

This looks like a huge code change but it's just removing meaningless elses and reformatting

dev/financial#163 Rename / remove internal references to contributionTypeID

https://lab.civicrm.org/dev/financial/-/issues/163

Merge pull request #19159 from colemanw/api4DynamicJoinFix

APIv4 - Fix dynamic bridge joins (used by Search Kit)

Extract function to retrieve the membership labels.

Rather than construct an array (membershipTypes) early on with a db lookup
and pass it around we retrieve the labels when needed using a helper function
to retrieve the values from the already-available array on membership type
details

Switch to non-static functions

I realise these functions are static so that the formRule can call them - but
the formRule has access to the form object and the intention was obviously always
that the formRule can call functions on the form. This cleans that up:

Merge pull request #19161 from seamuslee001/5.33

Add release-notes/5.32.1

Add release-notes/5.32.1

Merge pull request #19152 from MegaphoneJon/financial-160

financial#160 - set correct from_financial_account_id on an edited fee amount FinancialTrxn

APIv4 - Fix dynamic bridge joins

Move ON clause conditions to the first join when they reference the bridge entity itself
Ensure bridge joins work with joining the same table to itself (e.g. Contact to Contact via RelationshipCache)

dev/core#2159 Handle exceptions in Mail:send class

This adds handling for exceptions - allowing the class to be more effectivel swapped out
- need to think a bit more about the messages though

Merge pull request #19157 from jgrygierek/feature/add-missing-south-korea-state

Add missing state for South Korea

Merge pull request #19158 from demeritcowboy/old-from-to

Remove code deprecated +1 year ago - case_from_XX / case_to_XXX search parameters

remove long-deprecated code

financial#160 - set correct from_financial_account_id on an edited fee amount FinancialTrxn

Merge pull request #19155 from eileenmcnaughton/except2

Remove meaningless calls to ignoreException

Merge pull request #19156 from eileenmcnaughton/vars

Remove some unused variables

add blank line

add missing state for South Korea

Remove some unused variables

Remove meaningless calls to ignoreException

The way ignoreException works is the exception mode is called until it is
deconstructed. In other words the result needs to be assigned to a variable
and when it is unset then the exception mode reverts back.

However, over time cleaner-uppers spot these unused variables & remove them -
meaning the line no longer actually does anything. Since these lines are already
borked they can go....

Merge pull request #18855 from eileenmcnaughton/record

[REF] Simplify membership form code towards simplifying BAO

[REF] Simplify membership form code towards simplifying BAO

On digging into the handling for the param relate_contribution_id in the BAO I found it is called
only from the membership form & it's really a hack for the way price sets were super-imposed on the form.

This has really solid testing in testTwoInheritedMembershipsViaPriceSetInBackend (Thanks @davejenx)
and stepping through there is the best way to make sense of it.

What is happening is that if there are 2 memberships in a price set Membership.create is called twice.

However, because the parameter contribution_status_id is passed in it attempts to create the contribution
twice. If we don't add this the second time then we wind up with the second membership payment not created
because that membership didn't exist the first round. So, an extra param relate_contribution_id got
added which says 'if this is passed in create the MembershipPayment record that didn't get created
the first time with the contribution id that we just did a lookup on.

All of this is needed because we create the contribution between the first Membership create
and the second. This removes both magic params from the create, and instead calls
recordMembershipContribution after both are done. I thought about calling Order.create or
just Contribution.create here but there is stuff to unravel around the soft credit creation
being done in RecordMembershipContribution and I am not yet clear if tests are adequate there

Merge pull request #19154 from civicrm/5.33

5.33

Merge pull request #19115 from totten/master-api4-perm

APIv4 - Add "Permission.get" for listing available permissions

Merge pull request #18794 from eileenmcnaughton/need_less

Membership BAO - do not require date fields to be passed in on update

Merge pull request #19146 from JMAConsulting/issue-2174

core#2174: Activity Summary report is missing pagination

Merge pull request #18838 from eileenmcnaughton/vrenew

dev/core#2024 extra line item issue on membership renewal

Merge pull request #19120 from eileenmcnaughton/fee

Handle possibility of fee_amount = ''

Merge pull request #19149 from seamuslee001/pager_notice_fix

Fix notice error on using max() with only one variable

Merge pull request #19109 from eileenmcnaughton/cull

Remove extraneous handling duplicated from createProfileContact

Afform - Export list of synthetic permissions

APIv4 - Add "Permission.get" test. Include BD/D7/D8/WP variants.

APIv4 - Add "Permission.get" for browsing available permissions.

CRM_Core_Permission_* - Add getAvailablePermision(). Implement on BD/D7/D8/WP.

Merge pull request #19153 from civicrm/5.33

5.33

Merge pull request #19150 from colemanw/searchKitJoins

Search kit joins

Remove extraneous handling duplicated from createProfileContact

I checked and tag and group are not available as import fields

While it's possible to add a contact to a group or tag them that is done elsewhere
- see
https://github.com/civicrm/civicrm-core/blob/0a66a182969ae3fde9f8e54308c4a35fa415685b/CRM/Contact/Import/ImportJob.php#L248

Merge pull request #19148 from civicrm/5.33

5.33

Merge pull request #18995 from lucky091588/2199_ufgroup_pre_post_hook

Add pre() and post() hooks for ufgroup entity

Search kit: bump version

Search kit: Improve labels & relevance of smart group field selection

Search kit: Display join name as part of default label

Fix notice error on using max() with only one variable

Merge pull request #19125 from eileenmcnaughton/ev533

CiviEvent - Error registering participants via search task

Merge pull request #19147 from civicrm/5.33

5.33

dev/core#2199: Add pre and post hooks for ufgroup entity