(NFC) Civi/CCase - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Core - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Payment - Update for Drupal.Commenting.VariableComment.IncorrectVarType

(NFC) Civi/Test - Update for Drupal.Commenting.VariableComment.IncorrectVarType

Merge remote-tracking branch 'origin/5.22' into 5.22-master-2020-01-24-13-18-00

Merge pull request #16370 from mattwire/membershipstatuscalc_params

NFC Add parameter definition to membershipstatus.calc

Merge pull request #16335 from colemanw/exportui

Move exportui extension into core

Merge pull request #16371 from yashodha/typo

Remove unused code

Merge pull request #16363 from eileenmcnaughton/deextract

Duplicate code (for real)

remove unused code

Add parameter definition to membershipstatus.calc

Merge pull request #16369 from eileenmcnaughton/export_house

dev/core#1364 Merge all addresses on export should INCLUDE merging households

Merge pull request #16129 from mattwire/gettotalamount_0

Make getTotalPayments return 0 instead of NULL

Merge pull request #16364 from eileenmcnaughton/export

[REF] Extract function to export header row

Merge pull request #16368 from yashodha/typo

fix more typo's in the text

dev/core#1364 Merge all addresses on export should INCLUDE merging households

This is an alternate to https://github.com/civicrm/civicrm-core/pull/15725 that still achieves the goal in
https://github.com/civicrm/civicrm-core/pull/15725#issuecomment-572408173 of making is so that the
mergeSharedAddress option mergesHouseholds first & then merges sharedAddresses.

In #15725 @monishdeb was fixing the code that differentiated between how master_id was treated
versus otherwise merged addresses. However that code 'broke' a long time ago and the arguably
broken behaviour is locked in by unit tests. Fixing it has not been requested - in this
issue or any other in the past year or 2 when it has been broken so I opted to remove the broken
code entirely & to open up the next query such that it does not exclude household addresses.
This didn't break any tests (woot) and also allowed me to enable the test rows @monishdeb wrote
to test this issue

fix more typo's in the text

Merge pull request #16358 from MegaphoneJon/event-30

event#30 - don't allow multiple waitlist registrations

event#30 - don't allow multiple waitlist registrations

Extract function to export header row

Merge pull request #16356 from totten/5.22-gencode-ts

(dev/core#1546) Fix translation of seed data

Duplicate code (for real)

This is going to seem unintuitive but I'm sure the path to cleaning up the participant form starts with
increasing code duplication.

Currently the code is sharing 2 functions with the front end form. The functions don't do a lot of actual processing
but they do do a lot of preparing parameters. The calling functions do a lot of pre-work to call them - so
we are not getting much benefit from the sharing. On the other hand having this code tied up with another form makes
it hard for us to do the 'right' sharing - ie. currently the code to create a contribution paid by credit card & participant record is totally different to another payment method & we should work to share these bits of functionality

Return 0 if total amount is 0 instead of NULL.

Merge pull request #16353 from pradpnayak/functionFix

Wrong parameter passed to executeQuery function

Merge pull request #16311 from MegaphoneJon/event-template-reminders

event#28: Don't send scheduled reminders for event templates

Don't send scheduled reminders for event templates

Merge pull request #16361 from yashodha/typo

fix typo's in the text

Merge pull request #16189 from eileenmcnaughton/acl

Update BAO_Acl internal functions to protected & unused to deprecated

fix typo's in the text

(NFC) LocalizedDataTest - Add tips on running

Merge pull request #16017 from mattwire/setentitysubtype

Add function to set entitySubType for entity forms

Merge pull request #16354 from mlutfy/translation35

dev/translation#35 Display the CiviCRM Resources dashlet in the user's language

(dev/core#1546) Fix translation of seed data

Overview
--------

This fixes a regression in which seed data is not translated.

Before
------

The `civicrm_data.*.mysql` files are not translated.

After
-----

The `civicrm_data.*.mysql` files are translated.

There's a unit-test to check this.

Comments
--------

This regression stems from #15411, which aimed to allow extensions to define
custom variants of `ts()`.  The crux of the issue is "What happens if you
try to translate a string before the system is bootstrapped - before the
extension is loaded?  What's your fallback behavior?"

In #15411, it used a fallback behavior of "do no translation".  In theory,
you shouldn't really get into this scenario since UIs are pretty much always
generated post-boot.

However, it turns out that there is a situation where you have an un-booted
system and need to translate strings -- i.e. when generating the localized
`civicrm_data.*.mysql` data.  Hence the bug.

This patch preserves most of the changes from #15411, but it changes the
fallback behavior from "do no translation" to "use the built-in/default
translator".

Merge pull request #16334 from eileenmcnaughton/ev_tpl

[REF] Move assignment of BalanceAmount

Wrong parameter passed to executeQuery function

dev/translation#35 Display the CiviCRM Resources dashlet in the user's language

Merge pull request #16351 from colemanw/debug

Add debug info to api4 output

Merge pull request #16260 from eileenmcnaughton/ids_2

[REF] Fix CRM_Core_BAO_UFGroup::createUFJoin to not receive by reference.

Add debug info to api4 output

Merge pull request #16349 from eileenmcnaughton/credit

Remove unreachable code lines.

Merge pull request #16307 from seamuslee001/dev_mail_59

dev/mailing#59 Add in provider option IMAP_XOAUTH2 to assist with goo…

Remove unreachable code lines.

We were very precautionary in Barcelona & deprecated rather than removing these line. However, we've had
several releases including a security release & an ESR since then with no one reporting hitting these so
they can go now

Merge pull request #16326 from wmortada/core#1537

dev/core#1537 Don't append multiple copies of the eligible related contacts text

Merge pull request #16286 from alifrumin/offlineContributor

Add "Contributor Name" to Offline Contribution Receipts

Merge pull request #16343 from colemanw/abstract

Api4 - Make abstract function abstract

Merge pull request #16346 from colemanw/entityGet

Api4 - Improve Entity::get

Merge pull request #16347 from demeritcowboy/name-label-yet-again

dev/core#1116 Wrong linebreaks viewing Inbound Emails when activity type label is changed

Merge pull request #16337 from eileenmcnaughton/ev_static

Change function buildEventFeeForm to non-static

wrong linebreaks when activity type label is changed

Merge pull request #16344 from eileenmcnaughton/ev_test

Fix unit test so price set params are valid

Merge pull request #16340 from jitendrapurohit/dev-1540

Add user friendly error message on merge error

Fix unit test so price set params are valid

Add user friendly error message on merge error

Merge pull request #16345 from eileenmcnaughton/ev_title

Simplify event title retrieval, use cache, add test

Merge pull request #16342 from JKingsnorth/patch-13

[NFC] Remove unused local variables from CRM_Contribute_Form_AdditionalPayment::processCreditCard

Api4 - Improve Entity::get

Ensure that every php file found by the scanner is_a class inheriting from AbstractEntity.
Remove the CustomValue class by virtue of its class type instead of manually remove it.
Improve performance when fetching entities by name.

Simplify event title retrieval, use cache, add test

Merge pull request #16341 from mattwire/membership_overridden

When viewing a membership show if the status is overridden

Api4 - Make abstract function abstract

Remove unused variable $now

When viewing a membership show if the status is overridden

Remove unused local variable $fields

Merge pull request #14591 from JKingsnorth/dev/core#1064

dev/core#1064 Allow personalised 'view in browser' links for mass emails

Changee buildEventFeeForm to non-static

I searched on thee full function name & also 'buildEvent' & 'feeForm' and I don't believe this is called
from outside this function. I want  to call it from tests to  fix some test set up issues butt
there are a bunch of undefined properties so I concluded I should set this up first.

Merge pull request #16315 from eileenmcnaughton/var

[REF] Remove setting of unused function.

Update help to reflect new Export UI

Merge pull request #16331 from eileenmcnaughton/cont_soft

[NFC] Code formatting

Clean up buildMappingForm which is no longer used by Export

buildMappingForm is now only used by Search Builder

Merge pull request #16329 from eileenmcnaughton/ev_one

Start to use function rather than multiple queries for event details add test

Merge pull request #16333 from eileenmcnaughton/bounce

[REF] Move the bounce to the start of the form submit

Merge pull request #16332 from seamuslee001/event_confirm_text_extend

[NFC] Extend unit test to cover for recent refactor by eileen to ensure sup…

[REF] Move assignment of BalanceAmount

This moves the assignment of balanceAmount to where the partial_payment_total is  determined.

Template assignments happen all over the place so it makes more sense to group this
which where the values are determined

Move the bounce to the start of the form submit

Start to use function rather than multiple queries for event details, add test

[NFC] Code formatting

Extend unit test to cover for recent refactor by eileen to ensure support for CRM-4196 remains

Merge pull request #16330 from eileenmcnaughton/process_event

[REF] Eliminate silly parameter from function signature.

Merge pull request #16018 from mattwire/manualcontributionstatus

Fix retrieving contribution_status_id for manual payment processor

Move exportui extension into core

Add code docs to Api4/CustomValue

Fix retrieving contribution_status_id for manual payment processor

Eliminate silly parameter from function signature.

Removes the isAdditionalAmount parameter which is only used to adjust the amount key on the value variable.

Value is reset each participant so we don't need to worry about it being 'cumulative'

Merge pull request #16316 from eileenmcnaughton/parent

Remove function that exactly matches parent class.

Merge pull request #16327 from eileenmcnaughton/pp

Extract code for isPaymentOnExistingContribution.

Extract code for isPaymentOnExistingContribution.

This allows us to make the meaning of the IF clause clearer

Merge pull request #16325 from eileenmcnaughton/ev

[REF] Extract code that assigns event variables to the tpl

Merge pull request #16314 from eileenmcnaughton/nfc

[NFC] Code cleanup

Extract code that assigns event variables to the tpl

Don't append multiple copies of the eligible related contacts text

Merge pull request #15992 from eileenmcnaughton/utf_temp

Update temp table handler to support utf8mb4 if that is the db collation

dev/mailing#59 Add in provider option IMAP_XOAUTH2 to assist with google's changes to imap access

Apply PHP Syntax fix patch and update tests

Bump up to 1.9.1 of zetacomponents/mail to fix syntax issue

[REF] Fix CRM_Core_BAO_UFGroup::createUFJoin to not receive by reference.

This function is called from one place. It gets passed a params variable that is 're-used' so this
change makes it clear what actually needs to be passed on. We have an anti-pattern in our code whereby params get
passed all sorts of places with it being confusing which ones actually need to receive what

Remove function that exactly matches parent class.

I copied it over the parent class fn & apart from minor comment block differences it matches

Merge pull request #15383 from michaelmcandrew/lab-1286

also skip report instance header and footer (lab-1286)

Merge pull request #16320 from mattwire/pcp_pagecleanup

NFC Convert Manage PCP page to short array syntax

[NFC] Fix some missing exceptions etc

[NFC] Minor code tidy up

Declare Exceptions, use strict comparison

Merge pull request #16321 from colemanw/CheckBox

dev/core#1535 - Fix missing address is_primary field

dev/core#1535 - Fix missing address is_primary field

Merge pull request #16317 from eileenmcnaughton/duplicate

Remove duplicate line of code