Merge pull request #16117 from seamuslee001/dev_core_1424

dev/core#1424 Support MS Excel for Diacritic Characters when exporting a report to CSV

Merge pull request #16154 from colemanw/getFields

Allow values to be passed to api4 GetFields

Merge pull request #16161 from civicrm/5.21

5.21

Merge pull request #16160 from seamuslee001/5.21

5.20.3 Release Notes

Update 5.20.3.md

Add release-notes/5.20.3.md

Merge pull request #16157 from seamuslee001/dev_drupal_99

dev/drupal#99 Fix E-notice on undefined property id of non object

Merge pull request #16156 from civicrm/5.21

5.21

Allow values to be passed to api4 GetFields

So far this only filters custom fields by contact type,
but values is now being passed to allow more filtering.

dev/drupal#99 Fix E-notice on undefined property id of non object

Merge pull request #16155 from seamuslee001/5.21

dev/core#183 Ensure that having a rule where each field is the same w…

dev/core#183 Ensure that having a rule where each field is the same weight with the threshold being the total sum of the weights does not cause a fatal error when searching for dupes

Merge pull request #16152 from colemanw/multi

Api4 - Fix auto serialize array input for CheckBox/MultiSelect fields

Merge pull request #15135 from mattwire/case_links_refactor_report

 Add case activity action links to activity report

Merge pull request #16079 from eileenmcnaughton/email

Remove unnecessary, and possibly incorrect query from email update

Merge pull request #16135 from eileenmcnaughton/pay_total

[REF] simplify recordAdjustedAmt function

Api4 - Fix auto serialize array input for CheckBox/MultiSelect fields

Merge pull request #16151 from eileenmcnaughton/seamus

[REF] extract member form batch update & add test

Merge pull request #16131 from eileenmcnaughton/acl

[Ref] improve readability of acl code

[REF] extract member form batch update & add test

Merge pull request #16112 from seamuslee001/remove_zend_validator

Remove unneeded zendframwork/zend-validator and zendframwork/zend-stdlib packages as not required by …

Merge pull request #16140 from eileenmcnaughton/part_name

dev/event#25 Fix regression on billing name overwrite

Merge pull request #16136 from eileenmcnaughton/member_test

[REF] remove CRM_Core_Error check

Merge pull request #16145 from eileenmcnaughton/index_api

Add api to check for missing indices

Merge pull request #16144 from eileenmcnaughton/acl_index

dev/core#1486 Add index to civicrm_acl_cache.modified_date

Merge pull request #16143 from eileenmcnaughton/deadlock_err2

Fix misleading error message on deadlock

Add api to check for missing indices

Add index to civicrm_acl.modified_date

Fix misleading error message on deadlock

We were seeing error messages like
API_Exception : financial_type_id is not valid : Donation
when a deadlock was hit. The error made not sense as 'Donation' DID exist and did not help us
to identify the issue as a deadlock. It's still helpful with a deadlock to know if
a real constraint violation is hit as per https://lab.civicrm.org/dev/core/issues/1481 sometimes
a disturbing amount of queries have been rolled back.

Let's fix to handle pseudoconstants here & give a useful message.
The previously converted params did not get to this point but we can re-do that work in this low
volume function rather than do a lot of re-thinking

Merge pull request #16142 from eileenmcnaughton/deadlock_err

[NFC] Declare exceptions, use single quotes where possible, use strict comparison

Merge pull request #16141 from eileenmcnaughton/fatal

dev/core#560 Remove instances of fatal errors

[NFC] Declare exceptions, use single quotes where possible, use strict comparison

General coding standards. If we know one side is a string we can switch comparison to strict

Merge pull request #16132 from eileenmcnaughton/member

[REF] step towards wiping out  $ids array in membership bao functions

Remove instances of fatal errors

Merge pull request #16134 from eileenmcnaughton/member_legacy

Remove legacy_redirect_path handling

Merge pull request #16122 from demeritcowboy/always-true

[NFC] If statement is always true

Merge pull request #16138 from eileenmcnaughton/static

Switch static var to using Civi::statics

Merge pull request #16137 from eileenmcnaughton/less_dep

[REF] Minor cleanup - declare exceptions, use single quotes, strict comparison

dev/event#25 Fix regression on billing name overwrite

https://lab.civicrm.org/dev/event/issues/25 was caused by https://github.com/civicrm/civicrm-core/commit/b1b7f4e35bbe566d9efccac1055d311245420974
where it started overwriting the name in that function.

In digging in the code I found that it seems that as long as we call this function before the processor all is
good so we should call right before the processor rather than broader contamination.

Also note ideally the settings bag would be used.

Switch static var to using Civi::statics

This static variable is actually problematic. Moving it to statics will allow us to stop reseting it
in the test suite & to hone in on where it is actually broken rather than the test suite
is not flushing it

[REF] Minor cleanup - declare exceptions, use single quotes, strict comparison

[REF] remove CRM_Core_Error check

I'm convinced there is no reason why find should return a core_error & hence if it did we wouldn't want
to keep calm & carry on - smells a lot like copy & paste. Also makes code hard to understand

[REF] step towards wiping out  array in membership bao functions

Merge pull request #16133 from eileenmcnaughton/member_test

[NFC] Test clean up.

[REF] simplify recordAdjustedAmt function

Makes IF statement more readable & casts to float to explicity handle it

[NFC] Test clean up.

Good test - nightmare to read. This helps.

Remove legacy_redirect_path handling

This has been hanging around for a long time & no-one has logged a JIRA (or pointed out that JIRA is now
defunct) so the initial analysis of it being unreachable makes sense

Merge pull request #16130 from colemanw/ext

Fix output display in api4 explorer

[Ref] improve readability of acl code

Variables are used in this code for table names, making the queries hard to read.

Since there is quite a bit I've only tackled 2 patterns
1) using  where an alias within mysql would do
2) using  where an alias withing mysql would do

Ideally we want to switch to using CRM_Core_DAO::executeQuery & ditch the variables forr
table names altogether but since there is a lot I've stuck to one pattern for this change

Fix output display in api4 explorer

Merge pull request #16125 from eileenmcnaughton/cont_sub

dev/core#1482 Remove lines of code that appear to relate to legacy logic

Merge pull request #16124 from eileenmcnaughton/acl_deadlock

Remove function marked as deprecated & unused by core nearly a year ago

Merge pull request #16127 from eileenmcnaughton/mem_test

Unit test for https://github.com/civicrm/civicrm-core/pull/16120

Unit test for https://github.com/civicrm/civicrm-core/pull/16120

Merge pull request #16126 from civicrm/5.21

5.21 to master

Merge pull request #16120 from seamuslee001/dev_core_1485

dev/core#1485 Fix field name for join_date to be membership_join_date…

dev/core#1482 Remove lines of code that appear to relate to legacy logic

As discussed in https://lab.civicrm.org/dev/core/issues/1482 the subscription history table appears to still be used for
recording group history but this business of creating a row with no attached group appears to be from a logic flow that
is long-superceded by change log & db logging

Merge pull request #16123 from civicrm/5.21

5.21 to master

Remove function marked as deprecated & unused by core nearly a year ago

dev/core#1485 Fix field name for join_date to be membership_join_date in line with the DAO schema

Also include membership_join_date into the mapping as per Jamie's comments

Merge pull request #16121 from mlutfy/core1490-rc

dev/core#1490 Fix syntax error in Membership Receipt

Merge pull request #16119 from eileenmcnaughton/acl_deadlock

[Ref] Remove php4 support from BAO_Acl class

if statement is always true

dev/core#1490 Fix syntax error in Membership Receipt

Remove php4 support from BAO_Acl class

Removes instances of php support involving & in the function signature. In each instance the function is only ever called once
(I think all are internal too) & the calling function doesn't do any weird stuff with references that might make it needed.
Also remove one instance of  unnecessarily passed by ref

Support MS Excel for Diacritic Characters when exporting a report to CSV

Merge pull request #15968 from eileenmcnaughton/bom

dev/core#1424 Support MS Excel for diacritic characters when exporting to csv

Also remove zendframework/zend-stdlib

Merge pull request #15972 from aydun/report#24

report#24 - Case Detail report - fix 'Active role?' when no relationships

Merge pull request #16101 from jitendrapurohit/dev-1474

dev/core#1474 - (On Hold) text missing from email column when custom …

Merge pull request #16115 from eileenmcnaughton/deprecate

Remove vestige of apiv2

Merge pull request #16114 from eileenmcnaughton/pear

[NFC] cleanup to declared exceptions, quotes, strict mode.

dev/core#1474 - (On Hold) text missing from email column when custom search profile is used in Advanced Search

Add test

Remove vestige of apiv2

[NFC] cleanup to declared exceptions, quotes, strict mode.

Merge pull request #16111 from eileenmcnaughton/pf

dev/core#1472 fix price field money clean

Remove unneded zendframwor/zend-validator package as not required by any other package

Merge pull request #16109 from seamuslee001/utils_grant_bin_fatal_exception

[REF] Switch CRM_Utils_Error::fatal calls for CRM_Core_Exception in G…

Merge pull request #16110 from seamuslee001/api4_country_state_entities

Add in Country and StateProvince APIv4 Entities

Add in Country and StateProvince APIv4 Entities

[REF] Switch CRM_Utils_Error::fatal calls for CRM_Core_Exception in Grant. utils, bin/ and Civi/Angular/ folders

Merge pull request #16108 from civicrm/5.21

5.21

Merge pull request #16107 from demeritcowboy/freezy-5.21

dev/core#1477 Fix fatal error when editing reserved option group settings

fix fatal error when editing reserved option group settings

fix price field money clean

Merge pull request #16103 from civicrm/5.21

5.21 to master

Merge pull request #16052 from eileenmcnaughton/desc

Fix unreleased regression where processors using getPaymentDescription get a fatal

Merge pull request #16096 from eileenmcnaughton/pf

E notice fix on Price set form

Merge pull request #16078 from colemanw/snippets

Allow Angular modules to provide dynamic HTML snippets

hook_angularModules - Accept 'partialsCallback' for dynamic partials

This is similar to the antecedent commit which used a field 'snippets' that
contained a list of HTML files and their content. However, this
uses a callback function instead of providing the full HTML content.

This makes it safer to work with the Angular metadata without needing to
load the content of all partials into memory.

E notice fix on Price set form

Merge pull request #16088 from eileenmcnaughton/paypal_clean

Paypal class cleanup

Merge pull request #16094 from pradpnayak/ActivityTypeName-Case

dev/core/issues/1470, show labels instead name for activity type

dev/core/issues/1470, show labels instead name for activity type

Merge pull request #16093 from civicrm/5.21

5.21

Merge pull request #16092 from seamuslee001/5.21

Fix Release note header

Fix Release note header

Merge pull request #16091 from civicrm/5.21

5.21

Merge pull request #16057 from eileenmcnaughton/act_perm521

dev/core#1444 Permit activities with campaigns on contact tab for con…

Merge pull request #16090 from seamuslee001/5.21

Add release-notes/5.20.2