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

This reverts commit 1297a574fd3e28d4714c145627b35c08e5933675.

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

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).

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

[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

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.

Merge pull request #23476 from eileenmcnaughton/import_dupe_more

[Import] Simplify checking contact type is valid

SearchKit - Use POST to send contact ids to action forms

Merge pull request #23489 from colemanw/filterCustomFields

APIv4 - Filter custom fields based on supplied values

Validate subtype change

[Import] Simplify checking contact type is valid

[Import] Duplicate finding cleanup

Text user supplied dedupe rule is used during import.

[REF] [Import] Extraction of get related contacts function

Merge pull request #23484 from eileenmcnaughton/import_get_type

Cleanup gender handling in contact import + add test

APIv4 - Filter custom fields based on supplied values

This allows targeted getfields for a particular entity or type of entity

Merge pull request #23461 from eileenmcnaughton/import_woohoo

[REF] [Import] Remove now-unused code

Cleanup gender handling in contact import

Merge pull request #23470 from eileenmcnaughton/import_dataform

[REF] [Import] Stop calling Parser in MapField mode (contact import)

Merge pull request #23483 from eileenmcnaughton/import_get

Painstaking unravelling of function

Merge pull request #23475 from eileenmcnaughton/imp_fold

[Import] Add tests for contact type & contact subtype handling

Merge pull request #23452 from colemanw/exportCustomFieldWithOptions

APIv4 - Export options along with custom field

Fold array building in

Set up related Contact Key earlier

Further simplification

Simply value building

Simplify related contact assignment

Remove always-true-if

Merge pull request #23481 from alifrumin/rn5.50

[NFC] First pass at 5.50 release notes

[NFC] first stab at 5.50 release notes

Merge pull request #23480 from demeritcowboy/too-many-questions

Remove extra question mark from url

remove extra question mark

Merge pull request #23469 from mlutfy/schemaTranslation

Fix Schema System Check translation and show missing Custom Field ID

APIv4 - Export option groups along with custom fields

Fix Schema System Check translation

Merge pull request #23446 from colemanw/searchKitDefaultAgg

SearchKit - Pick better default for aggregate functions

Merge pull request #23472 from colemanw/exportMaxLentgh

APIv4 - Respect max length for name with export action