Merge pull request #19336 from eileenmcnaughton/nfc

[NFC] Code reformat pledge class

Merge pull request #19298 from eileenmcnaughton/pp_mult

Rename PledgePayment create function

Merge pull request #19335 from eileenmcnaughton/up

DB changes for already-made schema changes

Merge pull request #19338 from seamuslee001/master

release-notes/5.33.0.md - Last few PRs

release-notes/5.33.0.md - Last few PRs

[NFC] Code reformat

IDE code reformat

DB changes for already-made schema changes

Alters financial type schema by making the following boolean fields required:
is_reserved (default 0)
is_deductible (default 0)
is_active (default 1)

And pledge schema
installments (default 1)
status (no default)
is_test (default 0)
frequency_unit (default 'month')

Merge pull request #19334 from civicrm/5.33

5.33

Merge pull request #19333 from mlutfy/fixTs533

Fix ts usage for CiviCRM 5.33

Merge pull request #19319 from mlutfy/wpMailingSettingsUrl

Mailing default domain error: force a backend URL for WP

Merge pull request #19331 from demeritcowboy/php74-squiggly-bracket

php 7.4 compatibility - Array and string offset access syntax with curly braces is deprecated

Fix ts usage for CiviCRM 5.33

php 74 compat

Merge pull request #19329 from civicrm/5.33

5.33

Merge pull request #19288 from eileenmcnaughton/group3

Add index on civicrm_group.cache_date

Merge pull request #19328 from colemanw/duplicateButtonFix

Correctly check button to allow duplicate contact saving from profiles

Merge pull request #19256 from demeritcowboy/is-development

dev/core#2240 and dev/core#2241 - Move non-compliant trigger_error out of logging

Fix dev/core#2287 correctly check button to allow duplicate contact saving from profiles

Merge pull request #19327 from agh1/5.33.0-releasenotes-final

5.33.0 release notes: final edits

Merge pull request #19321 from colemanw/profileGetFieldsFix

Fix APIv3 profile.getfields to return correctly keyed phone field

5.33.0 release notes: final edits

5.33.0 release notes: added late changes

Merge pull request #19324 from demeritcowboy/no-port

dev/core#2286 - Avoid notice on missing db port during install

handle missing port

Merge pull request #19311 from seamuslee001/dev_core_2028

dev/core#2028 Add in a status check for checking if the wkhtmltopdf p…

dev/core#2028 Add in a status check for checking if the wkhtmltopdf package is installed in the expected location, Also ensure if it isn't then the pdf generation falls back on the current default which is DOMPDF

Merge pull request #19322 from eileenmcnaughton/nfc

[NFC] Cleanup in test class

Merge pull request #19302 from eileenmcnaughton/act_map

[REF] extract and share code to determine if required contact fields are present

Merge pull request #19315 from mattwire/smtpsettingsignoreexception

Remove use of ignoreException from SMTP settings form

Merge pull request #19296 from eileenmcnaughton/fbool

Removes default for is_active & is_reserved since they now have DB defaults

Fix APIv3 profile.getfields to return correctly keyed phone field

This fixes the phone profile field to have the correct field key to match profile forms.
Fixes https://github.com/civicrm/org.civicrm.contactlayout/issues/91

Remove use of ignoreException() from SMTP settings form.

Merge pull request #19251 from totten/master-crypt-rekey

(dev/core#2258) Add API+hook to rotate keys for encrypted fields

Merge pull request #19301 from eileenmcnaughton/act_construct

[REF] Fix import signature on activity parser, add preliminary test

[NFC] Cleanup in test class

Removes default for is_active & is_reserved since they now have DB defaults

Is deductible already had a default but it has been fixed so we can rely on it.

FinancialType Schema: make is_deductible db default 0, make bool fields required

Merge pull request #19318 from colemanw/apiJoinTest

[tests] Fix join syntax conversion for APIv4 in v3 unit tests

Mailing default domain error: force a backend URL for WP

Merge pull request #19314 from mattwire/importnoignoreexception

Remove use of ignoreException on import form

Fix join syntax conversion for APIv4 in v3 unit tests

Merge pull request #19309 from eileenmcnaughton/pledge_req

Pledge: mark needed fields as required in schema

Merge pull request #19316 from ahed-compucorp/dev/core#2282

dev/core#2282 Use the proper content type for ICalendar link

Don't check for create temporary table permissions on import form. CiviCRM in general requires support for temporary tables

dev/core#2282 Use the proper content type for ICalendar link

Merge pull request #19307 from eileenmcnaughton/534

dev/core#2251 provide default receive_date

Merge pull request #19313 from civicrm/5.33

5.33

Merge pull request #19310 from seamuslee001/port_poc_c_533

(POC) MembershipRenewalTest - Address assertions that started failing…

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 '…

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

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

Pledge: mark needed fields as required in schema

This marks all the fields that are not optional as required. I've been having a bit of a battle adding
a v4 api & I hope this will help - but if not, it's correct for them to be marked required anyway

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

dev/core#2251 provide default receive_date

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

[REF] extract and share code to determine if required contact fields are present

[REF] Fix import signature on activity parser, add preliminary test

The construct class does not need to receive mapperKeys as a reference - this
was removed for Membership Parser here https://github.com/civicrm/civicrm-core/commit/14b9e069607c6cbcbbaf21a209f647ba422b8e04#diff-36f5db3555445d26df2de015a18598fd5d5682b76c62aee4f14dde95b7948274L65

In addition 2 places that instantiate this class pass in unused params (go copy & paste) so
that is removed. The test tests very little so far....

Rename PledgePayment create function

Plege payment has a non-standard create function. It is only called from one place in universe. This updates that
1 place to use a new name and puts in the create function in a more correct way.

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

(dev/core#2258) Add System.rotateKey API

(dev/core#2258) CryptoToken - Add rekey method

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

Add index on civicrm_group.cache_date

Per https://github.com/civicrm/civicrm-core/pull/19287 I also considered an index on refresh_date but
I couldn't see any evidence the field is useful

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