Merge pull request #23909 from civicrm/5.51

5.51 to master

Merge pull request #23893 from eileenmcnaughton/user_two

dev/core#3514 Add interface to allow extensions that deal with imports to cope with 5.51

Merge pull request #23907 from eileenmcnaughton/up

5.51 to master

Merge branch '5.51' of github.com:civicrm/civicrm-core into up

Fix Parser classes to use ClassScanner

Merge pull request #23888 from eileenmcnaughton/user_import

Convert the type on the UserJob entity to be a string

Import Summary - Update to match final schema ('UserJob.job_type')

eg When running "Import Activities", the page-title for the summary is incorrect.

Merge pull request #23903 from eileenmcnaughton/551

Allow tests to pick up hook too

Merge pull request #23902 from civicrm/5.51

5.51 to master

Merge pull request #23889 from eileenmcnaughton/offline_cur

Add second currency example

Merge pull request #23901 from totten/nfc-install-docblock

(NFC) Expand docblocks for hook_install and hook_postInstall

Allow tests to pick up hook too

Merge pull request #23900 from totten/5.51-sticky-classloader

(dev/core#3660) CRM_Extension_ClassLoader - Defend against redundant refreshes

Merge pull request #23894 from eileenmcnaughton/off_it

Minor test cleanup

Examples - Title should differentiate EUR/CAD examples

Merge pull request #23891 from eileenmcnaughton/smarty

Reset smarty-security after an exception

(NFC) Expand docblocks for hook_install and hook_postInstall

Add second currency example

(dev/core#3660) CRM_Extension_ClassLoader - Defend against redundant refreshes

Overview
--------

This is a follow-up to #23824 (c24dd7db7e1e91120fd7daeb7e151f856d6b78c3) which addresses a
regressive edge-case.

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

* Write an extension like `wmf-civicrm` which (a) calls `System.flush` (`rebuildMenuAndCaches()`)
  during `hook_install` -- and then (b) loads some class from the same extension.
    ```php
    function foo_civicrm_install() {
      civicrm_api3('System', 'flush', []);
      CRM_Foo_Helper::doStuff();
    }
    ```
* Try to install the extension.

Before
------

Crashes on loading the class `CRM_Foo_Helper`

After
-----

Loads the class `CRM_Foo_Helper`.

Comments
--------

(1) To see what's happening, consider `CRM_Extension_Manager_Module::onPreInstall()`.
This registers the new classloader and then fires `hook_install` which eventually
fires `rebuildMenuAndCaches()`. With c24dd7db, this resets the classloader again.
But the extension isn't fully installed yet - so it forgets about the new extension.

(2) Is it safe to have some (temporarily) sticky items?  Ish.  You might
say: "Ah, but what if we need to remove an extension?  Won't this static
variable retain stale things?" Doesn't matter.  In PHP, classloading is a
one-way-street.  (You cannot unload.) So you'll still have old classes in
memory, regardless of whether the `ClassLoader` has some old metadata
about where to find classes.

(3) I'm on the fence about whether this patch is a good idea.  Calling
`System.flush` explicitly in this context seems like an invitation to
trouble.  OTOH, it worked before #23824, so it can be called a regression.

Merge pull request #23898 from civicrm/5.51

5.51 to master

Merge pull request #23897 from agh1/5.51.0-releasenotes-initial

5.51.0 release notes initial run

Set version to 5.51.beta2. Retargeg FiveFiftyOne::updateUserJobTable to beta2.

5.51.0 release notes: added boilerplate

Convert the type on the UserJob entity to be a string

Currently we are storing a numeric ID - however if we
permit non-core classes to register types we find numeric ids
quickly become hard to manage as uninstalling an extension could
change the id. This switches to using a string type

Minor test cleanup

Merge pull request #23852 from IProSoft/master

New domain token base_url

Merge pull request #23892 from eileenmcnaughton/order

Ensure apiv3 params are converted at the line item level

New domain token base_url

Change url retrival method for base_url token

Change base url token title. Fix test error

PHPUnit TokenConsCRM_Utils_TokenConsistencyTest update

Ensure apiv3 params are converted at the line item level

We switched to using apiv4 but at least one test seems to be passing in a string
at the line item level which is a bit weird but we should handle it

Reset smarty-security after an exception

This stops tests from getting befuddled

Merge pull request #23887 from civicrm/5.51

5.51 to master

Merge pull request #23854 from totten/master-mixin-wfmsg

Scan for classes based on the PHP interface (WorkflowMessageInterface, ExampleDataInterface)

Merge pull request #23886 from totten/master-act-dao

Activity - Regenerate DAO with current description

Merge pull request #23885 from totten/master-wfmsgtest

WorkflowMessageTest - Re-enable skipped test

Activity - Regenerate DAO with current description

ExampleDataInterface - Switch to ClassScanner (part 3, swap scanner)

ExampleDataInterface - Switch to ClassScanner (part 2, use new names)

ExampleDataInterface - Switch to ClassScanner (part 1, rename *.ex.php => *.php)

WorkflowMessage - Switch to `ClassScanner` (was: bespoke `include_path` search)

mixin/scan-classes@1 - Allow extensions to opt-in to class scanning

ClassScanner - Add utility for scanning classes based on PHP interfaces

CRM_Utils_System::flushCache - More consistent behavior. Prepare for interface-php test.

* The interface-php test registers an instance of WorkflowMessageInterface.
* The list of WorkflowMessageInterface's is stored in the 'long' cache.
* When you enable/disable an extension, it should clear WorkflowMessage list (and other things).
* Before this patch, interface-php's `LifecycleTest` is inconsistent: it passes with
  `testLifecycleWithSubprocesses()` and fails with `testLifecycleWithLocalFunctions()`.
* After this patch, interface-php's `LifecycleTest` is consistent: it passes with both
  `testLifecycleWithSubprocesses()` and  `testLifecycleWithLocalFunctions()`.
* The problem - while executing `testLifecycleWithLocalFunctions()`, it evidentally uses
  `Arraycache` as the backend -- and thus skips the flushes. However, I cannot fathom why
  one would want to clear-caches for SQL+memcache+redis but keep them for Arraycache.

WorkflowMessageTest - Re-enable skipped test.

This is a follow-up to #23811 (409b1e0c30d28179f6244c1f180838a473d7582c). The prior commit
changed the way it found examples - and caused it to skip some examples.

This re-enables the example and adds an assertion to ensure that it actually
finds some examples.

Merge pull request #23879 from mattwire/wpnotify

Enable the 'notify' parameter for WordPress user creation

Merge pull request #23881 from civicrm/5.51

5.51

Merge pull request #23874 from AlainBenbassat/empty-weight-column-during-upgrade

returns weight 0 when weight column name is empty

Merge pull request #23722 from jensschuppe/fix/multipleLogfilesMultilanguage

Avoid multiple log files in multi-language environments

Merge pull request #23878 from herbdool/issue-3691

dev/core#3691: better description of activity table columns

Merge pull request #23876 from colemanw/checkRecentItemsPerms

RecentItems - Enforce permissions before adding item

Merge pull request #23602 from kainuk/core-3369

Fixes core#3369 scheduled Reminder limited by Participant Role fails if any participant has multiple roles

Enable the 'notify' parameter for WordPress user creation

Issue 3691: better description of activity table columns

5.51.0 release notes: raw from script

RecentItems - Enforce permissions before adding item

Merge pull request #23856 from eileenmcnaughton/pref

dev/core#2866 ignore preferred_mail_format, send both

returns weight 0 when weight column name is empty

Merge pull request #23873 from colemanw/uiDialogBackground

dev/core#3432 UI Dialog - Fix transparent background caused by certain other plugins

dev/core#2866 ignore preferred_mail_format, send both

Merge pull request #23872 from demeritcowboy/valuestyle

Unassigned variable in invoice message template

UI Dialog - Fix transparent background caused by certain other plugins

Fixes dev/core#3432
It was reported that certain Drupal modules that include a copy of jQuery UI
cause the background to become transparent. This should prevent that.

Merge pull request #23831 from mlutfy/purgeMenus

navigation: cleanup less frequently used New items

Unused variable. Meaningless comment.

Merge pull request #23829 from eileenmcnaughton/inv_lines

Replace more assigns with tokens in invoice, add tests

Swap out some additional assigns for tokens, improve tests

Merge pull request #23868 from civicrm/5.51

5.51 to master

Merge pull request #23867 from demeritcowboy/currency551

dev/core#3680 - Currency incorrectly displayed for price set when default is not USD

navigation: cleanup less frequently used New items

use the actual currency if there is one not the default

Merge pull request #23866 from civicrm/5.51

5.51 to master

Merge pull request #23865 from colemanw/behalf

dev/core#3675 Fix Permissioned Relationship pseudoconstant

dev/core#3675 Fix Permissioned Relationship pseudoconstant

Fixes dev/core#3675 Permissioned Relationship no longer set up
for "On Behalf of Organization" Membership Contribution Form submissions

Merge pull request #23860 from civicrm/5.51

5.51

Merge pull request #23847 from eileenmcnaughton/sync

Attempt to make function more readable by extracting in-line function

Merge pull request #23839 from eileenmcnaughton/import_outy

dev/core#3665 import summary fixes

Merge pull request #23849 from eileenmcnaughton/import_55

dev/core#3673 do not attempt to import empty related contact

Merge pull request #23855 from mlutfy/fixAccentsIcalAgain

dev/core#1541 Fix ICalendar random invalid utf8 (PHP <7.4 compat) and tests

Merge pull request #23836 from stesi561/dev/core/#3671

Refs: dev/core#3671 Fix regression involving CiviCRM Webform + Cases.

Attempt to make function more readable by extracting in-line function

Merge pull request #23858 from seamuslee001/guzzle_658

[REF] Update Guzzle to be 6.5.8

Merge pull request #23846 from seamuslee001/fix_warning_karma

[REF] Follow on from #23749 and remove refernece to ui-utils from kar…

[REF] Update Guzzle to be 6.5.8

Merge pull request #23850 from demeritcowboy/rsspubdate

Alternate to 23826 - Events RSS feed does not output a pubDate for each Event

Merge pull request #23853 from pradpnayak/updatecasefixes

Fixed fatal error on update multiple case

dev/core#1541 Fix ICalendar random invalid utf8 (PHP <7.4 compat) and tests

Fixed fatal error on update multiple case

dev/core#3673 do not attempt to report empty related contact

Merge pull request #23834 from colemanw/deleteCustomOption

Call hooks when deleting an option value from CustomOption BAO

Merge pull request #23840 from mlutfy/fixAccentsIcal

dev/core#1541 Fix ICalendar random invalid utf8

CIVICRM-1998 CiviCRM Events RSS feed does not output a pubDate for each Event, use the Event Start Date as the pubDate which is generally how other Event systems use RSS

Merge pull request #23822 from MegaphoneJon/3661-551

fixes core#3661: Don't check API permissions in CRM_Mailing_BAO_Mailing::getGroupNames()

[REF] Follow on from #23749 and remove refernece to ui-utils from karama conf

Merge pull request #23843 from seamuslee001/master

5.51

Merge branch '5.51'

Merge pull request #23842 from seamuslee001/regen_3658

[REF] dev/core#3658 Regenerate civicrm_generated to ensure that group…

[REF] dev/core#3658 Regenerate civicrm_generated to ensure that grouping is added on installs

Merge pull request #23841 from civicrm/5.51

5.51 to master

Add related contacts to the group too, per prior version

dev/core#3665 import summary fixes

m

fixes core#3661: Don't check API permissions in CRM_Mailing_BAO_Mailing::getGroupNames()

dev/core#1541 Fix ICalendar random invalid utf8