NFC: Cleanup recordAdditionalPayment

Merge pull request #11495 from yashodha/CRM-21637

CRM-21637 - Search Criteria for Card Type ID and Card Number not resp…

Merge pull request #11907 from mattwire/contribution_financialtype_fix

dev/core#88 Make sure financial_type_id is set when contribution is created

Merge pull request #12303 from totten/master-redis-errmsg

(dev/core#178) Redis - Report error messages

Merge pull request #11881 from yashodha/CRM-21854

CRM-21854 - Contribution start date and end dates are not respected

(dev/core#178) Redis - Be a little more conservative about sharing error info

Merge pull request #12309 from jaapjansma/issue_180_deleting_custom_values

Fix for #180: passing the entity ID to the custom hook when deleting a custom value

Merge pull request #11733 from eileenmcnaughton/sched

Fix access to scheduled reminders for admins without event access

Fix for #180: passing the entity ID to the custom hook when deleting a custom value record.

Merge pull request #12308 from eileenmcnaughton/getty

Only treat a request as a get request (and hence use GET) if it starts with get

Merge pull request #12304 from totten/master-redis-anon

(dev/core#179) Redis - Only send AUTH if there's a password

Only treat a request as a get request (and hence use GET) if it starts with get

Current code looks for the word 'get' anywhere in the action so the action 'forget' is also picked up.

Further down we explicitly look for 0 so we should here too.

Merge pull request #11830 from mattwire/devcore24_case_api_contact_fix

dev/core#24 Passing an array for contact_id/client_id to Case.Create API when updating an existing case causes case to be "reassigned"

Merge pull request #11481 from jitendrapurohit/CRM-21622

CRM-21622: Allow multiple parents to be created from 'New Group' form.

Merge pull request #12276 from eileenmcnaughton/memory_more

CRM-19798 fix memory leak on EntityTag.get & other calls in the DB (generic fix)

Merge pull request #12302 from totten/master-redis-null

(dev/core#177) Redis::get() should return NULL for undefined cache keys

 This is a combination of 2 commits.

Fix access to scheduled reminders form.

The current logic is that if the context is not event (empty) and the user does not
have administer CiviCRM they get bounced. if they pass that check they are then
passed into the check that should only be applied when the context IS event. This results
in a bounce for a user without any manage event access.

Remove code duplication

Declare  as a class property & get rid of e-notice handling for it.

There is no functional change here but the code change looks a little large due to the removal of a pointless if

(dev/core#179) Redis - Only send AUTH if there's a password

(dev/core#178) Redis - Report error messages

(dev/core#177) Redis::get() should return NULL for undefined cache keys

The docs for  `CRM_Utils_Cache_Interface::get()` specify that the return value should be NULL
if the key does not exist.

However, `CRM_Utils_Cache_Redis::get()` was returning FALSE in that circumstance
(because `unserialize(FALSE)===FALSE`). This fixes it comply.

Merge pull request #12301 from jitendrapurohit/core-175

dev/core#175 - Smart group involving relationship type filter display…

Merge pull request #12281 from eileenmcnaughton/cont_detail_report

Add testing to contribution detail report, covering dev/core/issues#170

Merge pull request #11883 from jitendrapurohit/core-32

core-32 - Add option to disable cancel button on a profile form

Merge pull request #11736 from jitendrapurohit/CRM-21815

CRM-21815 - re-opening a civicase - Case Coordinator (and other roles…

Merge pull request #12298 from eileenmcnaughton/notice

Fix e-notice, remove legacy code pattern

Merge pull request #12299 from eileenmcnaughton/cancel_recur

Remove unused code variable

Move upgrade function to 5.4

core-32 - Add option to disable cancel button on a profile form

add unit test

dev/core#175 - Smart group involving relationship type filter display incorrect results

Merge pull request #11964 from scardinius/core-56

(dev/core/56) Cancel Recurring Contribution activity should has a source record id

Merge pull request #12300 from civicrm/5.3

5.3

Merge pull request #12277 from seamuslee001/dev_core_163

dev/core#163 Improve inclusion of disabled groups when getting all ma…

Remove unused code variable

dev/core#163 Improve way of getting all groups for use in getting all mailing accessable by a user by removing is_active filter alltogether

Fix e-notice, remove legacy code pattern

Merge pull request #12295 from eileenmcnaughton/mailing

Confirm & lock in group.get handling of is_active

Merge pull request #12297 from civicrm/5.3

5.3

Merge pull request #12294 from seamuslee001/remove_payflow_recur

Remove unused function getRecuringTransactionStatus from PayflowPro c…

Merge pull request #12293 from colemanw/arrayUtil

Improve CRM_Utils_Array::recursiveBuild to work with existing arrays.

Merge pull request #12224 from eileenmcnaughton/templates

Add upgrade function for message templates that does not involve copying the whole template

Confirm & lock in group.get handling of is_active

Remove unused function getRecuringTransactionStatus from PayflowPro class

Improve CRM_Utils_Array::recursiveBuild to work with existing arrays.

dev/core#170 fix regression on soft_credits with unit test

Merge pull request #12288 from eileenmcnaughton/export

Function extraction BAO_Export class

Extract build buildRelatedContactArray function

Further minor function extraction on Export class

Merge pull request #12289 from eileenmcnaughton/frank

Partial code cleanup & testability improvement towards CRM-21177

Cleaned up code style. No expected functional change.

Merge pull request #11726 from scardinius/crm-21808

CRM-21808 Install custom group for Contribution or ContributioRecur s…

Merge pull request #12107 from eileenmcnaughton/save_report

CRM-20430 - Permission 'save Report Criteria'.

Merge pull request #11337 from WeMoveEU/CRM-21460

CRM-21460 Add job execution hooks

CRM-21177: Added unit test to lock in behavior of properly looking up
membership term from the line item.

Merge pull request #11945 from mattwire/devcore47_clone_scheduled_jobs

dev/core#47 Add clone scheduled job functionality

CRM-20430 - Permission 'save Report Criteria'.

If you don't have the permission, the 'save report' actions aren't shown
on the form of the report instance.

CRM-20430 - Check 'save Report Criteria' permission before saving a report instance.

Coding style issues.

Added a pre-commit message about the new permission.

Merge pull request #12067 from michaelmcandrew/CRM-21576-sms-permission-extra

Make Send SMS permission independent of Edit Contact permission

Merge pull request #11956 from MiyaNoctem/CRM-50-sub-tabs-for-contributions

(dev/core/50) Add Separate Sub-tabs for Contributions and Recurring Contributions

Merge pull request #12078 from compucorp/101-conditionals-in-search-form

dev/core/#101: Allow further customization of search form in hooks

@eileenmcnaughton
Add contact greeting to membership online receipt text message template

Add upgrade function for message templates that does not involve copying the whole template

Merge pull request #11840 from mfb/smtp-reconnect

CiviMail: If SMTP connection error is detected, disconnect so we can reconnect and retry

Merge pull request #12287 from eileenmcnaughton/5.3

core/#170 minimal fix for fatal on soft_credit field

Merge pull request #11724 from lemacarl/CRM-21779

CRM-21779 - ang\crmMailing - Fix adding empty mailing to recipients field

dev/core#101 Implement field spec pattern for advanced search form.

By implementing the field spec pattern as detailed in dev/core#115 extension developers have more control over
 what is displayed in this template without needing to resort to
 overriding it

Merge pull request #12284 from jitendrapurohit/mail-1-message

dev/mail#1 - Modify message text for pause mailing

Merge pull request #12271 from eileenmcnaughton/membership

Partial refactor of completeMembershipFromContribution

core/#170 minimal fix for fatal on soft_credit field

Merge pull request #12285 from eileenmcnaughton/master

merge to master

Merge branch '5.3' of https://github.com/civicrm/civicrm-core

Merge pull request #12283 from eileenmcnaughton/cust_test

Add unit test for CRM_Core_BAO_CustomField::getCustomFieldID

Merge pull request #12282 from eileenmcnaughton/5.3

core/#170 minimal fix for fatal on soft_credit field

dev/mail#1 - Modify message text for pause mailing

Merge pull request #12279 from colemanw/dev/core#165

dev/core#165 - Fix case activity breadcrumb

Merge pull request #12091 from eileenmcnaughton/paypal_exp

dev/financial#14 Fix enotices & inability to cancel recurring on paypal express

Merge pull request #12181 from JMAConsulting/dev_core_134

dev/core#134 Search Builder broken filter for Source Contact ID

Merge pull request #12280 from eileenmcnaughton/line_endings

[NFC] Fix line endings (Jenkins where were you)

Merge pull request #11803 from jitendrapurohit/mail-1

Mail#1 - Add pause/resume functionality to civicrm bulk mailing

Merge pull request #12195 from michaelmcandrew/dev/core#140

dev/core#140 add missing pseudoconstant for option_group_id in CustomField.xml

Merge pull request #12209 from JMAConsulting/access-3-event

dev/accessiblity#3 Remove orphan label tag from Event pages

Add unit test for CRM_Core_BAO_CustomField::getCustomFieldID

core/#170 minimal fix for fatal on soft_credit field

[NFC] Fix line endings (Jenkins where were you)

dev/core#165 - Fix case activity breadcrumb

When opening a case activity (without popups) this will correctly set the breadcrumb.

Merge pull request #12278 from eileenmcnaughton/5.3

Fix incorrect operator on previous Export fix

Fix incorrect operator on previous Export fix

Merge pull request #12272 from eileenmcnaughton/export_house

Towards fixing household merge export, extract function, add test, fix prev

Set version to 5.4.alpha1

Set version to 5.3.beta1

CRM-19798 fix memory leak on EntityTag.get

This is an alternate methodology using __destruct on the DAO object.

Note that I have found some places bypass this - ie.

```
   $rule = new CRM_ACL_BAO_ACL();
   $rule->query($query);
```

But I think that is a pretty clumsy construct & we should swap to
CRM_Core_DAO::executeQuery();

Merge pull request #12275 from JMAConsulting/dev_mail_11-1

dev/mail#11 add CRM_Mailing_BAO_MailingJob::del()

Merge pull request #12193 from eileenmcnaughton/dedupe

Fix non-display of conflicts after batch dedupe from dedupe screen

dev/mail#11 add CRM_Mailing_BAO_MailingJob::del()

Partial refactor of completeMembershipFromContribution

Merge pull request #12274 from civicrm/5.2

5.2 to master

Merge pull request #12273 from eileenmcnaughton/notices

dev/core#158 group contacts list and export limits results to 500 records

Fix e-notices on export coming from https://github.com/civicrm/civicrm-core/pull/12110/files

Towards fixing household merge export, extract function, add test, fix prev