Merge pull request #14986 from eileenmcnaughton/luciano

[REF][Import] add formatted parameter to formatInput

[REF][Import] add formatted parameter to formatInput

5.16.0 release notes - late changes (#14973)

* Add release-notes/5.15.1.md

* Add release-notes/5.15.2.md

* Update 5.15.2.md

* 5.15.2 release notes: added missing bug fix

* 5.16.0 release notes: late changes

* 5.16.0 release notes: contributors for late changes

Merge pull request #14944 from colemanw/color

Better support for hookable menubar colors

Improve color utils

Better support for hookable menubar colors

Merge pull request #14931 from eileenmcnaughton/mysql516

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

Merge pull request #14924 from colemanw/menubar5.16

Backport menubar fixes to 5.16

Merge pull request #14926 from eileenmcnaughton/conf

dev/core#1084 Contribution confirmation page: Translation won't work

Update Confirm.tpl

Merge pull request #14920 from eileenmcnaughton/url_search

Fix accidental move of sort_name field on advanced search

Merge pull request #14904 from eileenmcnaughton/activ_516

dev/core#1042 fix  duplicate activities on custom data block

Menubar - get rid of hardcoded pixel values in crm.menubar.js

Menubar - Use class instead of id for flexibility

Fix accidental move of sort_name field on advanced search

The code that made this field url-enterable ALSO re-ordered them by accident - this fixes

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

dev/core#1042 fix  duplicate activities on custom data block

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