Merge pull request #17588 from artfulrobot/artfulrobot-property-bag-support-empty

Improve PropertyBag handling of offsetGet and custom properties; add …

Merge pull request #17644 from eileenmcnaughton/export_fix_standalonne

Export fix

Merge pull request #17672 from mattwire/onbehalfof

Fix validation of select2 fields in "On behalf of Organisation" block

Export - further cleanup export class to remove unnecessary switch

Fix issues with select2 validation when it is part of hidden 'On behalf of organisation' block

Whitespace cleanup

Merge pull request #17671 from seamuslee001/select2_error_message

[REF] Fix jquery error message display on select2 field validation

Merge pull request #17670 from seamuslee001/fix_allow_clear_radio

[REF] Fix allowing users to clear values for raido custom fields when…

Merge pull request #17665 from eileenmcnaughton/ct

Fix master regression on inactive contact types

[REF] Fix jquery error message display on select2 field validation

[REF] Fix allowing users to clear values for raido custom fields when not required

Merge pull request #16488 from mattwire/quickform_requiredfields

Pass required attribute for quickform fields through to form

Fix master regression on inactive contact types

Removing the WHERE type.name = NULL here makes sense

https://github.com/civicrm/civicrm-core/commit/351e8d470a95e1a610280731706c816033fb79fa#diff-ae3cfb8afd3fe1ab8f471b3a069563cdL244

but a few lines later an extra condition is added - we need the WHERE there to prevent it being
added to the join clause.

Note that I did something a bit yuck with the static. I want to properly re-write the function
with consistent caching but after it's under testing, hence a quick fix

Merge pull request #17615 from seamuslee001/phpunit7_new

Use PHPUnit7 as the primary test runner for PHP versions >= 7.1 when using the phpunit script

Merge pull request #17642 from eileenmcnaughton/trxn

[REF] Extract getTestTrxnID

Merge pull request #17654 from eileenmcnaughton/cancel

Fix A.net to retrieve processor id from property bag

[REF] Use PHPUnit7

Fix tap printer

Merge pull request #17663 from seamuslee001/prep_phpunit7

[NFC] Fix tests in prepartion for phpunit7

[NFC] Fix tests in prepartion for phpunit7

Merge pull request #17660 from seamuslee001/fixflexmailer

[REF] Fix Compatability issue with flexmailer caused by the PHP7.4 fix

Merge pull request #17662 from seamuslee001/master

Revert "[NFC] Fix tests in prepartion for phpunit7"

Revert "[NFC] Fix tests in prepartion for phpunit7"

This reverts commit 11a56cfe5f918a54284b7614a9489d53c9c41208.

Merge pull request #17661 from seamuslee001/prep_phpunit7

[NFC] Fix tests in preparation for phpunit7

[NFC] Fix tests in prepartion for phpunit7

[REF] Fix Compatability issue with flexmailer caused by the PHP7.4 fix

Merge pull request #17608 from seamuslee001/dev_translation_48

dev/translation#48 Implement Brick/Money to better handle money values

Merge pull request #17612 from eileenmcnaughton/format

[NFC] Remove instances where html is passed to crmMoney

dev/translation#48 Implement Brick/Money to better handle currecny values

Added in test case as per Eileen comment so we know we can handle large floats

Fix A.net to retrieve processor id from property bag

Merge pull request #17611 from eileenmcnaughton/renew

[REF] Consolidate code in processMembership

Merge pull request #17656 from civicrm/5.27

5.27

Merge pull request #17008 from ivan-compucorp/CPS-70-fix-radio-value

dev/core#1566 Fix display value for money radio custom field

Merge pull request #17652 from twocs/patch-1

Misspelled word

Merge pull request #17651 from colemanw/domainSer

Add serialize metadata to domain.locales field

Merge pull request #17653 from MegaphoneJon/dynamicreference-fix-527

handle less common dynamic reference cases

Merge pull request #17648 from seamuslee001/manual_processor_set_paymentProceesor

[REF] Ensure that the Manual Payment Processor sets the _paymentProc…

Merge pull request #17649 from seamuslee001/fix_array_acess_non_array

[REF] Fix PHP7.4 Test fails when we are accessing array keys when the…

handle less common dynamic reference cases

Misspelled word

receieve -> receive

Add serialize metadata to domain.locales field

CPS-70: Add unit test for Money/Radio field values

PropertyBag now uses deprecatedFunctionWarning instead of log; is silent during cast() calls

Merge pull request #17647 from seamuslee001/php74_query_fix_2

[REF] Fix parts of code where curly brackets were being used for array or string access which is deprecated in PHP7.4

[REF] Fix PHP7.4 Test fails when we are accessing array keys when the value is not an array

[REF] Ensure that the Manaual Payment Processor sets the _paymentProcessor variable like other Processors to fix issues in PHP7.4 and UnitTests

[REF] Patch electrolinux/phpquery and also core files to fix using {} when doing array or string access when it should be []

Merge pull request #17646 from colemanw/isMultilingual

Fix isMultilingual to use static caching and respect current domain

Fix isMultilingual to use static caching and respect current domain

Merge pull request #17637 from mlutfy/isFormInViewOrEditMode

Add CRM_Core_Form::isFormInViewMode and CRM_Core_Form::isFormInEditMode

Adds CRM_Core_Form::isFormInViewMode and CRM_Core_Form::isFormInEditMode (useful for extensions implementing buildForm)

WIP - fix for export

[REF] Extract getTestTrxnID

Merge pull request #17640 from samuelsov/bugreportcivigrant

Bug in Grant statistics report

Merge pull request #17639 from mlutfy/pfSetEntity

Price Field Form: save the fid for the postProcess hook

Price Field Form: save the fid for the postProcess hook

Bug in Grant statistics report

Merge pull request #17516 from eileenmcnaughton/pf_field

Add EntityFormTrait to pricefieldForm - stdised getEntityId()

Set CRM_Core_Form::isFormInViewOrEditMode as public (useful for extensions implementing buildForm)

Merge pull request #17634 from civicrm/5.27

5.27

Merge pull request #17635 from seamuslee001/5262_notes

5.26.2 Release Notes

Update 5.26.2.md

Update 5.26.2.md

5.26.2 Release Notes

Merge pull request #17632 from seamuslee001/dev_core_1806

dev/core#1806 - Fix import of "Radio"-style custom fields using option "label"

dev/core#1806 Fix issue with importing radio custom data field using the option label

Merge pull request #17631 from seamuslee001/master

5.27

Regenerate DAO

Merge in 5.27

Merge pull request #17628 from colemanw/customOptionsFallback

dev/core#1816 - Legacy support for matching custom field options by label

Merge pull request #17627 from demeritcowboy/deprecate-allactivitytypes

dev/core#1433 - Add deprecation warning for CRM_Case_XMLProcessor::allActivityTypes

dev/core#1816 - Legacy support for matching custom field options by label

Merge pull request #17625 from demeritcowboy/missing-activitytype-5.27

dev/core#1822 - Fix missing newly created activity types from the dropdown on the new activity form

add deprecation warning

Merge pull request #17622 from seamuslee001/smart_group_error

Fix issue with building smart groups when  is empty

fix missing newly created activity types

Merge pull request #17619 from seamuslee001/dev_core_1823_alt

dev/core#1823 Fix building the Price Field visibility options

Merge pull request #17616 from demeritcowboy/xml-acttype-more-more

dev/core#1433 - Replace last instances of calls to CRM_Case_XMLProcessor::allActivityTypes()

Fix issue with building smart groups when  is empty

dev/core#1823 Fix building the Price Field visibility options

[REF] Consolidate code in processMembership

Now we have test cover from https://github.com/civicrm/civicrm-core/pull/17605 and other
recents we can consolidate this code & be comfortable the test cover will pick up issues.

test for case search form

replace function that doesn't cache properly with the one it ends up calling which does cache properly

Merge pull request #17610 from eileenmcnaughton/tax

dev/core#1603 remove places where taxAmount is rounded

PropertyBag call deprecatedFunctionWarning on array access to custom props

Merge pull request #17614 from demeritcowboy/xmlproc-acttypes-more

dev/core#1433 - Replace call to CRM_Case_XMLProcessor::allActivityTypes() that doesn't cache properly

Payment PropertyBag now issues warning if array access used to get custom prop

replace call to function that doesn't cache properly

Merge pull request #17613 from civicrm/5.27

5.27 to master

Merge pull request #17607 from eileenmcnaughton/pi

Fix payment instrument bug by using correct payment instrument

dev/core#1603 remove places where taxAmount is rounded

[NFC] Remove instances where html is passed to crmMoney

After grepping I'm pretty confident the other params are never passed so removed

Merge pull request #17600 from seamuslee001/null_array_access

[REF] Fix issue with using array access tools on NULL values

[REF] Fix issue with using array access tools on NULL values

Merge pull request #17604 from seamuslee001/incorrect_array_key_exist_group

[REF] Replace incorrect usage of array_key_exists with property_exist…

Merge pull request #17605 from eileenmcnaughton/membertest

[NFC] Add assertions to existing test to lock in current behaviour

Merge pull request #17606 from seamuslee001/php74_mock

[NFC] Update mocking to use getMockBuilder instead instead of createMock

Fix payment instrument bug by using correct payment instrument

Per https://github.com/civicrm/civicrm-core/pull/17589 it is necessary that we do NOT pass the wrong payment_instrument_id
to the payment processor. BUT we have to provide accurate defaults for manual payments. This is a narrower fix than

- back office membership form
- back office participant form
- back office contribution form

and in all cases the Manual processor still loaded correctly. I don't feel I can say this is the last fix in this
area but it stands alone as a sensible fix to do and also one that should address the immediate issue.

Note there is some weirdness in a second function in EventFees - that function should GO IMHO - but I have not yet reached it in
UI testing to confirm if other changes need to be made.

[NFC] Update mocking to use getMockBuilder instead instead of createMock

[NFC] Add assertions to existing test to lock in current behaviour

Per standard tdd refactoring I'm adding tests before I alter the code that is tested. Note that the code
has quite a bit I think can go

[REF] Replace incorrect usage of array_key_exists with property_exists in Group BAO Class