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

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

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