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

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

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

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] 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.

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

Remove always true parameter

Fix lab issue 2254: cannot set is_bulkmail from UI

Remove legacy tpl assigns

These really look like they were copied over from participant forms -
they are not used in the membership tpl receipts

Merge pull request #19216 from laryn/patch-3

dev/core#2211 Make sure addressee field fits column

dev/core#2211 Make sure addressee field fits column

Truncate if addressee field goes beyond the available 255 characters in an export that merges contacts by shared address.

Greenwich: Fix Select2 free-tagging css bug

Merge pull request #19223 from civicrm/5.33

5.33

Merge pull request #19220 from eileenmcnaughton/nfc

Minor code cleanup

Merge pull request #19222 from eileenmcnaughton/mem_form

Remove all handling related to pledge, cms user from newly separated function

Merge pull request #19219 from colemanw/targetTableFix

Fix Invalid argument PHP warning

Merge pull request #19221 from civicrm/5.33

5.33

dev/financial#163 Comment all the places where removal of contributionTypeID proposed

Remove all handling related to pledge, cms user from newly separated function

Merge pull request #19217 from totten/master-assert-perm

DispatchPolicy - Actively report any upgrade problems with hook_civicrm_permission

Merge pull request #19211 from eileenmcnaughton/form_move

Duplicate processFormContribution only Membership form

Minor code cleanup

Fix Invalid argument PHP warning

DispatchPolicy - Actively report any upgrade problems with hook_civicrm_permission

Overview
--------

This is a preventive/diagnostic revision which would bring to light potential
problems with firing `cleanupPermissions()`/`hook_civicrm_permission` at the
wrong moment during an upgrade.

Before
------

If `cleanupPermissions()` (and its `hook_civicrm_permission`) are fired too
early during an upgrade, then the cleanup runs quietly but omits important
results (because the hook is dropped).  On several UF's, the
`cleanupPermissions()` **revokes access** for any omitted permissions.  The
sysadmin would have manually re-grant access.

After
-----

If `cleanupPermissions()` (and its `hook_civicrm_permission`) are fired too
early during an upgrade, then it raises an error during the upgrade.

Surely, it's better to report the error rather than silently drop the data.

Comments
--------

* This revision is preventive/diagnostic/speculative.  It's based on rumor that
  somebody had a problem with permissions in an upgrade.  I don't actually have
  steps to reproduce a probelmatic case.

* To see this preventive mechanism in action, I provoked a trivial violation:
    * Setup a DB with 5.32
    * Checkout the code for 5.34.
    * Add a local hack https://gist.github.com/totten/e1448b343f94ff9c3d971402a1b3db3d
      which has the effect of running `cleanupPermissions()` prematurely.
    * Observe: `cv upgrade:db` fails (cv v0.3.5+)

* It's tempting to think of `hook_civicrm_permission` as returning a static list of
  strings -- in which case, there's no real harm to firing during the defensive
  phase.  The problem is that `hook_civicrm_permission` can also be used for
  dynamic scenarios.  (The power of hooks!) That's useful for any site-building
  extension (e.g.  bespoke forms/views/APIs/dashlets) where you generate new
  permissions based on site configuration.  That will depend on the
  correct-functioning of the extension+configuration...  which cannot be
  ensured during the defensive upgrade-phase...  and which creates a parallel
  choice between incorrect-results (data-loss) or fatal-error.  This is the
  real reason why one shouldn't run `cleanupPermissions()` during the defensive
  phase.  (Of course, we should run it...  during the liberal phase...)

Remove functions from EmailCommon that were moved to the trait

Merge pull request #19019 from eileenmcnaughton/total_fail

dev/core#927  Fully remove cancel & fail from Contribution BAO

Merge pull request #19207 from eileenmcnaughton/member_ids

[REF] Clean up on $ids['contribution']

Duplicate processFormContribution only Membership form

Merge pull request #19212 from eileenmcnaughton/form_move2

Move processConfirm function from Utils file back to form class

Merge pull request #19215 from totten/master-dispatch-doc

(NFC) DispatchPolicy - Add comments to docblock

Merge pull request #19213 from eileenmcnaughton/dep

Remove deprecated function

(NFC) DispatchPolicy - Add comments to docblock

This updates some of the docblocks to reflect https://github.com/civicrm/civicrm-core/pull/17126

Remove deprecated function

Move processConfirm function from Utils file back to only form still using it

This function is really part of the Confirm form. People thought sharing it would
be good once. They were wrong. More specifically sharing large blocks of
code that attempt to service many diferent needs is not helpful

Merge pull request #19208 from eileenmcnaughton/mem_deb

Remove some more variable variables + some test cleanup

Merge pull request #19181 from colemanw/api4DateRange

APIv4: Support relative date range input

Add an eventID to the pre/post Insert/Update events so they can be matched together

Merge pull request #19206 from seamuslee001/migrate_print_array

#REF Migrate the print_array smarty plugin from in packages into core…

Cleanup on test classes to support more flexible use of price set work

Remove some more variable variables

Merge pull request #19205 from eileenmcnaughton/mem_forms

[REF] Move function to shared parent so MemberForm can use it too