Adds missing join on the custom table

Merge pull request #15535 from eileenmcnaughton/payment_simp

[REF] Rename balanceTrxnParams variable to paymentTrxnParams

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 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

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

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

Fix typo in Manager.php

Merge pull request #15515 from eileenmcnaughton/event_flow

[REF] remove obsolete code.

Merge pull request #15517 from jitendrapurohit/tax-recur-test

dev/core#1317 - Fix total_amount on repeattransaction when tax amount is involved.

Merge pull request #15485 from demeritcowboy/role-name-heading

[NFC] dev/core#1046 - more accurate column heading

Merge pull request #15514 from eileenmcnaughton/order_participant

Order api updates to fix participant handling & deprecate creating 'completed

dev/core#1317 - Fix total_amount on repeattransaction when tax amount is involved.

Order api updates to fix participant handling & deprecate creating 'completed'

Per https://docs.civicrm.org/dev/en/latest/financial/OrderAPI/

1) we want to deprecate creating orders with a status other than 'Pending'
2) we don't want to have to require status is passed in for Participant.

[REF] remove obsolete code.

On trying to look into a bug around this code I realised that there is quite a bit of code that does nothing.

These lines exist to build the  variable - which is never used. Hence they can go.

I also spotted scriptArray is never used either - this stuff came from svn before everyone was using IDEs that
highlighted unused lines

Merge pull request #15358 from colemanw/relDesc

dev/core#1257 Make relationship description searchable

Unit test to replicate dev/core#1317

Merge pull request #15510 from civicrm/5.19

5.19 to master

Merge pull request #15502 from eileenmcnaughton/payment_bug

dev/financial#69 Fix misrecording of payments against non pay_later Pending contribution.

Merge pull request #15507 from civicrm/5.19

5.19

Merge pull request #15506 from seamuslee001/ignore_own_extension

Do not check for the extension we are trying to download itself

Do not check for the extension we are trying to download itself

Merge pull request #15492 from eileenmcnaughton/credit

[REF] Deprecate calls to createCreditNoteId

Merge pull request #15499 from eileenmcnaughton/pp

Test  calling CRM_Financial_BAO_PaymentProcessor::getPaymentProcessors

Mark test incomplete for now

Merge pull request #15501 from eileenmcnaughton/order

dev/financial#73 Update Order.create so that total_amount is not required.

dev/financial#73 Update Order.create so that total_amount is not required.

As test shows it is otherwise created from the line items

Merge pull request #15500 from eileenmcnaughton/part_cleanup

Switch creation of ParticipantPayment to use API

dev/financial#69 Fix misrecording of payments against non pay_later Pending contribution.

Fixes bugs when payments are added against a non-pay later contribution. There were 2 different bugs
1) bug in form layer when the payment is fully paid
2) bug in BAO layer for partial payments.

This fixes both. It may be a regression, hence 5.19

Merge pull request #15497 from seamuslee001/frontend_title_receipts

Ensure that the frontend title is used for contribution page details …

Switch creation of ParticipantPayment to use API

Switch creation of ParticipantPayment to use API

Ensure that the frontend title is used for contribution page details in receipts if avaliable

Add in strict checking and simplify code block in contribution page and fix comment style as per EIleen's comments

Merge pull request #15498 from civicrm/5.19

5.19

Test  calling CRM_Financial_BAO_PaymentProcessor::getPaymentProcessors

Add test to ensure calling with Test capability only returns testMode processors

Merge pull request #15399 from eileenmcnaughton/pending

Update tests to simulate labels that are not the same as names

Merge pull request #15450 from mattwire/extensionupgrade

Check requirements before upgrading extensions via the UI

Merge pull request #15495 from civicrm/5.19

5.19 to master

Deprecate calls to createCreditNoteId

createCreditNoteId is called from BAO_Contribution::add whenever the contribution
status is 'Cancelled' or 'Refunded.

A few lines later recordFinancialAccounts is called, so by the time recordFinancialAccounts is called
it is already set & the empty check will prevent these lines being hit.

All 4 places where updateFinancialAccounts are called in the
code are from recordFinancialAccounts.

Record financial accounts is called from Contribution Create & Payment create.

contribution.create does not rely on these lines to set the creditnote_id, as discussed.

In the BAO_Payment there are 2 scenarios
1) payment is positive
2) payment is negative - in this case recordRefundPayment is called.

It's logically OK for the code to  NOT to create a credit note when being called from payment.create
 because payment.create is NOT cancelling the contribution or refunding it and the funtion NEVER changes
 the contribution status to Refunded, Cancelled, Chargeback (those are 'business' statuses).

Payment.create updates contribution status to reflect a payment has been made - eg change from
Pending to partially paid or various payment-related-statuses to 'Completed'.

ie Payment.create is
- only adding payments (or refunds) to an order
- and the decision as to whether something is being credited back & warrants a credit note id or
whether the payment is being returned due to overpayment or other payment related concerns does
not belong in the payment code.

Payment create never changes contribution
status to Refunded, Cancelled or ChargeBack so the lines would never be hit from payment.create

This leaves us with the conclusion the lines are never hit. For safety/sanity we can deprectate them rather
than remove them by now. That way if the above analysis is wrong a test will fail.

Less likely (given financial test coverage) the deprecation notice will show up in the wild in the next few months.

(It's rather unclear how it DOES fit in since it feels like creditnote_id only applies if the entire
contribution is being refunded & really should be generated at the point at which, for example, the event
fees associated with it are changed - however, complaints to date about creditnote_id have all focussed
on performance issues, not logic issues

Merge pull request #15494 from eileenmcnaughton/519_mem

Fix 5.18 regression on membership handling

Merge pull request #15467 from eileenmcnaughton/mess

dev/core#371 Remove template refs to 'print for your records'

Fix 5.18 regression on membership handling

Fixes a bug caused by
https://github.com/civicrm/civicrm-core/commit/5b9d3ce80f1a64b39bdd63160ad0a1b960f2faed

And picked up in test with https://github.com/civicrm/civicrm-core/pull/15399

Rename (some) labels to differ from names to simulate the fact they are different in internationalised extensions

Adjust tests for renamed labels

Remove template refs to 'print for your records'

We agreed to remove these a while back. Since https://github.com/civicrm/civicrm-core/pull/15466
is already upgrading them it's a good chance to rip our the rest:

Merge pull request #15144 from JKingsnorth/copying-events-contribution-pages

Improvements to copying events and contribution pages

Merge pull request #15315 from MegaphoneJon/reporting-20

reporting-20: don't display soft credits by default when force=1

Merge pull request #15323 from elisseck/dev/core/1266

dev/core/1266 widget header tab next state to pcp tab

Merge pull request #15223 from seamuslee001/when_package_upgrade

Upgrade When package to the lastest version

Merge pull request #15472 from ejegg/contribStatusUpdate

dev/financial#72 Filter contribution status lists for batch creation and update

Merge pull request #15470 from ejegg/filterContribStatus

dev/financial#72 Filter contribution status dropdowns in search and report

Merge pull request #15458 from eileenmcnaughton/excetp

[REF] move definition of important values & retrieval outside payment…

Merge pull request #14919 from eileenmcnaughton/mem_review

[Membership-backoffice] Reduce reliance on multiple specific but confusing class variables

Merge pull request #15293 from ixiam/dev#issue-826

dev/core#826

Merge pull request #15421 from artfulrobot/queue-safety

Fix bug in SQL queue that can cause tasks to be run twice in a multiprocess environment

Merge pull request #15468 from aydun/add_order_reference

dev/financial#57 Add order_reference field

Merge pull request #15488 from eileenmcnaughton/fin_acc

[REF] Minor cleanup to determine taxterm with a helper function.

Merge pull request #15489 from eileenmcnaughton/credit_dep

[REF] Replace deprecated function call with a more readable alternative

Merge pull request #15490 from eileenmcnaughton/fn_ref

[REF] improve function signature on updateFinancialAccountsOnContributionStatusChange

[REF] improve function signature on updateFinancialAccountsOnContributionStatusChange

This private function is only called from one place. The conditional before it is called is
if ($context == 'changedStatus') {

Ergo $context is ALWAYS changedStatus when this function is called.

As a result I have removed context from the function signature & the if that checks it is
equal to 'changedStatus'

Replace deprecated function call with a more readable alternative

This adds a wrapper to getting financial accounts with the type. We could call the other directly but it's less
readable & requires the account name to be specified which feels like a look-up for devs

Minor cleanup to determine taxterm with a helper function.

Simple cleanup to reduce the lines of code and make it more greppable

Merge pull request #15476 from mattwire/paymentprocessorpay_typo

Fix typo in parameter description for PaymentProcessor.pay

minor consistency change

more accurate column heading

Merge pull request #15482 from seamuslee001/fix_merge_array

Fixes an issue where the two arrays weren't always being merged correctly

Fix merge hook arrays

Fix typo in parameter description for PaymentProcessor.pay

Merge pull request #15474 from artfulrobot/fix-missing-contribution-charts

Fix missing/broken contribution charts

Merge pull request #15339 from seamuslee001/hook_before_boot

Do no invoke Hooks via UF unless container has been built or it is a …

Merge pull request #15473 from demeritcowboy/too-loopy

The loop doth process too much, methinks

Fix missing/broken contribution charts

Merge pull request #15442 from artfulrobot/fix-select2-height-followup-from-15403

Fix CSS for public select2 elements regardless of parent theme box-si…

be less loopy

dev/financial#72 Filter contribution status in create/update

Replacing pseudoconstant calls in batch creation/update using
CRM_Contribute_BAO_Contribution_Utils::getContributionStatuses().

This actually changes the list of options offered, but all the
ones it removes should probably already have been gone.