Merge pull request #19683 from colemanw/searchDisplayFixes

Search display fixes

Merge pull request #19708 from J0WI/patch-1

Fix migrateUtf8mb4 for special database names

Merge pull request #19675 from demeritcowboy/casedashlets

dev/core#1400 - Put dashlets back to only showing open cases by default

Fix migrateUtf8mb4 for special database names

Merge pull request #19702 from seamuslee001/domain_petition_emails

[NFC] Add in unit test to prove that domain tokens are able to be inc…

Merge pull request #19706 from civicrm/5.35

5.35

Merge pull request #19705 from seamuslee001/update_schema_handler_test

[NFC] Update Schema Handler to use the standard create table syntax o…

Merge pull request #19704 from colemanw/removeDeprecatedAngularJs

Angular - Remove deprecated js file

[NFC] Update Schema Handler to use the standard create table syntax of using ROW_FORMAT=dynamic

[NFC] Add in unit test to prove that domain tokens are able to be incldued in peition emails

Merge pull request #19698 from eileenmcnaughton/custom

dev/core#2423 Fix quasi-regression around serialized custom fields

Angular - Remove deprecated js file

Merge pull request #19701 from civicrm/5.35

5.35

Bring back some madness

dev/core#2423 Fix quasi-regression around serialized custom fields

https://lab.civicrm.org/dev/core/-/issues/2423

Merge pull request #19692 from demeritcowboy/case-custom-money-5.35

dev/core#2394 - Don't crash when saving custom case fields of type money

Merge pull request #19699 from colemanw/controlField

APIv4 - Rename 'controlField' property to 'control_field'

Merge pull request #19695 from civicrm/5.35

5.35

APIv4 - Rename 'controlField' property to 'control_field' for consistency

By convention, all keys returned from the api should be lowercase.

Merge pull request #19690 from colemanw/api3Operators

APIv3 - Improve array-based apis to support sorting and operators

Merge pull request #19680 from eileenmcnaughton/money

dev/financial#166 Fix for inconsistency around currency symbol

Merge pull request #19696 from eileenmcnaughton/user

dev/core#2427 Fix user creation regression

Merge pull request #19629 from MegaphoneJon/core-2386

core#2386 - metadata-driven chain-select fields

APIv3 - Improve array-based apis to support sorting and multiple operators

This backports some APIv4 code to v3, for the purpose of supporting
entityRef widgets for Afform.

Merge pull request #19668 from eileenmcnaughton/empty_rows

dev/core#2420 Add text to report template saying no results found if that is the case

dev/core#2427 Fix user creation regression

Merge pull request #19693 from eileenmcnaughton/if

[REF] clean up if

dev/financial#166 Fix for inconsistency around currency symbol

From https://lab.civicrm.org/dev/financial/-/issues/166 we learn that the existing code
(tested via testFormatLocaleNumericRoundedByCurrency) is not consistent across
all platforms. I think this may be

Merge pull request #19685 from eileenmcnaughton/535

dev/core:2394 Fix for number formatting regression

Merge pull request #19689 from demeritcowboy/my-cases-listtype

dev/core#1207 - My Cases dashlet shouldn't crash for users with only my cases access

[REF] clean up if

This just makes 2 lines simpler. The issue is that if checks the same thing
twice - presumably the if empty check got added for the e-notice but
the other was not removed

don't crash when saving custom fields of type money

Merge pull request #19688 from demeritcowboy/lottery-win

Fix for missing function CodeGen_Util::removeDir

don't crash for users with only my cases access

missing function

Merge pull request #19686 from colemanw/searchKitClearSelection

SearchKit - Fix clearing selection behavior

SearchKit - Fix clearing selection behavior

The variables `allRowsSelected` and `selectedRows` were sometimes getting out-of-sync.
This adds a function to help ensure they both get updated at the same time when the selection needs to be cleared.

Merge pull request #19674 from demeritcowboy/caseclosed

dev/core#1400 - Failing test showing that My Cases users can't view their own closed cases

dev/core:2394 Fix for number formatting regression

This function has been picking up the formatting for the locale
meaning that the currency separator replacement is already done and
is swapped back by formatLocaleNumericRoundedByPrecision

Doing it via brickmoney is better than our custom
separator replacement  - but we need a quick fix for the rc

Merge pull request #19682 from colemanw/dashlet-spacing

Reduce wasted space in civicrm dashboard margins

SearchKit - Fix js error when adding label to buttons/links in List display

SearchKit - More compact display column editing

SearchKit - Add numbers to links when > 1 of the same entity

Merge pull request #19666 from kainuk/dev_core_1400

dev/core#1400 Users with 'view own cases' permission can't open resolved cases

Merge pull request #19673 from colemanw/searchKitLinks

SearchKit - Add links/menus/buttons to search displays

Reduce wasted space in civicrm dashboard margins

This removes the extra margins to the top and sides of the home dashboard page, only placing margins between the dashlets.

Merge pull request #19641 from eileenmcnaughton/no_part

[REF] Load participant id within completeOrder rather than passing it in

Merge pull request #19658 from eileenmcnaughton/member

[REF] Move code that relates to receipts when paying by credit card out of the shared function

Merge pull request #19677 from totten/master-joomla-syncuser

Joomla::synchronizeUsers - Fix notice due to old style reference

Merge pull request #19679 from civicrm/5.35

5.35 to master

Merge pull request #19678 from totten/master-guzzle-url

GuzzleMiddleware::url() - Distinguish frontend and backend routes

GuzzleMiddleware::url() - Distinguish frontend and backend routes

Before: The `route://` scheme assumed backend access.

After: The `frontend://` and `backend://` schemes are unambiguous.
       The `route://` scheme will make a best-guess by determining
       if the menu item has `is_public`.

SearchKit - Add links/menus/buttons to search displays

Joomla::synchronizeUsers - Fix notice due to old style reference

This fixes a PHP notice when synchronizing all users/contacts:

PHP Notice:  Only variables should be assigned by reference in ...CRM/Utils/System/Joomla.php:855

Add test to report template saying no results found if that is the case

Note this is done to create minimum white space change. I actuallY
think the criteria around charts should be in the tpl that
makes the decision to include Table.tpl rather than in Table.tpl
and this would become {if }{/else}{ts}No results found{/ts}{/if}

I can do that as a follow up if agreed

put dashlets back to only showing open cases by default

test showing non-admins can't see closed cases

Merge pull request #19672 from colemanw/contentEditableUI

Afform - move contentEditable directive into crmUI library for reusability

Merge pull request #19662 from colemanw/searchDisplayRewrite

SearchKit - cleanup display configuration and add "rewrite" feature

Merge pull request #19671 from jaapjansma/dev_2424

Fixes dev/core#2424 navigation items order

Afform - move contentEditable directive into crmUI library for reusability

Fixes dev/core#2424 navigation items order

Merge pull request #19670 from eileenmcnaughton/aff

Switch afform back to '=' to unbreak deduper

Switch afform back to '=' to unbreak deduper

https://github.com/eileenmcnaughton/deduper/pull/9

Merge pull request #19669 from eileenmcnaughton/cont_status

Metadata: Add html:label for Contribution Status ID

Metadata: Add html:label for Contribution Status ID

Merge pull request #19667 from eileenmcnaughton/recurse

Fix master-only regression

Fix master-only regression

It seems the tests ran before the test that covered this
was merged giving a false positive

See https://github.com/civicrm/civicrm-core/pull/19478#issuecomment-785388559 for rationale

Merge pull request #19638 from colemanw/entityRefId

 EntityRef - Search by id when an integer is entered

Merge pull request #19664 from demeritcowboy/case-rel-minor-cleanup2

[NFC] minor cleanup in testCaseCreateWithResolvedStatus

Merge pull request #19663 from demeritcowboy/case-rel-minor-cleanup

Minor cleanup - ChangeCaseStatus

Merge pull request #19665 from seamuslee001/date_search_select

Afform: Only permit Date fields to be made into a select list if it i…

EntityRef - Fix merging api params when searching by id

dev/core#1400 Users with 'view own cases' permission can't open resolved cases

Afform: Only permit Date fields to be made into a select list if it is a search form

Merge pull request #19632 from colemanw/afformDateRange

Afform - Support date range filters for search displays

minor cleanup

minor cleanup

SearchKit - add option to rewrite field output in displays

Merge pull request #19478 from eileenmcnaughton/ref

[REF] do not needlessly pass as reference, enforce valid param

Merge pull request #19610 from ahed-compucorp/dev/core#2372

dev/core#2372 Enable submitOnce for some forms

Merge pull request #19661 from civicrm/5.35

5.35 to master

Merge pull request #19659 from demeritcowboy/revert-18782

revert 18782

Merge pull request #19655 from mlutfy/tr32fix

dev/translation#32 Contribution ThankYou: partial fix for recurring units

SearchKit - More efficient use of space when configuring display fields

SearchKit - Move common displayAdmin fields into shared templates

Merge pull request #19643 from colemanw/searchKitButtons

SearchKit - Don't save on enter key

dev/translation#32 Contribution ThankYou: partial fix for recurring units

revert 18782

Merge pull request #19650 from tapashdatta/Enable-"Do-no-trade"-filter-1

Enable-"Do-no-trade"-filter

Merge pull request #19645 from colemanw/afformLink

Afform - Add 'View Page' link from afform edit screen

Merge pull request #19657 from totten/master-aff-perm-blank

dev/core#2404 - Afform - Resolve ambiguity of omitted permissions

[REF] Move code that relates to receipts when paying by credit card out of the shared function

The emailReceipt function is shared between the Batch Entry form and the backoffice
credit card form, while the emailMembershipReceipt is only used in this function.

I aim to copy the function onto the Batch Entry form & split them up but
am first moving this out of the shared code as it only
relates to credit card payments, which are not reachable from the Batch Entry form

Merge pull request #19654 from eileenmcnaughton/member

[REF] Simplfy parameter

Merge pull request #19648 from eileenmcnaughton/mm

dev/core#778 Do not increment tax on validation error

(dev/core#2404) afform_core - If "permission" is empty-string, then use default

(dev/core#2404) afform_admin - Treat "permission" as a required field

[REF] Simplfiy parameter

Rather than pass in contributionParams & params & combine in the function,
combine when passing in. Note receive_date is always 'now' in this
workflow as it is specifically the card payment workflow (for recurring)
being altered here & that field is not displayed on
credit card payments

Merge pull request #19656 from civicrm/5.35

5.35 to master

Merge pull request #19653 from eileenmcnaughton/535

dev/core#2360 - Escape the word `rows` in sql query