Merge pull request #23111 from eileenmcnaughton/silly3

Cleanup unnecessary variables

Merge pull request #23110 from eileenmcnaughton/silly2

Remove unnecessary pass-by-ref

Merge pull request #23113 from eileenmcnaughton/silly5

Remove unused variable

Merge pull request #23112 from eileenmcnaughton/import

Test cleanup (minor)

Merge pull request #23108 from eileenmcnaughton/silly

Remove double return statement

Remove unused variable

Test cleanup

Simplifies test set up of options & ensures they are deleted

Merge pull request #23106 from eileenmcnaughton/import

Test cleanup, deprecate ContributionSoft::retrieve

Cleanup unnecessary variables

Merge pull request #23109 from seamuslee001/master

5.48

Merge in 5.48

Remove unnecessary pass-by-ref

Merge pull request #23102 from seamuslee001/dompdf_update

Update DOMPDF to 1.2.1 and update composer-compile-plugin as per master

Merge pull request #23076 from eileenmcnaughton/cont_status

dev/financial#186 Do not display partially paid & partially refunded for selection on edit

Remove double return statement

Merge pull request #23105 from colemanw/grantViewOptions

CiviGrant - Remove non-functional setting for grants tab

Test cleanup

This removes the only call to ContributionSoft::retrieve that I could
find in core, hence I added a deprecation notice to the function.

There could be some extension uses - although it would never have
been supported to call this rather than the api

Merge pull request #23040 from eileenmcnaughton/display_opt

Stop passing settings into getTaxLabel

Merge pull request #23008 from eileenmcnaughton/temp_selected

Re-fix enotice on templateSelected

CiviGrant - Remove non-functional setting for grants tab in `contact_view_options`

Fixes dev/core#3159

The status of the tab can be controlled at a coarse level by disabling/enabling the CiviGrant extension,
and at a finer level with the Contact Layout Editor extension, making this option unnecessary.
Since it's broken, it seemed best to remove it.

Merge pull request #22992 from eileenmcnaughton/billingnot

Fix smarty notices in billing form

Merge pull request #23039 from eileenmcnaughton/inv2

Access invoice_prefix setting directly

Merge pull request #23038 from eileenmcnaughton/assign3

Assign totalTaxAmount more consistently

Merge pull request #23077 from eileenmcnaughton/js_cancel

Prevent escaping on cancelInfo_show_ids

Merge pull request #23096 from eileenmcnaughton/ev_paid

Test usability fix

Merge pull request #23018 from colemanw/afformDeepSearch

SearchKit - Ensure filters work with multiple search displays on a form

Merge pull request #23100 from civicrm/5.48

5.48

Update DOMPDF to 1.2.1 and update composer-compile-plugin as per master

Merge pull request #22986 from colemanw/afformFixBlocksAgain

Afform - repeat block metadata update to fix errors when saving blocks

Merge pull request #23081 from eileenmcnaughton/paypal

First recurring payment (paypal ipn) - remove redundant status set, start_date change

Merge pull request #23094 from colemanw/contactTypeIconUI

dev/user-interface#26 Use font-awesome icons for contact types and recent items

RecentItems - Use font-awesome icons in recent items sidebar

Remove redundant status seting in IPN

Merge pull request #23041 from christianwach/lab-core-2103

Event Location fixes

Merge pull request #23052 from konadave/r25088

Don't lose search criteria for smart group built with search builder

Merge pull request #22223 from kurund/pcp-account-improvements

add image upload support for pcp account

Merge pull request #23083 from jaapjansma/issue_3034

dev/core#3034

Fix for issue 3034

Fix for issue 3034

Merge pull request #23079 from colemanw/financialAccountCleanup

Cleanup FinancialAccount BAO to use `writeRecord` and deprecate `add`

Test usability fix

Having the dummy data in the past causes debugging pain

Merge pull request #23095 from seamuslee001/array_key_first_fix

[REF] Create polyfill function for array_key_first which only exists …

ContactType - Prefer icons over image_URL

This is an intermediary step toward removing the image_URL field.
The UI now uses css-based icons when available, and only falls back to image_URL when
an icon hasn't been set for a given sub-type.

Fixes dev/user-interface#26

Update contactType form to support icons and deprecate image_URL

Add system check for contact type image

[RFF] Use Symfony Polyfil to fix issue with missing array_key_first in ealier php versions

Merge pull request #23093 from braders/feature/activity-search-notices

Avoid PHP notices in activity search

Merge pull request #23073 from colemanw/contactTypeIcon

Add civicrm_contact_type.icon column

Avoid PHP notices in activity search

Merge pull request #22376 from colemanw/searchKitInPlaceCreate

SearchKit - Allow creation of new records via in-place edit

Merge pull request #23090 from civicrm/dependabot/npm_and_yarn/minimist-1.2.6

Bump minimist from 1.2.5 to 1.2.6

Merge pull request #23049 from pradpnayak/addUniquieId

Add membership_id as a reference to the row

Bump minimist from 1.2.5 to 1.2.6

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

---
updated-dependencies:
- dependency-name: minimist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #23088 from seamuslee001/update_composer_compile_plugin

[REF] Update Composer compile plugin

[REF] Update Composer compile plugin

Merge pull request #23085 from civicrm/5.48

5.48

Merge pull request #23086 from demeritcowboy/report-filter

dev/core#3153 - Fix missing filter summary on CiviReports

Merge pull request #23034 from colemanw/searchKitIcons

SearchKit - Add icon support

Merge pull request #23042 from eileenmcnaughton/sane

Finish conversion of static function to non-static

Merge pull request #23069 from eileenmcnaughton/triggers

Avoid trailing spaces in trigger output

fix missing filter summary

SearchKit - Add icon support

This allows each column to have one or more icons, based on a field value or a conditional rule.

SearchKit - Fix deleting css rule

Missing argument to splice was causing all rules to be deleted instead of just the one clicked on.

Also updates style to match other delete buttons in searchKit.

jQuery - Switch from deprecated `done()` to more standard `then()`

Fix unit test - don't use empty string as name

Merge pull request #23051 from adixon/3031-smart-subscribe-check-sql

split sql select into two selects for performance reasons

Merge pull request #22996 from colemanw/optionGroupShape

Add option_value_fields metadata

split sql select into two selects for performance reasons

Add membership_id as a reference to the row

Merge pull request #23075 from civicrm/5.48

5.48

Merge pull request #23078 from eileenmcnaughton/ft

Ensure formType is assigned to the template

Cleanup FinancialAccount BAO to use `writeRecord` and deprecate `add`

Sets default values for boolean fields

Merge pull request #23067 from MegaphoneJon/financial-account-improvements

Add support for financial account custom fields

Merge pull request #22971 from eileenmcnaughton/pay_later

Ensure pay_later variables assigned to template

Merge pull request #22972 from eileenmcnaughton/notice_send

Ensure some variables are assigned to the template

Merge pull request #23037 from eileenmcnaughton/assign2

Ensure soft credit types are assigned to the template

Add civicrm_contact_type.icon column

Toward dev/user-interface#26
This adds an icon column and populates it with data but does not yet expose it in the UI.
The next step is to switch the UI from using the image_URL column over to using the icon column.

Merge pull request #23070 from colemanw/searchKitGroupByFix

SearchKit - Fix broken group-by caused by non-aggregated id column

Ensre formType is assigned to the template

Prevent exscaping on cancelInfo_show_ids

Merge pull request #23072 from eileenmcnaughton/isset

Remove more isset from offline receipt

dev/financial#186 Do not display partially paid & partially refunded for selection on edit

I'm pretty sure these were actually removed at one point and got over-added
back because the original fix over-removed them - ie it took them out
when the contribution's current status was 'Partially paid' etc,
making a no-save change impossible

Merge pull request #23071 from demeritcowboy/token-time

dev/core#3072 - Put contribution dates in message template back to not including time

Remove more isset from offline receipt

Merge pull request #23064 from eileenmcnaughton/isset

Remove isset from event registration receipt

put dates back to not including time

SearchKit & Afform - Add option_value_fields info to managed option groups

Add option_value_fields metadata

This allows an optionGroup to declare which columns of the OptionValue table it uses.
Previously this was hard-coded into the form.

SearchKit - Fix UI to allow aggregation of custom fields

Fixes dev/core#3143

SearchKit - Fix broken group-by caused by non-aggregated id column

Before: The `id` column was being unconditionally added, breaking FULL_GROUP_BY sql
After: Let the Api4SelectQuery take care of it for DAO entities

Merge pull request #23066 from colemanw/relationshipCacheJoins

SearchKit - Fix joins when RelationshipCache is base search entity

Avoid trailing spaces in trigger output

Merge pull request #23068 from seamuslee001/master

5.48

Merge branch '5.48'

Add support for financial account custom fields

SearchKit - Fix joins when RelationshipCache is base search entity

Merge pull request #23058 from eileenmcnaughton/ext_notice

Do not default-escape upgrade linkes + Enotice fix on Extension screen

Merge pull request #23063 from demeritcowboy/timezone-multilingual

dev/core#3151 - Don't localize event_tz field when backing up the field during upgrade

Remove isset from event registration receipt

Remove remaining empty checks on pay_later offline receipt

Merge pull request #23060 from eileenmcnaughton/isset

Remove issets from event_offline_receipt pay_later_text