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

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

Figure out & test what happens with  ext identifier & type, subtype combos

Merge pull request #23464 from eileenmcnaughton/import_on_dup

[REF] [Import] Minor simplification

APIv4 - Respect max length for name with export action

Merge pull request #23459 from darrick/dev/core#2300

dev/core#2300 Can not use Custom Fields defined on a contact_sub_type in dedupe…

[REF] [Import] Stop calling Parser in MapField mode

Merge pull request #23463 from eileenmcnaughton/import_proc

[REF] [Import] Remove unused variable

Merge pull request #23466 from totten/master-timeout-guzzle

CRM_Utils_Check - Catch Guzzle exception, same as core exception

Merge pull request #23457 from eileenmcnaughton/import_more_part

Consolidate datasource form - event import

CRM_Utils_Check - Catch Guzzle exception, same as core exception

Overview
--------

`checkExtensions()` is supposed to show status-messages which report on
any out-of-date extensions.  This requires a list of available updates.

Alas, the Internet is a big and scary place, and sometimes the list is
unavailable.  DNS servers, HTTP servers, timeouts, yadda yadda.  If it can't
get the list of extensions, it should show a status-message about this
problem.

Before
------

Throws an exception, causing a full failure in the page-view that does
the status-check.

After
-----

Catches the exception. Reports a status-message about the failure.

Comments
--------

It appears that this was probably the behavior before (ie `checkExtension()`
already has a try/catch for this kind of problem). However, the underlying
HTTP request was changed a versions back (ie from `HttpClient` to `Guzzle`)
which means that errors can get reported in other ways.

To simulate errors, I did this:

1. Run buggy/placeholder TCP service (`nc -l 8199`)
2. Hack `CRM_Extension_Browser::__construct()` to set `$this->repoUrl` to `http://localhost:8199`
3. Run `cv api System.check`

Merge pull request #23289 from eileenmcnaughton/import_summ

[REF] [Import] Activity - clean up copy & paste