Merge pull request #20862 from eileenmcnaughton/leg_setting

dev/core#2719 [REF] Remove some legacy references to contribution_invoice_settings

Merge pull request #20967 from colemanw/nf

NFC - Reformat JSON file whitespace in Afform mock ext

Merge pull request #20968 from colemanw/fixDeprecatedJoin

SearchKit - fix deprecated join

SearchKit - fix deprecated join

Fixes dev/core#2689

NFC - Reformat JSON file whitespace in Afform mock ext

Merge pull request #20962 from eileenmcnaughton/float

Cast to float when setting overrideTotalAmount - regression in master

Merge pull request #20885 from eileenmcnaughton/param_replace

Invoice template: Add function to update message tokens during upgrade

Merge pull request #20927 from alifrumin/priceplaceholder

Improve accessibility for price fields of the type select by adding the price field label to the placeholder.

Merge pull request #20837 from colemanw/customACLs

 Distinguish custom field VIEW from EDIT permissions

Merge pull request #20941 from eileenmcnaughton/ord

Fix Payment.create to update financial_item.status_id

Merge pull request #20919 from JMAConsulting/issue_2710

dev/core#2710 Cannot disable contact type/sub-type

dev/core#2710 Cannot disable contact type/sub-type

Fix Payment.create to update financial_item.status_id

When I try to switch to the order->create flow for membership forms it turns
out we are leaving the financial_item.status_id as 'unpaid' when adding a payment.

No one has noticed because this field is kinda unused - but it needs to work
to pass tests

Fix order api to set price field value for default price set

This makes it such that when using the default price set for contributions it not only
fills the default price field id. I originally had this in the code but I pulled it out
because the line item test failed. However, I'm seeing cases where not
having a price_field_value_id fails to add tax and I think the issue with the line item
test is that it is, incorrecly, trying to add 2 line items on the price field in
the default price set - which is not actually the expectation on the default price set

I do, separately, wonder if we need the unique id for price_field_id + contribution_id
+ price_field_value_id - but out of scope on this

Merge pull request #20964 from eileenmcnaughton/cast

[REF] [Super trivial] -tighten up some casting

[REF] [Super trivial] -tighten up some casting

Merge pull request #20308 from masetto/master

Fix unit translation for recurring contributions

fixed unit translation for recurring contributions

small fix

 double "every" and `p` tag fixed, removed final `else`

Merge pull request #20482 from MegaphoneJon/translation-69

fix missing 'on behalf of' and honoree labels in multilingual

Add function to update message tokens during upgrade

In conjunction with https://github.com/civicrm/civicrm-core/pull/20867

Cast to float when setting overrideTotalAmount

This is a regression in master whereby submitting a new contribution and leaving total_amount empty
causes a fatal before it can get to the validation error - back office contribution form

Merge pull request #20957 from demeritcowboy/smarty-tags

Smarty notices for New Activity / New Case for tags for attachments

Merge pull request #20955 from eileenmcnaughton/sql3

dev/core#2721 [Ref] simplify passed parameters

Merge pull request #20956 from eileenmcnaughton/valid_fix

Fix test to use valid financials

Merge pull request #20954 from eileenmcnaughton/sql2

dev/core#2721 [REF] Further divide savedSearchParam loading into the sql functions

Merge pull request #20960 from civicrm/5.40

5.40

Merge pull request #20959 from alifrumin/rn5.40

[NFC] Release notes for 5.40

Merge pull request #20685 from jaapjansma/dev_financials_6_contactsummary_3

 dev/financial#6: Button for create/view template contribution on contact summary

Improve accessibility for price fields of the type select by adding the field label to the placeholder text

[NFC] 5.40: first stab at release notes

Merge pull request #20918 from eileenmcnaughton/cust2

dev/core#2709 Enable logging for custom data tables with non-standard names

smarty notices

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

dev/financial#6 exclude template contributions from the contact summary and add a button to view them to the recurring contributions tab

Merge pull request #20867 from eileenmcnaughton/display_name

Swap {$displayName} smarty for {contact.display_name} token

Merge pull request #20939 from eileenmcnaughton/num_term

[Test] Add cover for setting num_terms to 2

Additional opt ins & outs on validateFinancials

Ideally we opt in the class & opt out specific tests as an interim. Moving
towards opt in everything & opt out specific tests

[Test] Opt a couple more classes into validateFinancials

Fix test to use valid financials

Merge pull request #20948 from demeritcowboy/smarty-repdet

Smarty notices on logging detail report

[Ref] simplify passed parameters

I think if we want to make this some sort of hook or listener we don't really
want to pass out 'add select' and 'exclude clause' when they are just group id
with a bit of string. It seems cleaner to specify the sql within the function

[REF] Further divide savedSearchParam loading into the sql functions

[REF] start unravelling the way we retrieve the saved search

We have 3 types of saved searches
- search kit
- legacy core searches
- legacy custom searches

The only information these 3 need to load is the savedSearch details and
the group ID (to add in the add & exclude). The wrangling of the params should
happen in the getSql functions - which we can think about being in a listener once
they have standard inputs & outputs. However, to get to that point
we want to standardise those inputs & outputs. This removes
only point of randomness - ie savedSearch has a consistent value & the wrangling
of what is in it is pushed closer to the relevant functions

Merge pull request #20923 from totten/master-phptype

CRM_Utils_Type::validatePhpType - Helper to validate PHP type expressions

Merge pull request #20949 from demeritcowboy/smarty-reppager

Smarty warning about pager location on all CiviReports

smarty warning - rows per page filter missing

smarty warnings

Merge pull request #20946 from mattwire/managedentitydeleteerror

Fix error message when unable to delete ManagedEntity

Fix error message when unable to delete managedentity

Merge pull request #20857 from scardinius/translation-status-bounces

Add ts() function within CRM_Core_Error::statusBounce() method

Merge pull request #20943 from eileenmcnaughton/cc

[NFC] Remove exceptions from comments

[NFC] Remove exceptions from comments

callApiSuccess now uses fail rather than throwing an exception so
we don't need these here

Merge pull request #20877 from kurund/patch-1

add support for profile map view

Fix spelling

Update CRM/Contribute/Page/ContributionRecur.php

Co-authored-by: demeritcowboy <demeritcowboy@hotmail.com>

[Test] Add cover for setting num_terms to 2

Merge pull request #20882 from eileenmcnaughton/hh

Extract getCurrentMembership

Merge pull request #20937 from civicrm/5.40

5.40

Merge pull request #20925 from demeritcowboy/install-linemode

dev/drupal#161 - (not just drupal) Can't install sample data using UI installer

Merge pull request #20924 from JMAConsulting/cust_deadlock

Reduce deadlock while inserting custom data for Address

Swap displayName smarty for {contact.display_name token}

This swaps over the smartyToken for the token token on the template added on new installs, and adds a test.

Separately I will look at an upgrade script and other templates

To test make sure you enable invoicing & then generate an invoice from search results.
use the demo site due to aforementioned lack of upgrade and check the display name is still on
the left hand side

Merge pull request #20932 from eileenmcnaughton/mgd

Add v4 managed api entity

Merge pull request #20835 from MegaphoneJon/core-2687

fixes core#2687: regression on group rebuild scheduled job

Merge pull request #20934 from demeritcowboy/lybunt-param

[NFC] Typo in parameter for lybunt test

Merge pull request #20926 from mlutfy/fixInclusiveTaxTs

Fix 'includes tax' translation for PriceSet Checkbox fields

incorrect parameter

Merge pull request #20556 from ixiam/dev#2642

Add filter by custom fields in Accounting Batch. https://lab.civicrm.…

Add filter by custom fields in Accounting Batch. https://lab.civicrm.org/dev/core/-/issues/2642

Merge pull request #20910 from colemanw/searchKitPickLists

[REF] SearchKit - Refactor pick-lists to use onCrmUiSelect directive

Add v4 managed api entity

Note that this is searchable since it seems useful to me to be able to search these - albeit
for more advanced users!

Merge pull request #20930 from eileenmcnaughton/total

Enable range search for money fields in search kit + afform

Merge pull request #20931 from civicrm/5.40

5.40

Enable range search for money fields

Allow total_amount to be exposed for searching by range

dev/core#2709 Enable logging for custom data tables with non-standard names

Merge pull request #20928 from demeritcowboy/20914-5.40

dev/core#2699 - 5.40 version of Attempt regression fix

Fix for cust field

Merge pull request #20904 from mattwire/pcpcurrency

Display configured currency on PCPPage instead of default

Merge pull request #20920 from eileenmcnaughton/cleanup2

Add check to see if these indices are actually removed + call the function which should

Merge pull request #20868 from eileenmcnaughton/load_acls

[REF] Extract the function to load from the acl table

Extract getCurrentMembership

Note I've moved the fixmembershipStatusBeforeRenew here - it is just compensating for the possibility of
the job not being run. On the Membership form we run it in preProcess but this feels like our first
chance in this form

Fix 'includes tax' translation for PriceSet Checkbox fields

[REF] Extract the function to load from the acl table

linemode not needed

Merge pull request #20854 from eileenmcnaughton/response

Remove obsolete code

Reduce deadlock while inserting custom data from Address

Merge pull request #20911 from colemanw/select2css

Greenwich - fix select2 font size

CRM_Utils_Type::validatePhpType - Helper to validate PHP type expressions

Add check to see if these indices are actually removed

This check for indices seems to be much more reliable - the question is whether we can
run it against a range of dbs to confirm

Merge pull request #20913 from colemanw/fixCapitalizationOfSqlFunction

Capitalization Fix

Capitalization Fix

Greenwich - fix select2 font size

SearchKit - Refactor pick-lists to use onCrmUiSelect directive

Merge pull request #20908 from colemanw/fixApiCalculatedBridgeFields

APIv4 - Fix calculated fields in bridge entities

Distinguish custom field VIEW from EDIT permissions

Before: CustomGroup ACLs made no distinction between VIEW and EDIT

After: User with VIEW can see a custom group but not edit it, both in the UI and the API.

APIv4 - Fix calculated fields in bridge entities

Fixes a crash-inducing bug in the use of calculated fields on a bridge entity,
such as RelationshipCache.is_current.

Merge pull request #20886 from colemanw/searchKitRelationshipLinks

SearchKit - Add links to view/edit/delete relationships

Merge pull request #20866 from totten/master-pathmove

CRM_Utils_Array::pathMove - Add helper to move an item within array tree

Merge pull request #20906 from eileenmcnaughton/api4

Add test & fix for failing OR clause in apv4

Merge pull request #20900 from colemanw/angularTraits

[REF] Refactor SearchKit displays to inherit traits from a common base

Add test for failing OR clause

This fails because when the field is being concatenated like
`a`.`first_name`='x' OR`a`.`last_name`='x'

The lack of a space between OR and `a` is fugly but it
is parsed by mysql. However, when the value requires utf8mb4
to be supported to do the comparison it returns '0=1' if the
database does not support utf8mb (or it thinks it doesn't per

0 = 1 OR0 = 1

Merge pull request #20870 from totten/master-tokensmarty

(REF) Extract TokenSmarty::render() from MessageTemplate::renderMessageTemplate()