Merge pull request #16390 from artfulrobot/tame-propbag-deprecation-logs

Prevent PropertBag from being so noisy about deprecation warnings

Merge pull request #16395 from eileenmcnaughton/cont_setting

Finish removal of deferred_revenue_enabled from non-standard setting

Merge pull request #16271 from mattwire/pcp_extractsupportertext

REF Extract code to build pcp_supporter_text and enable translation

Merge pull request #16396 from eileenmcnaughton/cont_setting2

[REF] Remove unused variables

Merge pull request #16394 from demeritcowboy/extdir-links

dev/core#1550 Invalid links to extension directory in popup help text

[REF] Remove unused variables

Finish removal of deferred_revenue_enabled from non-standard setting

Merge pull request #16372 from colemanw/api4Magic

Api4 - Use explicit adder functions rather than magicMethod

invalid links

Prevent PropertBag from being so noisy about deprecation warnings for #16390

Merge pull request #16389 from eileenmcnaughton/ev_static

Fix tests to use buildFeeFn

Merge pull request #15408 from ejegg/l10n

dev/translation#30 Move l10n resource dir under private files

(dev/translation#30) Undo new MagicMerge::$l10nDir property

The previous commit added a new property `l10nDir` to `MagicMerge`.

The property doesn't seem to be needed, so omit it.

Merge pull request #16352 from mlutfy/translation34

dev/translation#34 Fix Contact Reference option for Postal Code

Merge pull request #16367 from MegaphoneJon/event-32

event#32: Fix bad variable in message template

dev/translation#30 Move l10n resource dir under private files

Looks under [civicrm.private]/l10n if that exists, falling back to
[civicrm.root]/l10n.

Since I18n::getResourceDirectory needs to be called from GenCode,
where we don't have the base directory defined, we make the l10n
path factory function consult new #define CIVICRM_L10N_BASEDIR first.

In GenCode itself, keep using the old location for now, but make
it possible to change the location with an env var as soon as we
update buildkit.

Assert on fail

Fix tests to use buildFeeFn

Merge pull request #16348 from eileenmcnaughton/ev_online

Remove irrelevant code.

Merge pull request #16375 from seamuslee001/dev_core_1538

Update Export unit test to pass on MySQL 8

Merge pull request #16387 from demeritcowboy/blank-google-recaptcha

dev/translation#33 Fix incorrect quoting in a link and an incorrect link

Merge pull request #15912 from mydropwizard/d8-language-empty-prefix

dev/drupal#85 Drupal8: Fix bug with empty language prefix mangling https:// to http:/

fix invalid quoting and incorrect link

Merge pull request #16386 from civicrm/5.22

5.22

Merge pull request #16377 from ejegg/serviceFileFinder-5.22

loadServices: tighten up file match regex

Merge pull request #16385 from civicrm/5.22

5.22

Merge pull request #16384 from totten/5.22-nfc

(NFC) Civi, CRM, tests - Update for Drupal.Commenting.VariableComment.IncorrectVarType

[NFC] Fix var declarations in the test folder

Update Var delcartion to be standard in CRM folder

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

(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 pull request #16382 from seamuslee001/crm_var_update

(NFC) Update Var declaration to be standard in CRM folder

Update Var delcartion to be standard in CRM folder

Merge pull request #16355 from eileenmcnaughton/ev_part

Add helper for getting participantValues

Merge pull request #16379 from totten/master-coder-ivt

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

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

(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 pull request #16373 from colemanw/api4Stuff

Api4 - improve docblocks & display links in explorer

loadServices: tighten up file match regex

Before: Civi code living under a path containing the characters php
(such as ~/src/php/civicrm) will crash trying to load services with
a message like the following:
   ReflectionException: Class Civi\Api4\Service\Spec\Provider\src
   does not exist in ReflectionClass->__construct()

After: Civi loads services normally no matter what the enclosing path

Update Export unit test to pass on MySQL 8

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

dev/translation#34 Fix Contact Reference option for Postal Code

Add @see annoations to api4 docblocks and show as links in api explorer

event#32: Fix bad URL in message template

Api4 - Use explicit adder functions rather than magicMethod

Api Actions had been relying on a magic method to provide getter/setter functions for params.
That works fine for simple get/set operations but it was also attempting to provide adder functions.
Those are more nuanced and the magic method didn't do a good job of understanding whether the param
was an indexed or unindexed array, and didn't do strict type checking.
Making our own adder functions gives better documentation, stricter checking of inputs and also
the convenience of variadic functions for adding several values at once.

Better looking checkboxes in api explorer

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

Add helper for getting participantValues

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

Remove irrelevant code.

In making this fn non-static we established it is only called from this  function  so
this cannot be true

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