Merge pull request #19232 from eileenmcnaughton/friend

[REF] Extract code that assigns isPendingOutcome variable on thank yo…

Merge pull request #19199 from seamuslee001/dev_core_2242

dev/core#2242 Ensure that when a custom field is deleted any associat…

Merge pull request #19273 from eileenmcnaughton/complete

Simplify handling of ids in Authorize.net now that related_contact is no longer used

Merge pull request #19286 from eileenmcnaughton/group

[REF] Minor cleanup on contactGroup function

Merge pull request #19284 from eileenmcnaughton/mem_r

[REF] Remove some variables not relevant to backoffice member form

Merge pull request #19280 from eileenmcnaughton/params

[REF] Rationalise financialType variable

Merge pull request #19287 from eileenmcnaughton/group2

Stop using refresh_date in civicrm_group table

Merge pull request #19293 from eileenmcnaughton/campindex

Add index to campaign.name

Merge pull request #19299 from eileenmcnaughton/sunil2

fix email receipt flag for recurring record

Merge pull request #19308 from eileenmcnaughton/ev

dev/core#766 fix failure to save custom data on template-based-event

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

Test class tidy up

fix email receipt flag for recurring record

added test case

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#766 fix failure to save custom data on template-based-event

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

Add index to campaign.name

[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

Stop using refresh_date in civicrm_group table

I was looking to add indexes to civicrm_group.cache_date & civicrm_group.refresh_date - but I couldn't figure out why
the latter exists / is used. I went through the places it's used and it is simply a calculated version of
cache_date + smartGroupCacheTime and I can't see any value.

I can follow up on this with a removal if no-one else can see the point of it

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

[REF] Minor cleanup on contactGroup function

This function is only called from one place so we can be sure that contactID is an integer and
showHidden is false

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

[REF] Remove some variables not relevant to backoffice member form

The back office member form permits limited contribution fields to be entered - notably not including
- contribution_page_id
- thankyou_date
- cancel fields
- amount_level

This removes

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

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