Merge pull request #16684 from civicrm/5.23

5.23

Merge pull request #16686 from demeritcowboy/install-seqcreditnotes

Fix parameter format for upgrade call to install/enable sequentialcreditnotes

Merge pull request #16687 from demeritcowboy/letter-h

[NFC] Add missing letter "h" in upgrade script for task description

Merge pull request #16685 from mlutfy/fixSettingsYesNo

Fix SettingTrait YesNo translation

Merge pull request #16680 from magnolia61/Contribution_Invoice_Enhancements

Style & layout clean up

Merge pull request #16683 from demeritcowboy/disable-xoauth2

dev/mail#59 - Don't release 5.23 with XOAUTH2 option enabled since it isn't ready yet

fix parameter format

add missing letter h

Merge pull request #16020 from mattwire/setentityid

Add setEntityId() to entityForm

Fix SettingTrait YesNo translation

Style & layout clean up

Merge pull request #16682 from colemanw/remove

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

Update unit test now that the xoauth2 option is disabled for the time being

Merge pull request #16681 from alifrumin/rn5.23secondpass

rn-5.23: adding late changes to the 5.23 release notes [nfc]

xoauth2 isn't ready yet

Remove deprecated function CRM_Contact_BAO_GroupContactCache::remove

adding late changes to the 5.23 release notes

Merge pull request #16675 from christianwach/lab-1628

Fix display of administrator permissions in WordPress Multisite

Fix display of administrator permissions in WordPress Multisite

Merge pull request #16676 from christianwach/lab-1629

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16679 from pradpnayak/priceSetSchema

added sudo constant for sms api type

added sudo constant for sms api type

Fix synchronisation of Users to Contacts in WordPress Multisite

Merge pull request #16672 from MegaphoneJon/core-1627

Activity Details report: fix filtering by is null/is not null

Activity Details report: fix filtering by is null/is not null

Merge pull request #16667 from eileenmcnaughton/acl

[REF] Only call getACLs when contact_id is present, remove handling

Merge pull request #16566 from eileenmcnaughton/setting_fix

Remove helper function now that contribution settings is not weirdly stored

Merge pull request #16647 from JMAConsulting/core-1623

core#1623: My Case dashlet doesn't sort by name but contact_id instead

Merge pull request #16669 from demeritcowboy/audit-tpl-2

dev/core#1366 - CRM_Case_XMLProcessor_Report::run() is never called anymore

Merge pull request #16668 from eileenmcnaughton/php

Increase php min recommended version

remove unused function

core#1623: My Case dashlet doesn't sort by name but contact_id instead

Increase php min recommended version

We seem to not be doing a good job of communicating about dropping php 7.0
but given php 7.1 is now out of support https://www.php.net/supported-versions.php
it seems pretty clear it meets no longer meets the criteria for 'minimum recommended'

https://www.php.net/supported-versions.php

[REF] Only call getACLs when contact_id is present, remove handling

getACLs is called from one place in the code. Within the function there are 2 places that
handle the possibility it might be null - this moves it to the calling function
& requires contact id to be an int

Merge pull request #16665 from pradpnayak/priceSetSchema

Add pseudoconstant for priceset

Merge pull request #16664 from totten/master-ext-hidden

Extensions - Change new tag "hidden" to "mgmt:hidden"

Merge pull request #16624 from mattwire/tokencompatsubscriber

TokenProcessor - fix greetings tokens

Add pseudoconstant for priceset

Extensions - Change new tag "hidden" to "mgmt:hidden"

Overview
--------

This is follow-up from the discussion about how to name extension tags

https://github.com/civicrm/civicrm-dev-docs/pull/761

It's a follow-up #16531 (dev/financial#84) within the same 5.24.alpha1 dev cycle.

Before
------

The `sequentialcreditnotes` extension is tagged `hidden` and omitted
from display in the web UI.

After
-----

The `sequentialcreditnotes` extension is tagged `mgmt:hidden` and omitted
from display in the web UI.

Merge pull request #16662 from seamuslee001/tcpdf_depreaction

[REF] Deprecate _html2pdf_tcpdf function in favour of _html2pdf_dompdf

Merge pull request #16663 from eileenmcnaughton/acl

[REF] Removed unused function

Merge pull request #16655 from seamuslee001/hook_uf_fields

[Feature] Add in new hook alterUFFields to allow extensions to modify…

Add in code doucmentation of the format of the fields array and add in a unit test

[REF] Removed unused function

I did some searches - this isn't called

Merge pull request #16660 from demeritcowboy/audit-tpl

dev/core#1366 - Case Audit.tpl file no longer used

[REF] Deprecate _html2pdf_tcpdf function in favour of _html2pdf_dompdf

Merge pull request #15964 from mlutfy/financial109

dev/financial#109 Fix country/province assignation in the contribution invoice

Merge pull request #16648 from pradpnayak/priceSetSchema

Add pseudoconstant for priceset

Merge pull request #16661 from seamuslee001/typo_fix

Fix unreleaesd typo error

Fix unreleaesd typo error

Merge pull request #16658 from mattwire/fixtypo

Fix typo (unrelease regression)

Add pseudoconstant for priceset

dev/financial#109 Fix country/province assignation in the contribution invoice

remove files no longer used

Fix typo (unrelease regression)

Merge pull request #16657 from mattwire/fixcomment

Fix mistake in comment

Fix mistake in comment

[Feature] Add in new hook alterUFFields to allow extensions to modify which fields can be added to a profile

Merge pull request #16652 from UshaMakoa/patch-2

Update contributor-key.yml

Merge pull request #16653 from seamuslee001/hook_pre_post_uf_field

[REF] Add in pre and post hooks to UFField Entity

Update TokenTest.php

Merge pull request #16644 from seamuslee001/extension_notices

[NFC] Use insert ignore for inserts into civicrm_extension to stop warnings on duplicate entry for sequential credit notes extension

[REF] Add in pre and post hooks to UFField Entity

Update contributor-key.yml

Merge pull request #16632 from colemanw/dash

News dashboard - Code cleanup to update js & css

Merge pull request #16645 from colemanw/null

APIv4 - Correctly return null values from DAO save actions

Add basic test for TokenCompatSubscriber

APIv4 - Correctly return null values from DAO save actions

Make sure the value "null" is not returned as a string
Don't return null values from saved DAO objects unless it is in params or reload is set to true

[NFC] Use insert ignore for inserts into civicrm_extension to stop warnings on duplicate entry for sequential credit notes extension

Merge pull request #16615 from MegaphoneJon/selfsfc-refactor

[REF] Extract self-service eligibility code into its own function

Merge pull request #16627 from seamuslee001/aug_custom_fields_fix

[REF] Apply fix for CRM-607 for when building the select section of t…

Merge pull request #16638 from UshaMakoa/patch-1

Contact Type Values with Cap in order to be well translated with ts()

Merge pull request #16643 from demeritcowboy/activity-summary-groupby

dev/core#1619 - Fatal error when grouping Activity Summary report by activity date

Merge pull request #16642 from eileenmcnaughton/ss2

[REF] simple function extraction

Merge pull request #16595 from seamuslee001/refactor_payment_processor_radio

[REF] Refactor adding payment processor radio section onto register a…

Merge pull request #16637 from eileenmcnaughton/ss

Remove unused columns from civicrm_saved_search

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

[REF] simple funnction extraction

[REF] Extract self-service eligibility code into its own function

fatal error when grouping by activity date

Remove unused columns from civicrm_saved_search

We identified these as unused in 5.20 & removed the places still saving to them -
https://github.com/civicrm/civicrm-core/pull/14891

This removes the actual columns. A couple of other tables still have similar columns, not sure
if they are required still

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`

Contact Type Values with Cap in order to be well translated with ts()

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'

News dashboard - Code cleanup to update js & css

- Use css class instead of inline style for unread items
- Use `CRM.cache` instead of `window.localStorage`
- Longer preview snippet for wide screens

Merge pull request #16633 from alexymik/patch-1

CommunicationPreferences 'loclize' -> 'localize'