Add unit test for merge handling on custom data.

This also moves the start & stop of the transaction to being further out so a hard fail
later doesn't leave a half-merged state

Merge pull request #14303 from eileenmcnaughton/extract_cache

[REF] extract buildGroupTree function

Merge pull request #14301 from eileenmcnaughton/reltest

[REF] Test class cleanup - Extract function to create nuanced custom fields to a trait

Merge pull request #14102 from jitendrapurohit/core-888

dev/core#888 - Backoffice participant transfer form does not populate…

Merge pull request #14306 from eileenmcnaughton/doc_block

[NFC] fix comment blocks capitalising null

Merge pull request #14305 from eileenmcnaughton/bool

[NFC] fix more places where var is declared as boolean rather than bool

[REF] extract buildGroupTree function

I did this mainly to surface the fact that the variables involved are factored into the cachekey

[NFC] fix comment blocks capitalising null

From running the drupal code sniffer with type checking enabled it turns out the capitalisation of NULL
does not extend to docblocks.

See https://github.com/civicrm/coder/pull/7

[NFC] fix more places where var is declared as boolean rather than bool

See https://github.com/civicrm/coder/pull/7

Merge pull request #14304 from eileenmcnaughton/dao

[NFC] formatting & comment fixes - Regenerate DAO files

Merge pull request #14282 from eileenmcnaughton/profiletest

[NFC] ProfileTest class - formatting & general cleanup

Regenerate DAO files

Merge pull request #14295 from eileenmcnaughton/dao_2

[NFC] Add trailing comma to pseudoconstant array in DAOs

Merge pull request #14300 from colemanw/Fix

[REF] Fix pointless use of CRM_Utils_Array::value in Campaign BAO

Merge pull request #14294 from eileenmcnaughton/stds

[NFC] Fix variable types in DAO doc blocks

Merge pull request #14296 from eileenmcnaughton/bool

[NFC] fix docblock types from boolean to bool

Merge pull request #14297 from eileenmcnaughton/dedupe_cleanup

[code-cleanup] remove orderBy param from getDuplicatePairs, it is not used

Extract function to create nuanced custom fields to a trait

I want to write a test involving set up of multiple custom fields - this seems like the most
useful function for that, so moving to a trait for re-use

Fix pointless use of CRM_Utils_Array::value in Campaign BAO

Merge pull request #14283 from eileenmcnaughton/db_test3

dev/core#980 [dbunit] QueryTest Remove dbunit dependency on discontinued dbunit

Merge pull request #14291 from eileenmcnaughton/refund_ev

[NFC] array formatting, some comments

Merge pull request #14288 from MegaphoneJon/access-page-simplify

[REF] Simplify ACL page template

Merge pull request #14267 from eileenmcnaughton/setting_fin

dev/financial#54 Fix mishandling of deferred revenue settings

Merge pull request #14289 from eileenmcnaughton/event_dep

Fix deprecation notice on manage events page

[code-cleanup] remove orderBy param from getDuplicatePairs, it is not used

[NFC] fix docblock types from boolean to bool

This is the correct type name we should be using

Add trailing comma to pseudoconstant array in DAOs

Code change only includes sample - full regenerate will follow after merge

Fix variable types in DAO doc blocks

Phpcs (stricter) reports

  79 | ERROR   | [x] Expected "intunsigned" but found "int unsigned" for @var tag in member variable comment
 156 | ERROR   | [x] Expected "bool" but found "boolean" for @var tag in member variable comment

We are declaring int unsigned instead of int & boolean instead of bool

Note this commit only contains sample of resulting change - will add more complete set after

[NFC] ProfileTest class - formatting & general cleanup

[NFC] array formatting, some comments

Merge pull request #14285 from civicrm/5.14

5.14 to master

Fix deprecation notice on manage events page

[REF] Simplify ACL page template

dev/core#980 [dbunit] Remove dbunit dependency on discontinued dbunit

Merge pull request #14277 from seamuslee001/dev_core_976

Remove htmlpurifier from composer and move it back to within IDS to f…

Merge pull request #12152 from JMAConsulting/dev_drupal_19

dev/drupal#19 Drupal8: Implement set UF locale/language (affects mailing tokens)

Drupal#38 - allow creating absolute URLs from CLI in D8

additional fixes

override languageNegotiationURL for Drupal8

(dev/drupal/19) Drupal8: Implement set UF locale/language (affects mailing tokens)

Merge pull request #14278 from seamuslee001/nfc_langauge_cleanup

[NFC] Update Language to be more user friendly when checking logs

dev/financial#54 add routine to convert contribute settings

dev/financial#54 check the correct place for deferred_revenut settings

Currently we have a non-std store for invoicing settings that we are checking, yet the form saves it using our stds.

This updates checks for these settings to actually check the right place

Update Language to be more user friendly when checking logs

Merge pull request #14276 from eileenmcnaughton/dbunit_2

Test cleanup to remove references to discontinued package dbunit

Remove htmlpurifier from composer and move it back to within IDS to fix breakage on drupal with the htmlpurifier module

Merge pull request #14241 from colemanw/enableApi4

Try enabling api4 extension in test suite

Merge pull request #14274 from seamuslee001/protected_fields_dao

Declare API Key as a protected field for future use

Merge pull request #14266 from seamuslee001/dev_core_369

dev/core#369 Prevent hard fail of API Job when SMS provider has been deleted

[TEST] cleanup MailingContactTest to remove dbunit dependency

Remove obsolete code

Clean up groupNesting Test to not use dbunit

Merge pull request #14271 from eileenmcnaughton/db_test_towards

[TEST SUITE] Remove some use of dbunit from tests

Merge pull request #14273 from eileenmcnaughton/test_dupe

[NFC] array formatting in CRM_Dedupe_MergerTest

Declare API Key as a protected field for future use

Merge pull request #14254 from yashodha/add_dev_tab

add developer tab for summary report

[NFC] minor commits cleanup

[NFC] array formatting in CRM_Dedupe_MergerTest

Try enabling api4 extension in test suite

Clean up groupNesting Test to not use dbunit

Rename agpl-3.0.txt to License (#14255)

* Rename agpl-3.0.txt to License

This is the standard name for the license file and will improve this project's community score from GitHub.

* Rename License to LICENSE

Merge pull request #14270 from eileenmcnaughton/dedupe

[ref] Remove reference symbol from 2 variables

Merge pull request #14137 from eileenmcnaughton/pay_up

[REF][Test] Remove call to CRM_Contribute_BAO_Contribution::recordAdditionalPayment in favour of payment create

More dbunit removal

Remove some use of dbunit from tests

Remove reference symbol from 2 variables

I checked & these are not altered in the function - so no need

Remove call to CRM_Contribute_BAO_Contribution::recordAdditionalPayment in favour of payment create

Add test of hard fail following SMS provider deletion and expand action schedule testing to include SMSes

dev/core#369 Fix hard failure caused by setting of NULL in database for sms_provider_id when provider is deleted

Minor improvment to test ensure that is_error is actually 0

Merge pull request #14260 from eileenmcnaughton/spec

[REF+test] clean up code for getting labels for merge screen, stdise

Merge pull request #14264 from colemanw/settingsOptions

[REF] Centralize logic for resolving settings pseudoconstants

Merge pull request #14259 from colemanw/cacheOptions

Remove wasteful double-caching of settings metadata

Merge pull request #14262 from francescbassas/patch-16

dev/core#964 Add start date and end date values on page contribution widget ajax response

Centralize logic for resolving settings pseudoconstants

dev/core#964 Add date start and end values on page contribution widget ajax response

Merge pull request #14263 from elisseck/dev/core/966

dev/core/966 skip urlencode for email in elavon xml

[REF] code cleanup on handling on presentation of merge fields for display

This code was really confusing - all that 'special values' stuff. Turned out it
didn't have to be....

Add unit test for handling of 'special values' when retrieving merge data

This is preliminary to a code cleanup.

m

Merge pull request #14261 from eileenmcnaughton/master

[REF] Remove extraneous variable

dev/core/966 skip urlencode for email in elavon xml

Remove wasteful double-caching of settings metadata

Settings metadata was being cached once per domain and one extra time for good measure.
hook_civicrm_alterSettingsMetaData() was invoked even when fetching cached data,
even though the hook-altered data was stored in the cache.

Now that the hook is invoked only once and the results cached, tests have been tweaked
to cache after setting the hook instead of before.

Merge pull request #14258 from eileenmcnaughton/tidy

[REF] Super Minor code readability fix

Remove extraneous variable

Delete Pair is only called from one place in the code & it does not pass the $entityTable
variable - so remove it & reduce complexity.

In addition the viceVersa variable is always true so we can remove this too.

[REF] Minor code readability fix

Merge pull request #14257 from civicrm/5.14

5.14

Jenkins style fix

Update 5.13.4.md

Update date of 5.13.3 release and add in APIv4 SA into 5.13.4

Update 5.13.4.md

WIP Release Notes for security release

Update Release notes based on Tim's Review and likely drop of 5.13.3

civicrm/file - Be forgiving about old image hyperlinks

Previous versions of Civi sometimes generated URLs for contact-images with incorrect `&mime-type` values:

http://dmaster.bknix:8001/civicrm/file?reset=1&filename=Hello_cca4153cb14beab37c68ab7f07162425.jpg&mime-type=image/jpg

The recent security update will generate an error if the mime-type is incorrect, so this patch relaxes it
to allow the old links to continue working.

Add whitelist back in and validate extension of file is permtted for the mime-type supplied and use mime-type from db if supplied with an fid and eid

Switch to different libary that is php5.6 compatable

Strength mime checking by comparing mime-type to the file path mime-type if we have entity_id and file id otherwise only permit image mime_types to be accepted if going via the filename route

Ensure mimetype is set in the case where we are passing it through and its valid

Remove Whitelisting of mime-types as not useful and only check mime-types if we have had one passed in

Resolve security/core#45 by validating the mimeType of the file with the supplied mime-type

Resolve security/core#52 by validating that an installtype is sensible

security/core#49 Ensure that only intergers are passed to the IN build options in address

Fix Rule checking and add a unit test

Add in unit test on building country_id options too

Add in a unit test for building county options with a state_province_id filter

Also escape subtype to fix POC#2 found by Patrick

escape alphanumeric/checkbox custom data

Fix security/core#51 by paramatising the Event Type part of the wuere clause

Resolve security/core#53 by updating tcpdf to latest version of 6.2.x

(security/core#50) Update jQuery to address prototype pollution