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

Merge pull request #14646 from colemanw/pledgeSpec

Removing unused spec function

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

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

Fix Batch searching on receive date follwoing conversion and handle receive_date param for the moment as not properly namespaced

Merge pull request #14590 from colemanw/i18nApi

Support api3 & 4 language syntax & improve test

Merge pull request #14592 from civicrm/5.15

5.15 to master

Merge pull request #14572 from seamuslee001/dev_core_1052

Fix dev/core#1052 Activities causing a fatal error if target_contact is not accessable to logged in user

Support api3 & 4 language syntax & test

Merge pull request #14460 from eileenmcnaughton/ajax_test

Add default receive_date for contributions at BAO level

Merge pull request #14480 from JMAConsulting/core-553

dev/core#553: Creating new event takes value from default value not from saved template for custom fields

dev/core#1052 fix fatal error when first target contact is not visible to the logged in user and update test appropriately

Add in unit test demonstrating the fatal error generated