Merge pull request #19291 from eileenmcnaughton/prof

dev/core#527 use public title for on behalf profile and confirm/thankyou pages

Merge pull request #19277 from yashodha/dev-2273

(dev/core#2273) Contact type incorrectly set to Contribution due to '…

Merge pull request #19294 from totten/master-invasive

(REF) Civi\Test\Invasive - Add helper for checking protected/private members

Merge pull request #19300 from eileenmcnaughton/import

[REF] Move another deprecated function back to the only class that calls it

[REF] Move another deprecated function back to the only class that calls it

Merge pull request #19305 from totten/master-renewtest-c

(POC#C) MembershipRenewalTest - Address assertions that started failing circa Jan 1, 2021

(POC) MembershipRenewalTest - Address assertions that started failing circa Jan 1, 2021

Merge pull request #19285 from eileenmcnaughton/money_defaults

Use specific function when formatting money for a default

Merge pull request #19283 from eileenmcnaughton/facl

Move financial acl warning from FinancialType BAO to extension.

Merge pull request #19281 from eileenmcnaughton/bool

Add default for boolean fields on financial_type

(REF) versionCheckTest - Use "Invasive" helper

It's shorter/prettier.

(REF) CiviReport TestCaseTest - Use "Invasive"  helper

It's shorter/prettier

(REF) PaymentTest - Use "Invasive" helper

It's shorter/prettier.

(REF) MailingJobTest - Use "Invasive" helper

It's shorter/prettier.

(REF) ActivityTest - Use "Invasive" helper

It's shorter/prettier.

Civi\Test\Invasive - Add helper for checking protected/private members

Merge pull request #19292 from eileenmcnaughton/camp_index

[REF] Minor cleanup, remove variable variables

Merge pull request #19290 from ahed-compucorp/dev/core#2269

dev/core#2269 Use contribution's currency instead of the default

[REF] Minor cleanup, remove variable variables

Merge pull request #19272 from seamuslee001/more_add_radio

REF use centralised addRadio function more

dev/core#527 use public title for on behalf profile and confirm/thank you pages

This is a partial reviewer's commit of https://github.com/civicrm/civicrm-core/pull/19144

That PR is stale. I tested one flow & added in the 2 changes from there that I could see in my test
& which I was comfortable would not affect anything else. The flow I tested was a contribution
page flow with an on behalf profile. I did not check the email as that would need
a test anyway

dev/core#2269 Use contribution's currency instead of the default

Merge pull request #19057 from ixiam/dev/core#2173

Adapt filters with empty and null values in alphanumeric custom fields

REF use centralised addRadio function more

Use specific function when formatting money for a default

It seems that there are really only 2 ways money::format is called
1) for presenting money - with symbols - in this case only 2 params are passed in
2) for prefilling the form value - in which case some combination of the last 3 are passed in

We have an alternate function that works in the latter case - at which point we could deprecate all
3 in the former case (after a bit more grepping).

The function name is formatLocaleNumericRoundedForDefaultCurrency

I have to admit I'm tempted to remove 'locale' from that name - although
I guess it denotes the swapping of thousand separators

Merge pull request #19241 from demeritcowboy/dev-core-2127

dev/core#2127 - Don't accidentally trim à characters when importing files

Merge pull request #19279 from eileenmcnaughton/go_mem

[REF] Remove unreachable code

Move financial acl warning from FinancialType BAO to extension.

It turns out this is not currently triggered by the api as the api passes in the key FinancialType in
ids and it is looking for financialType. Regardless I think this makes sense in the extension
and setting in the session as currently sometimes works makes sense (at least enough to
move rather than remove)

Add default for boolean fiels on financial_type

[REF] Remove unreachable code

This code was copied from the previously shared function & supports some really weird stuff for the contribution form.

I tested & was unable to reach this from the back office membership form - note it is only reachable
when processing recurring which in turn relies on the membership type being configured and the
processor. It seems that recurring is not available with price sets

Merge pull request #19266 from demeritcowboy/is-development-2

dev/core#2240 - Convert remaining two deprecatedWarning calls

Merge pull request #19275 from colemanw/apiJoinConditions

APIv4 - Smarter check for adding default ON clause to explicit joins

dev/core#2271 - APIv4 - Smarter check for adding default ON clause to explicit joins

When adding explicit joins in APIv4, the API will automacially add an ON clause linking it
to the base entity unless it detects some other link already present in the ON clause supplied.
Previously it would determine this by checking for the join entity ID;
now it looks for the join entity ID OR any other field in the join entity with an FK.
This gives more leeway for people to add creative joins without the defaults getting in the way.

Merge pull request #19274 from eileenmcnaughton/tax

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

style fix

(dev/core#2273) Contact type incorrectly set to Contribution due to 'Honoree Profile'

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

convert remaining calls, and php errors/exceptions don't support
placeholders

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