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.

dev/financial#72 Filter contrib status search selects

Replacing pseudoConstant calls with BAO buildOptions in 'search'
context.

This commit is only intended to touch places where we are building
a list of options to offer to the user, not places where we are
displaying results.

In practice this doesn't remove any currently available statuses,
but now we can filter our the proposed 'Template' status.

Merge pull request #15407 from mattwire/membershippayment_2

Further cleanup and clarification on MembershipPayment

Merge pull request #15291 from mattwire/formatpaymentparams_addccip

REF Move ipAddress and CC expiry date to prepareParamsForPaymentProcessor

Merge pull request #15469 from wmortada/patch-1

Update my organisation

Update my organisation

Merge pull request #15283 from christianwach/lab-1200-3

Restrict inactive Dashlet query to Dashlets in the current domain

Merge pull request #15465 from eileenmcnaughton/dep_fn

[REF] remove call to deprecated function

Merge pull request #15462 from eileenmcnaughton/event_api

[REF] Fix typos and remove unused variables

Merge pull request #15456 from aydun/change_getTemplateContrib

dev/financial#72 Modify getTemplateContribution() to use is_template field

Merge pull request #15463 from eileenmcnaughton/status_fix

[REF] Improve signature on updateFinancialAccountsOnContributionStatusChange

[REF] remove call to deprecated function

getPartialPaymentType basically just calls CRM_Contribute_BAO_Contribution::getContributionBalance
& then returns the  balance in a different array key depending on the value.

This switches to calling it directly & completes the work of removing all calls to it & more heavily deprecating it

Merge pull request #15461 from civicrm/5.19

5.19

Merge pull request #15464 from seamuslee001/5.19

5.18.2 release notes

Fix typos and remove unused variables

Merge pull request #15448 from artfulrobot/replace-openflash-charts

Replace openflash charts with charts that work

Update release-notes.md

Update 5.18.2.md

Update release notes following recent merges

Update release notes for fix for dev/core#1269

Add in release notes for 5.18.2

Merge pull request #15460 from eileenmcnaughton/event_transact

[REF] Remove transaction as part of minor code cleanup

Change getTemplateContribution to prefer is_template contrib
Add test

Improve signature on updateFinancialAccountsOnContributionStatusChange

Merge pull request #15459 from eileenmcnaughton/event_clean

[ref] Simplify function signature as parameter is not used

Pass contact ID as param to executeQuery()

Tidy up charts code, including avoiding js error on tabular data view

[REF] Remove transaction.

@mattwire proposed moving this in
https://github.com/civicrm/civicrm-core/pull/15446/commits/8f76cfacf156cfbd11e280b9a86926e0a5082c27

However, I believe removing it is the correct action. The usefullness of transactions is to maintain integrity
by rolling back the creation of a bunch of related entities together if one fails.

However in this case it's not rolling back the contribution or participant record or non discount line items so
if it worked (which I doubt) it would reduce rather than increase db integrity

Make getTemplateContribution match is_test
- non-test recurs should only pick up non-test templates
- test recurs should only pick up test templates
Add test

Simplify function signature as parameter is not used

Merge pull request #15453 from eileenmcnaughton/import_campaign

dev/core#1285 import support for campaign_id (& other fields where it could be an id, a name or a label.

Merge pull request #15454 from demeritcowboy/custom-fieldspec-bao

dev/core#1305 - fieldspec bao not defined for custom fields

Merge pull request #15444 from civicrm/5.19

5.19

Convert getTemplateContribution to api4

dev/core#1305

dev/core#1285 import support for campaign_id (& other fields where it could be an id, a name or a label.

After digging into https://lab.civicrm.org/dev/core/issues/1285 it feels a little arguable what we should support but issues in
gitlab indicate there is at least some demand for id & label so it now will
- accept id if input matches an id
- accept name if input matches a name
- accept label if input matches a label.

This means for payment instrument it would accept '1', 'Check' or 'Cheque' (in that order if name were renamed).

This should apply to a few fields - payment instrument, constribution status & per the issue campaign
(on contribution). There are 'some' on membership too I think

WIP checkin

WIP: bar chart sort of working now

Pie and Bar charts working, with download in SVG and PNG links

oops remove accidental inclusion

remove accidentally included crap

remove 3d barchart as unnecessary (and unimplemented)

Merge pull request #15321 from yashodha/dev_1065

(dev/core#1065) Pledge in Edit mode needs to be shown consistently

Merge pull request #15443 from eileenmcnaughton/ex_merge

dev/core#1302 fix regression on exported when merging addresses with specified fields

Merge pull request #15433 from aydun/add_contribution_is_template

dev/financial#72 Add is_template field to contribution entity

 dev/core#1302 fix regression on merge export

Fix example

Merge pull request #15447 from seamuslee001/dev_core_1269

dev/core#1269 Add in fix for undefined index bao in parsePseudoConst…

Merge pull request #14891 from eileenmcnaughton/saved_search

[Code-quality] remove references to where_tables, where_clause

Merge pull request #15439 from eileenmcnaughton/ex_bug

dev/core#1293 Fix regression on  export filtering for postal address only

dev/core#1269 Add in fix for underfined index bao in parsePseudoConstantField for Import Parser

Add ipAddress and creditcard expiry date to prepareParamsForPaymentProcessor

Add is_template field to contribution entity
Update tests and examples

Merge pull request #15440 from eileenmcnaughton/ex_zero

dev/core#1293 Still output csv file on export, even if no rows are in it

Merge pull request #15438 from demeritcowboy/emailprocessor-still-hungry-alt

dev/core#1270 - fix email processor dropping attachments - ALTERNATE

Merge pull request #15281 from mattwire/formatpaymentparams_useprepare

REF Use the new prepareParamsForPaymentProcessor function in more places

dev/core#1293 Fix regression on  export filtering for postal address only

Still output csv file, even if no rows are in it

Merge pull request #15425 from eileenmcnaughton/loc_clean

[NFC] minor simplification

Merge pull request #15418 from seamuslee001/frontend_contribution_page_title

Add in frontend_title field for Contribution Pages to allow site mana…

fix email processor dropping attachments

[NFC] minor simplification

This just changes how return vars are handled to make it clearer that ONLY contact::Create
cares about location_block::create return values

Merge pull request #15434 from civicrm/5.19

5.19 to master

Merge pull request #15428 from greenpeace-cee/fix-status-check

Fix status check not rendering before 5.19 migrations

Merge pull request #15411 from systopia/dev_l10nx

Allow custom ts functions in extensions; defer custom ts calls until booted

Merge pull request #15430 from civicrm/5.19

5.19

Merge pull request #15412 from demeritcowboy/fly-backwards

Minor a_b/b_a mixup on case type edit screen

Merge pull request #15420 from eileenmcnaughton/live_notice

dev/core#1281 fix e-notice on isLiveMode

Merge pull request #15359 from fkohrt/master

Improve item order inside Online Contributions section of configuration checklist; see lab issue dev/core#1278

Fix status check not rendering before 5.19 migrations

This fixes an issue where the status check page does not render when
migrations to 5.19 are pending because the is_active column hasn't been
added yet.

The rationale for this change is to avoid scenarios where users forget
to run upgrades because there's no status check warning.

Merge pull request #15424 from eileenmcnaughton/email_form

[NFC] code formatting

Merge pull request #15423 from greenpeace-cee/fix-is-required-address-field

Fix E_NOTICE for is_required in address form

Add in frontend_title field for Contribution Pages to allow site managers to specify a title for a page different to that for use from the backend

[NFC] code formatting

Merge pull request #15419 from eileenmcnaughton/event_filter

dev/financial#72 Make is_template=0 the default for api get requests.

Set version to 5.20.alpha1

Set version to 5.19.beta1

Fix E_NOTICE for is_required in address form

This fixes an E_NOTICE that's triggered for optional address custom
fields in CRM_Contact_Form_Edit_Address.

dev/core#1281 fix e-notice on isLiveMode

Looks like a regression from https://github.com/civicrm/civicrm-core/commit/752cee47213094c29b8e6ae5d0b20f0800520d80
which hit in 5.15 - seems good to merge to rc / about to cut to rc

dev/financial#72 Make is_template=0 the default for api get requests.

We are planning on adding is_template to the civicrm_contribution table - following the precedent from event.

By default it is NOT desirable to get template entities returned in a get request - this fixes that at the generic
apiv4 level and means that it will be a default filter when present

Merge pull request #15417 from eileenmcnaughton/matt

Add trxn_id as a parameter on Payment.get

Merge pull request #15414 from eileenmcnaughton/fix_log_text

dev/core#1283 fix inability to export more than 255 chars

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

Update fix - should not be localisable

Accept trxn_id to Payment.get

Alternative to #15166 (incorporates #15229 )

Unit test for #15166 fix

Merge pull request #15381 from eileenmcnaughton/w_test

Add test for uF group one bug

Update fix - should not be localisable

dev/core#1283 fix inability to export more than 255 chars

don't prevent I18n init, @totten says it's not too costly...

fixed style error

Merge pull request #15409 from eileenmcnaughton/status

dev/core#1295 Add is_active field to civicrm_status_preference

run strings through ts(), but skip actual translation

Merge pull request #15413 from eileenmcnaughton/default_sub_type

[REF] don't handle non-array, pass by reference

Heed is_active value if set to zero

Note this is done BEFORE doing checks in case it is for performance reasons

Add test for uF group one bug

[REF] don't handle non-array, pass by reference

This is only called from one place & it is definitely coming in as an array & using the return values.

minor a_b/b_a mixup

Merge pull request #15352 from colemanw/extCompat

Improve obsolete extension management

implemented @totten's approach to fix custom ts functions in extensions

Ensure contribution_id is always set and clarify what functions do

Merge pull request #15404 from seamuslee001/dev_core_861

dev/core#861 Ensure that when processing mailings that no emails are …

Merge pull request #15384 from mattwire/membership_updaterecur

Remove call to updateRecurMembership which is handled by Membership::create BAO

Merge pull request #15406 from mattwire/morecontributionstatuslabel

Fix checking of isSubscriptionCancelled when label is changed

Merge pull request #15405 from MikeyMJCO/nomoreyahoo

Remove legacy Yahoo references from the Mapping/Geocoding settings/code.

Fix check for isSubscriptionCancelled in contribute