Merge pull request #16516 from colemanw/request

APIv4 - merge ActionUtil with Request::create

Merge pull request #16523 from yashodha/header

add column for report

Merge pull request #16537 from civicrm/5.23

5.23

Merge pull request #15824 from mfb/email-search-speed-boost

Speed boost for civicrm/ajax/checkemail

Merge pull request #16467 from ixiam/dev-1110

fix contribution summary report's statistics when grouping and having

Merge pull request #16451 from mattwire/settingreadonly

Settings Fix setting readonly attribute

Merge pull request #16535 from eileenmcnaughton/523rel

dev/core#1592 fix regression on relation active period

Merge pull request #15330 from mattwire/paymentprocessor_testmodelivemode

Fix issues with retrieving supportsTestMode/supportsLiveMode for payment processors

Merge pull request #16534 from eileenmcnaughton/523m

dev/core#1551 regression fix

Merge pull request #16526 from eileenmcnaughton/valid_calls

Fix calls to Request::retrieve

dev/core#1592 fix regression on relation active period

Per https://lab.civicrm.org/dev/core/issues/1592 the custom query  for this field is being
ignored now it's using datepicker with relevant format changes. This  causes the
custom function for the field to  be called

dev/core#1551 regression fix

Now that mailing_id is defined as field metadata the form main metadata fn is trying to handle it, However
the form is casting it to the wrong array format -https://github.com/civicrm/civicrm-core/blob/86f20cc17dde028977130f84feaf3b1d760ca57d/CRM/Contact/Form/Search.php#L649

We could fix  this & rip it out of Mailing_BAO_Query::where but there is still a gap  in  that
there is no pseudoconstant recorded for the field so the qill doesn't work & there is a gap in
pre-parsing for the form 'IN'

Hence this seems like a good approach to remove the regression in the immediate term

Merge pull request #16530 from civicrm/5.23

5.23

Merge pull request #16521 from seamuslee001/dev_core_1489_2

dev/core#1489 Include hook_civicrm_managed into the list of upgrade f…

Merge pull request #16477 from demeritcowboy/unit-test-env

[TEST] dev/core#1572 Unit test environment not working on windows after recent CodeGen updates

Fix calls to Request::retrieve

Do not pass in values that are set by default. Use retrieveValue  rather than passing inn the NULL object

APIv3 - Handle input errors

add column for report

Merge pull request #16522 from totten/master-simplecache-patchurl

dev/core#1562 - composer.json - Fix E2E tests run on D8 build (via "patches") #16427

Merge pull request #16512 from yashodha/header

Relationship report - add sort order for end date

(dev/core#1562) composer.json - Fix E2E tests run on D8 build (w/"patches" and URLs)

dev/core#1489 Include hook_civicrm_managed into the list of upgrade freindly hooks

Remove patch files that are now loaded from URL

The `composer.json` was updated to get these patch files via URL instead of
via local/relative path.

Future maintainers who wish to revise these patches will need to post new
revisions somewhere with a durable URL *before* they can update
`civicrm-core`.

Keeping the files in here wrongly implies that you can revise the files
in-situ - their presence is misleading.

Merge pull request #16519 from civicrm/5.23

5.23

Merge pull request #16520 from agh1/5.23.0-release-notes

5.23.0 release notes

Merge pull request #16517 from agh1/5.23.0-release-notes

5.23.0 release notes: initial run

Merge pull request #16518 from greenpeace-cee/fix-xoauth-imap

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

dev/mail#59 - Fix use of useXOAUTH2 constant instead of var

This fixes an issue where a missing $ before the useXOAUTH2 variable
prevents IMAP authentication with XOAUTH2 from working.

5.23.0 release notes: added boilerplate

Merge pull request #16513 from eileenmcnaughton/credit_note_setting

Move  settings definition on contribution settings form to metadata.

APIv4 - merge ActionUtil with Request::create

The Request::create function was broken for v4, and ActionUtil had been serving the same purpose but in a non-broken way.

5.23.0 release notes: raw from script

Move credit_note_definition on contribution settings form to metadata.

This moves the code that allows settings to be added to forms by metadata from the Generic setting form to the setting trait.

This means many more forms will support injecting settings by metadata. Specifically we switch across the Contribution preferences form,
making credit_note_prefix a metadata-added field. This is preliminary to moving that to an extension & allows us to inject it
in by simply defining the metadata in a hook

add sort order for end date

Merge pull request #16511 from colemanw/kernel

API Kernel - cleanup deprecated fn & unused param

Merge pull request #15859 from jitendrapurohit/dev-mem-17

dev/mem#17 - Deleting memberships does not delete its related line item

API Kernel - cleanup deprecated fn & unused param

Merge pull request #16510 from colemanw/ws

crmSelect2 - fix whitespace in tooltip

crmSelect2 - fix whitespace in tooltip

Merge pull request #16508 from colemanw/escape

Common.js - remove duplicate function

Merge pull request #16507 from colemanw/title

dev/core#1587 Select2 - Use description as title for each option

Common.js - remove duplicate function

This function did the same thing as the one in lo-dash

Select2 - Use description as title for each option

Merge pull request #16504 from acrollet/master

dev/financial#117 - Add link metadata for payment edit link

Merge pull request #16499 from pradpnayak/caseAuditReport

Added conditional check so that it can be altered by hook

Merge pull request #16505 from eileenmcnaughton/part_pay_dep

Add deprecation notices on PartialAmount params

Merge pull request #16497 from totten/master-pear-mail-delg

(REF) Move CIVICRM_MAIL_LOG logic from patch-files to wrapper-class

composer.json - Patch URLs should be more concrete, less symbolic

The previous commit made used patch URLs in the format:

`https://raw.githubusercontent.com/civicrm/civicrm-core/master/{FILEPATH}`

This is symbolic in that the `master` does not refer to a specific piece of content; it refers
subjectively to the current developmental code. This creates a few problems:

* When `civicrm-core` is tagged or branched, the `composer.json` still points
  to `master`.
* If you need to update the patch-file, you send in a PR with a different
  patch URL, right? But any builds based on this PR would still use the
  canonical `master` rather than the proposed revision.

Resolution: Don't point to patch-files symbolicly.  Instead, address them
specifically.  This revision uses past Github commits (because that's easy),
but it doesn't have to be in Github per se.  It should be somewhere that's
reliable and unlikely to change (preferrably with addressing by content
checksum).  In the Drupal community, you see a lot of examples pointing to
file-attachments in the issue-tracker.

LAB-1576 - Use URLs for patches, to avoid local path issues with composer-patches

Trying Github raw URLs this time ...

CRM_Utils_Mail::send - Adjust hack to work with filtered mailer

(REF) CRM_Utils_Mail - Split LoggingMailer into FilteredPearMailer and Logger

1. Captures more context (i.e. the original driver and params)
2. Changes various property names to avoid potential for conflict with delegated properties
3. Adds `addFilter($id, $func)` so that one can move more filters in here
4. Consolidates the various references ot CIVICRM_MAIL_LOG* into one file.

Merge pull request #16360 from artfulrobot/case-activities-setting

Add CiviCase option for showing case activities in normal views

Merge pull request #15410 from herbdool/cloud-21

dev/cloud-native#21 Don't cache the full path of extensions so they don't break with dynamic paths

Add deprecation notices on PartialAmount params

We've removed all the places that pass these in so these 2 places handling them are the only remaining references.

All calling code now uses Payment.create to add the payments - yay. Removing these fully will clean up the code
notably

CRM_Admin_Form_Setting_Smtp - Use the same mailer as CRM_Utils_Mail

When civicrm-core used patches to change the behavior of `Mail_mail` (etal),
this was fine.  The preceding commit removes the patches and uses a wrapper
instead - so it's more important to get the same mailer as the
regular CRM_Utils_Mail logic.

Add CiviCase option for showing case activities in normal views

Different aproach to tackling #16353

Fix array-to-string notice; Improve activity action links

remove upgrader - as per Pradeep's suggestion

update version added

tweak way description is gotten

update activity tests to check civicaseShowCaseActivities setting

Fix another array-to-string conversion error

Remove File On Case from activity View; add Manage Case

fix case activity links when using caseShowCaseActivities

remove ctx parameter for case activity links

various

US-Englishized Case settings texts.

fix permission name

Add link metadata for payment edit link

Merge pull request #16502 from yashodha/header

remove unnecessary file

Merge pull request #16487 from mlutfy/core1019

dev/core#1019 Calculate.tpl: fix the Total Amount currency formatting.

Merge pull request #16442 from eileenmcnaughton/pree

Fix backoffice participant partial payments to be stdised & not miscalculate net_amount

remove unnecessary file

Merge pull request #16500 from eileenmcnaughton/fatal

Remove fatal  from  form

Remove fatal  from  form

Merge pull request #16475 from JMAConsulting/core_1577

dev/core#1577 : Custom Group Types not filterable

Merge pull request #16465 from albionbrown/#1565-API-Exception

Throwing API_Exception is file fails to copy when creating attachment…

Added conditional check so that it can be altered by hook

(REF) Move CIVICRM_MAIL_LOG logic from patch-files to wrapper-class

Overview
--------

This aims to improve maintainablity and deployment workflows by reduing the
scope of the needed files.

It is an alternative approach for #16481.

Before
------

`civicrm-core` included patch files for `pear/mail`.  These files modified
the behavior of `Mail_sendmail`, `Mail_smtp`, and `Mail_mail` to ensure that
the `send()` function would abide by the `CIVICRM_MAIL_LOG` constant.

After
-----

`civicrm-core` includes a wrapper class `LoggingMailer`.  It takes an
instance of `Mail_sendmail`, `Mail_smtp`, or `Mail_mail` and mixes-in the
`CIVICRM_MAIL_LOG` logic.

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

There is a `hook_civicrm_alterMailer`, for which consumers could potentially
see a change in behavior (because the concrete-class of `$mailer` may
change and support different properties). Mitigating factors:

* There is only one public method in the `Mail` contract: `send()`.
* The only example of `alterMailer` in the documentation suggests
  that you wholly replace the object with your own implementation.
  This was the original purpose. If you use it this way, then it doesn't
  matter what's happening inside the old object.
  https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_alterMailer/
* In public `universe`, there is only one extension (Cividesk's Sparkpost)
  which uses this hook. And it does swap in that expected way.
* Just in case... the `LoggingMailer` passes-through all calls to read/write properties.
  You can see this in action in a specific configuration, e.g.
  ```
  ## Before patch
  $ cv ev '$m=Civi::service("pear_mail"); return [get_class($m), $m->host];'
  [
      "Mail_smtp",
      "127.0.0.1"
  ]

  ## After patch
  $ cv ev '$m=Civi::service("pear_mail"); return [get_class($m), $m->host];'
  [
      "CRM_Utils_Mail_LoggingMailer",
      "127.0.0.1"
  ]
  ```

Merge pull request #16494 from mattwire/enablevalidate

Enable jQuery validate on register/contribution forms

Merge pull request #16493 from mattwire/unusedcode

Remove unused code

Merge pull request #16492 from yashodha/header

fix headers

Enable jQuery validate on register/contribution forms

Remove unused code

fix contribution summary report's statistics when grouping and having

Merge pull request #16490 from seamuslee001/dompdf_update_script

[REF] Remove patch from dompdf cleanup script that is no longer needed

fix headers

[REF] Remove patch from dompdf cleanup script that is no longer needed

Merge pull request #16489 from civicrm/5.23

5.23

Merge pull request #16483 from seamuslee001/utf8mb4_min

[NFC] Skip utf8mb4 test on MySQL versions lower than 5.7

Merge pull request #13078 from agh1/contactdetail-no-or2

Improve activity query performance in Constituent Detail Report

Merge pull request #16459 from eileenmcnaughton/rec_ver

Make php 7.3 the recommended php version

Merge pull request #16486 from mfb/year-typo

Fix year typo.

dev/core#1019 Calculate.tpl: fix the Total Amount currency formatting (code reformatting).

dev/core#1019 Calculate.tpl: fix the Total Amount currency formatting.

Fix year typo.

dev/core#1577 : Custom Group Types not filterable

dev/mem#17 - Deleting memberships does not delete its related line item

Set version to 5.24.alpha1

Set version to 5.23.beta1

[NFC] Skip utf8mb4 test on MySQL versions lower than 5.7

Merge pull request #16485 from seamuslee001/master

release-notes: Late changes for 5.22 release notes

release-notes: Late changes for 5.22 release notes

Merge pull request #16457 from colemanw/api4doc

APIv4 documentation & code cleanup

avoid backslash problems

Update api/v3/Attachment.php to remove concatenation

Co-Authored-By: colemanw <coleman@civicrm.org>

Merge pull request #16472 from eileenmcnaughton/cn_for_now

Revert "Remove unreachable code lines."

Merge pull request #16466 from mattwire/recurringentity_copy

Call hook_civicrm_copy for RecurringEntity

Merge pull request #16471 from totten/master-psr-log

composer.json - Relax psr/log constraint. Improve D8 compatibility.

Revert "Remove unreachable code lines."

In the course of https://github.com/civicrm/civicrm-core/pull/16462
I concluded that there might be (untested scenarios) where this would still be hit.

In #16462 I adopted a cleaner (tested) approach. However, I think it's unlikely that
will be merged before 5.23 is cut so I propose reverting this change & re-reverting it
once the rc is cut as a more conservative approach. We can focus on getting #16462
wrapped up for 5.24

This reverts commit d7943ca7f7107c1466e687e876bddaf0f1998d99.

Merge pull request #16460 from colemanw/noInclude

APIv3 - Only scan files for deprecation checks in Entity.get

composer.json - Relax psr/log constraint. Improve D8 compatibility.

Overview
--------

This addresses a composer conflict that's reported when trying to install
on Drupal 8.7. For example, run these commands:

```
drush8 dl drupal-8.7.x
cd drupal-8*
composer require civicrm/civicrm-core:5.22.x-dev
```

This is a port of #16470 for `master`.

Before
------

The install fails because `civicrm-core` requires `psr/log:~1.1`, and something else
is prodding us to use `psr/log:1.0.2`.

```
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for civicrm/civicrm-core 5.22.x-dev -> satisfiable by civicrm/civicrm-core[5.22.x-dev].
    - Conclusion: remove psr/log 1.0.2
    - Conclusion: don't install psr/log 1.0.2
    - civicrm/civicrm-core 5.22.x-dev requires psr/log ~1.1 -> satisfiable by psr/log[1.1.0, 1.1.1, 1.1.2, 1.1.x-dev].
    - Can only install one of: psr/log[1.1.0, 1.0.2].
    - Can only install one of: psr/log[1.1.1, 1.0.2].
    - Can only install one of: psr/log[1.1.2, 1.0.2].
    - Can only install one of: psr/log[1.1.x-dev, 1.0.2].
    - Installation request for psr/log (locked at 1.0.2) -> satisfiable by psr/log[1.0.2].

Installation failed, reverting ./composer.json to its original content.
```

After
-----

It should work.  However, this is hard to demonstrate via `r-run` without merging.

Comments
--------

The substantive differences between `psr/log` in v1.0 and v1.1 relate to `LoggerInterfaceTest` and `TestLogger`:

https://github.com/php-fig/log/compare/1.0.2...1.1.2

However, `civicrm-core` does not use `LoggerInterfaceTest` or `TestLogger`, so it shouldn't matter.

For the standard tarballs which use `composer.lock`, this does have the side-effect of bumping up from 1.1.0 to 1.1.2.