"Admin => Misc" - Fix validation of "Maximum File Size"

The form "Administer => System Settings => Miscellaneous" has a field for "Maximum File Size".

I was on a workstation where the PHP-default and the Civi-default were disagreeable, so
(by default) I couldn't submit the form. So I noticed some quirks -- fixed below.

Before
------

* The error message tells you that the "Maximum File Size" disagrees with a setting in `php.ini`.
  Hopefully, you know what+where of "php.ini", otherwise you'll be at a loss for what value is acceptable.
* The validation assumes that `upload_max_filesize` is expressed in unit-megabytes. But `php.ini` actually
  allows different units (`2m`, `2048k`, `1g`, `2097152`). The comparison is incorrect
  if any other unit is used. (ex: If `php.ini has `upload_max_filesize=1g`, and if the requested
  limit is `2` megabytes, then it should accept - but it rejects due to mismatched units.)

After
-----

* Error message tells you what you need to know.
* Validator correctly interprets the units used in `php.ini`'s `upload_max_filesize`.

Merge pull request #19278 from colemanw/relationshipTypePseudoconstants

Add pseudoconstant for Relationship.relationship_type_id

Merge pull request #19365 from eileenmcnaughton/dep

Move another deprecated utils function

Merge pull request #19325 from colemanw/cssFix

Add explicit box-sizing to crm-flex-box style

Merge pull request #19366 from eileenmcnaughton/import

[NFC] minor code cleanups

Merge pull request #19380 from civicrm/5.34

5.34

Merge pull request #19377 from civicrm/bump_composer_compile_version

REF Bump the composer-compile-plugin version

Merge pull request #19378 from ahed-compucorp/Add-Ahed-to-contributors-file

[NFC] Update contributor-key.yml with Ahed

Add Ahed to contributor-key.yml file

REF Bump the composer-compile-plugin version

Merge pull request #19375 from civicrm/5.34

5.34 to master

Merge pull request #19374 from agh1/5.34.0-releasenotes-initial

5.34.0 release notes initial run

Merge pull request #19370 from eileenmcnaughton/cleanup2

Use php to require an array

Merge pull request #19367 from eileenmcnaughton/import2

[NFC] Minor code cleanup

5.34.0 release notes: added boilerplate

5.34.0 release notes: raw from script

Merge pull request #19369 from eileenmcnaughton/cleanup

[REF] Remove function that is the same as parent

Use php to require an array

Removes is_array check in favour of php strict typing

[REF] Remove function that is the same as parent

Merge pull request #19364 from civicrm/move_upgrade

[NFC] Move Upgrade for navgation from 5.34.alpha1 to 5.35.alpha1 as t…

[NFC] Minor code cleanup

[NFC] minor code cleanups

Move another deprecated utils function

Function is only called from one class, relocate there

[NFC] Move Upgrade for navgation from 5.34.alpha1 to 5.35.alpha1 as the 5.34 RC has been branched

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