Merge pull request #17991 from eileenmcnaughton/merge_single

[REF] Extract function to get locations to merge, rename 'operation' …

Merge pull request #18275 from eileenmcnaughton/linepre

dev/core#1980 Move (now tested) delete acl check from v3 api to pre delete hook

Move (now tested) delete acl check from v3 api to pre delete hook

Merge pull request #18280 from yashodha/drop_tables

remove dropped tables for activity target and assignment

remove dropped tables for activity target and assignment

Merge pull request #18274 from eileenmcnaughton/linetests

Add line item acl tests

Merge pull request #18276 from civicrm/5.29

5.29

Merge pull request #18268 from sunilpawar/report_47

report#47 Report Bookkeeping add time field for date filter.

Add line item acl tests

Merge pull request #18273 from seamuslee001/master

Fix regen and update civicrm_generated

Merge pull request #18271 from eileenmcnaughton/tax529

dev/core#1972 Fix tax_amount calclation on renewal form

Merge pull request #18270 from seamuslee001/auto_dsn

[REF] Move Auto DSN Switching into a core function

added custom clause under whereClause and removed where function from bookkeeping, re-added receipt_date

Fix regen and update civicrm_generated

Merge pull request #18269 from eileenmcnaughton/finacl

Shell Financial ACLs extension

dev/core#1972 Fix tax_amount calclation on renewal form

[REF] Move Auto DSN Switching into a core function

Shell Financial ACLs extension

This is intended as a hidden extension while we work through the process of migrating core functionality into it.

I'm expected there to be a bit of work to unravel this into hook interactions - but the core extension mechanism gives us that space

report#47 Report Bookkeeping add time field for date filter.

Merge pull request #18266 from sunilpawar/dev_912

dev/core#912 update payment instrument of main contribution record

Merge pull request #18265 from eileenmcnaughton/objects

Use already determined value for contributionRecurID

Merge pull request #18107 from demeritcowboy/mysql-ssl-install

dev/core#1926 - Allow SSL mysql connections for civicrm-setup

fix test case warning

Remove additional line

dev/core#912 update payment instrument of main contribution record

Merge pull request #18244 from colemanw/drop

Remove old dropped items from the schema xml

Use already detemined value for contributionRecurid

Merge pull request #18257 from eileenmcnaughton/relob

Remove 1 of 2 remaining places where relatedObjects is called in completeOrder

Merge pull request #18252 from JKingsnorth/core-1971-fix

dev/core#1971 Caching of domain-specific option values

Remove 1 or 2 remaining places where relatedObjects is called in completeOrder

If we remove this & the other place we no longer need to call loadRelatedObjects before this
function and only message related places call it

In this case the lines can go as

1) trxn_id should always be used if provided. Casting to isoDate has not been required for a few
years as the DAO now handles

Merge pull request #18258 from eileenmcnaughton/refer

Remove last pass-by-reference in completeOrder signature

Merge pull request #18249 from mattwire/utf8mb4moretables

Support more table names for utf8mb4 conversions and database name

dev/core#1971 Fix test breaking subsequent tests when it fails

Merge pull request #18250 from seamuslee001/ref_entity_id_group

[REF] Ensure that the form param _id is set when adding a new group w…

Merge pull request #18248 from JMAConsulting/optimise_proximity_search

Optimise proximity custom search, by reducing addGeocodingData fn call

Remove last pass-by-reference in completeOrder signature

ids is now only referred to once in the function & never altered. I have assigned that value to
a param & unset ids after that to make that clear

Merge pull request #18255 from colemanw/fieldLabel

Expose field label to APIv4 and Search creaor

Merge pull request #18256 from colemanw/fieldTitles

Separate titles from labels in exportable fields

Separate titles from labels in exportable fields

Follow-up to #18255 and #18114 - this separates field titles (used in export) from labels (used in forms and other displays).
The previous approach was to try to use the title for both, and if that didn't work, override it somewhere.

Merge pull request #18254 from pradpnayak/casetypeselection

Fix js for case type status change on load

Expose field label to APIv4 and Search creaor

Fix js for case type status change on load

dev/core#1971 Domain-specific option value caching

Support more table names for utf8mb4 conversions and database name

dev/core#1971 Test for domain-specific option values

[REF] Ensure that the form param _id is set when adding a new group with the newly created group id so that consumers of the hook_civicrm_postProcess can access the id

Merge pull request #18235 from pradpnayak/dev-1946

Batch Update via Profile does not supply data for editing when custom fields created after participant added to event

Merge pull request #18236 from mattwire/crmregion_notyou

Wrap "not you" message in crmRegion

Optimise proximity custom search, by reducing addGeocodingData fn call

Merge pull request #18243 from eileenmcnaughton/static

Do not pass by reference for createRelatedMemberships

Merge pull request #18242 from eileenmcnaughton/dep

Remove 2019 deprecated function

Merge pull request #18118 from artfulrobot/artfulrobot-msgtpl-disable-smarty

Add disable_smarty option to MessageTemplate.send API

Add disable_smarty option to MessageTemplate.send API

Remove old dropped items from the schema xml

Removes unused fields and indexes which were dropped prior to 4.4.7
which is currently the min upgradable version.

Do not pass by reference for createRelatedMemberships

There are 2 parameters passed by reference to this function
1) dao - this is not altered
2) params - this IS altered but the calling function (create) does not use it again. While it's
possible a function that calls create uses it again the idea that they would want the params
to have been altered to refer to a different related relationship does not seem probable.

Tests on this are in JobTest & apiv3_MembershipTest

Merge pull request #18229 from totten/master-patchset

distmaker - Autogenerate civicrm-*-patchset.tar.gz

Remove 2019 deprecated function

We stopped using this in core & deprecated it in 5.16

Merge pull request #18153 from larssg-wildsight/patch-1

dev/core#1950 Update help text and description for the profile add to groups setting

Merge pull request #18221 from civicrm/eileenmcnaughton-patch-1

Remove ref to logging time

Merge pull request #18240 from demeritcowboy/deprecated-RID

Replace deprecated Drupal 8 constants

Merge pull request #18234 from pradpnayak/dedupeTrim

trim white space when checking dedupe

Merge pull request #18238 from agileware/separator-not-seperator

Spelling mistake "separator" not "seperator"

Merge pull request #18239 from demeritcowboy/gencode-checksum

[NFC] Some DAO checksums not updated

deprecated constants

checksums not updated

Global spelling mistake "separator" not "seperator"

(NFC) distmaker - More comments

Merge pull request #18231 from civicrm/5.29

5.29

Merge pull request #18230 from seamuslee001/5.29

Update contributor key for Andrew

Wrap not you message in crmRegion

Batch Update via Profile does not supply data for editing when custom fields created after participant added to event

trim white space when checking dedupe

Update contributor key for Andrew

Merge pull request #18228 from seamuslee001/5.29

5.28.2 & 5.29.3 Release Notes

distmaker - Autogenerate civicrm-*-patchset.tar.gz

Add release-notes/5.28.2.md

Add release-notes/5.28.3.md

Merge pull request #18226 from civicrm/5.29

5.29

Merge pull request #18223 from eileenmcnaughton/529deupe

dev/core#1964 Fix regression bug on deduping contacts with dedupe_exception matches

[REF] Extract function to get locations to merge, rename 'operation' to is_replace'

This is a preliminary refactor. I have a bug to fix in this code but cannot yet make sense of it. The mergeHandler class
is simply an object to refactor the functions onto. Much of this code is hard to work with as the use of static functions
necessitates a lot of compilation and compiling so the process of cleaning it up involves extracting functionns to this
new class (which may one day replace the Merger class). As it is a refactoring process the functions reflect the old code
more than the ideal code. However, it provides the change to give some documentation regarding the locBlock array
and also to rename the confusing field 'operation' to 'is_replace'

Merge pull request #18222 from civicrm/5.29

5.29 to master

dev/core#1964 Fix regression bug on deduping contacts with dedupe_exception matches

I want to do another pass on this & do more SELECT queries & less UPDATE queries so we are doing
less locking - but for now this should get us past the logged regression

Merge pull request #18214 from eileenmcnaughton/dedupe

Fix dedupe regression whereby deleted contacts are found

Merge pull request #18136 from demeritcowboy/deprecated-hook-invoke

[Test framework] - Remove test for deprecated way of calling hook invoke

Merge pull request #18208 from mattwire/contribregister_clean

Cleanup contactID variables in event/contribution register buildQuickForm

Merge pull request #18209 from mattwire/repeattransaction_cleanup

Remove unused variables from repeatTransaction

Merge pull request #18216 from eileenmcnaughton/meta

Metadata fix in activity search

Remove ref to logging time

The page is not anon accessible & I think we don't do this now - @joshgowans ?

Merge pull request #18217 from mattwire/orderapicleanup

Minor code cleanup to Order API

Minor code cleanup to Order API

Fix dedupe regression whereby deleted contacts are found

This affects api calls where check_permissions = TRUE and getduplicates is called. This can be
done via the api (per this test) or ann easy UI way is with the deduper extension but it should
also affect the 'normal' dedupe screen.

Note that there can be cases where the dedupe results are cached into prevnext cache to hide this

Merge pull request #18215 from civicrm/5.29

5.29

Metadata fix in activity search

This fixes it such that even if the pseudofield (activity_type) has a 'type' the 'type'
from the 'real field' is used in the WHERE query. This mostly prevents a test fail per

https://github.com/civicrm/civicrm-core/pull/18147

if we improve the metadata. It's a bit of a non-fail fail - ie we are checking the where
clause has activity_type_id = 3 vs '3' and my testing has not shown a performance difference
but we don't want the test to be changed to the less good version

Merge pull request #18206 from chamilwijesooriya/issue-1959

dev/core#1959 Brick\Math\Exception\RoundingNecessaryException

Merge pull request #18213 from eileenmcnaughton/is_deleted

Fix regression whereby deleted contacts are in quicksearch results

Fix regression whereby deleted contacts are in quicksearch results

dev/core#1959 Brick\Math\Exception\RoundingNecessaryException

Merge pull request #18211 from totten/master-resg

(NFC) Add some tests to the `resources` group

Merge pull request #18210 from civicrm/5.29

5.29

(NFC) Add some tests to the `resources` group