Merge pull request #19274 from eileenmcnaughton/tax

[REF] Fix tax_amount to be consistent & load from the templateContribution

Merge pull request #19269 from eileenmcnaughton/mem_form

[REF] Unshared another function back onto Membership_Form

Merge pull request #19276 from civicrm/5.33

5.33

Merge pull request #19267 from alifrumin/rn-5.33

[NFC] Release notes 5.33 - First Pass

[REF] Unshared another function back onto Membership_Form

As with the other functions there is more not in common than in common & subsequent cleanup can remove
much of the copied function

Remove code unrelated to membership form

[REF] Fix tax_amount to be consistent & load from the templateContribution

This leverages the work done in https://github.com/civicrm/civicrm-core/pull/15517/files (the test in particular)
but alters it to be more consistent. The original fix relied on the tax_amount on the contribution object
passed in - this fixes it to get the amount from the templateContribution (which is more consistent
now than when the patch was likely written

Merge pull request #19271 from eileenmcnaughton/mem_form2

[REF] Unshare code to build an array of params for the recurring contribution

Merge pull request #19236 from totten/master-crypt-svc

dev/core#2258 - Add services to support encryption

Merge pull request #19270 from eileenmcnaughton/import

Fold deprecated function into the only function that calls it

Merge pull request #19268 from seamuslee001/dev_core_2272

dev/core#2272 Fix a DB error when merging contacts into the same hous…

Merge pull request #19200 from eileenmcnaughton/act_contact

Fix preferred repeattransaction flow to correctly create the activity contacts for the contribution

Merge pull request #19265 from seamuslee001/use_add_radio

REF use addRadio function to add in the radio fields to these forms

Release Notes 5.33

[REF] Unshare code to build an array of params for the recurring contribution

Merge pull request #19262 from eileenmcnaughton/recur_test

  Remove code to retrieve premium data

Fold deprecated function into the only function that calls it

Merge pull request #19248 from eileenmcnaughton/dep_utils

[REF] Relocate another deprecated utils function to the only class that calls it.

Add in unit test to lock in the fix for dev_core_2272

dev/core#2272 Fix a DB error when merging contacts into the same household on contact export and only exporting selected contact fields

Fix creating of activity for repeat transactions to respect those form the template contribution, if on behalf

Add test to ensure correct assignment of activities for on-behalf repeattransactions

REF use addRadio function to add in the radio fields to these forms

Fixes following review by Dave D

Merge pull request #19257 from ixiam/dev#2270

dev/core#2270 - Editing a smartgroup created through the search builder renders the new block by force

Merge pull request #19174 from eileenmcnaughton/conttype

dev/financial#163 Removal of contributionTypeID

Merge pull request #19263 from civicrm/5.33

5.33

Merge pull request #19264 from agh1/5.33.0-releasenotes-initial

5.33.0 release notes initial run

5.33.0 release notes: added boilerplate

5.33.0 release notes: raw from script

Merge pull request #19259 from eileenmcnaughton/confirm

Convert contributionSoft to an array

Merge pull request #19260 from colemanw/searchKitInput

Search kit: Improve token support

Merge pull request #19261 from colemanw/searchKitFix

Search kit - fix bugs introduced during refactoring

  Remove code to retrieve premium data

    After a bit of digging I can't see an evidence we ever have premiums on the backoffice membership forms

Merge pull request #19245 from seamuslee001/dev_core_2263

dev/core#2263 Do not try and store items in the session if the sessio…

Merge pull request #19258 from eileenmcnaughton/recur_test

[REF] Convert previously shared function to non-static, remove unrelated code

Search kit - fix bugs introduced during refactoring

Search kit: Add token selector in display admin UI, allow tokens in tooltips

Also includes some cleanup of the crmSearchAdminLinkSelect component, making it more self-contained

Convert contributionSoft to an array

This is a partial of #19096 which requires contributionSoft to be an array & just simplifies that commit (which
is still failing tests

[REF] Convert previously shared function to non-static, remove unrelated code

There is no facility for creating on_behalf on back office membership forms
so this code, copied from the shared function, does not relate.

The activity IS created by Contribution.create - the extra code was
just to hack it around to doing on_behalf

Likewise, the back office membership form does not collect
data on contribution custom fields or notes so removing these
also makes sense here.

Merge pull request #19227 from eileenmcnaughton/mem_form

Remove or hard-code variables from previously shared function

Merge pull request #19238 from jitendrapurohit/pledgefix

Fix pledge on contribution page when the site has a WR for "contribution"

Merge pull request #19202 from eileenmcnaughton/act_cont2

Fix Contribution.create to not attempt to set contacts on activity update

Rework on the detection of new vs existing smartgroup

Merge pull request #19255 from seamuslee001/concat_within_group_concat

[APIv4] Permit using other SQL functions such as CONCAT within a GROU…

Merge pull request #19116 from eileenmcnaughton/pay_edit

Fix Payment edit form to use Payment.cancel & payment.create api

[APIv4] Permit using other SQL functions such as CONCAT within a GROUP_CONCAT

Merge pull request #19253 from eileenmcnaughton/depp

[REF] Move another deprecated function to the class that uses it

Merge pull request #19246 from eileenmcnaughton/import

[REF] Switch to using shared function to call deprecated function

[REF] Move another deprecated function to the class that uses it

[REF] Relocate another deprecated utils function to the only class that calls it.

I removed the tests because they didn't test much & what they did test
seemed like stuff we should alter

Merge pull request #19249 from eileenmcnaughton/depit

[REF] Relocate another function from DeprecatedUtils to the calling c…

Merge pull request #19252 from mattwire/contributionupdatedlogs

Improve logging when a contribution is created/updated

Merge pull request #19250 from jitendrapurohit/addgroupby

Fatal error on contribution summary report (and probably others) when adding contacts to group

Improve logging when a contribution is created/updated

Fix fatal error on contribution summary report -> add to group action

(REF) CryptoToken - Extract method 'parse()'

(dev/core#2258) CryptoRegistry - Add findKeysByTag()

(dev/core#2258) CryptoRegistry - Allow multiple plaintext entries

This allows the plaintext entries to do have different prioritizations among
different keys.

[REF] Relocate another function from DeprecatedUtils to the calling class

This is only called from one class so relocate to there

Merge pull request #19247 from eileenmcnaughton/dep_utils

[REF] Relocate function from DeprecatedUtils to the class that actually calls it

[REF] Relocate function from DeprecatedUtils to the class that actually calls it

Switch to using shared function to call deprecated function

With this done all the functionality in the deprecated function can be moved into the shared function

Merge pull request #19165 from eileenmcnaughton/pdf

dev/financial#162 Simplify decision as to whether to use a pdf on membership emails

Merge pull request #19225 from colemanw/select2Tweak

Greenwich: Fix Select2 free-tagging css bug

dev/core#2263 Do not try and store items in the session if the session is currently empty

Store the locale in the session even if we are in a single lingual instance

Merge pull request #19244 from eileenmcnaughton/strict

dev/core#2252 remove all handling of strict mode as it has aged out of relevance

Add an explicit alias for sql functions

dev/core#2252 remove all handling of strict mode as it has aged out of relevance

Merge pull request #19237 from colemanw/stripTagsFromOptionDescription

APIv4: Normalize option list descriptions as plain text

Merge pull request #19229 from colemanw/searchKitInput

Search kit: Rewrite input widget to support IN sets, relative dates, BETWEEN groups, etc.

Merge pull request #19209 from mattwire/daoeventid

Add a unique event ID so we can match pre/post Insert/Update

Merge pull request #19235 from colemanw/flex

Add min-width to flex columns for responsive layout on small screens

Merge pull request #19240 from MikeyMJCO/patch-6

(NFC) Fix issue reporting link to go to the corresponding `core` project.

Fix issue reporting link to go to the corresponding `core` project.

(dev/core#2258) CryptoToken - Change notation

This is pre-merge change to the notation in the token. Two things:

* Use only one control character instead of multiple.
* Use URL-style key-value notation. It should be easier to skim and to tweak.

(NFC) CRM_Utils_Hook - Tweak description of crypto()

(dev/core#2258) CryptoRegistry - Remove fallback option based on CIVICRM_SITE_KEY.

This was included under the expectation it might make a nicer upgrade. But
I don't think it buys a whole lot:

1. You run the upgrader. The SMTP password is converted from
   rj256-ecb-sitekey to aes-cbc-sitekey.  All other credentials are left
   unencrypted.  Afterward, you set CIVICRM_CRED_KEY and run the rekey.

2. You run upgrader. The SMTP password is decrypted. All other credentials
   are left unencrypted. Afterward, you set CIVICRM_CRED_KEY and run the rekey.

Additionally, I think there's a question of risk-management when we get to
encrypting more things in the Setting and API layers.  If we go with path 2,
then we can ramp-up adoption progressively, e.g.

* Release 1: Add support as non-default option
* Release 2: Enable by default on new builds
* Release 3: Display alert to existing sites that don't have encryption keys

(dev/core#2258) CryptoToken - More defense against malformed input

(dev/core#2258) CryptoRegistry - Don't parse factory code unless it's being used

This moves the factory function from `Container.php` (which is loaded on all
page-views on all configurations) to `CryptoRegistry.php` (which is only
loaded if the site actually used encrypted fields).

(dev/core#2258) CryptoToken - Don't build registry if we only read plaintext

Example: You have a basic site that has not enabled encrypted fields, but the
integration with settings/APIs causes one to use CryptoToken::decrypt()

(dev/core#2258) CryptoToken - Add service for processing stored ciphertext

(dev/core#2258) PhpseclibCipherSuite - Add default crypto provider

(dev/core#2258) CryptoRegistry - Keep track of available keys+suites. Hookable.

CRM_Utils_String - Add URL-style base64 codec

Merge pull request #19214 from eileenmcnaughton/dep2

Remove functions from EmailCommon that were moved to the trait

Merge pull request #19228 from eileenmcnaughton/inv_assign

Remove duplicated tax assignments from copied code

Fix pledge on contribution page

APIv4: Normalize option list descriptions as plain text

Our schema is inconsistent about whether `description` fields allow html,
but it's usually assumed to be plain text, so we strip_tags() to standardize it.

Merge pull request #19224 from artfulrobot/artfulrobot-fix-bulkmail

Fix lab issue 2254 is_bulkmail cannot be set through UI

Add min-width to flex columns for responsive layout on small screens

The .crm-flex-box class is new and only used in 2 places: Search Kit & the Dashboard.
This sets a min-width on those layouts so the 2 columns collapse to 1 on small screens.

Search kit: Rewrite input widget to support IN sets, relative dates, BETWEEN groups, etc.

This deletes the crmSearchValue widget (and the now-empty crmSearchKit module),
which had originally been copied from the API Explorer, and replaces it with a more flexible
set of components with separate templates for each data type.

Merge pull request #19231 from eileenmcnaughton/processConfirm

Convert previously shared function from static to non-static

Merge pull request #19234 from eileenmcnaughton/cms

Only do cms account create from the one relevant place

Merge pull request #19226 from eileenmcnaughton/odd

Remove unused tpl assigns

Merge pull request #19233 from civicrm/5.33

5.33

Only do cms account create from the one relevant place

This function is called from 3 places - create CMS user is not applicable
to the back office form and the other place actually blocks it...

Convert previously shared function from static to non-static

This no longer needs to be static as it is no longer shared with other forms
(although it needs to be public to support the test class

Merge pull request #19230 from totten/master-phpseclib-cxnrpc

composer.json - Update civicrm-cxn-rpc and phpseclib

composer.json - Update civicrm-cxn-rpc and phpseclib

Before
------

Require `civicrm-cxn-rpc` v0.19 (with `phpseclib` v1.x)

After
-----

Require either of:

* `civicrm-cxn-rpc` v0.20 (with `phpseclib` v2.x)
* `civicrm-cxn-rpc` v0.19 (with `phpseclib` v1.x)

Technical Details
-----------------

* The public interfaces from civicrm-cxn-rpc are the same in 0.19+0.20.
  They only differ in which verison of `phpseclib` is used.
* As pointed out in https://github.com/civicrm/civicrm-cxn-rpc/issues/9, we're not
  the only folks using phpseclib, so some flexibility on that seems good.
* The primary change in phpseclib 2.x is the use of PHP namespaces
  (e.g. `Crypt_AES` => `\phpseclib\Crypt\AES`).
* There are newer versions of both v0.19 and v0.20 which bundle an updated certificate.

Remove or hard-code variables from previously shared function

This function was separated from the shared function for cleanup.

This removes 3 variables
- isRecur (always true)
- billingId - this is elsewhere handled on the form
- online (always false)

For billing id this is handled through back office form
shared functions. The handling might be slightly different but it seems
more important that the form is internally consistent with how
it creates billing addresses (between recur & non-recur) than with other forms

Remove duplicated tax assignments from copied code

The assignment of tax data is happening twice on the form - once in generic
code and once in code only reached for recurring contributions that
was in previously-shared code. We can be fairly comfortable that
in this latter case we don't need it as this is a marginal flow
on this form whereas the main flow is being used 90% of the time
& is doing the assignment