Menubar - Use class instead of id for flexibility

Merge pull request #14918 from totten/5.16-prevnext

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Overview
-------------------

This reverts commit a5ca2ad8e16b145edf3a5604fab923d1e2386b37
and fixes a bug in management of the prev/next cache. The
bug is visible when following this procedure:

1. In browser, use "Quick search" to search for text "am"
2. In browser, select checkboxes next to two items
3. In MySQL CLI, run `select * from civicrm_prevnext_cache where is_selected =1;`
4. In browser, continue to next page. Observe that there are no more selection.
5. In MySQL CLI, re-run SQL query. Observe that the selections have disappeared.

Before
-------------------

Proceeding to another page of search results causes loss of selections

After
-------------------

Selections are preserved

Merge pull request #14900 from eileenmcnaughton/regress

Fix unreleased regression - fatal on financial account screen

Fix unreleased regression - fatal on financial account screen

Merge pull request #14876 from eileenmcnaughton/is_front

dev/core#378 Determine front end pages in drupal

Determine front end pages in drupal

Alternative to https://github.com/civicrm/civicrm-drupal/pull/546/files that I think narrows the scope
since it means we are not changing an existing property

Merge pull request #14866 from alifrumin/5.16.0

Annotating and Sorting 5.16.0 release notes

Annotating and Sorting 5.16.0 release notes

Merge pull request #14722 from JKingsnorth/fix-public-url-link-in-browse

Fix public URL link with hashed URLs in mailing browse screen

Merge pull request #14764 from elisseck/dev/report/15

dev/report/15 Add fix and tests for contact subtype report filter

add tests for contact subtype null and not null report filters

apply reporting#15 fix against master

Merge pull request #14789 from seamuslee001/fast_array_caches

Ensure recently converted groups cache matches previous behabiour my …

Ensure recently converted groups cache matches previous behabiour my setting withArray as fast for it

Add in code comment as per Eileen

Merge pull request #14783 from agh1/5.16.0-releasenotes

5.16.0 release notes first run

5.16.0 release notes: include César Ramos
per
https://github.com/civicrm/civicrm-core/pull/14518#issuecomment-501881835

5.16.0 release notes: add boilerplate

5.16.0 release notes: raw from script

Contributor key: fix invalid YAML

Merge pull request #14775 from eileenmcnaughton/bgm

dev/drupal#75 Drupal8: fix call to languageNegotiationURL() when called from cv

dev/drupal#75 Drupal8: fix call to languageNegotiationURL() when called from cv

Merge pull request #14746 from eileenmcnaughton/order_report

dev/core#1081 fix for error on contribution detail when using custom data order by without select

dev/core#1081 fix for error on contribution detail when using custom data order by without select

Merge pull request #14732 from eileenmcnaughton/backoffice

dev/report#16 Unreleased regression - fee levels incorrectly show sol…

Merge pull request #14733 from eileenmcnaughton/payment_reg

Fix refund payment not recording from additional payment form

Fix refund payment not recording from additional payment form

Revert participant portion of https://github.com/civicrm/civicrm-core/pull/14244

Fix public URL link with hashed URLs in mailing browse screen

Set version to 5.16.beta1

Merge remote-tracking branch 'origin/5.15'

dev/report#16 Unreleased regression - fee levels incorrectly show sold out

Merge pull request #14730 from alifrumin/5.15.0-releasenotes

5.15.0 releasenotes

Set version to 5.15.0

Update 5.15.0.md

cleanup

finished annotating

annotating

finishing changes with git lab issues

linking and annotating

annotating

annotating

contributors

organizing by git lab issue and condensing as well as pulling out obvious misc

5.15.0 release notes: added boilerplate

5.15.0 release notes: raw from script

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