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

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

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

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

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

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.

Merge pull request #16458 from twomice/1573_extra_columns_contribution_detail_report

Fix for #1573: Extra columns for Contribution Detail report.

APIv4 Organize debug sql into a single array

Merge pull request #16469 from civicrm/5.22

5.22 to master

Merge pull request #16464 from WeMoveEU/core-1564-against-5.22

dev/core#1564 Fix category name for a/b test temporary table

Merge pull request #16463 from eileenmcnaughton/obscure

Fix obscure bug

Call hook_civicrm_copy when we copy entities using RecurringEntity

Throwing API_Exception is file fails to copy when creating attachment with APIv3

dev/core#1564 Fix category name for a/b test temporary table

Fix obscure bug

After upgrading to MacOS Catalina & MAMP 5.6 (latest) with php version 7.4.1 or 7.3.9
my site was unable to load CiviCRM with a segfault error. I traced it to this line & found that
phpstorm was highlighting the line it red as possibly subject to https://bugs.php.net/bug.php?id=62577

- taking phpstorm's recommendation allowed the site to load - the failure had been on parsing info.xml files

Merge pull request #16461 from eileenmcnaughton/api_test

Add require_once to api3TestTrait

Add require_once to api3TestTrait

I used this trait from an extension but it was used with apiv4 before apiv3 (it supports both).

I got an error because utils.php is not included by this file. It 'freeloads' off the fact apiv3 initialises
the file - but in fact the file that includes a function requiring a function
should have the require once - reduces risk & cognitive load

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

Make php 7.3 the recommended php version

We definitely should be recomending the latest version we know it works on

APIv4 - Pluralize entity name in docblocks

Fix for #1573: Extra columns for Contribution Detail report.

Add additional test and fixes to ensure that results are the  same when

1) creating a partially paid registration with a pending contribution & then adding a payment to  that contribution
2) creating a partially paid registration with a completed contribution for a partial amount
3) created a pending registration with a pending contribution & adding a payment to the contribution.

Note that the following bugs previously affected these flows
 - flow 1 & 2 net_amount was incorrect
 - flow 1, an extraneous payment was added
 - flow 3, participant status was not updated from pending to partially paid on adding a payment

Use correct...er flow to create partial payments on events

This fixes the methodology for recording partial payments on the backoffice contribution form to create a pending contribution (order)
and then add a payment. The previous method was our first cut at supporting partial payments (before the Order+Payment flow) and
relies on some handling in the BAO which is crufty & has proven unreliable. This allows us to deprecate that handling & remove the
now unused addPayments function.

Prior to this PR I wrote a test for current behaviour & I had to alter that test for this as follows

1) prior to this change net_amount was clearly miscalculated & hence this fixes & I updated the test
2) prior to this change a financial transaction was created for the contribution. Now only the payment financial transaction is made.
3) prior to this change the financial item was set to partially paid, not it is still 'unpaid'

Items 1 is clearly a bug fix whereas 2 & 3 are a bit borderline.

Item 2 is the fact that for pending contributions (in general) we don't create financial items when creating them. We do for (some) other
statuses. This was  discussed here https://github.com/civicrm/civicrm-dev-docs/issues/712#issuecomment-545164618 & the summary was
'it would be better to change this but this is what was the scope /how it is'

Item 3 is discussed in https://docs.civicrm.org/dev/en/latest/financial/financialentities/ with the summary  being that in any other path
to creating a partial payment the financial item would be left at 'Unpaid' - hopefully until it's fully paid.

Item 1 is a clear bug fix & I think the fact no-one picked it up is material because it gives us a clue as to how many
people use this rather obscure flow that requires an admin user to alter the status to 'partially  paid' and alter the amount.

Given the above we have a choice
1) Make this flow behave like other flows, despite this flow being arguably correct or
2) Hack away to keep the current quirks

I've come down on the side of 1 as a standardisation. I think changing it so items 2 & 3 behave differently is worth pursuing but it's better
for this edge case flow to do the same as our approved flow & if we want to iterate on that we can

Merge pull request #16441 from eileenmcnaughton/event_pre

[REF] Interim code cleanup - make the usage of addPayments clearer

Api4 Replace - simplify by removing unnecessary fallback

Clearer docblock description of the defaults param

APIv4 Explorer - use OOP format for chained api calls

Passing an array into setChain works, but the more "pure" way is to pass an object into addChain.

Merge pull request #16446 from mattwire/wp_passlanguage

Pass language code to CiviCRM for Wordpress

Merge pull request #16397 from eileenmcnaughton/cont_setting3

dev/core#1558 retrieve setting credit_notes_prefix as  a 'real' setting.

Merge pull request #16449 from colemanw/docVars

APIv4 - Variable substution in docblocks

Merge pull request #16454 from colemanw/explorerDefaults

APIv4 Explorer - Better UI for defaults param

APIv4 Explorer - Better UI for defaults param

Merge pull request #16452 from mattwire/tokenrow_docblock

NFC Fix TokenRow docblock

Fix TokenRow docblock

APIv4 - Variable substution in docblocks

Replaces $ENTITY and $ACTION in docblocks to improve help text in the api explorer.

Merge pull request #16450 from eileenmcnaughton/ev_test

[NFC] Minor test cleanup.

Minor test cleanup.

Merge pull request #16438 from eileenmcnaughton/ev_clean

[REF] Pass params into function rather than this weird check for whether params exists.

Merge pull request #16447 from samuelsov/lab#1319

dev/core#1319 Fix regression with shared household address

dev/core#1319 UI - different label for household and employer

Pass language code to CiviCRM

Merge pull request #16267 from jitendrapurohit/dev-1471

dev/core#1471 - Add system status alert for deleted custom fields in …

[REF] Interim code cleanup - make the usage of addPayments clearer

This brings the condition back to the calling function which clarifies what is going on with it

Merge pull request #16437 from eileenmcnaughton/ev_back

Add unit test to back office form when a payment is made against a partially paid contribution

Merge pull request #16439 from eileenmcnaughton/part_odd

Respect form value for register_date on partipant form

dev/core#1558 retrieve credit_notes_prefix as a 'real' setting.

    This is part of work to convert the non-standard contribution_invoice_settings array of settings into settings in line
    with our standards

Extend checks in test

Merge pull request #16425 from colemanw/markdown

Use markdown in php docblocks & display in APIv4 Explorer

Merge pull request #16098 from MegaphoneJon/event-23-correct

event#23 - don't count non-counted participants on Confirm Registrati…

Respect form value for register_date on partipant form

The backoffice event registration form has a register_date field. In credit card mode it
does an awful lot of handling which is mostly for legacy code reasons but the net result is to ignore it
& set it to now. However, the field defaults to now & if someone actually changed it they would expect their change to
be respected. This removes that handling & just uses the field value entered. The field is always present from my
digging

Array formatting

[REF] Pass params into function rather than this weird check for whether params exists.

This function used to be shared with the online form & hence this form had to work to make itself 'look' like the other form.

It is now free to be it's own form

Merge pull request #16434 from eileenmcnaughton/cont_form_settings

[REF] Remove unused variable

Add unit test to back office form when a payment is made against a partially paid contribution

Merge pull request #16374 from colemanw/api4Delete

APIv4 - don't throw exception when updating/deleting 0 items

Merge pull request #16432 from colemanw/comments

Cleanup API_Exception usage