Merge pull request #19355 from demeritcowboy/php74-more3

php74 compatibility - Backwards arguments to implode are deprecated

Merge pull request #19069 from eileenmcnaughton/test

Remove unreachable customGroup section in online event receipt

backwards arguments are deprecated

Merge pull request #17889 from mattwire/freezerecur

Disable frequency/interval fields if not required on backend contribution forms

Merge pull request #19351 from mlutfy/fixSoftLoc

[NFC] Fix Contribution Soft Credit entity translation

Merge pull request #19354 from demeritcowboy/php74-more-more

php 7.4 compatiblity - Clarify order for nested ternary operator

Merge pull request #19353 from demeritcowboy/php74-more

php 7.4 compatibility - more curly braces

php 7.4 compatiblity - clarify order

php 7.4 compatibility

Merge pull request #19349 from totten/master-crypt-setup

dev/core#2258 - Define CIVICRM_CRED_KEYS during installation

Merge pull request #19350 from demeritcowboy/php74-more

php 7.4 compatibility - squiggly brackets

Fix Contribution Soft Credit entity translation

deprecated in php 7.4

dev/core#2258 - Setup API should define %%credKeys%%

This updates the the civicrm-setup API to generate CIVICRM_CRED_KEYS (%%credKeys%%) on
t new installations (based on web-installer or cv installer).

dev/core#2258 - Add %%credKeys%% to civicrm.settings.php.template

There are multiple installers distributed across different git repos, and it
make take a bit before they're all updated.  The convoluted ternary
expression ensures that CIVICRM_CRED_KEYS is well-formed regardless
of whether the particular installer knows how to set %%credKeys%%.

Merge pull request #19239 from totten/master-crypt-smtp

dev/core#2258 - Read+write SMTP password using 'crypto.token'

(dev/core#2258) SMTP Password - If CRED_KEYS is defined during upgrade, use it

(REF) CryptoToken - Allow optional injection of $registry

(dev/core#2258) SMTP Password - Support key rotation

(dev/core#2258) SMTP Password - Read+write via 'crypto.token'

Before
------

The format of the 'smtpPassword' subfield depends on the available PECL extensions:

* The field is ciphertext... if PHP has `mcrypt` enabled
* The field is plaintext... if PHP has `mcrypt` disabled

After
-----

The format of the `smtpPassword` subfield is specified by `crypto.token` which means:

* The field is ciphertext... if it begins with `chr(2)`
* The field is plaintext... if it begins with any printable character

(dev/core#2258) SMTP Password - Remove old encryption during upgrade

Merge pull request #19306 from totten/master-renewtest-d

(POC#D) (dev/core#2284) MembershipRenewalTest - Address assertions that started failing circa Jan 1, 2021

Merge pull request #19243 from demeritcowboy/remove-old-xoauth2

dev/core#2264 - Remove never-used IMAP_XOAUTH2 option value before it gets more confusing

(REF) CRM/Member - Change time() to CRM_Utils_Time::time()

(REF) CRM/Member - Change date() to CRM_Utils_Time::date()

(REF) CRM/Member - Change strtotime() to CRM_Utils_Time::strtotime()

MembershipRenewalTest - Use mocked time. Fix assertions that fail in 2021

Circa Jan 1, 2021, this tested started failing reliable -- most likely
because the example values only make sense in a certain time-frame (ie after
Apr 2020 but before Jan 20201).

This patch makes the time-frame explicit by mocking the 'current time'.

CiviTestListener - Ensure that mocked times are always cleaned up

CRM_Utils_Time - Prefer function names which match the stdlib counterparts

There are a large number of calls to `time()`, `date()`, and `strtotime()`
in the codebase.  For someone doing a conversion, it's easier to understand
if the names match.

* Harder conversion
    * `time()` => `CRM_Utils_Time::getTimeRaw()`
    * `date()` => `CRM_Utils_Time::getTime()`
    * `strtotime()` => `strtotime(..., CRM_Utils_Time::getTimeRaw())`
* Easier conversion
    * `time()` => `CRM_Utils_Time::time()`
    * `date()` => `CRM_Utils_Time::date()`
    * `strtotime()` => `CRM_Utils_Time::strtotime()`

FiveThirtyFour - DRY message. Tweak prose.

Merge pull request #19340 from maynardsmith/master

Accept PHP memory_limit of -1 as meeting requirement

Merge pull request #19347 from seamuslee001/5331_rn

5.33.1 Release Notes

Merge pull request #19346 from seamuslee001/master

Fix hook_civicrm_permission upgrade failure. Defer system-flush to 'upgrade.finish' phase.

5.33.1.md - Add upgrader PR

5.33.1 Release notes

CRM_Upgrade_Form - Defer system-flush under 'upgrade.finish' phase

Fix recurring fields on backend contribution forms

Merge pull request #19343 from seamuslee001/master

Fix Contribution.tpl mismatched ts.

Merge pull request #19330 from demeritcowboy/why

Stop preventing test runs and dev sites from seeing PHP deprecation notices

Fix Contribution.tpl mismatched ts.

Accept PHP memory_limit of -1 as meeting requirement

A PHP memory_limit of '-1' is specified as 'unlimited'.

Before the change the checkmemory() function generates a Warning if the limit is set to -1.  This shows up as a Warning in the Drupal Status Report, but more seriously also prevents the Drupal database update process (mydomain.com/update.php) from running.

After the change the value '-1' is accepted as meeting the requirement.

why

remove never-used option value

Merge pull request #19282 from eileenmcnaughton/ftype_add

Add v4 api for financial type, financial account

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