Merge pull request #18903 from eileenmcnaughton/aip2

Remove deprecated code

Merge pull request #18807 from jvos/postProcess-MessageTemplates

Add postProcess hook to MessageTemplates

Merge pull request #18897 from eileenmcnaughton/ip3

[REF] Remove silly if

Merge pull request #18902 from totten/master-altermailstore

dev/core#2141 - Add hook_civicrm_alterMailStore

Remove deprecated code

These few lines have had any meaning removed already....

Merge pull request #18852 from eileenmcnaughton/aip

[Ref] Move sending the email back out of the recur function

dev/core#2141 - Add hook_civicrm_alterMailStore

Overview
--------

The CiviCRM "MailStore" layer is used for importing email messages from
various sources (IMAP, POP3, Maildir, etc).  There is a built-in list with a
handful of drivers (`CRM_Mailing_MailStore_Imap`, etc).

This patch adds a hook for manipulating those drivers.

Before
------

It's not possible for an extension to add a driver, modify a driver, etc.

After
-----

It is now possible to add/modify/replace drivers. Here are two examples.

(1) To supplement the IMAP authentication with a dynamic token for XOAuth2:

```php
function hook_civicrm_alterMailStore(&$mailSettings) {
  if (...$mailSettings requires oauth...) {
    $mailSettings['auth'] = 'XOAuth2';
    $mailSettings['password'] = $myOauthSystem->getToken(...);
  }
}
```

(2) To add a new protocol `FIZZBUZZ`, you would:

1. Register a value in the OptionGroup `mail_protocol`.
2. Create a driver class (eg `CRM_Mailing_MailStore_FizzBuzz` extends `CRM_Mailing_MailStore`)
3. Use the hook to activate the class:

  ```php
  function hook_civicrm_alterMailStore(&$mailSettings) {
    if ($mailSettings['protocol'] === 'FIZZBUZZ') {
      $mailSettings['factory'] = function ($mailSettings) {
        return new CRM_Mailing_MailStore_FizzBuzz(...);
      };
    }
  }
  ```

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

This adds a unit-test with examples of basic/non-hook behavior and hooked behavior.

In reading the diff for `getStore()`, note that it  previously had a long
'switch()' to handle instantiation.  I tried to make the change in a way that
you could see some continuity - e.g. it's still the same basic `switch()`.

The change is to basically wrap the bits inside a function:

* Before: `case 'FOO': return new Foo(...)`
* After: `case 'FOO': return ['factory' => function(...) { return new Foo(...); }]`

[REF] Remove silly if

Checking component AND contributionRecur is silly - the latter will only
be set if relevant so remove the extra if

Merge pull request #18893 from civicrm/5.31

5.31 to master

Merge pull request #18877 from seamuslee001/dev_core_2149

dev/core#2149 Ensure that if a chained select field is required and t…

Merge pull request #18881 from eileenmcnaughton/cancelb

dev/core#927 Move handling of participant.cancel from BaseIPN to  contributioncancelactions

Merge pull request #18882 from eileenmcnaughton/pps

[Ref] Use direct version of participant id

Merge pull request #18892 from eileenmcnaughton/uf

Hack away at false negative test fails

Merge pull request #18891 from civicrm/5.31

5.31

Merge pull request #18890 from seamuslee001/dev_core_2154

dev/core#2154 Fix the output of the full text custom search form

Merge pull request #18887 from colemanw/entityPaths

Add entity paths to schema & APIv4

Hack away at false negative test fails

Merge pull request #18889 from demeritcowboy/fulltext-attributes

dev/core#2154 - Pass correct parameters to CRM_Core_Form::add() in fulltext custom search

Merge pull request #18876 from colemanw/searchDisplay

Lotsa new features for the Search Kit extension

dev/core#2154 Fix the output of the full text custom search form

Merge pull request #18888 from colemanw/deprecateCrudLink

Deprecate hook_civicrm_crudLink

Merge pull request #18885 from totten/master-add-mail

dev/core#2141 - "Add Mail Account" - Allow hookable listing of setup links

pass correct parameters

Move handling of participant.cancel to contributioncancelactions extension

Deprecate hook_civicrm_crudLink

The hook is not used anywhere in civicrm_universe, and the function calling it is only used in 1 place,
full-text search for generating links to file entities.

Merge pull request #18886 from mlutfy/moneyTypo

(NFC) Fix typo in Money valueFormat depretation warning

Mark version beta2 & add upgrade script

Add entity paths to schema & APIv4

Entity paths can be used to construct a url to create/view/edit/update a given entity.
The paths are local and contain tokens which would need to be replace, e.g. [id] or [contact_id].
The token name corresponds to the field name whos value is requred.

Merge pull request #18884 from jaapjansma/dev_2152_master_2

/dev/core#2152: Merge contacts keep case roles

Merge pull request #18883 from eileenmcnaughton/ppp

Remove always-true & otherwise silly if

(NFC) Fix typo in Money valueFormat depretation warning

dev/core#2141 - "Add Mail Account" - Allow hookable listing of setup links

Overview
--------

For certain types of mail accounts -- such as Google Mail and Microsoft
Exchange Online -- the setup process may require interaction with a remote
web-service.  The web-service provides some critical details (like
authentication tokens) and some handy details (like their username / email
address).

If the supports one of these services, then this revision will enable a
setup process.  It alters the "Add Mail Account" action to use a more
fine-tuned procedure.

Before
------

* Navigate to "Administer => CiviMail => Mail Accounts".
* Below the table, there is a singular button "Add Mail Account".
* Click the button.
* It opens an empty form for configuring the account.

After
-----

By default, the UX is the same.  However, if you have an extension like
`oauth-client`, then it changes:

* Navigate to "Administer => CiviMail => Mail Accounts".
* Below the table, there is a select box for "Add Mail Account". It lists different account types.
* Choose one of the options from the dropbox. The step depends on...
    * If you choose "Standard Mail Account", it opens the empty config form.
    * If you choose "Microsoft Exchange Online", it redirects to MS to get authorization
      from the user. Then, it redirects and prefills the config form.

Fix for dev/core#2152

Remove always-true & otherwise silly if

[Ref] Use direct version of participant id

ids['Participant'] is taken from the url. It is then passed to loadRelatedObjects
where
https://github.com/civicrm/civicrm-core/blob/0bd5e6bfbe8f339dbe066e3e96e00760c93a57e2/CRM/Contribute/BAO/Contribution.php#L2890
loads an object based on it. We then set ids too the object's id - which is a long way around to
using the paramter we started from. This does that

Merge pull request #18879 from colemanw/fixDAOStyle

Fix extension generated DAO files to pass civilint

Fix extension generated DAO files to pass civilint

Search ext: Add option to format table cell as link in search display

Search ext: decouple actions, add searchKit base module, rename searchAdmin module to crmSearchAdmin

Search ext: Improve styling

Search ext: Dynamic templates and a preview button for displays

Search ext: Support for embedding search displays in afforms

Search ext: Link to displays from admin screen

Search ext: Exclude settings from being html-encoded in db

Search ext: Add searchDisplay and searchPage modules

Search ext: group tab

Search ext: Rename component crmSearch to crmSearchAdmin

Search ext: Add vertical tabs

Search ext: Rework URL routing in preparation for search display management

Having the API params in the URL bar (like the API Explorer) was kinda neat but isn't going to work
with more complex management of multiple displays on the same screen.
This simplifies the routing to standard CRUD routes of /list, /create/:type, & /edit/:id

Search ext: Improve wording of update & delete actions

Search ext: Add list view

Search ext: Add SearchDisplay database entity

dev/core#2149 Ensure that if a chained select field is required and there are no valid options found ensure that jquery validation doesn't block submission

Merge pull request #18875 from colemanw/5.31-merge

Merge branch 5.31 into master

Merge pull request #18874 from mlutfy/totalParticipantTs

Participant Calculate/Fees: fix ts usage, simplify wording

Merge branch '5.31' into master

Merge pull request #18712 from eileenmcnaughton/locev3

dev/core#1879 Fix inconsistent ability to view event information

Merge pull request #18867 from sunilpawar/sms_provider

Show only Active SMS provider List on Mass SMS form

Merge pull request #18873 from colemanw/title_plural

APIv4 - Fix nonstandard camelCase of title_plural field

Participant Calculate/Fees: fix ts usage, simplify wording

Merge pull request #18872 from seamuslee001/dev_core_2150

dev/core#2150 Re-enact the recapture validation by validating the res…

APIv4 - Fix nonstandard camelCase of title_plural field

dev/core#2150 Re-enact the recapture validation by validating the response token on recapture

Ensure the form rule is actually triggered not just registered

Merge pull request #18871 from totten/master-api-single

APIv4 - Add `$result->single()` helper

Merge pull request #18870 from totten/master-api-al

ClassLoader - Fix autoloading of `API_Exception`

APIv4 - Add $result->single() helper

ClassLoader - Fix autoloading of API_Exception

Merge pull request #18306 from mattwire/refactororderapientityparams

Refactor entityParams in Order.Create API so it is easier to understand/modify

Merge pull request #18840 from yashodha/activity_report

(core/dev#2104) Add gender column for target contact to activity report

Merge pull request #18853 from eileenmcnaughton/base

dev/core#927 Move cancelling of related memberships to extension from BaseIPN

Merge pull request #18849 from eileenmcnaughton/finacl2

Move filtering of unpermitted options for reports/ search select to financialacl extension

Show only Active SMS provider List on Mass SMS form

Move filtering of unpermitted options for reports/ search select to financial acl extension

Merge pull request #18862 from demeritcowboy/mb-substr

dev/core#2146 - Long unicode contact names get truncated badly causing a crash

Merge pull request #18860 from colemanw/fixEncode

APIv4 - Fix type coersion of non-string input

Merge pull request #18856 from eileenmcnaughton/int_amount

Remove meaningless legacy code

truncate unicode strings better

Merge pull request #18859 from masetto/italian_provinces

updated italian provinces (CRM-5048)

Merge pull request #18861 from civicrm/5.31

5.31

Merge pull request #18792 from eileenmcnaughton/status2

Fix campaign_id handling for batch entry

Merge pull request #18857 from eileenmcnaughton/sort2

dev/core#2132 Fix for sort on search results

Refactor entityParams in Order.Create API so it is easier to understand/modify

dev/core#2132 Fix for sort on search results

Simpler fix - accept bad sql but catch it :-(

Ideally we would always be in exception mode

APIv4 - Fix type coersion of non-string input

Merge pull request #18808 from jvos/set-id-postProcess-mailing-Component

Set id after save for the mailing component in the postProcess

updated italian provinces (CRM-5048)

updated italian provinces (CRM-5048)

Add postProcess hook to MessageTemplates

Added comments and removed withespace.

Set id after save for the mailing component in the postProcess

Added comment and removed whitespace

Removed empty line.

Remove meaningless legacy code

I looked around & I am pretty sure int_amount is never set and init_amount is never used

Move sending the emaill back out of the recur function

This allows us not to pass in extra ids that are only required for that use and
steps us closer to not loading them

Merge pull request #18848 from eileenmcnaughton/finacl

Move CRM_Member_BAO_MembershipType::getPermissionedMembershipTypes to financial acl extension

Fix campaign_id handling for batch entry

Fixes a bug whereby campaign_id is not updated on batch entry if it has been added to the profile.

The underlying issue appears to be an  attack of copy and paste. I tidied up the variable
in both the places that call the processMembership function such that it is passed in
as an array of pass-through params. The function can be simplified & eventually removed.

Merge pull request #18745 from seamuslee001/backdrop_session

backdrop#116 Override sessionStart function and use backdrop function…

dev/core#927 Move cancelling of related memberships to extension from BaseIPN

This moves the code to cancel memberships on a related contribution cancel to the contributioncancelactions
core extension.

Before

The cancellations are done using convoluted & elsewhere duplicated logic in BaseIPN
using convoluted input params

After
When a contribution is updated & the status_id of 'Cancellled' is set then
the hook will kick in, look for any related pending memberships and cancel them.
The test demonstrates that the api call will ensure related pending memberships
are cancelled.

Note that I am using line_items table to get the membership - we have been storing this
membership_payment link there for log enough it should be reliable but if there
are remaining data issues then this is a lower risk flow to flush them out.

Also note that once this is cleaned up I'll move to the 2 other flows I'm aware of
Order.create api & Contribuion form edits

Merge pull request #18842 from ixiam/dev-#2138

Get phone_type by name. dev/core#2138

Merge pull request #18850 from aydun/user-interface#34

dev/user-interface#34 Replace <th> with <td> on ContributionPage configuration Amount tab

Merge pull request #18835 from eileenmcnaughton/aip

dev/financial#152 [REF] Clean up interaction with contribution recur object

Merge pull request #18813 from eileenmcnaughton/base

dev/core#927 preliminary cleanup

Replace <th> with <td> on ContributionPage Amount tab