Merge pull request #20951 from eileenmcnaughton/act

Activity tasks - don't add invalid tasks

Merge pull request #20872 from eileenmcnaughton/fprocessor

dev/core#2715 [REF] Start the process of moving financial processing to own class

Merge pull request #20905 from eileenmcnaughton/utf8

Fix databaseSupportsUTF8MB4

Merge pull request #20980 from demeritcowboy/upgrade-enotice

PHP E_NOTICE during upgrade

php notice

Merge pull request #20096 from larssandergreen/copy-template_type-when-cloning-mailing

Make final mailing from A/B test same template type and options as A/B so it can be copied

Merge pull request #20952 from eileenmcnaughton/cs

dev/core#2721 Move custom search framework to a core extension

Merge pull request #20974 from erawat/core#2762

dev/core#2726 - Set ssl option when using DSN with SSL

dev/core#2721 Move custom search framework to a core extension

Put back the handling for custom in StateMachine_Search

Update civicrm_generated.mysql

Move the rest of the menu over

Merge pull request #20961 from eileenmcnaughton/status_name

dev/core#2650 Add support for names & labels for token pseudoconstants

Merge pull request #20973 from demeritcowboy/test-20858

Unit test for invalidateChecksum hook

dev/core#2726 - Set ssl option when using DSN with SSL

Merge pull request #20858 from mattwire/checksumhook

Add hook for validating checksum

test for invalidateChecksum hook

Add hook for invalidating checksum

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

Caching fixes

dev/core#2650 Add support for names & labels for token pseudoconstants

This is per https://lab.civicrm.org/dev/core/-/issues/2650 -
it
- ensures that the 4 existing functions that deal with tokens handle tokens for the
name and label for the (only) field that has had this treatment so far -contribution_status_id

Hence both CRM_Core_SelecetValues::contributionTokens and CRM_Contribute_Tokens->tokenNames
are tested to ensure they both return the same keys and labels for
{contribution.contribution_status_id}
{contribution.contribution_status_id:name}
{contribution.contribution_status_id:label}

And both rendering CRM_Contribute_Tokens via scheduled reminders and
using CRM_Contribute_BAO_Contribution::replaceContributionTokens are tested to ensure
they render them the same.

In the context of this PR no existing tokens are altered or removed & there is
only addition. However, the next step would be to remove the following token
from  {contribution.status}. Since there is no UI availability
of this token it is likely unused - but that step would entail an upgrade
script to remove it from the saved scheduled reminders.

With those parts in place it should be possible to reconcile the remaining tokens,
lock that parity in with tests and move on to exposing the contribution tokens
to message templates.

It would be nice to fully remove CRM_Contribute_BAO_Contribution::replaceContributionTokens
or make it a wrapper for  - however, I fear that might be quite
challenging due to the way it's used with group bys & some pretty intense hackery.

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

Activity tasks - don't add invalid tasks

Instead of 'handling the task code building invalid tasks - #20945' - this proposes to only attempt
to add a page if it is for a valid task. This keeps the grouchiness in the generic controller
to warn us of other mistakes.

This code is pretty fugly! However, it really is just called internally within this class.

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

Fix databaseSupportsUTF8MB4

stripos returns 0 on success, not true.

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