[REF] Remove most interaction with

With this change we are only looking up
1) objects['paymentProcessor'] - we should probably just pass in the id
2) objects['contribution'] - we could make this a param in it's own right & remove objects
3) objects['event'] - just used to get event title - we could do a query off participant

Merge pull request #18303 from eileenmcnaughton/event_batch

[REF] Clean up return variables on updateContributionStatus, updatePendingOnlineContribution

Merge pull request #18279 from eileenmcnaughton/ordertest

Fix test set up to call Order.create to create the correct line items

Merge pull request #18294 from eileenmcnaughton/linecreate

[REF] Separate and move line-item specific portion of checkTaxAmount to LineItem api

Merge pull request #18305 from civicrm/5.29

5.29

Merge pull request #18293 from eileenmcnaughton/fin

Move LineItem acl handling from v3 api to financialacls core extension

Merge pull request #18290 from eileenmcnaughton/tax

dev/core#1983 Fix to tax calculation on multi-line-item

Merge pull request #17810 from seamuslee001/negative_payment_create

[REF] Fix financial item allocation of negative payments against completed payments

[REF] Clean up return variables on updateContributionStatus, updatePendingOnlineContribution

The return variable from updatePendingOnlineContribution is never used - ergo don't return it. This means
the return value from updateContributionStatus is also never used - ergo - you get it. In addition don't call the function
with empty variables - I've moved the check as to whether they are empty into the calling function - although likely
in a second pass the checks will be removed or change again

Merge pull request #18237 from mattwire/mailingsearchcolumns

Remove 'hack' that overwrites result of searchColumns hook in mailings list

Merge pull request #18302 from civicrm/5.29

5.29 to master

Merge pull request #18301 from demeritcowboy/dev-core-1982-5.29

dev/core#1982 - 5.29 version of PR 18282

[REF] permit negative payments to be made against completed payments

Add in unit test for adding additional refund payment to contribution already refunded and also adding a negative payment to a contribution that is negative already

faulty check for simplexml node value - see also PR 18282

Merge pull request #18067 from MegaphoneJon/event-34-reroll

event#34: allow negative self-service/transfer time

Merge pull request #18292 from eileenmcnaughton/receipt

Add test to check, remove unnecessary lines

Merge pull request #18300 from MegaphoneJon/clean-non-deductible

Clean money for non-deductible amount

Merge pull request #18291 from eileenmcnaughton/depit

[REF] [test] Slight increase in test cover & make functions more re-usable

Merge pull request #18296 from eileenmcnaughton/curr_member

[REF] simplify retrieval of existing membership on membership renewal form

Clean money for non-deductible amount

tests

Add test to check, remove unnecessary lines

Per the code comment it was likely the removed lines were not needed but tests were required to confirm.

This does that

Merge pull request #18288 from KarinG/extend_testSubmitContributionPageWithPriceSetQuantity

extend_testSubmitContributionPageWithPriceSetQuantity

[REF] simplify retrieval of existing membership on membership renewal form

After stepping through the code I determined that if membershipID is passed into getContactMembership then it either
1) returns the membership id much like membership.get but with is_current_member calculated or
2) returns the owner membership in a similar format

However, the renewal form is not visible for a non-owner membership so getting the owner membership is cruft
from another form & only 1 applies. Calling the api is much more transparent

Merge pull request #18283 from aydun/class_api_tweak

Improve robustness of api/class.api.php

[REF] Separate and move line-item specific portion of checkTaxAmount to LineItem

This is not intended to be the final place but making sense of checkTaxAmount is hard so I wanted to keep this simple and
iterate.

checkTaxAmount is called from 2 places - once from v3 LineItem.create with isLineItem = TRUE
and once from Contribution.create with isLineItem = FALSE

Most of the function is accessible dependent on isLineItem - meaning it's 2 separate functions.
However, the first chuck is called 'whenever id is set'. BUT it makes no sense for line items. It
uses the 'id' - regardless of whether it is a line item id or a contribution id - to get the previous
contribution. It then sets tax amount based on a comparison - but
that really makes zero sense. I'm pretty sure it was never intended to be used for LineItems

Move LineItem acl handling from v3 api to financialacls extension

This makes it available for the v4 api

dev/core#1983 Fix to tax calculation on multi-line-item

This is similar to https://github.com/civicrm/civicrm-core/pull/18284 - it differs in that the totals are calculated by iterating
through the line item array afterwards, rather than expecting the 'getLine' function to calculate totals. Some
obvious follow ups suggest themselves but I will look against master.

This is difficult to test (Karin gave it a really good shot) because of the weird way it's calculated in Main and thenn
used in Confirm. Cleanup should resolve the testability issue too

extend_testSubmitContributionPageWithPriceSetQuantity

Slight increase in test cover & make functions more re-usable

Just preliminary to increasing cover in another test class

Merge pull request #18289 from MegaphoneJon/actionschedule-serialize

add serialize data to actionschedule

Merge pull request #18285 from colemanw/contains

dev/event#37 Add CONTAINS operator for APIv4 & Search

add serialize data to actionschedule

Merge pull request #17991 from eileenmcnaughton/merge_single

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

allow negative self-service hours

event #34 message improvement

fixes to message template

fixes

Search ext - Show translated labels for operators

Fix test set up to call Order.create to create the correct line items

This test is not creating valid line items - so when we expect them to be correct - they aren't

Merge pull request #18275 from eileenmcnaughton/linepre

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

APIv4 - Add CONTAINS operator and add to search extension

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

APIv4 - Move list of accepted query operators to CoreUtil

Use !empty() instead of isset() in constructor so that empty strings don't confuse the logic
Update array syntax in comments
Add test for class.api.php

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"