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

Move api_key write permission checks from api to BAO

Move api_key read permission checks from api to BAO

Merge pull request #14645 from colemanw/noAmp

Remove superfluous pass-by-ref in api3

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

Merge pull request #14612 from seamuslee001/fix_contribution_status_default_search_form

Ensure that completed status is selected by default on search contrib…

Ensure that completed status is selected by default on search contribution form

Use PseudoConstant function as label could change (good point thanks pradeep)

Follow up to receive_date convert to datepicker - update test

Cast = to IN as tests show this PR otherwise breaks on resolving parent-child

Merge pull request #14609 from seamuslee001/5.15

[NFC] Ensure that test is properly cleaned up

Merge pull request #14604 from colemanw/dev/core#1066

dev/core#1066 Fix mailing permissions

Merge pull request #14613 from seamuslee001/dev_core_1059

dev/core#1059 Replace deprecated start and end url params with receiv…

Merge pull request #14611 from eileenmcnaughton/url_support

Follow up fix on start as a url parameter

Merge pull request #14605 from eileenmcnaughton/cust_deadlock

Reduce deadlocks on inserting custom data by only using 'ON DUPLICATE' when it is not a new row

Merge pull request #14623 from eileenmcnaughton/receive

Add query object support for receive_date_high & receive_date_low and generically date fields

Add query object support for receive_date_high & receive_date_low and generically date fields

[NFC] Ensure that test is properly cleanuped

Merge pull request #14619 from mattwire/pseudoconstant_types

NFC These pseudoconstant functions return array or string

dev/core#1059 Replace deprecated start and end url params with receive_date_low and receive_date_high

Copy assignProportional Line items back into Payment.create function

This is a case where (unintuitively) I feel reducing code sharing and extraction makes sense. The reason being
I think the 'needs' of adding a payment are not obviously the same as the needs of 'changing a payment instrument'
and the code feels confusion and like the extractions make less sense now than they did. In the interests
of improving it bringing it back together seems the right path to me here. We can then edit for the purposes
of payment create code very specifically (and all functions that add payments need to call that)

Fix regression when searching on empty in group search

dev/core#926 Fixes bug on searching for removed members of smartgroups

dev/core#1048 test replicating bug

These pseudoconstant functions return array or string

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

Follow up fix on start as a url parameeter

Merge pull request #14606 from eileenmcnaughton/format

[NFC] code formatting

Merge pull request #14594 from eileenmcnaughton/url_support

Fix url support for receive_date_high & receive_date_low

Merge pull request #14580 from seamuslee001/convert_prevnext_cache

Convert prevNextCache to standard Cache Defintiion format

[NFC] code formatting

Ensure that contact groups caches are cleared if memory backed

Reduce deadlocks on inserting custom data by only using 'ON DUPLICATE' when it is not a new row

ON Duplicate is used in the customData.create function so that a row can be inserted and if it exists
mysql adapts and updates the existing row. This is more expensive and more prone to deadlocks than a straight
'INSERT' but is probably better than figuring it out at the php layer when you don't know if it could be
an update rather than an insert.

However, in many of the cases we already know this information - ie. if we are creating a new contribution
the custom data is created afterwards so we can use this information from Contribution.create to opt for the
cheaper & less deadlocky version.

We deployed this fix before our main fundraiser due to handful of daily deadlocks on this under peak load
and this form of deadlock did not bother us again during our main fundraiser when we were processing large volumes.
The patch has been in production around 6 months at this point.

Note that the reason deadlocks are encountered is that the 'next row' index is locked when inserting, and
it's either locked for longer or more aggressively when it;'s also checking for a deadlock at the same time (not
sure which)

Merge pull request #14600 from seamuslee001/util_function_for_resetting_ACL_System_cache

[REF] Add in utility function for resetting ACL and System Level Caches

Merge pull request #14603 from mattwire/event_comments

NFC Cleanup comments on BAO event

dev/core#1066 Fix mailing permissions

Cleanup comments

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

Merge pull request #14599 from eileenmcnaughton/record_partial_1

Decommission recordPartialPayment function

Add in utility function for resetting ACL and System Level Caches

Merge pull request #14598 from eileenmcnaughton/amp

[REF] do not pass  by reference to store & create functions

Merge pull request #14584 from seamuslee001/contact_group_cache_backend_convert

Convert Contact Groups cache group to standard cache definition

[REF] do not pass  by reference to store & create functions

After checking the code these are unchanged & not using pass-by-reference unecessarily is better practice

Ensure that if we are ever clearing keys from civicrm_cache for prevNext that we also clear from civicrm_prev_next_cache if sqlBacked or just deleteItem if only deleting a single cache Key

Convert prevNextCache to standard Cache Defintiion format

Merge pull request #14595 from colemanw/setup

Fix setup.sh crash when using the -a flag

Decommission recordPartialPayment function

While I would normally say extracting is good sometimes if feels like things are being broken down in the
wrong places. In this case I think the function name is misleading and having this in the body of Payment.create
makes the path to cleaning up that function clearer

Reset language at end of localized api call

Merge pull request #14593 from eileenmcnaughton/seamus

Towards Convert receive_date to use datepicker in search

Fix setup.sh crash when using the -a flag

Fix url support for receive_date_high & receive_date_low

This ensures that the following url params are supported
receive_date_high=20180101
receive_date_low....
contribution_cancel_date_high
contribution_cancel_date_low
invoice_number=

Note that in doing this I fixed the new & somewhat experimental default function to use field uniquenames
rather than field names (as this is how they are added to the form & it was required for cancel_date.

Support for receive_date_relative=this.day etc would be trivial but not yet tackled

Also I see that we historically support 'start' and 'end' I think we can tackle this by doing something like
if start is set then ->set('contribution_date_low' - probably with a deprecation notice)
I will leave for a follow up though