Merge pull request #21303 from civicrm/5.41

5.41

Merge pull request #21302 from seamuslee001/fix_conformance_tests

[NFC] Fix APIv4 Conformance tests on php8

Merge pull request #21291 from eileenmcnaughton/541r

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

[NFC] Fix APIv4 Conformance tests on php8

Merge pull request #21289 from eileenmcnaughton/tokens

[Ref] intial testing on case tokens, make knownTokens optional

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

Merge pull request #21300 from colemanw/imageField

SearchKit - Image field handler implementation

SearchKit - Image field handler implementation

Fixes dev/core#2781

Merge pull request #21298 from eileenmcnaughton/silly

[REF] Remove duplicate IF

Merge pull request #21292 from eileenmcnaughton/camps

Fix for new prefetch key

[REF] Remove duplicate IF

Earlier in the function there was already a RETURN if
so this IF is always true

Merge pull request #21296 from eileenmcnaughton/auth2

[REF] Minor extraction

Fix for new prefetch key

Fixes a bug in
https://github.com/civicrm/civicrm-core/pull/21184
which was masked in the test by the lack of campaigns to load.

The ?? operator handles FALSE differently to the ?: operator
so this was casting FALSE to TRUE

Merge pull request #21294 from eileenmcnaughton/cont_form

[REF] Remove unreachable code

[REF] Minor extraction

Merge pull request #21293 from eileenmcnaughton/auth2

[Ref] Minor extraction

Merge pull request #21290 from eileenmcnaughton/pdftrait

[REF] dev/core#2790 Initial creation of pdf trait

[REF] Remove unreachable code

[Ref] Minor extraction

Simply extracts getting the subscription id & since the retrieve function throws
defaults to 'abort' the next line is unreachable

Merge pull request #21268 from totten/master-ml-optlabel

(dev/translation#70) Multilingual - Fix loading multiple translations within same page-view (OptionValues, ContactTypes)

[Ref] intial testing on case tokens, make knownTokens optional

There is a perception in the token code that passing around knownTokens is more performant

This may or may not be true - but since the goal is to migrate the code to the
token processor making the code simpler is
a higher priority as any performance change will change again.

This makes knownTokens optional in the replaceCaseTokens and adds test cover.
It is a step towards simplification in the PdfLetter code
which could be simplified if not needing to pass in knowntokens here

[REF] dev/core#2790 Initial creation of pdf trait

Only one function copied over for now - setDefaultValues

Merge pull request #21220 from agileware/CIVICRM-1812

Change the default PDF file name from "CiviLetter.pdf" to use the Activity Subject, if available

Merge pull request #21139 from totten/master-msgtpl-class

(dev/mail#83) Workflow Messages - Introduce class contracts

Merge pull request #21281 from civicrm/5.41

5.41

Merge pull request #21275 from eileenmcnaughton/541

Fix token deprecation to be a check not an upgrade notice

Merge pull request #21280 from civicrm/5.41

5.41

Merge pull request #21150 from colemanw/afformUpload

Afform - support file uploads

dev/translation#70 - Cache separate ContactType lists per-locale

Notes:

* If you grep to see how `contactTypes` cache is used, it appears that they never
  access this row individually outside this function.
* There are coarse-grained `clear()` invocations all `contactTypes`-related data.
  These should hit the old+new keys the same way.
* I made the cache-lookup a little more micro-optimal. :shrug:

To verify that this fixes the bug, I enabled multilingual with fr_CA+es_MX and manually translated the labels for "Individual" contacts.
Then used the following command:

```
cv ev -U admin 'function go($l){ CRM_Core_I18n::singleton()->setLocale($l); return \Civi\Test\Invasive::call(["CRM_Contact_BAO_ContactType","getAllContactTypes"])["Individual"]; } return [go("fr_CA"),go("es_MX")];'
```

Before the patch, it returns the en_US labels ("Individual"..."Individual"...).

After the patch, it returns the fr_CA and es_MX labels that I had created ("Particulier"..."Persona"...).

OptionGroup - Fix loading of localized labels

LocaleTest - Add example for loading localized labels

Merge pull request #21278 from seamuslee001/5.41

Add release-notes/5.40.4.md

Add release-notes/5.40.4.md

CIVICRM-1812 Change the default PDF file name from "CiviLetter.pdf" to use the Activity Subject, if available

Merge pull request #21259 from demeritcowboy/better-file

dev/core#2789 - Filename is not non-english-friendly in print/merge document

Afform - support file uploads

Adds a new API action Afform.submitFile which receives uploaded files one at a time after the form is submitted.

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 #21274 from mattwire/contributionviewid

REF Don't check if id is set in ContributionView form - it's required

Merge pull request #21273 from eileenmcnaughton/mem

[REF] Remove meaningless if

Don't check if id is set in ContributionView - it's required

[REF] Remove meaningless if

This if is meaningless but I left it in last time to reduce
visual change

Merge pull request #21126 from eileenmcnaughton/batch

dev/core#2717  - further cleanup

APIv4 - Add util function to get name of id field (primary key)

Merge pull request #21269 from mattwire/removeisdevelopment

Remove deprecated isDevelopment() function

Remove deprecated isDevelopment() function

Merge pull request #21221 from agileware/CIVICRM-1813

Change PDF file name from "civicrmContributionReceipt.pdf" to use the standard "receipt.pdf" file name

Merge pull request #21266 from civicrm/5.41

5.41

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 #21260 from eileenmcnaughton/act_search

Do not enable custom activity search on new installs

Merge pull request #21235 from seamuslee001/dev_core_2768

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

Merge pull request #21264 from civicrm/5.41

5.41 to master

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'

Merge pull request #21243 from mattwire/contributionpageerrors

Logging improvements for "Failed to update contribution in database"

Do not enable custom activity search on new installs

In general we want to get away from maintaining the custom searches. Reading the docs I realised
this search does not do anything not done through advanced search and activity search (even
before we start talking about search kit) so this is 'one down

make filename non-english-friendly

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 #21258 from civicrm/5.41

5.41

Cleanup input params on legacyProcessMembership

Changed support to dev/core from ext/oauth

Merge pull request #20935 from eileenmcnaughton/hhhh

dev/core#2717 Simplify batch membership renewal

Merge pull request #21106 from eileenmcnaughton/order

dev/core#2634 Add v4 Membership api, access it via order

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

FIx order api regression

Merge pull request #21257 from eileenmcnaughton/cset

Template fixes - notices, syntax

Merge pull request #21255 from demeritcowboy/not-so-hidden

Fix invalid parameter giving E_WARNING

E notice fixes

fix invalid parameter

Settings.tpl - Fix syntax error

Add order  fix

Merge pull request #21252 from civicrm/JoeMurray-patch-1

Provided standard links in ext/oauth-client/info.xml, fixed typo

Merge pull request #21251 from eileenmcnaughton/email

[Ref] Copy emailcommon function back to email trait

Merge pull request #21101 from eileenmcnaughton/inv3

[REF] Update a few references to invoicing

Provided standard links, fixed typo

Merge pull request #21217 from colemanw/searchKitButtonTokens

SearchKit - Allow tokens in menu button text

Merge pull request #21134 from eileenmcnaughton/tok_nearly

 dev/core#2745 - Contribution Tokens - Support 'contributionId'

[Ref] Copy emailcommon function back to email trait

This is part of ending the email common trait. It is currently 'shared'
with a couple of other places - but most of the functionality is
not that shared & could be easily cleaned up when separated out.

I want to create a trait for PdfTrait like the email trait so
getting the email trait better separated for that

Merge pull request #21006 from deb1990/rename-pdf-file

dev/core#2121 Ability to change pdf filename before downloading

Merge pull request #21246 from eileenmcnaughton/master

5.41 to master

Merge pull request #21244 from demeritcowboy/angular-loader

[NFC] - Replace deprecated function in AngularLoaderTest

Merge pull request #21245 from agileware/CIVICRM-1816

CiviCRM Mailing, function unsub_from_mailing has spelling error, "experiement" impacts A/B Mailing unsubscribes

Revert "Re-fix test to work with new check"

This reverts commit 77136dc6d4216211998729cea305d81868309809.

Merge branch '5.41' of github.com:civicrm/civicrm-core

CIVICRM-1816 CiviCRM Mailing, function unsub_from_mailing has spelling error, "experiement" impacts A/B Mailing unsubscribes

Merge pull request #21169 from eileenmcnaughton/email

dev/core#2769 use php email validation not hacked & bad quickform function

replace deprecated function

Set error level for mail configuration errors

Log actual error message when contribution page submission fails to complete

Merge pull request #21212 from eileenmcnaughton/541-orig

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

Merge pull request #20898 from JMAConsulting/issue_2701

(Smart Group) is being constantly added while editing the smart group title from 'Manage Group' page

dev/core#2769 use php email validation not hacked qf

Per https://lab.civicrm.org/dev/core/-/issues/2769 we have had problems over the years with
quickform's email validation and we now have a hacked version that is
problematic from a maintenance pov & also doesn't work
with the string I have just encountered: name.-o-.i.10@example.com
(which I am told is valid and which passes the php filter).

We already have an email rule which calls a php native function
which is better maintained than our layers of hacks. This
PR registers our email rule - which overrides the quickform
one. If we merge this we can revert quickform back to
unhacked which will improve debugging
and maintenance (although it's actually bypassed
now with this change)

Merge pull request #21240 from civicrm/5.41

5.41 to master

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

Merge pull request #21105 from colemanw/afformSubmission

Afform - Store submissions in a new database table

Merge pull request #21231 from civicrm/5.41

5.41

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