Merge pull request #18182 from colemanw/severity

Remove unnecessary transformation of upgrade msg severity

Merge pull request #18114 from eileenmcnaughton/phone

Metadata fix - phone_type_id, location_type_id, gender_id

Merge pull request #18168 from twocs/patch-4

Check if $post defined before trying to use its field values.

Merge pull request #18193 from civicrm/5.29

5.29 to master

Merge pull request #18180 from eileenmcnaughton/recur

dev/core#1945 Fix recur access regression

Merge pull request #18188 from demeritcowboy/mysqli-mysql

Put mysql back in civicrm.settings.php template

dev/core#1945 Fix recur access regression

Merge pull request #18189 from demeritcowboy/widget-notice

E_NOTICE on contribution page widget tab

e_notice on widget tab

put mysql back

Merge pull request #18183 from colemanw/versionCheck

CRM_Utils_VersionCheck - respect force param

Merge pull request #18174 from demeritcowboy/d8-upgrade-warning

dev/core#1937 - Upgrade message about needing composer patching turned on and updating mysql in DSN strings

Merge pull request #18187 from seamuslee001/master

5.29

Merge in 5.29

Merge pull request #18185 from civicrm/revert-18091-xbutton-529

Revert "Swap out button/submit inputs for button elements"

Merge pull request #18184 from totten/master-dao

(NFC) Update various DAO checksums

Revert "Swap out button/submit inputs for button elements"

upgrade message about composer patches

Update various DAO checksums

Merge pull request #18169 from demeritcowboy/no-discount

E_WARNING when saving event fees admin page if there's no discounts set

CRM_Utils_VersionCheck - respect force param

Remove unnecessary transformation of upgrade msg severity

Merge pull request #18181 from colemanw/upgrademsg

Upgrade screen - show success instead of error if already upgraded

Upgrade screen - show success instead of error if already upgraded

Merge pull request #18177 from eileenmcnaughton/gross

Processors - remove gross_amount param from processors

Merge pull request #18173 from civicrm/5.29

5.29

Merge pull request #18176 from agileware/CIVICRM-1465

{contribution.receipt_date} token does not use any CiviCRM date formatter, output in YYYY-MM-DD HH:MM:SS format and {contribution.receive_date} also uses a non-standard format

Processors - remove gross_amount param from processors

The gross_amount param is returned by a bunch of processors but is not used so this removes it.

The core processors are often used as a starting point for extensions - we can see for example
this parameter is present in IATS so removing it helps clarify it as unnecessary & unused

Merge pull request #18175 from demeritcowboy/installation-doclinks

Installation doclinks not getting url-rewritten

Merge pull request #18170 from eileenmcnaughton/testc

Minor cleanup in test class

installation doclinks

Merge pull request #18171 from demeritcowboy/unused-UTF8

[REF] Deprecate unused SQL temptable functions

Merge pull request #18166 from demeritcowboy/cvv-required

cvv required html attribute should depend on backoffice setting

Merge pull request #18154 from highfalutin/patch-5

Mass SMS: Limit group selection to mailing groups

Merge pull request #18172 from demeritcowboy/utilsarray-isset

[REF] Update recent replacement for CRM_Utils_Array::value

deprecate unused functions

update replacement for utils_array::value

Merge pull request #18165 from civicrm/5.29

5.29

Minor cleanup in test class

Merge pull request #18157 from mattwire/utilsarraycontributionmembership

Replace CRM_Utils_Array::value in contribution/confirm and bao/membership

element doesn't exist if there's no discounts set

Check if $post defined before trying to use its field values.

Admin pages (e.g. in a custom plugin) may see $post as undefined. The existing usage to get permalinks for these pages can fill debug logs with PHP Notices.

required

Merge pull request #18163 from seamuslee001/5.29

[REF] Remove unnecessary comma

[REF] Remove unnecessary comma

Merge pull request #18159 from civicrm/5.29

5.29

Replace CRM_Utils_Array::value in CRM_Member_BAO_Membership

Merge pull request #18133 from eileenmcnaughton/utf529

Fix 5.29 (unreleased) regression using temp tables

Replace CRM_Utils_Array::value in CRM_Contribute_Form_Contribution_Confirm

Merge pull request #18155 from eileenmcnaughton/member_order

Remove legacy handling for 'fixing' line_item.entity_id

Merge pull request #18156 from eileenmcnaughton/renew

Remove deprecated code ids['userId']

Remove deprecated code ids['userId']

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

Remove legacy handling for 'fixing' line_item.entity_id

I tried to see what was happening with the test in https://github.com/civicrm/civicrm-core/pull/18113
and spotted that it wasn't using Order.api so the line items were likely wrong. However, once I set
it up to use the Order api I found the code we added back in 2018 to attempt to cope with other code
setting entity_id incorrectly was actually itself setting entity_id incorrectly. The issue
happens when there are 2 Memberships linked to one contribution & the removed code 'decides'
one is wrong and alters it. https://github.com/civicrm/civicrm-core/pull/11816

This line of code throws a deprecation notice - which has not been reported / apparently seen in the last
couple of years so I'm assuming it's not actually doing any good and without removing it this test actually
fails

Mass SMS: Limit group selection to mailing groups

To populate the selector for mass SMS recipients, the form calls CRM_Core_PseudoConstant::nestedGroup(). However the function call was missing an argument, so all groups were being returned, not just mailing groups as intended. This PR corrects the mistake.

Merge pull request #18142 from seamuslee001/ref_space_currency

[REF] Remove usages of CRM_Utils_Money::format that pass in a blank s…

Merge pull request #18148 from civicrm/5.29

5.29

Merge pull request #18151 from eileenmcnaughton/money

Use more appropriate money format function

[REF] Remove usages of CRM_Utils_Money::format that pass in a blank space as a currency

Use formatLocaleNumericRoundedForDefaultCurrency instead as that achieves the same result

Merge pull request #18149 from seamuslee001/dev_core_1952

dev/core#1952 Remove uncessary component checking when exporting all …

Fix 5.29 regression using temp tables

I just pushed 5.29rc through our WMF CI & it failed on a situation where it created a temp
table and that got joined on a non-temp table with a collation mix error. The issue is that the
DB default is set to utf8_general_ci (at least on our development DBs) whereas the
civicrm tables are utf8_unicode_ci. Until https://github.com/civicrm/civicrm-core/pull/18012/files
there was a list of permitted matches for db collation but now any 'utf' string matches.

From what I can tell utf8_general_ci  is a pretty common collation on drupal DBs and from my tests
it causes problems if not excluded here

Use more appropriate money format function

dev/core#1952 Remove uncessary component checking when exporting all activities

Merge pull request #18144 from seamuslee001/dev_core_1953

dev/core#1953 Ensure that Contribution pages do not fail validation o…

dev/core#1953 Ensure that Contribution pages do not fail validation on credit cards when a zero dollar price is offered

Metadata fix - phone_type_id, location_type_id, gender_id

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