[REF] minor refactor towards removing complexity.

This change actually makes the foreach & if following it redundant. I left them out for legibility
but basically all the wrangling is on the 'other' contact. We do just one thing with the 'main'
contact so the whole foreach & if is silly. Will follow up with removal of those

Merge pull request #15564 from eileenmcnaughton/deprectate

dev/financial#79 minimal deprecation of Contribution.transact.

Merge pull request #15591 from eileenmcnaughton/noisy_deprecate_transact

Add noisy deprecation warning to Contribution.transact

dev/financial#79 minimal deprecation of Contribution.transact.

It seems urgent that we so some deprecation of this api - this is the absolute minimum.

More would involve creating some noise when people use it but there
is a separate issue for that. This is just to geet it out of being 'promoted'
by the api explorer

Merge pull request #15589 from mfb/obsolete-method

Remove obsolete supportStorageOfAccents() method.

Add noisy deprecation warning to Contribution.transact

Note that the Order api has been an adequate replacement for over a year - per efforts by wordpress integration

Remove obsolete method.

Merge pull request #15504 from mattwire/transact

Extract deprecated Contribution.transact API into it's own file

Merge pull request #15587 from eileenmcnaughton/dedupe

[REF] minor extraction in dedupe code

Merge pull request #15581 from mountev/street-address-sorting-contact-detail-report

Support street address sorting for contact detail report

[REF] extract part of the function to build the location blocks

Who hates the variable 'moniker'?

Merge pull request #15584 from civicrm/5.19

5.19

Merge pull request #15583 from seamuslee001/5.19

5.18.4 Release notes

Merge pull request #15579 from mattwire/additionalpayment_fixstatusbounce

Fix parameters for statusBounce in AdditionalPayment

Update 5.18.4.md

5.18.4.md

There was some ambiguity between original problem-report and PR-description.
Update per MM.

release-notes/5.18.4.md

Merge pull request #15548 from eileenmcnaughton/order

Support chaining Payment.create from Order api

street address sorting for contact detail report

Merge pull request #15577 from mattwire/translate_additionalpayment

Translate two string in AdditionalPayment form

Translate two string in AdditionalPayment form

Fix parameters for statusBounce in AdditionalPayment

Merge pull request #15520 from compucorp/fix-date-format-values

Make sure labels match the actual date input format

Merge pull request #15572 from civicrm/5.19

5.19 to master

Merge pull request #15569 from eileenmcnaughton/pay_notice

Enotice fix (unreleased enotice)

Merge pull request #15537 from eileenmcnaughton/pay_check

Fix a regression whereby payment details are not saved from the AdditionalPayment form

Enotice fix

per https://github.com/civicrm/civicrm-core/pull/15545#issuecomment-544531438

Support payment related fields on Payment.Create

On some digging I found that various payment related fields like pan_truncation & card_type_id were
being quietly ignored rather than saved by Payment.create

The additional payment form now uses this api so it is a regression on that form.

This fixes the metadata, tests & support for payment-related trxn fields

Merge pull request #15567 from yashodha/report_cleanup

CRM-21677 - report clean up for case demographic report

Merge pull request #15568 from eileenmcnaughton/test_pp

Combine IF clause for readability.

Merge pull request #15565 from eileenmcnaughton/payment_processor_label

Fix  bug whereby cidZero does not prepopulate billing details for selected contact for pay later.

Merge pull request #14894 from eileenmcnaughton/date_fisc

Add freeform relative date for 'This Fiscal Year'

Combine IF clause for readability.

I'm still making sense of https://github.com/civicrm/civicrm-core/pull/15330 but these
lines are simply a readability change & 'hiving them off' into a separate PR & getting that
merged should make the actual changes in that PR more parsable.

CRM-21677 - report clean up for case demographic report

Merge pull request #15550 from eileenmcnaughton/api3_template

Filter out is_template contributions by default

Merge pull request #15566 from civicrm/5.19

5.19

Merge pull request #15561 from eileenmcnaughton/payment_amount

dev/core#1333 Fix incorrect balance amount on fully paid payment

Fix  bug whereby cidZero does not prepopulate billing details for selected contact for pay later.

To replicate
1) create a contribution page  & select  pay later   (and no other processor)  with 'billing  address required'
2) access the contribution page with cid=0 in the url
3) select a contact - witness  the contacts  detaisl have NOT loaded  into the billing block
4) follow the same steps  on event

I was trying to reduce complexity on  this form for another  change when  I realised  that  the variable
to determine  whether to show the block  was  wrong  when  billing details  were  required  for
pay later and  on testing  there  seemed now downside (other than a very minor performance
hit in  obscure  circumstances) in just always loading the billing profile  details.

I saw no js error  when the billing block was  NOT present  and we are already checking
the person has access  to see the contact so that is not an issue here (Profile.get
does a permission check

Merge pull request #15563 from civicrm/5.19

5.19  to master

Merge pull request #15558 from eileenmcnaughton/report_page519

(dev/core#1252) Fix paging on contact summary

dev/core#1333 Fix incorrect balance amount on fully paid payment

Per https://lab.civicrm.org/dev/core/issues/1333 this is resulting in an incorrect balance.

The code was previously marked as highly dubious

Support chaining Payment.create from Order api

Attempt to fix paging on contact summary

Merge pull request #15539 from eileenmcnaughton/pay_back

Rename balanceTrxnParams to paymentTrxnParams [REF]

Merge pull request #15528 from eileenmcnaughton/report_page

dev/core#1252 Attempt to fix paging on contribution summary

Merge pull request #15554 from demeritcowboy/casetype-xml-array

dev/core#1335 - Prevent php warning when saving case type

prevent php warning

Merge pull request #15551 from civicrm/5.19

5.19  to master

Merge pull request #15545 from eileenmcnaughton/pay_email_p

dev/core#1322 Fix regression whereby Additional Payment Form not setting from when sending email confirmations

Filter out is_template contributions by default

Merge pull request #15431 from ejegg/templateContrib

dev/financial#72 add 'Template' contribution status

Attempt to fix paging on contact summary

Ensure a default FROM address is used when sending email confirmations

Merge pull request #15542 from demeritcowboy/upgrade-convert-autoassignee-2

Upgrade script to flip autoassignees using bidirectional relationship in older civicase configs

Merge pull request #15483 from demeritcowboy/say-my-name-2

dev/core#1046 - Fix name vs label for case roles - alternate with case type edit screen code cleanup

Merge pull request #15491 from magnolia61/Additional_Email_Greetings_in_workflow_message_templates

dev/core#781 Additional email greetings in message templates

Merge pull request #15540 from wmortada/unfreeze_membership_end_date

dev/core#1126 - Don't freeze the end date if the membership is linked to a recurring payment

Merge pull request #15544 from civicrm/5.19

5.19 to master

Merge pull request #15538 from eileenmcnaughton/ppal_std

Regression fix on Paypal Std with memberships not being finalised correctly.

Merge pull request #15543 from MegaphoneJon/reporting-22

fix 'balance due' on Pledge Detail for non-US installs

update script to flip the autoassignee direction

fix 'balance due' on Pledge Detail for non-US installs

Don't freeze the end date if the membership is linked to a recurring payment

Additional email greetings in message templates

Rename balanceTrxnParams to paymentTrxnParams [REF]

This array needs some fixes but I wanted to rename the parameter first as balanceTrxnParams is misleading.

The parameters are for the financial_trxn entry corresponding to the payment

Merge pull request #15535 from eileenmcnaughton/payment_simp

[REF] Rename balanceTrxnParams variable to paymentTrxnParams

Regression fix on Paypal Std with memberships not being finalised correctly.

Per https://github.com/civicrm/civicrm-core/pull/15053#issuecomment-542770130

Rename balanceTrxnParams to paymentTrxnParams [REF]

This array needs some fixes but I wanted to rename the parameter first as balanceTrxnParams is misleading.

The parameters are for the financial_trxn entry corresponding to the payment

Merge pull request #15533 from eileenmcnaughton/confirm_tpl

Simplify logic for displaying card details for Event online registraion Confirm, Thankyou page

Merge pull request #15532 from eileenmcnaughton/event_cont_mode

Fix logic determining whether (masked) credit card details are displayed in event online receipts

Simplify logic for displaying card details for Confirm and Thank you page

Fix Event online receipts to display credit card info if available.

We currently have a very complex IF clause which includes some deprecated parameters.

However, we can simplify this to simply display the information if availble. We only collect
credit card information if we require it & hence the presence of it is an adequate indicator
as to whether to display it.

We could check for credit_card_number instead but actually there is precedent in another tpl
for checking credit_card_type & it's pretty neutral as both should be present or missing

Merge pull request #15531 from eileenmcnaughton/silly_template

Remove extraneous full stop, line

Merge pull request #15525 from eileenmcnaughton/evetn_template

Add test for participant receipts, super minor consistency fix.

Remove extraneous full stop, line

I noticed in testing the template looks like

Dear p,  Thank you for your participation.  This letter is a confirmation that your registration has been received and your status has been updated to Registered.  .

===========================================================
Event Information and Location
===========================================================
Annual CiviCRM meet

Add test for participant receipts, super minor consistency fix.

This is mostly adding testing for participant receipts. The one change to core code is to move the assignment
of contactID to the template from a specific path into the shared code. This parameter is not required by
the templates but it is somewhat common for sites to want to have it assigned to templates for
customisation which is presumably why it's there. Since it is there in one path we should either remove or
have it in all & for a field like contactID all makes sense. Tests lock in that var

Merge pull request #15529 from civicrm/5.19

5.19

Merge pull request #15527 from eileenmcnaughton/merge

Fix default mode on Contact.merge api

Merge pull request #15521 from agh1/5.19.0-releasenotes-initial

5.19.0 release notes initial run

Merge branch '5.19' into 5.19.0-releasenotes-initial

Merge pull request #15526 from civicrm/5.19

5.19 to master

fix name vs label for case roles

Fix default mode on Contact.merge api

I believe this was a change I made in error & only the Contact.get_merge_conflicts api should
receive an array for 'mode'. The mode is part of a loop within that api

Merge pull request #15524 from seamuslee001/5_18_3_release_notes

5 18 3 release notes

Merge pull request #13365 from mattwire/caseformatcustomdata

Format details for case custom data activity in a human readable format

Update 5.18.3.md

Add in release notes for 5.18.3

Merge pull request #15522 from MegaphoneJon/core-1323

Fix error when adding activities from Search Builder

Fix error when adding activities from Search Builder

5.19.0 release notes: added boilerplate

Make sure labels match the actual date input format

5.19.0 release notes - raw from script

Merge pull request #15486 from demeritcowboy/displaylabel-vs-displayLabel

[NFC] dev/core#1046 - minor consistency change

Merge pull request #15493 from artfulrobot/final-death-of-openflash

Remove references to open flash chart, including uploadchartimage fun…

Remove references to open flash chart, including uploadchartimage function, calls and routes

regen sql/civicrm_generated.mysql

Merge pull request #15519 from eileenmcnaughton/dedupe

[REF] minor extraction with code to build dedupe arrays

Merge pull request #15516 from eileenmcnaughton/part_mult

[REF] basic extraction of sendMails functionality along with a small test extension

[REF] minor extraction with code to build dedupe arrays

Merge pull request #15184 from eileenmcnaughton/dedupe9

dev/core#1230 [Ref] Rationalise dedupe code loop.

Merge pull request #15518 from alexymik/manager-typo

Fix typo in Manager.php