[NFC] Minor code cleanup

Merge pull request #19363 from seamuslee001/regen_generated

(NFC) Regenerate civicrm_generated.mysql

(NFC) Regenerate civicrm_generated.mysql

Merge pull request #19342 from alifrumin/mailmenu2

Update permissions for "CiviCRM Admin Menu -> Mailings -> Message Templates" nav item

Merge pull request #19362 from eileenmcnaughton/anet2

Set contribution_page_id from template contribution

Merge pull request #19323 from seamuslee001/default_handler_exception

REF Update CiviCRM default PEAR Error handling to be exception rather…

Merge pull request #19361 from demeritcowboy/arrested-development

dev/core#2241 - Deprecate direct calls to isDevelopment()

Merge pull request #19360 from agileware/CIVICRM-1638

When two contacts, Contact A and Contact B have the same email address, if Contact A is deleted then Contact B is incorrectly excluded from mailings, does not receive emails

Set contribution_page_id from template contribution

This is tested in AuthorizeNetIPNTest::testIPNPaymentRecurNoReceipt

deprecate direct calls to isDevelopment

Merge pull request #19358 from colemanw/dashboardStatus

Dashboard: Improve status messages when moving/deleting dashlets

Set version to 5.35.alpha1

Set version to 5.34.beta1

Merge pull request #19359 from totten/master-dao-update

Regenerate DAOs

CIVICRM-1638 When two contacts, Contact A and Contact B have the same email address, if Contact A is deleted then Contact B is incorrectly excluded from mailings, does not receive emails

Mailing Recipients - Update to match

This is a follow-up to https://github.com/civicrm/civicrm-core/pull/19351 which captures the
spelling fix ("Recipientses").

Regenerate DAOs

This is a follow-up to https://github.com/civicrm/civicrm-core/pull/19351 to
reflect recent changes in some schema files.

Merge pull request #19357 from totten/master-joomla

distmaker - Don't require dummy config file for building Joomla

Dashboard: Improve status messages when moving/deleting dashlets

(REF) distmaker - Simplify joomlaxml.php input-parsing with "??" notation

Makes this much easier to read.

Honestly, I'm not sure it should even be checking $_SERVER like this --
$argv is probably enough, and I suspect that $_SERVER won't give the
expected values in most PHP environments.

distmaker - Don't require dummy config file for building Joomla

BEFORE: To run distmaker and produce Joomla ZIPs, you need to create a dummy settings file.
For example, this is how `cividist` does it:

https://github.com/civicrm/civicrm-buildkit/blob/v19.07.0/bin/cividist#L119-L121

It seems that is just a long-winded way of setting up the Civi classloader. (It wouldn't
make sense to use any other Civi services this is a pre-bootstrap environment.)

AFTER: The dummy file is neither needed nor used. We just get the classloader directly.

Merge pull request #19356 from demeritcowboy/add-timeline-buttons-fix

dev/core#2292 - Add Timeline dropdown on Manage Case not working after buttons UI changes

timeline fix after buttons changes

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

upgrader script

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.

update permissioning for new sites

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