Merge pull request #16008 from civicrm/5.20

5.20

Merge pull request #16007 from seamuslee001/5.20

Add release-notes/5.19.3.md

Add release-notes/5.19.3.md

return manager info during api call

Merge pull request #15970 from eileenmcnaughton/ex_easy

[REF] clean up merge array.

Merge pull request #16001 from agileware/CIVICRM-1383

Code style, remove typo, double ;; from code where it makes sense to do so

Merge pull request #15937 from seamuslee001/dev_core_1405

dev/core#1405 Strip any spaces from Option Group name parameter and r…

CIVICRM-1383 Remove typo, double ;; from CiviCRM code

Merge pull request #16004 from civicrm/5.20

5.20

Merge pull request #16000 from seamuslee001/dev_core_1434

dev/core#1434 Fix table alias used in select when ouputting as a chart

Merge pull request #16002 from seamuslee001/handle_custom_field_deleted

Fix regression from fixing relative key handling where custom field no longer exists in the database

Catch CiviCRM APIv3 Error if custom field no longer exists in the database

Add in unit test demonstrating the regression where smart group fails to load if a custom field has been deleted which was stored in the form values array

dev/core#1434 Fix table alias used in select when ouputting as a chart

Merge pull request #15996 from eileenmcnaughton/act_test_priority

Add testing for various params that pass through Contact_BAO_Query::optionValueQuery

[REF] minor code simplification. Test where handling for gender_id etc

Add test & reduce special handling for gender_id, suffix_id, prefix_id, communication_style_id

Merge pull request #15995 from eileenmcnaughton/act_test_qill

Remove calls to deprecated pseudoconstant activityType, test

Merge pull request #15994 from eileenmcnaughton/act_test

[NFC] cleanup on QueryTest

Merge pull request #15945 from seamuslee001/dev_core_874

dev/core#874 [UI] Fix styling of the menubar colour picker in joomla

Update the name of Macedonia, Republic of to North Macedonia following ISO 3166-1 update

[NFC] cleanup on QueryTest

Merge pull request #15990 from eileenmcnaughton/act_search

Fix activity search to work by url params

dev/core#1405 Strip any spaces from Option Group name parameter and remove the name field from front end form as will be auto generated

Ensure that option group names are lower csae

Ensure that option group names are created as lower case and fix as per Justin's review

Merge pull request #15981 from eileenmcnaughton/setting_fix2

dev/core#1425 Replace deprecated setting function

Merge pull request #15993 from eileenmcnaughton/acti_priority

Add import & export metadata to activity.priority_id

[Test] fix test setup on ANet tests

Per https://github.com/civicrm/civicrm-core/pull/15706 the setup for this test
is creating invalid transactions. This fixes - I had to do an order fix in the process

Remove calls to deprecated pseudoconstant activityType, test

Add import & export metadata to activity.priority_id

There is a lot of special handling for this field we can eliminate. Given no other tables have
priority_id I'm not thinking to add the uniqueName & just add import & export to add it
to metadata. I'll follow up with removals / fixes to support silliness

Fix activity search to work by url params

It seems to be missing the call to the parent fn

Note I feel like the parent should also handle some of the rows below in future

dev/core#523 get the Lybunt charts working again

Also ensure that the lifetime total and last year total field boxes look correct in the sample report

dev/core#1425 More aggressively deprecate old setItem & setOptionValue functions on BAO_Setting

There is still one core place that calls setItem - but it's BAO_Navigation::resetNavigation so it's called very infrequently & I think it's
OK to deprecate first - although I suspect tests will fail & then....

Merge pull request #15988 from jensschuppe/dev_core_1425

dev/core#1425 Remove misleading type hint for Setting value parameter

dev/core#1425 Remove misleading type hint for setting value parameter in CRM_Core_BAO_Setting::setItem().

Merge pull request #15983 from eileenmcnaughton/search_nfc

[NFC] Cleanup comments, string comparison in grant search

Merge pull request #15987 from seamuslee001/master_metadata

Only add in the metadata for advanced search if use has access to the…

Only add in the metadata for advanced search if use has access to the relevant component

CiviPledge is separate to CiviContribute

Merge pull request #15986 from civicrm/5.20

5.20

Merge pull request #15845 from jitendrapurohit/dev-1390

dev/core#1390 - Search builder error on Membership source field

Merge pull request #15923 from eileenmcnaughton/act_order

Fix fatal error when sorting by status in activity search

Merge pull request #15985 from seamuslee001/master

5.20 port to master

Merge in 5.20

Merge pull request #15962 from seamuslee001/dev_core_1422

dev/core#1422 Ensure that the form values are correctly passed onto t…

Merge pull request #15966 from seamuslee001/permission_access_metadata

Conditionally add metadata for advanced search only if the user has a…

[NFC] Cleanup comments, string comparision in grant search

Merge pull request #15931 from seamuslee001/standardise_grant_search

[REF] Standardise the Grant Search form handling

dev/core#1390 - Search builder error on Membership source field

use handleWhereFromMetadata()

Merge pull request #15980 from eileenmcnaughton/setting_fix

dev/core#1425 Replace deprecated settings fns in test suite

Merge pull request #15979 from eileenmcnaughton/search

Remove CRM_Contact_Form_Search_Custom_Basic from searches added on install.

Merge pull request #15982 from civicrm/5.20

5.20

dev/core#1425 Replace deprecated settings fns in test suite

Merge pull request #15977 from seamuslee001/fix_issue_number_range_smart_group_empty_values

Ensure that Relative key is not added for non Select Date is_search_range custom fields

Replace deprecated setting function

Note we can't use Civi::setting here because only the api handles options & arrays well - I wonder what apiv4 does

Only add in relative key when its a custom date field

Ensure that we strip the _to _from _high _low from the element name first

Regenerate sql

Remove CRM_Contact_Form_Search_Custom_Basic from searches added on install.

From https://github.com/civicrm/civicrm-core/pull/15963 this search appears to do what the main basic search does but
we can see that to make it work with sort etc we have to make changes to the BAO_Query object - doing
that to support a custom search scares me & we have a big picture position that we should move these
from core to an extension. However, an easy first step is not to add on new installs. In the case of
this search it seems like an easy call as it doesn't add anything discernable

Merge pull request #15978 from civicrm/5.20

5.20

Merge pull request #15976 from eileenmcnaughton/5.20

Fix 5.20 regression on retrieving template transaction with no logged  in user

Merge pull request #15901 from eileenmcnaughton/matt

[REF] further removal of unnecessary  params

Fix 5.20 regression on retrieving template transaction with no logged in user

Merge pull request #15973 from kcristiano/5.20-msg-tpl

Link to docs.civicrm.org instead of wiki

Make else statement more legible

This just condenses the statement since  != '' is so broad we know the final
else is only when that is true - ergo it's the same as above

Link to docs.civicrm.org instead of wiki

Merge pull request #15877 from demeritcowboy/casereport-unittest

[TEST] dev/core#1366 Unit test for case audit

Merge pull request #15909 from eileenmcnaughton/payment

Fix api Payment.create to support overpayments

Merge pull request #15951 from eileenmcnaughton/export_bom

CSV Export: Add deprecation warning

Fix dev/report#23
When filtering by contact sub-type using 'is not one of' do not exclude where sub-type is null.

Add unit tests for contact sub-type 'is one of' and 'is not one of'
Tighten the matching on sub-type to avoid problems where one contact sub-type is a substring of another.

Fix fatal error when sorting by status in activity search

[REF] clean up merge array.

This code has been doing my head in because it is just like 'construct complex array & then another
complex array & iterate them together in nasty ways.

However, after another round of misery I'm pretty sure that this cleanup works. Basically the whole removed
section of code amounts to 'the array built in the first pass takes precedence over the one in the
second pass'. By using a property & only setting the array values when not-yet-set we can
allow the second pass to not clobber the first while populating it rather than as follow up wrangling

Merge pull request #15967 from seamuslee001/dev_core_1329

dev/core1329 Reduce number of deceased contacts in the demo data from…

Make utf8mb4 status check language more for the general consumer

Fix grammar and change title of check to be a bit more human related

dev/core1329 Reduce number of deceased contacts in the demo data from 31 to 12

Limit number of deceased contacts to 4 and add in age checks as per Justin

Merge pull request #15882 from demeritcowboy/audit-timeline

dev/core#1366 - Make case activity audit print report work again

Merge pull request #15965 from colemanw/ufMatchTest

Fix potential test glitch when repeatedly calling createLoggedInUser

Conditionally add metadata for advanced search only if the user has access for searching to that perticular component

Fix potential test glitch when repeatedly calling createLoggedInUser

adding extractCustomPropertiesForDoPayment() method proof of concept

Merge pull request #15960 from eileenmcnaughton/ex_param

[REF] Remove unused parameter

dev/core#1422 Ensure that the form values are correctly passed onto the Badge label task and others

Merge pull request #15958 from civicrm/5.20

5.20

Remove unused parameter

This parameter is always TRUE so it adds no value, removing

Merge pull request #15939 from demeritcowboy/xml-file-status-check-520

dev/core#1046 Status check for external case xml files

Merge pull request #15953 from eileenmcnaughton/export_bom2

Remove unused parameter

Merge pull request #15952 from eileenmcnaughton/ps_ex

Fix deprecation warning on Price Set report

Merge pull request #15947 from eileenmcnaughton/act

Rename activity search field from status_id to activity_status_id

Merge pull request #15954 from eileenmcnaughton/ex_silly

[REF] CSV Export: Remove impossible checks on  var

status check for external case xml files

Merge pull request #15955 from JMAConsulting/core-1420

core#1420 Quicksearch with phone filter doesn't work with non-numeric character

Merge pull request #15956 from colemanw/removeFn

Remove unused functions

Merge pull request #15861 from JMAConsulting/core-1398

core#1398: Option to open navigation item in new window (if present)

Introduce PropertyBag to help standardise payment params

WIP: proof of concept for payment property bag

WIP: keep civilint happy

WIP: keep civilint happy again

Add other getters, setters, tests

Add require() method and test

some wip code

linter fixes

remove :void from function declaration in Civi/Payment/PropertyBag

fix PropertyBagTest.php  tests

fix linter

Re-do payment code lost in rebase

linting fixes

More WIP on payment property bag

one day i'll run civilint before committing

WIP checkin

Correct test which referred to Credit card instead of Credit Card causing a fail with new property bag which is stricter

Payment PropertyBag: allow week as recurFrequencyUnit; accept ZLS for feeAmount for legacy sake

CRM_Core_Payment: remove stuff cut from release

remove test no longer needed

remove un-needed code because feature dropped before it was released

remove comment

Remove/disarm payment instrument setter

restore getPaymentInstrumentID behaviour

remove not required function

set x

Merge pull request #15734 from seamuslee001/remove_activity_option_join_custom_search

Remove join to civicrm_option_value in favour of using getLabel funct…

Remove unused functions

These functions are no longer needed now that KAM is merged into core.

core#1398: Option to open navigation item in new window (if present)

core#1420 Quicksearch with phone filter doesn't work with non-numeric character

Merge pull request #15950 from eileenmcnaughton/export_ref

[REF] Further cleanup on address handling in merge code.

Merge pull request #15928 from seamuslee001/case_advanced_search

Enable Case search metadata on Advanced Search Form

[REF] Remove checks on  var

This variable is hard-coded to a double quote so checks to see if it is a single quote will never be true.

I've checked this pretty thoroughly & I think the removed IFs are dead code

Merge pull request #15941 from seamuslee001/dev_core_560_sms

dev/core#560 Replace instances of CRM_Core_Error::fatal with Exceptio…

Remove unused parameter

There are 3 calls to writeCSVFile - none of them pass in saveFile so it can be removed

Fix deprecation warning on Price Set report

In 5.20 we added a deprecation warning on searches that are borked WRT filling the prev_next
cache & hence doing searches. The price set search falls into this camp & while it has
been broken forever the deprecation notice is new (& the fix is safe) so targettin 5.20