Additional test

Merge pull request #23535 from eileenmcnaughton/import_prefix

Extend import option handling to prefix, suffix, language, communication_style

Merge pull request #23550 from eileenmcnaughton/import_cust

Clean up function signature

Merge pull request #23532 from eileenmcnaughton/import_depo

Remove extraneous deprecated check

Extend import option handling to prefix, suffix, language, communication_style

This extends a pattern thoroughly tested for gender to
other single-option-value fields & removes duplicate handling.

Test cover in

`CRM_Contact_Import_Parser_ContactTest::testPrefixLabel`
`CRM_Contact_Import_Parser_ContactTest::testPreferredLanguageImport`

Clean up function signature

Merge pull request #23533 from eileenmcnaughton/import_if

Remove always true if conditions (whitespace in next commit)

Merge pull request #23515 from eileenmcnaughton/no_prof

[REF] Simplify contact.create

Merge pull request #23545 from eileenmcnaughton/import_location2

Remove never-true fields, and impossible property handling from previously shared function

Merge pull request #23488 from colemanw/afformGetHook

Afform - dispatch event when loading afforms

Merge pull request #23536 from eileenmcnaughton/import_cust

[REF] Minor simplification on input

Merge pull request #23537 from eileenmcnaughton/greet_cust

Greeting handling - if email_greeting_custom (etc) isset & email_greeting_id is not, set to 'Customized'

Merge pull request #23540 from eileenmcnaughton/import_location

Add location import testing

Merge pull request #23538 from eileenmcnaughton/import_regression

Fix unreleased regression from strict typing

Merge pull request #22682 from agileware/CIVICRM-1922

🪳🪳🪳 Bug Fix: Merging an Organisation contact and de-selecting the merge Membership checkbox causes all related Membership to not be removed and the Primary Membership is linked to the deleted Organisation

Merge pull request #22487 from mattwire/repeattransactiontemplatecontribution

dev/financial#6 Use template contribution for Contribution.repeattransaction

Merge pull request #23467 from christianwach/event-caps

Capitalise event setting title

Merge pull request #23541 from eileenmcnaughton/import_provider

E-notice fix on import

Capitalise event setting title

Remove never-true fields, and impossible property handling from previously shared function

Merge pull request #23542 from eileenmcnaughton/import_location2

Duplicate function - divide & conquer

Merge pull request #23544 from colemanw/revertPost

Revert "SearchKit - Use POST to send contact ids to action forms"

Revert "SearchKit - Use POST to send contact ids to action forms"

This reverts commit 1297a574fd3e28d4714c145627b35c08e5933675.

Duplicate function - divide & conquer

Add location import testing

E-notice fix

Merge pull request #23539 from demeritcowboy/huh

[NFC] Remove weird file

weird file

Merge pull request #23404 from colemanw/getFieldsFilters

SearchKit - Make inline edit only available when applicable

Fix unreleased regression from strict typing

[REF] Minor simplification on input

Greeting handling - if email_greeting_custom (etc) isset & email_greeting_id is not, set to 'Customized'

Merge pull request #23502 from eileenmcnaughton/import_test_ex

Add unit tests to cover date field imports

APIv4 GetFields - Ignore invalid fields rather than fatal error in getFields action

SearchKit - ensure only relevant fields are editable

Fixes dev/core#3423
Ensures that custom fields for one entity sub-type (e.g. Individual) are not
presented as editable for other sub-types (e.g. Organiztion).

Remove always true if conditions (whitespace in next commit)

Merge pull request #23531 from eileenmcnaughton/import_goned

[Import] Remove empty if

Merge pull request #23530 from eileenmcnaughton/import_gone

Remove always-true if-check

Merge pull request #23496 from colemanw/fixCustomGroupJoinable

SearchKit - Fix in-place editing custom relationship fields

Switch now-tested birth & deceased date handling to be metadata based

Add unit tests to cover date field imports

[REF] Minor simplification on input

Merge pull request #23509 from eileenmcnaughton/import_except

[Import] Remove handling for unavailable field current_employer

Remove extraneous deprecated check

[Import] Remove empty if

Remove always-true if-check

Merge pull request #23508 from eileenmcnaughton/import_subkid

[Import] Fix & test check on contact subtype change

Merge pull request #23526 from eileenmcnaughton/import_sub

[Import] Check subtype validity in validate rather than wait for 'imort'

SearchKit - Fix in-place editing custom relationship fields for related contacts

Before: In-place edit updates the wrong record for custom relationship fields
After: Fixed

Merge pull request #23525 from aydun/yesno

Fixes dev/core#3475 - Allow 'yesno' as a valid value of `html_type`

Merge pull request #23331 from colemanw/post

SearchKit - Use POST to send contact ids to action forms

[Import] Check subtype validity in validate rather than wait for 'import', test

Fixes dev/core#3475 - Allow 'yesno' as a valid value of `html_type`I

Merge pull request #23521 from eileenmcnaughton/dead

Ensure setting deceased date sets is_deceased in the BAO

Merge pull request #23524 from mattwire/contributiontestfixes

Payment forms test fixes

Merge pull request #23523 from eileenmcnaughton/import_little

Re-use code in test

Build form properly for CRM_Financial_Form_PaymentEditTest::testSubmitOnCheckNumberChange

Build form properly for PaymentEditTest::testSubmitOnPaymentInstrumentChange

Build form properly for FinancialItemTest::testGetPreviousFinancialItemHavingTax

Re-use code in test

Merge pull request #23518 from eileenmcnaughton/import_meta

[REF] Move list of fields that have been upgraded

Ensure setting deceased date sets is_deceased in the BAO

Merge pull request #23517 from eileenmcnaughton/import_comments

[NFC] Improve comment blocks for mapper

[REF] Move list of fields that have been upgraded

Improve comment blocks for mapper

Merge pull request #23511 from eileenmcnaughton/import_unused

[Ref] [Import] Cleanup function return

Merge pull request #23498 from eileenmcnaughton/import_cust_extract

Ref Extract import validation

Merge pull request #23512 from eileenmcnaughton/part_count

Cleanup sql query

[REF] Simplify contact.create

Merge pull request #23504 from eileenmcnaughton/import_replace

[Import] Remove svn whimsy - DUPLICATE_REPLACE

Merge pull request #23501 from eileenmcnaughton/import_json

Fix use of not-universally supported mysql field type

Merge pull request #23510 from yashodha/ckeditor_issue_5

fix to get around link issues in ckeditor when opening scheduled remi…

Cleanup sql query

On digging into this query I found that is uses protected words like value & count
so I have fixed the query to no longer use those & re-formatted to add comments.

I suspect the actual fix might need to sit around pending community input so I think it is good to
get this tidy up merged to make it all clearer

[Ref] [Import] Cleanup function return

fix to get around link issues in ckeditor when opening scheduled reminder in popup

Merge pull request #23507 from eileenmcnaughton/import_match2

[REF] Remove unreachable code

Merge pull request #23495 from eileenmcnaughton/import_unused

[REF] [Import] Remove unused parameter

Merge pull request #23493 from eileenmcnaughton/import_rel

[Import] Remove silly param

[Import] Remove handling for unavailable field current_employer

[Import] Fix & test check on contact subtype change

[REF] Remove unreachable code

[Import] Remove svn whimsy - DUPLICATE_REPLACE

Add unit tests to cover date field imports

Fix use of not-universally supported mysql field type

Merge pull request #23500 from civicrm/5.50

5.50 to master

Merge pull request #23497 from totten/5.50-schedrem-bool

Restore `civicrm_action_schedule.limit_to` (5.50)

[Import] Remove silly param

Merge pull request #23494 from eileenmcnaughton/import_if

[Import] Remove `if (1) {` (whitespace intensive)

Ref Extract import validation

Merge pull request #23492 from totten/master-silly-method

(REF) CRM_Upgrade_Form - Remove silly method `runQuery`

FiveFortyNine - Tweak task title

FiveFortyNine - Revise message about "limit to"

1. This fixes a problem where the message shows at the opposite-of-correct time
   (ie it should display on builds that have already run 5.49.{beta,0,1}, but
   it actually displayed on builds that have not run 5.49.{beta,0,1})

2. This makes the pre+post messages match.

FiveFortyNine - Don't munge soft credit 'limit-to'

I'm not sure what it even means to limit a scheduled-reminder to a "soft_credit_type".

But it's a thing - I can use the 5.48 GUI to create it. Here's how it looks in the DB (in 5.48):

+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| id | title                                    | recipient        | limit_to | recipient_manual | recipient_listing | group_id |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+
| 11 | Limit To - Soft Credit                   | soft_credit_type |        1 | NULL             | in_memory_of      |     NULL |
+----+------------------------------------------+------------------+----------+------------------+-------------------+----------+

The `changeBooleanColumnLimitTo()` was coercing down to `limit_to=null` because it lacked `recipient_manual` and `group_id`.

FiveFortyNine - Fix guard on `changeBooleanColumnLimitTo()`

add pre+post upgrade message

FiveFortyNine - Draft message for 5.49.2

Revert civicrm_action_schedule.limit_to boolean column to be NULL

[REF] [Import] Remove unused parameter

[Import] Remove if (1) {

(REF) CRM_Upgrade_Form - Remove silly method `runQuery`

This function just wraps `CRM_Core_DAO::executeQuery`, and it's only
used once.