[REF] Simple function extraction for buildMappingForm

Merge pull request #14693 from eileenmcnaughton/cust_field_sql

[REF] remove instances of pass-by-reference where no change takes place

Merge pull request #14503 from pradpnayak/ReportFix

$this->_selectedTables is not populated incase of boleen filters

Merge pull request #14702 from demeritcowboy/why-is-php-case-insensitive-for-classnames

dev/joomla#13 - followup - Some class names have wrong upper/lower case spelling

Merge pull request #14689 from eileenmcnaughton/cust_field_create

[REF] extract prepareCreate from CustomField.create

WIP

Merge pull request #14699 from eileenmcnaughton/activity_search

[NFC][test] reformat activity search test

Merge pull request #14700 from eileenmcnaughton/search

[NFC][test] code formatting only

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

[NFC][test] reformat activity search test

Merge pull request #14695 from eileenmcnaughton/revert

Remove failing assertion

Merge pull request #14698 from civicrm/5.15

5.15 to master

Merge pull request #14692 from agh1/pseudoconstant-caps-5.15

dev/joomla#13 lower-case C on CRM_Core_PseudoConstant breaks 5.14 upgrade on Joomla w/ CiviCase

Remove failing assertion

This was a 'never worked' as the original call was doing no check. I changed it to do a check but
got confused & didn't realise thee check was actually failing when I merged. This check was never successful
we should remove it

[REF] remove instances of pass-by-reference where no change takes place

I did a grep on these and the passed params are not altered - down with php4

Use correct caps for CRM_Core_PseudoConstant everywhere

Merge pull request #14688 from eileenmcnaughton/revert

 Fix erroneous variable passed to callAPISuccessGetValue

dev/joomla#13 CiviCase: use correct caps for CRM_Core_PseudoConstant

[REF] extract prepareCreate from CustomField.create

Straight forward extraction - cleaning up code with a view to offering a more efficient wrapping function

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.