[NFC][test] code formatting only SearchTest, SelectorTest

Merge pull request #14688 from eileenmcnaughton/revert

 Fix erroneous variable passed to callAPISuccessGetValue

Fix erroneous variabl passed to callAPISuccessGetValue

Merge pull request #14671 from eileenmcnaughton/cust_field

[REF] transform the setting of defaults in CustomField::create to be like (some) other entities

Merge pull request #14687 from eileenmcnaughton/matt

Deprecate some deprecated address functions: defaultCurrencySymbol

Merge pull request #14244 from greenpeace-cee/fix-contribution-overwrite

dev/financial#50 - Fix contributions and participants getting overwritten

Merge pull request #14587 from samuelsov/lab1058

Fixing the display of checkboxes in event confirm / thank you (dev/core#1058)

Merge pull request #14686 from eileenmcnaughton/search_cancel

Fix failing test by changing expected date format

Deprecate some deprecated address functions: defaultCurrencySymbol

Merge pull request #14654 from JMAConsulting/dev-1079

dev-core#1079: Improper character encoding breaks xml processor

Merge pull request #14682 from demeritcowboy/div-div-div-div-div

dev/core#1086 - Mismatched div and /div tags in caseTypeDetails.html

Fix failing test by changing expected date format

We have been seeing failures on testCancelledFilter on day one of the month - the reason seems to be
that we are expecting, but not receiving, a date format with a leading zero in the qill. This would
pass for much of the month but fail during the single digit days. This doesn't seem like a regreession to
me - just a test-error surfaced by changes in https://github.com/eileenmcnaughton/civicrm-core/commit/6e7cc0f578e814459b1e1b80c7a922684da46140#diff-984fd916f4708bb4b370841ae0ab69e5R465

Merge pull request #14683 from eileenmcnaughton/unserial2

Block classes in unserialize field for IDE cheer

Merge pull request #14681 from eileenmcnaughton/format

[NFC][test] reformat jobTest class

Block classes in unserialize field for IDE cheer

one too many divs

[NFC][test] reformat jobTest class

Merge pull request #14679 from colemanw/Improve

Improve protected field metadata

Improve protected field metadata

Merge pull request #14677 from civicrm/5.15

5.15 to master

Merge pull request #14656 from eileenmcnaughton/ad_hoc_mailing

dev/core#1030 fix mis-saving of hidden smart group when sending mail from search builder

[REF] transform the setting of defaults in CustomField::create to be like (some) other entities

This patter (massaging the params & then copying) is more consistent. PR also moves creating the transaction to the start of the fn

I am cleaning up towards adding a bulk function that can be utilised by api v4 to create multiple fields with some efficiency on
sql statements (which matters when adding 2 or more fields to a large table)

Merge pull request #14670 from eileenmcnaughton/cust_field_2

[REF] do not receive  by reference in CustomField::create

Merge pull request #14477 from eileenmcnaughton/part_defaults

URL support for some params in event search

[REF] do not receive  by reference in CustomField::create

I audited the places where this is called in core and the  object is not later interogated for changes

Merge pull request #14625 from eileenmcnaughton/cust_rel_fix

dev/core#389 Fix custom data relative date searching

Merge pull request #14669 from civicrm/5.15

5.15

Merge pull request #14657 from eileenmcnaughton/smart_15

Remove cached smart group entries when removing from a group

Store header to statics in unit tests

dev/core#1074 fix failure of actions from manage groups screen

dev/core#1030 fix mis-saving of hidden smart group when sending mail from search builder

Fixes a bug where sending a civimail from search builder was sending out to the wrong list

Merge pull request #14256 from greenpeace-cee/innodb-log-tables

dev/core#663 - Use InnoDB engine for extended log tables

Merge pull request #14665 from seamuslee001/mechanisenros_key

Add in Andrei Mondoc(mecachisenros) to contributor key

Merge pull request #14666 from demeritcowboy/finborg

dev/core#1082 - test demonstrating message template mixup

test demonstrating dev/core#1082

Add in Andrei Mondoc(mecachisenros) to contributor key

Merge pull request #13477 from mecachisenros/invoice-line-items

Fix incorrect display of Line Items created via API when printing invoice (for Participants)

Merge pull request #14664 from civicrm/5.15

5.15

Merge pull request #14659 from seamuslee001/5.15

dev/core#1082 properly regenerate civicrm_generated file

Merge pull request #14660 from colemanw/apiKeyCheck

 Move api_key read/write permission checks from api to BAO

Merge pull request #14622 from eileenmcnaughton/recordpartial

[REF] Copy assignProportional Line items back into Payment.create function

Merge pull request #14650 from eileenmcnaughton/cust_field_create

[REF] minor code cleanup - move indexExist calculation to the only place in the code that needs it

Merge pull request #14651 from eileenmcnaughton/cust_field_2

[REF] Remove columnName field

add test for invoice line items

Remove cached smart group entries when removing from a group

In 5.15 we have added fixes for smart groups regarding removed entries. Possibly related to this but also possibly pre-existing
I'm seeing contacts who have been removed from smart groups showing up after removal in search results for the duration of the smart group cache.

Regardless the fix feels safe, sensible and correct so I'm comfortable targetting the rc without further research as to whether it
is a regression. The fix being removal of the cache entry when the group entry is removed.

retrieve LineItems by contributionID regardless of component

Move api_key write permission checks from api to BAO

Move api_key read permission checks from api to BAO

dev/core#1082 properly regenerate civicrm_generated file

Merge pull request #14645 from colemanw/noAmp

Remove superfluous pass-by-ref in api3

dev-core#1079: Improper character encoding breaks xml processor

dev/core#389 Fix custom data relative date searching

This addresses a fairly old regression where smart groups based on relative dates stopped being relative

Merge pull request #14653 from eileenmcnaughton/test_clean

[REF + test] extract buildFieldChangeSql and add unit test

Merge pull request #14637 from seamuslee001/unit_test_dev_core_1069

[Test] Add in unit test attempting to demonstrate issue caused by dev…

[REF + test] extract buildFieldChangeSql and add unit test

Merge pull request #14649 from eileenmcnaughton/test_clean

[NFC] formatting on test class cleanup

[NFC] formatting on test class cleanup

Merge pull request #14652 from eileenmcnaughton/cust_field_3

[REF] extract createOptionValue function in CustomField::create

[REF] extract createOptionValue function in CustomField::create

Remove columnName field

This field always holds the same information as ['column_name']. It was introduced
before https://github.com/civicrm/civicrm-core/pull/1206/files which removed some of it's
use but not the variable. There is test coverage on passing in column_name
in testCreateCustomFieldColumnName

Merge pull request #14646 from colemanw/pledgeSpec

Removing unused spec function

[REF] minor code cleanup - move indexExist calculation to the only place in the code that needs it

Readability fix

Merge pull request #14643 from eileenmcnaughton/schemah

Remove a few places where pass by reference is used but does not need to be

Removing unused spec function

Remove superfluous pass-by-ref in api3

Remove a few places where pass by reference is used but does not need to be

Merge pull request #14642 from civicrm/5.15

5.15

Merge pull request #14571 from eileenmcnaughton/mem

dev/core#1038 partial - fix checksum url for cancelling recurring with mid in the url

Merge pull request #14641 from agh1/fix-cron-help-link

Scheduled jobs: replace outdated wiki link

Scheduled jobs: replace outdated wiki link

Merge pull request #14601 from seamuslee001/dev_core_1067

dev/core#1067 Clean Money before creating Campaign record and add test

Merge pull request #14626 from eileenmcnaughton/notice

Fix notice on editing contribution

Fix notice on editing contribution

Merge pull request #14499 from mattwire/extract_getdefaultroleid

REF Extract getDefaultRoleID for add participant

Merge pull request #14634 from colemanw/checklist

Add checklist-model angular module

Merge pull request #14636 from colemanw/revertDate

Partially revert "Update example dates from 2009 to current year"

Merge pull request #14597 from colemanw/Reset

Reset language at end of localized api call

Merge pull request #14638 from civicrm/5.15

5.15 to master

Merge pull request #14624 from eileenmcnaughton/force_defaults

dev/core#1059 Fix contribution search to work with url parameters in force mode

Merge pull request #14618 from eileenmcnaughton/5.15

dev/core#926 & dev/core#1048 Fix group search builder bugs

[Test] Add in unit test attempting to demonstrate issue caused by dev/core#1069

Merge pull request #14632 from colemanw/datePicker

Handle relative start & end dates passed to datepicker widget

Partially revert "Update example dates from 2009 to current year"

This reverts most of commit 86ef49af1894a4e68a9bf84d40bb5fa0cb98c586.

Merge pull request #14635 from colemanw/2019

Update example dates from 2009 to current year

Update example dates from 2009 to current year

Add checklist-model angular module

Merge pull request #14630 from mlutfy/activityStatus

Activity formRule status check cleanup

Handle relative start & end dates passed to datepicker widget

Merge pull request #14607 from seamuslee001/clear_memory_backed_groups_cache

Ensure that contact groups caches are cleared if memory backed

Merge pull request #14631 from seamuslee001/remove_legacy_prevnext_group

Remove handling for legacy PrevNextCache group as it has now been con…

dev/core#1059 Fix contribution search to work with url parameters in force mode

Note this is part of the general 'convert receive_date to datepicker' work

Support url defaults on event search

This adds url support for sort_name & participant_status_id & participant_register_date to event search

civicrm/event/search?reset=1&sort_name=p&participant_status_id=1&participant_register_date_low=20180101

Remove handling for legacy PrevNextCache group as it has now been converted

Merge pull request #14627 from eileenmcnaughton/cont_date

Follow up to receive_date convert to datepicker - update test

Merge pull request #14588 from colemanw/js

Improve utilities & tests for working with js notation

Merge pull request #14629 from seamuslee001/test_dev_core_357

[TEST] Update Email Common Test to incorporate testing for the fix fo…

Improve utilities & tests for working with js notation

Activity formRule status check cleanup

Under some circumstances, the activity_type_id from fields and from the getKey
call can both return NULL, causing the formRule to raise an error that one cannot
record a scheduled SMS activity.

Merge pull request #14602 from mattwire/event_selectedchild

Add a helper function to ensure we always set the correct tab for manage events

[TEST] Update Email Common Test to incorporate testing for the fix for dev/core#357 for signatures