Remove deprecated code ids['userId']

These lines were deprecated 7 months ago and were pretty edge case to start with

Merge pull request #18135 from colemanw/daoVersions

API - Add upgrade-safe checks to ensure table exists before reading

Merge pull request #18138 from eileenmcnaughton/dao

Deprecate 'trapException' in DAO::executeQuery

Merge pull request #18139 from totten/master-region-bits

(REF) CRM_Core_Region - Remove unused bits

Deprecate 'trapException' inn DAO::executeQuery

    We are pretty heavily moving towards using exceptions so we should deprecate this. It seems the
    fillWithSql is just converting the error to an exception so we shouldn't avoid it

Suppress errors when reading DAO version constants

DAO - only check version on fields from core tables

Log warning when attempting to access nonexistant table from api

Cleanup CRM_Utils_Check version safeguard with APIv4

Early return during APIv4::get if table doesn't exist

If there are pending upgrades, checks if table exists before attempting to read from it.

Regenerate DAO files

CodeGen - Add EXT and TABLE_ADDED to each DAO file

Merge pull request #17956 from eileenmcnaughton/export_phone

dev/core#1661 Allow phones with types longer than 16 chars to export

Merge pull request #18131 from demeritcowboy/move-isssldsn

[REF] Move isSSLDSN() function to avoid potential problems

Merge pull request #18134 from colemanw/deprecateCheckVersion

Deprecate/remove unused checkVersion() functions

Merge pull request #18132 from yashodha/dev-1944

(dev/core#1944) Add new columns to mailing summary report

Deprecate unused CRM_Core_DAO::checkVersion() and remove unused CRM_Upgrade_Form::checkVersion()

move function to avoid problems

Merge pull request #18094 from colemanw/ets

CodeGen - Add use statement to extensions DAO files

(dev/core#1944) Add new columns to mailing summary report

DAO.tpl - use tsFunctionName for extension support

Merge pull request #18128 from civicrm/5.29

5.29

Merge pull request #18126 from eileenmcnaughton/dupe529

dev/core#1934 fix regression on merging contacts with settings using contact_id

Merge pull request #18123 from seamuslee001/dev_core_1936_alternate

dev/core#1936 Make the price field value label field not required

dev/core#1934 fix regression on merging contacts with settings using contact_id

This is an interim fix to a reported regression. I'll look at more carefully in master when time permits

dev/core#1936 Make the price field value label field not required

Fix test and ensure default value is NULL

Merge pull request #18095 from demeritcowboy/no-db

dev/core#1926 - Towards supporting SSL for mysql connections - remove DB::connect that doesn't add anything

Merge pull request #18122 from seamuslee001/master

5.29

Merge in 5.29

Merge pull request #18116 from mattwire/phpnoticecontributionpage

Fix PHP notice on contribution page

Merge pull request #18119 from demeritcowboy/document-hidden-buttons

[NFC] Add code comments about hidden but still used buttons

Merge pull request #18115 from sunilpawar/dev_1943

dev/core#1943 add functionality to add civicrm log into Drupal access log

comments about hidden but still used buttons

Merge pull request #18117 from mattwire/529_master

5.29 to master

More radical cleanup of button CSS

Fix CSS for new buttons

Fix JS for input buttons that are now button buttons

Remove deprecated submitOnce js function

No need for crm-button wrappers for real buttons

Give buttons a value when the submitted value is used to identify them

Put icons inside of button elements

Swap out button/submit inputs for button elements

Correction to change priority level

Merge pull request #18091 from agh1/xbutton-529

Swap out button/submit inputs for button elements

Fix PHP notice on contribution page

small correction

dev/core#1943 add functionality to add civicrm log into Drupal access log

Merge pull request #18112 from eileenmcnaughton/paytest

[NFC] [Test] Initial testing on event payment forms.

[Ref] [Test] Update Authorize.net test to use the trait

[NFC] [Test] Intial testing on event payment forms.

I'm trying to work on improving our testing of payment forms. I started with 3 event forms but there were
enough challenges that for this commit I'm just adding tests on one form. As noted in the code comments
it makes sense for this test to be in the extension but I want to work through the challenges on the
other forms before finalising any helper functions to the point where they are available to extensions.

This is actually a bit of a break through as it's the first time we have testing on a form flow - ie
submitting the first form and then the second. It was quite painful

Merge pull request #18110 from eileenmcnaughton/xml

Add Serialize key to payment_processor field for Event & Contribution Page

Merge pull request #18109 from eileenmcnaughton/format

[NFC] Array formatting

Merge pull request #18108 from eileenmcnaughton/paytest

Remove unused variable

Add Serialize key to payment_processor field for Event & ContributionPage

The DAO can handle arrays if it knows the format now...

Remove unused variable

I spotted this variable as defined but unused - let's remove

[NFC] Array formatting

This is a straight up IDE reformat to get rid of old array syntax.

IDE had some whitespace thoughts with I'm neutral on - if Jenkins is too then all good IMHO

Merge pull request #18104 from civicrm/5.29

5.29

Merge pull request #18101 from demeritcowboy/eventcart-settingcheck

dev/event#40 - EventCart - Check legacy setting until extension is public

Merge pull request #18100 from civicrm/5.29

5.29

check legacy setting until extension is public

Merge pull request #18068 from seamuslee001/dev_wordpress_66

dev/wordpress#66 Re-instate newer variables but with more support for…

Merge pull request #18097 from demeritcowboy/mysqli

There is no "I" in mysqli - oh wait there is

Merge pull request #18098 from civicrm/5.29

5.29 to master

Merge pull request #18092 from seamuslee001/preimum_onbehalfof_fix

[REF] Fix jquery validation for on behalf of fields when combined wit…

Merge pull request #17919 from seamuslee001/nfc_locale_documentation

[NFC] Update locale over-ride documentation to mention that it is mos…

mysqli

REF - add helper function to get codeGen tpl

unneeded connect call

Merge pull request #18093 from colemanw/expFix

APIv4 Explorer - Fix possible undefined index

CodeGen - Add use statement to extensions DAO files

Merge pull request #17934 from eileenmcnaughton/conttype

[REF] Update subtypeInfo function to leverage getAllContactTypes

APIv4 Explorer - Fix possible undefined index

Merge pull request #18047 from eileenmcnaughton/ent_ref

[REF] Fix metadata label for contribution_page_id

[REF] Fix jquery validation for on behalf of fields when combined with a preimum

Merge pull request #18090 from civicrm/5.29

5.29

Merge pull request #17950 from lcdservices/dev-core-1895

dev/core#1895 fix first/last name adv search

More radical cleanup of button CSS

Fix CSS for new buttons

Fix JS for input buttons that are now button buttons

Remove deprecated submitOnce js function

No need for crm-button wrappers for real buttons

Give buttons a value when the submitted value is used to identify them

Put icons inside of button elements

Swap out button/submit inputs for button elements

Merge pull request #18088 from agh1/forcebackend

dev/core#1905 force backend links for new "configure" buttons

Merge pull request #18089 from colemanw/api4cv

APIv4 Explorer: Generate short cv code

Merge pull request #18085 from totten/5.29-periodic

dev/core#1932 - Make status-checks more polite during upgrade

dev/wordpress#66 Re-instate newer variables but with more support for legacy file systems

APIv4 Explorer: Generate short cv code

dev/core#1905 force backend links for new "configure" buttons

dev/core#1932 - Make status-checks more polite during upgrade

Before
------

If you happen to run a status-check (eg `showPeriodicAlerts()`) in the interim between
downloading code and running DB upgrades, then you'll get weird failures.

After
-----

The status-checks are able to complete.

Technical Details
-----------------

(1) I figure that this conditional is lightweight because it relies on data
that's read-once and then cached in memory:

```
CRM_Utils_System::version() !== CRM_Core_BAO_Domain::version()
```

(2) The easiest way to reproduce is to get a DB from before 4.7 and coerce
`CRM_Utils_Check::CHECK_TIMER`.

Merge pull request #18027 from seamuslee001/pear_db_composer

[REF] Deploy Pear DB package using composer

Set version to 5.30.alpha1

Set version to 5.29.beta1

Merge pull request #18080 from seamuslee001/dev_core_1928

dev/core#1928 Fix HTML5 error due to required attribute being set swi…

Merge pull request #18084 from civicrm/5.28

5.28

Merge pull request #18083 from totten/5.28-rn

(NFC) 5.28.0.md - Describe last minute PR. Random copyedits.

Merge pull request #18082 from seamuslee001/master

5.28

5.28.0.md - Describe last minute PR. Random copyedits.

Merge in 5.28