Merge pull request #21291 from eileenmcnaughton/541r

dev/core#2786 Ensure non-English installs have a default financial type for membership default price set

dev/core#2786 Ensure non-English installs have a default financial type for membership default price set

Merge pull request #21275 from eileenmcnaughton/541

Fix token deprecation to be a check not an upgrade notice

Merge pull request #21278 from seamuslee001/5.41

Add release-notes/5.40.4.md

Add release-notes/5.40.4.md

Fix token deprecation to be a check not an upgrade notice

It turns out the upgrade notice is calculated before the upgrade action runs
- this means that currently the upgrade notice is displayed even though
manual intervention will only be required in edge cases (since the
upgrade replaces the most common usage and it would require the
presence of an if or something like that for it to still exist).

Merge pull request #21265 from seamuslee001/5.41

[NFC] Do not generate a result cache file when running legacycustomse…

[NFC] Do not generate a result cache file when running legacycustomsearches phpunit tests

Merge pull request #21235 from seamuslee001/dev_core_2768

Resolve dev/core#2768 Reinstate code into APIv4 that handled magic fu…

Merge pull request #21250 from agileware/CIVICRM-1820

CiviCRM Scheduled Reminders, Effective Start Date and Effective End Date are incorrectly evaluated if these fields contain a value '0000-00-00 00:00:00'

CIVICRM-1820 CiviCRM Scheduled Reminders, Effective Start Date and Effective End Date are incorrectly evaluated if these fields contain a value '0000-00-00 00:00:00'

Merge pull request #21230 from eileenmcnaughton/541-w-order

FIx order api regression

Add order  fix

Merge pull request #21212 from eileenmcnaughton/541-orig

dev/financial#180 Fix line item calculation regression on line items (incorrectly treating as exclusive)

Revert test change that 'supported' incorrect code change

https://github.com/civicrm/civicrm-core/commit/e967ce8fe2b58b94e2163dde395542e55599da13#diff-44a947115564a688b2d6e0772ad25039b8d244225cf250dbc27ed32a134467a3L395-R396

Merge pull request #21239 from kcristiano/dev-2784

dev/core#2784 Export PRIMARY fields Fails on CiviCRM 5.40.2 and MariaDB 10.3 with row size issue

Move the tested but crazy tax calc to the contribution api

Fix line item calculation from exclusive to inclusive

This was added https://github.com/civicrm/civicrm-core/commit/e967ce8fe2b58b94e2163dde395542e55599da13#diff-a16d4d7449cf5f3a0616d1d282a32f27ab6d3f7d2726d076c02ad1d4d655af41R512

but incorrectly assumes the totalAmount is exclusive at this point - but
it is inclusive

Merge pull request #21238 from eileenmcnaughton/541-lines

Fix test to actually pass valid data

Merge pull request #21237 from seamuslee001/5.41

Add release-notes/5.40.3.md

Export PRIMARY fields Fails on CiviCRM 5.40.2 and MariaDB 10.3

with row size issue  https://lab.civicrm.org/dev/core/-/issues/2784

Re-fix test to work with new check

A new check was enabled for this test recently and it passed because we
were doing recalculation - but that recalculation was not always right
and there is no expectation that passing in the wrong total_amount
is OK so this fixes total_amount to be equal to the
sum of the line items - which is what is checked

Add release-notes/5.40.3.md

Resolve dev/core#2768 Reinstate code into APIv4 that handled magic functions

Add in test to assert Exception type and exception message that is thrown

Merge pull request #21224 from ufundo/oauth-systoken-refresh-fix

dev/core#2779 - Specify row fields to fetch in Api4 OAuthSysToken.Refresh

Merge pull request #21229 from eileenmcnaughton/cache

Test cleanup fix

Test cleanup fix

implement-basicbatchaction-getselect

Merge pull request #21195 from colemanw/UnknownFieldError

APIv4 - Properly show deprecated joins in unit tests (fixes dev/core#2771)

Merge pull request #21192 from jitendrapurohit/reminder

dev/core#2777 - Fix js break on reminder form

APIv4 - Properly show deprecated joins in unit tests

Fix js break on reminder form

Merge pull request #21189 from eileenmcnaughton/541b

dev/rc#14 handle api calls post schema change

dev/rc#14 handle api calls post schema change

Merge pull request #21163 from seamuslee001/5.41

Add release-notes/5.40.2.md release-notes/5.40.1.md

Add release-notes/5.40.2.md release-notes/5.40.1.md

Merge pull request #21151 from eileenmcnaughton/541m

Membership batch processing error

Membership batch processing error

This fixes a fatal because of incorrect typing

Affects 5.40

Merge pull request #21143 from seamuslee001/fix_enotice_php8

[REF] Fix e-notice in APIv3 Profile Test on PHP8

Merge pull request #21147 from eileenmcnaughton/541

dev/core#2758 fix task tiles

dev/core#2758 fix task tiles

[REF] Fix e-notice in APIv3 Profile Test on PHP8

Remove e-notice issue found by dave d

Merge pull request #21136 from agh1/5.41.0-releasenotes-initial

5.41.0 release notes initial run

5.41.0 release notes: added boilerplate

5.41.0 release notes: raw from script

Merge pull request #21099 from eileenmcnaughton/camp54

dev/core#2743 fix api v3 to not unnecessarily load options

Merge pull request #21102 from artfulrobot/ui/37

dev/user-interface#37 Reinstate jQueryUI visible focus effect in quicksearch

Merge pull request #21124 from MegaphoneJon/fix-event-validation

fix crmValidate() not loading on event registration pages

fix crmValidate() not loading on event registration pages

Merge pull request #21107 from MegaphoneJon/clean-override-total

delocalize total amount before passing to setOverrideTotal

Merge pull request #21108 from seamuslee001/5.41

[REF] Upgrade CKEditor to 4.16.2

Merge pull request #21104 from totten/5.41-typo

(NFC) release-notes - Fix typo

Merge pull request #21100 from eileenmcnaughton/exceptless

dev/core#2749 remove exception when no payment processor configured when no payment processor configured

[REF] Upgrade CKEditor to 4.16.2

delocalize total amount before passing to setOverrideTotal

(NFC) release-notes/5.39.1.md - Fix typo

Typo was autogenerated from a template. Template is also fixed.

Reinstate jQueryUI visible focus effect in quicksearch user-interface issue 37

dev/core#2749 remove exception when no payment processor configuredhen no payment processor configured

This is an attempt to fix a regression https://lab.civicrm.org/dev/core/-/issues/2749
when there are legitimately no payment processors assigned to a page.

I feel like the configuration screens are adequate to enforce this & having an exception like
this in the assign function is the wrong place.

The original issue is that the is_monetary box was not checked & hence the
processor wasn't assigned. This felt like asking users to understand that
a payment method when money options are configured

Fix one test & remove 2 that covered the error message

dev/core#2743 fix api v3 to not unnecessarily load options

Merge pull request #21001 from seamuslee001/fix_utf8mb4_test

[REF] Fix utf8mb4 test in APIv4 and re-enable the altering of databas…

[REF] Fix utf8mb4 test in APIv4 and re-enable the altering of database in the v3 version of the test and fix handling for first name field

Set version to 5.41.beta1

Merge pull request #21053 from eileenmcnaughton/group_odd

dev/core#2742 fix group type on group create from form

Merge pull request #21051 from eileenmcnaughton/allan

dev/core#2740 fix dashboard regression on links

Merge pull request #21050 from eileenmcnaughton/notice

Tpl notice fixes

dev/core#2742 fix group type on group create from form

dev/core#2740 fix dashboard regression on links

Merge pull request #21045 from colemanw/fixSearchKitTaskPermissions

SearchKit - Fix permission to access task list

Tpl notice fixes

These fields are both conditionally assigned to the template

Merge pull request #21044 from mlutfy/fixContribSymbiotic

contributor-key.yml: updates for Symbiotic

SearchKit - Fix permission to access task list

This regressed in 06ea32c, as a new api action was added but permissions were not declared.

contributor-key.yml: updates for Symbiotic

Merge pull request #21039 from eileenmcnaughton/tok_pseudo

Reconcile tokens misnamed or missing on legacy class

Merge pull request #21038 from eileenmcnaughton/tok_names

Switch to using apiv4 for metadata

Merge pull request #21041 from eileenmcnaughton/notice2

Enotice fix

Merge pull request #21040 from eileenmcnaughton/tpl

Enotice fix

Merge pull request #20752 from samuelsov/ReportGroup

Make report subtotals work for custom fields

Enotice fix

This one is a bit of a brain twister but I'm pretty sure the re-ordered IF has the
same result - but one less enotice

Merge pull request #20989 from mattwire/mailtasks

Fix mailing task when there are no tasks

Enotice fix

Reconcile tokens misnamed on legacy class

This switches to using the same metadata based record for both classes

Switch to using apiv4 for metadata

Merge pull request #21036 from eileenmcnaughton/act_mon

Reconcile contribution amount tokens

Reconcile contribution amount tokens

This adds non_deductible_amount, as identified in https://github.com/civicrm/civicrm-core/pull/21034

It also moves the metadata functions to a parent class - this class might wind up
being a temporary class as there are 3 other contenders for where this functionality
should go - the TokenTrait, the abstractProvider or the tokenRow class. The intent
is not to make that decision yet - just to provide visual clarity about
what is generic

Note - I've figured out how to switch over to v4 api for this but will do that in a separate PR

Merge pull request #21037 from eileenmcnaughton/act_source

Fix advertised token for source

Merge pull request #21035 from eileenmcnaughton/act_more

Update contribution_page_id token in sched reminders

Update contribution_page_id token in sched reminders

Fix advertised token for source

Merge pull request #21000 from eileenmcnaughton/act_more

 Reconcile and upgrade tokens for payment instrument, financial type

Reconcile and upgrade tokens for payment instrument, financial type

This switches the last 2 'alias' tokens from Contribute_Tokens to the preferred format,
adds tests & upgrade and switches the advertised tokens. I tested & the token
removed from the sent letter collection still works

Next step is to analyse the remaining differences between the 2 sets of tokens.
My suspicion is that we have enough metadata that we use in search kit that we
don't need a hard-coded white list of tokens in 2 different places & can just
expose 'all the fields' that are not acl or otherwise limited

Merge pull request #20999 from eileenmcnaughton/act_up

Switch contribution action schedule tokens to use advertised tokens for cancel_date & source

Merge pull request #21019 from ejegg/customFieldBulkLog

Fix bulk adding custom fields with logging on

Merge pull request #21033 from kurund/patch-2

Update contributor-key.yml

Update contributor-key.yml

Merge pull request #21032 from seamuslee001/master

5.40 up merge

Merge in 5.40

Merge pull request #21031 from totten/5.40-bump0

5.40.0 - Version bump. More release notes.

Set version to 5.40.0

Tweak release-notes/5.40.0.md