Merge pull request #19400 from eileenmcnaughton/pledge_ref

Fix pledge to not use pass-by-reference

Merge pull request #19403 from eileenmcnaughton/mem_order

[REF] Fix Order class to not reload if already loaded

Merge pull request #19407 from eileenmcnaughton/anet

Remove call to loadRelatedObjects

Merge pull request #19410 from eileenmcnaughton/load

[REF] Move code closer to where it is used

Merge pull request #19408 from eileenmcnaughton/old_params

[NFC] Whitespace fix up

[REF] Move code closer to where it is used

Simply moves a few lines of code so the actual usage is apparent

Merge pull request #19404 from eileenmcnaughton/mem_deb

[Test] Extend new membership form to cover multi-line renew + minor cleanup

Merge pull request #19406 from eileenmcnaughton/param

[REF] Param simplification

Fix pledge to not use pass-by-reference

We have deprecated this out of BAO classes

[NFC] Whitespace fix up

Merge pull request #19401 from eileenmcnaughton/mem_test

[NFC] Preliminary cleanup

Merge pull request #19405 from eileenmcnaughton/inv

[REF] Minor cleanup on emailReceipt

Remove call to loadRelatedObjects

None of the ids loaded in this function are used anymore so we can remove the call to it
The only reference to ids after this line is to contribution page id, which is loaded earlier

[REF] Param simplification

[REF] Param simplification

On looking at this param it can only be reached if recur->id is set and the other criteria can be
squashed to an empty check

[REF] Minor cleanup on emailReceipt

Fixes 2 things
1) empty check on line items when it is never set
2) starts to move code out of the shared-function to the form function
when it is only possible when called via the form function. The sharing here
is one of those cases where only a small part of the shared code is actually shared

[REF] Minor cleanup

[Test] Extend new membership form to cover multi-line renew

[REF] Fix Order class to not reload if already loaded

[NFC] Preliminary cleanup

Merge pull request #19399 from seamuslee001/mysql8_rlike

REF Ensure that when doing RLIKE BINARY the field is cast as BINARY t…

REF Ensure that when doing RLIKE BINARY the field is cast as BINARY to prevent issue on MySQL 8

Merge pull request #19393 from civicrm/5.34

5.34

Merge pull request #19398 from seamuslee001/5.34

Add release-notes/5.33.2

Add release-notes/5.33.2

Merge pull request #19384 from totten/5.34-distmaker-wpver

(dev/wordpress#85) Fix version number reported within WordPress admin UI

(dev/wordpress#85) Fix version number reported within WordPress admin UI

Merge pull request #19388 from demeritcowboy/nulls-all-around

dev/core#2297 - E_NOTICE on contribution thankyou_date_time since time component no longer a separate field

Merge pull request #19383 from eileenmcnaughton/civi_sett

Add setting default to speed up cache flush

Add setting default to speed up cache flush

Detailed discussion in https://lab.civicrm.org/dev/core/-/issues/2031 but note
this does not add vendor in to make it easier to get it merged (as
there is more complexity there

Merge pull request #19339 from kainuk/dev_core_2288_sensible

dev/core#2288 - Alternative fix search range for select/radio custom fields - now disabling the combination

Merge pull request #19382 from totten/master-maxfile

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

Update Miscellaneous.php

Merge pull request #19352 from colemanw/afformRef

Afform - code refactoring for angular best practices

Merge pull request #19385 from civicrm/5.34

5.34

time component no longer a field

Merge pull request #19381 from eileenmcnaughton/regress

Fix naming regression dev/core#2298

Fix naming regression dev/core#2298

"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

Afform - use core flex styles

Afform - refactor afForm directive to component

Afform - Ref directives to components in their own files

Afform - Fix scanning for input types

Afform - Refactor elements as components & move to their own files

Afform - Move components to their own files

Afform - Refactor icon picker to service

Afform - Refactor functions to services, move a couple directives to files

Afform - Refactor afGuiEntity to a component

Afform - Refactor afGuiEditor to a component and rename af files

Afform - fix link in breadcrumb

Afform_admin - cleanup ts() calls for best-practices

Rename afform_gui -> afform_admin

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