Merge pull request #16639 from seamuslee001/dev_core_1621

[NFC] dev/core#1621 Extend unit tests to ensure that entity financial account is co…

Merge pull request #16640 from JMAConsulting/dev-core-1158

core#1158: Unit Test to ensure primary addresses are printed on mailing label if searchPrimaryDetailsOnly is disabled

Merge pull request #16641 from colemanw/apiError

Api4 - Display sql errors in explorer

Merge pull request #16634 from seamuslee001/lab_core_1612

dev/core#1622 Fix unsubscribe when loading the unsubscribe form on a …

Api4 - Display sql errors in explorer

Api code minor cleanup

core#1158: Unit Test to ensure primary addresses are printed on mailing label if searchPrimaryDetailsOnly is disabled

[NFC] Extend unit tests to ensure that entity financial account is correctly deleted when financial type is deleted

dev/core#1622 Fix unsubscribe when loading the unsubscribe form on a different locale to the one the mailing was sent from

Add in comments as per request from Eileen

Merge pull request #16636 from totten/master-settings-tpl

civicrm.settings.php.template - Simplify examples of `$civicrm_setting`

Merge pull request #16635 from seamuslee001/dev_core_1615_distmaker

dev/core#1615 Update distmaker to handle the fact civicrm-setup is no…

dev/financial#84 - Update distmaker to include sequentialcreditnotes

civicrm.settings.php.template - Simplify examples of `$civicrm_setting`

Overview
--------

This simplifies the examples in `civicrm.settings.php.template`.

Before
------

Every reference to `$civicrm_setting` uses an English-looking group-name like

``php
$civicrm_setting['URL Preferences']['imageUploadURL'] = 'http://example.com/example-image-upload-url';
```

After
-----

The varying group names are just `domain`.

```php
$civicrm_setting['domain']['imageUploadURL'] = 'http://example.com/example-image-upload-url';.
```

Comments
--------

Since CiviCRM v4.7, expessions like `$civicrm_setting['URL Preferences']` have been aliases
for `$civicrm_setting['domain']`.  (See `SettingsManager::parseMandatorySettings()`.)

These examples were initially kept in the verbose 4.6 format so that users of 4.6 and 4.7
could continue to exchange examples with each other.  But 4.6 and 4.7 are pretty old, so I
don't think that's an issue anymore. We're now firmly in the 5.x world.

What does still matter is intuition - if the examples set an expectation that you should put
things under buckets like `URL Preferences`, then it implies that you should be thinking about
those buckets.  (Does the `extensionsURL` belong under `URL Preferences` or `Extension
Preferences`?  Should a search-related setting defined by an extension go under `Search
Preferences` or `Extension Preferences`?  Can I make up new groups?  If I know the name used
by the Setting API, then how do I figure out the group-name used for the `$civicrm_setting`
override?  ...  The answer to all of these questions is that it doesn't matter because they're
really the same group.)

Merge pull request #16597 from colemanw/drilldown

Menubar - Add "find menu item" search feature

dev/core#1615 Update distmaker to handle the fact civicrm-setup is now in core

Merge pull request #16631 from kartik1000/Fix_Invalid_parameter_doc

Removed Invalid Paramter from function doc

Merge pull request #16630 from breheret/master

Add ts() for a sentence 'Add to case as role'

Merge pull request #16633 from alexymik/patch-1

CommunicationPreferences 'loclize' -> 'localize'

'loclize' -> 'localize'

Removed Invalid Paramter from function doc

Signed-off-by: Kartik Kathuria <kathuriakartik0@gmail.com>

Add ts() for a sentence 'Add To case role'

Merge pull request #16628 from totten/master-setup-action

dev/drupal#4 - Add Civi\Setup::getPendingAction() helper

dev/drupal#4 - Add Civi\Setup::getPendingAction() helper

Overview
--------

This exposes a new piece of information for downstream consumers of the `civicrm-setup` API.

It doesn't change any behaviors or expectations - it merely tracks an extra piece of info.

Before
------

The `Civi\Setup` interface does not give any reporting about what's going on.

After
-----

The `Civi\Setup` interface has a method, `getPendingAction()`, which reports the pending action.

Technical Details
-----------------

(1) The pending action is strictly singular - it would not be sensible, for example, to have the
`installFiles` and `uninstallFiles` actions running at the same time. Thus, the guards within
each function.

(2) The basic gist of the change - in each of the functions
(`{install,uninstall}{Files,Database}`), set the `pendingAction` at the start and unset
it at the end.  The change looks slightly bigger than it should - because it wraps the
existing code within the `try ...  finally ...` block.

(3) This facilitates dev/drupal#4 and https://github.com/civicrm/civicrm-drupal-8/pull/37
- there were some use-cases in which the auto-install behaviors of `civicrm_install()`
and `FlushDrupal8.civi-setup.php` would provoke each other (causing a sort of
installer-loop).  This change makes it possible to avoid the loop without changing the
contracts.

Merge pull request #16560 from eileenmcnaughton/fgbvar

dev/core#1596 fix (unreleased) regression on contribution summary

dev/core#1596 fix (unreleased) regression on contribution summary

https://lab.civicrm.org/dev/core/issues/1596

Makes more use of parent function to fix (in general we've done fixes/ improvements to the report
generic functions that have not filtered down to reports that override them

Merge pull request #16503 from aydun/dev_core_1584

dev/core#1584 Add function to specify whether payment processor requires email address

Merge pull request #16338 from mattwire/note_timestamp

Convert civicrm_note.modified_date to timestamp

Merge pull request #16598 from eileenmcnaughton/setup

Enable the "sequentialcreditnotes" extension on new installations

Merge pull request #16621 from totten/master-setup-autoload

dev/core#1615 - Fix for recently migrated civicrm-setup

Merge pull request #16622 from seamuslee001/test_dev_core_1386

dev/core#1386 Add in unit test for #15834

dev/core#1386 Add in unit test for #15834

dev/core#1615 - Fix for recently migrated civicrm-setup

Overview
--------

Fix a recent (intra-alpha) regression from https://github.com/civicrm/civicrm-core/pull/16618

The search for `civicrm-setup` goes slightly awry because the migrated copy of civicrm-setup uses
the core's classloader rather than its own classloader.

Steps to Reproduce
------------------

* Create a site with WP and Civi(master).
* Ensure Civi is uninstalled. (ex: `cv core:uninstall -f`)
* In `wp-content/plugins/civicrm/civicrm`, touch the file `.use-civicrm-setup`.
  (*This is supposed to activate the new UI.*)
* Go to the WP admin console. In the left navbar, choose "CiviCRM Installer".

Before
------

It fails to load the new UI -- the search finds `.use-civicrm-setup` but does not
find `civicrm-setup-autoload.php`. It falls back to the old UI.

After
-----

It opens the new UI.

Regenerate civicrm_generated.mysql

bin/regen.sh - Allow the `sequentialcreditnotes` ext to propagate into generated-data snapshot

Add new core-extension to install sql

Per @totten there are some good future-thinking ways to address this issue of how to
install extensions on install. For now however, this should work & achieve our short-term goal

Merge pull request #16618 from totten/master-setup

dev/core#1615 - Import civicrm-setup

Merge pull request #16619 from demeritcowboy/remove-noop-translations

dev/translation#37 - Remove hardcoded translations from 13 years ago that no longer do anything

remove hardcoded translations that don't actually do anything anyway

(NFC) Update civicrm-setup to current civilint

(dev/core#1615) Load local copy of 'civicrm-setup' instead of downloading package

(dev/core#1615) Import civicrm-setup code

Note: This does not import the docs. Those should go to the dev guide.

Merge pull request #16616 from totten/master-scn

Move sequentialcreditnotes under `ext/` folder

Move sequentialcreditnotes under `ext/` folder

Maybe this is a bit of bikeshedding, but..

* `civicrm/ext/` is used for similar content in a few other cases (eg when adding `iatspayment` to the tarball; eg for
   the `drupal-case` and `wp-case` build-types).
* `civicrm/ext/` looks analgous to `sites/default/files/civicrm/ext`, which has similar content.
* Among the list of folders under `civicrm/`, `ext/` looks more consistent.  There are only two
  folders with capital letters (`CRM/` `Civi/`) and only two folders with multiple
  words (`bower_components/` `release-notes/`)... and 18 other folders which are
  lowercase+single-word (`ang/` `api/` `bin/` `css/` `distmaker/`
  `extern/` `i/` `install/` `js/` `l10n/` `packages/` `settings/` `sql/`
  `templates/` `tests/` `tools/` `vendor/` `xml/`).
* The file-hierarchy is additive (combining elements of the OS/host FHS, of the CMS, and of intra-extension content)
  and makes for a long total.

Menubar - Add "Find menu item..." search feature

Allows user to locate menu items by typing a few letters.
Find tool is located under the "Home" (Civi logo) menu.

Merge pull request #16613 from seamuslee001/nfc_custom_field_array

[NFC] Convert Custom Field BAO file to use short array syntax

[NFC] Convert Custom Field BAO file to use short array syntax

Merge pull request #15834 from francescbassas/patch-16

dev/core#1386 Allow advanced search for contributions without a soft credit related

Merge pull request #16608 from civicrm/5.23

5.23

Merge pull request #16607 from totten/5.23-pathurltest

Update recently-added PathUrlTest to be more representative

Update recently-added PathUrlTest to be more representative

Before
------

The test generates the URL for an example page (`civicrm/mailing/subscribe`)
by calling `url()`. It does not explicitly set the `url(...$frontend...)` option,
so it defaults to `FALSE`.

By contrast, the other places which make URLs for `civicrm/mailing/subscribe`
(eg `CRM/Mailing/BAO/Mailing.php` and `CRM/Utils/Token.php`) do set `$frontend` to `TRUE`.

After
-----

The test behaves the same way as other code.

Merge pull request #16545 from eileenmcnaughton/part_pend

[REF] Extract function to getTransactionInfo

Merge pull request #16606 from ejegg/API4Regex-5.23

Fix two more php-finding regexes

Fix two more php-finding regexes

Before: Civi code living under a path containing the characters php
(such as ~/src/php/civicrm) will fail to load the API4 explorer with
a message like the following:
    ReflectionException: "Class \Civi\Api4\src does not exist"

After: API4 explorer loads no matter what the enclosing path

Very similar to PR #16377

Merge pull request #16605 from seamuslee001/master

[REF] Update civicrm_generated following merge of #16362

Merge pull request #16604 from civicrm/5.23

5.23

[REF] Update civicrm_generated following merge of #16362

Merge pull request #16602 from mattwire/apiv3paymentspec

Add description to params for api3 Payment.get

Add description to params for api3 Payment.get

Merge pull request #16600 from mattwire/casesettingstpl

Remove hardcoded settings from form and use SettingForm.tpl for Case settings

Merge pull request #16362 from eileenmcnaughton/pp_disable

Do not enable core payment processor types that we believe likely don't work on new installs

Remove hardcoded settings from form and use SettingForm.tpl for Case settings

Merge pull request #16599 from seamuslee001/dev_translation_36

dev/translation#36 Ensure that we are only passing integers to the ng…

Merge pull request #16562 from eileenmcnaughton/notice

Move determination of year & month to formatCreditCardDetails

Add function to specify whether payment processor requires email address
Manual payment processor implements this capability
Test included

dev/translation#36 Ensure that we are only passing integers to the ngettext function for plurals

Merge pull request #16596 from civicrm/5.23

5.23

Merge pull request #16594 from alifrumin/rn-5.23

First pass at release notes for 5.23

First pass at release notes for 5.23

Merge pull request #16529 from mattwire/altermailing_templatetype

Pass template_type through to alterMailing hook

dev/core#1386 Allow advanced search for contributions without a soft credit related

Merge pull request #16085 from seamuslee001/dev_core_1466

[NFC] dev/core#1466 Update Documentation URLS to be the correct links…

Merge pull request #15940 from seamuslee001/dev_financial_105

dev/financial#105 Add CSS class onto the radio button payment processor options

Merge pull request #15316 from mepps/order-campaign-dashboard

Secondarily order campaign dashboard by id

Merge pull request #15884 from kainuk/issue-lab-1365

Related / Inherited Memberships: Custom fields not filled with data

Merge pull request #16359 from jensschuppe/dev_core_1547

dev/core#1547 Add DT_RowClass CSS classes to nested group markup

Merge pull request #16005 from magnolia61/Contribution_Invoice_Privacy

dev/core#1436 Do not CC or BCC Contribution invoice

Merge pull request #16580 from civicrm/5.23

5.23

Merge pull request #16588 from eileenmcnaughton/523partnotice

Fix enotice regression (unreleased)

Merge pull request #16589 from colemanw/copyValues

Cleanup copyValues DAO function

Cleanup copyValues DAO function

The $serializeArrays parameter was transitional and is now always enabled.
Passing $params by reference was unnecessary and has been removed.
Renamed a few variables for readability.

Merge pull request #16585 from eileenmcnaughton/dep_it

Add deprecation notice

Merge pull request #16581 from eileenmcnaughton/nfc

[NFC] Test cleanup.

Fix enotice regressionn

This was merged to 5.23 but I made a booboo in it
https://github.com/civicrm/civicrm-core/commit/a3aed42a453009733e8feb2c56fde5750b37cd14

I think the params are actually not used so it's just the notice it causes but this fixes

Merge pull request #16587 from seamuslee001/5.23

Add release-notes/5.22.1.md

Merge pull request #16575 from colemanw/savedSearch

Make savedSearch bao sane

Merge pull request #16583 from totten/5.23-gencode-no-l10n

xml/GenCode.php - Fix execution in default locale (without l10n data)

Add release-notes/5.22.1.md

Add deprecation notice

I've been trying to figure out when this line would be hit but I just can't find a way to trigger
it - removing it would help a lot with this code but let's deprecate for a bit.

Further test fix

Merge pull request #16582 from eileenmcnaughton/ids

(NFC) Remove `$Id$` from header

Make savedSearch bao sane

xml/GenCode.php - Fix execution in default locale (without l10n data)

Overview
--------

This fixes a recent regression in which `xml/GenCode.php` fails to execute in certain configurations.

Initial report: https://github.com/civicrm/civicrm-buildkit/issues/503

Before
------

* If the folder `l10n` exists in its traditional location, and if you run `./bin/setup.sh -g`, then it
  correctly executes.
* If the folder `l10n` does not exist in its traditional locacation, and if you run `./bin/setup.sh -g`,
  then it fails with an error like this:
  ```
  + php -d mysql.default_host=127.0.0.1:3306 -d mysql.default_user=dmasterciv_abcde -d mysql.default_password=abcd1234 GenCode.php schema/Schema.xml '' Drupal

  civicrm_domain.version := 5.23.alpha1

  Parsing schema description schema/Schema.xml
  Extracting database information
  Extracting table information
  PHP Fatal error:  Uncaught RuntimeException: Invalid configuration: the [cms.root] path must be an absolute URL in /home/me/.../Civi/Core/Paths.php:174
  Stack trace:
  #0 /home/me/.../Civi/Core/Paths.php(151): Civi\Core\Paths->toAbsoluteUrl('/', 'cms.root')
  #1 /home/me/.../Civi/Core/Paths.php(176): Civi\Core\Paths->getVariable('cms.root', 'url')
  #2 /home/me/.../Civi/Core/Paths.php(151): Civi\Core\Paths->toAbsoluteUrl('/', 'civicrm.root')
  #3 /home/me/.../Civi/Core/Paths.php(224): Civi\Core\Paths->getVariable('civicrm.root', 'path')
  #4 /home/me/.../Civi/Core/Paths.php(84): Civi\Core\Paths->getPath('l10n')
  #5 [internal function]: Civi\Core\Paths->Civi\Core\{closure}()
  #6 /home/jo in /home/me/.../Civi/Core/Paths.php on line 174
  ```

After
-----

You can run `./bin/setup.sh -g` with or without the `l10n` folder.

Comments
--------

There's an argument to be made that this shouldn't be necessary: when running
`GenCode`, it should only create PHP files, and none of the output should depend
on the CMS URL - because that's liable to change when you deploy the PHP code.
If someone did try to generate URL at such an early stage, it's arguably good to
generate an error.  In point of fact, `GenCode` is not actually using the CMS
URL.

The oddity stems from the contract of `CRM_Utils_System_*` (specifically,
`getCiviSourceStorage()` and `getDefaultFileStorage()`) which do double-duty
providing both path and URL.  To avoid duplicate work, `Civi\Core\Paths` uses
the same grain of information - it tracks the pairs of path+URL.

A more aggressive fix might be to split `getCiviSourceStorage()` and
`getDefaultFileStorage()` so that it's possible to get the path and URL
separately; then revise `Civi\Core\Paths` to take advantage of the finer-grained
contract.  However, splitting those things would be a more invasive patch,
and we're currently in RC.

[NFC] Test cleanup.

Cleaned up this test - re-used code etc

Merge pull request #16531 from eileenmcnaughton/cn_

dev/financial#84 Move sequential credit notes from 'deeply  embeded functions to separate structure

Merge pull request #16528 from mattwire/removeoldflexmailer

Remove old defines for flexmailer that haven't been required since CiviCRM 5.x

Remove  from header

I was a bit surprised to see the old drupal svn  value pop back up - did if come from this header.txt file

Merge pull request #16578 from seamuslee001/dev_core_1602

dev/core#1602 Fix join date filtering in search builder by standardis…

dev/core#1602 Fix join date filtering in search builder by standardising date field handling in Membership Query BAO

Merge pull request #16563 from eileenmcnaughton/selfsvc

[NFC] Minor code cleanup

Merge pull request #16577 from civicrm/5.23

5.23 to master