[REF] simple extraction of prepareCreateParams

Merge pull request #14723 from seamuslee001/ref_date_to_local_extract

Extract code converting a date object to local timezone object to own…

Extract code converting a date object to local timezone object to own function

Merge pull request #14717 from totten/master-rm-ser

(dev/cloud-native#3) SerializeCache - Remove unused, incomplete cache-driver

(dev/cloud-native#3) SerializeCache - Remove unused, incomplete cache-driver

The class `CRM_Utils_Cache_SerializeCache` provides a cache-driver for
storing cache records in the filesystem (under `CIVICRM_TEMPLATE_COMPILEDIR`,
using PHP `serialize()` format). Why remove it?

1.  As we work through cleanup in file management (e.g.  #12843,
   dev/cloud-native#3), having an unnecessary reference to
   CIVICRM_TEMPLATE_COMPILEDIR makes it harder to reason about the system.

2. The class is not used. I'm pretty sure it was added speculatively
   (i.e.  with an aim to try using it some day), but that never came to
   pass.  Grepping `universe`, I cannot find any usages or references to
   `SerializeCache`.

3. The implementation is incomplete -- parameters like `get(..., $default)`
   and `set(..., $ttl)` will generate exceptions if used.

Merge pull request #14714 from civicrm/5.15

5.15 to master

Merge branch 'master' into 5.15

Merge pull request #14690 from eileenmcnaughton/profile_copy

dev/core#1088 Fix fatal error when copying profile with civirules enabled

Merge pull request #14713 from pradpnayak/REF-1

Grab contribution status id from database

Merge pull request #14608 from eileenmcnaughton/csv_test

 [test] convert export test to handle exception rather than early return

Grab contribution status id from database

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

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

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

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

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

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

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

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

Merge pull request #14704 from pradpnayak/REF-1

Grab contribution status id from database

Merge pull request #14685 from eileenmcnaughton/mark_conflict

Consolidate handling of conflicts between the batch job and get_conflicts api

fix wrong spelling of classname

fix wrong spelling of classname

fix wrong spelling of classname

fix wrong spelling of classname

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

Grab contribution status id from database

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

Consolidate handling of conflicts between the batch job and get_conflicts api

This ensures that conflicts are stored during batch_merge to the prev_next cache with the same format as when the api
calls get_conflicts. The code doing this wrangling is moved from the api to the BAO layer.

We add a test to ensure the output is the same & use the previously added test to check the string is the same too.

Test cover here is pretty good

[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results

[REF] move formatting of conflict into markConflict function

The goal here is to make this information more usable to an api caller. Currently the 'conflict' key holds text formatted
for a specific form. I was going to change this to hold an array of data that an api could use. However, the
Contact.get_merge_conflicts already has a specific api-friendly format that is keyed by 'mode'
at the top level so my revised plan is to return api-friendly data as well (without messign with the
existing form's data).

This change simply moves the decisions about what data to store & how down to the function whose responsibility
it is (markConflict) and adds a test to ensure that no change results

Add test ensuring conflicts key existing behaviour is retained

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

dev/core#1088 Fix fatal error when copying profile with civirules enabled

Fixes a bug where the DAO name was not passed to the hook when copying a profile.

[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

[test] convert export test to handle exception rather than early return

This is part of switching to testing the eventual output rather than the temp table created. In most cases the temp
table is not truly needed so testing the real output makes more sense

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