Remove validation bypass

I was trying to understand why executeQuery allows validation to be bypassed. That seems
intuitively like something that should be handled by the calling function. This is one of the few
places that calls it - but I can't see a reason for it to by pass validation....

My best guess is that a parameter could legitimately be NULL - and the group properly is the
only one that seems to be a candidate

I also note the case for passing i18nRewrite seems dubious as the table has no localisation

Merge pull request #17330 from colemanw/dao_field

[REF] Cleanup and extract string functions specific to entity names.

Merge pull request #17304 from colemanw/endCode

Switch docblock code to use markdown syntax.

Merge pull request #17349 from eileenmcnaughton/validate

[REF] Stop passing  by reference.

Merge pull request #17347 from eileenmcnaughton/status

Call completeOrder directly from contribute status update form

Merge pull request #17343 from eileenmcnaughton/js_curr

Use status bounces, not fatal, in front end contribution form base class

[REF] Stop passing  by reference.

By checking in BAOContribution::loadRelatedObjects we can see it is not modified. The other 2 functions
only pass input to that function, so can also be non-variables

Call completeOrder directly from contribute status update form

We should really call completetransaction but that has been left for later unravelling with the focus here being
to clarify the purpose of the transaction. We don't need to pass a transaction around unless it has DB changes not
yet committed - which is not eally the case here. There is no reason to invoke BaseIPN but more unravelling required...

Merge pull request #17341 from seamuslee001/fatal_exception_status_bounce

dev/core#560 Replace deprecated CRM_Core_Error::fatal with either sta…

Use status bounces, not fatal, in front end contribution form base class

Merge pull request #17342 from eileenmcnaughton/event

Remove code that still seems unnecessary.

Merge pull request #17338 from demeritcowboy/test-from-to

[TEST] dev/core#1766 - Remove flakiness from testGetFromTo tests

Remove code that still seems unnecessary.

Per the code comment we should not need to unset these fields as they should not be present if not required.

I guess the test for this is whether the fields are unset if we start off with a payment processor & then deselect
it in favour of pay later

Merge pull request #17340 from demeritcowboy/not-yet-deprecated

[NFC] Add comment that deprecated code is actually still reachable

dev/core#560 Replace depreacted CRM_Core_Error::fatal with either statusBounce or Exceptions as appropriate in CRM/Core folder update langauge as per demeritcowboy

Merge pull request #17336 from seamuslee001/fatal_exception_status_bounce

dev/core#560 Replace some instances of CRM_Core_Error::fatal with an …

Merge pull request #17319 from agh1/icon-links

Extend icon helper support, including for formLink

code is still reachable

Merge pull request #17297 from agh1/treeminus-gone

Removed last TreePlus.gif and TreeMinus.gif spots

switch away from dataprovider for time-sensitive test

Merge pull request #17337 from civicrm/5.26

5.26

Merge pull request #17331 from colemanw/briefName

[REF] CRM_Core_DAO_AllCoreTables::getBriefName to accept BAO name

Merge pull request #17335 from demeritcowboy/missing-semicolon

[REF] Missing semicolon on smtp settings page, code cleanup in file-on-case, add test

Merge pull request #17334 from seamuslee001/fix_d8_install

[REF] Fix install on Drupal 8 using new setup code

dev/core#560 Replace some instances of CRM_Core_Error::fatal with an exception or status bounce

[REF] Fix install on Drupal 8 using new setup code

missing semicolon, other cleanup, add test

Merge pull request #17291 from mfb/tracking-default

Allow click-through and open tracking to be enabled/disabled by default

Fix CRM_Core_DAO_AllCoreTables::getBriefName to accept BAO name

This function was overly strict about not accepting BAO names, prompting end-users of the function
to do their own ad-hoc conversions before using it.

[REF] Cleanup and extract string functions specific to entity names.

The function CRM_Utils_String::convertStringToCamel was being stretched to also work in entity name oddities
And an api3 utility function was being over-used.
This adds symmetrical utility functions in AllCoreTables to covert entity names between the 2 formats.

Merge pull request #17323 from JKingsnorth/patch-14

dev/core#1762 Don't log subscription_history

Merge pull request #17296 from agh1/fa-accordions

Accordion open-close with Font Awesome icons

Merge pull request #17327 from demeritcowboy/poland-lower-case

[NFC] Add comment regarding lower case for Polish states

Merge pull request #17329 from civicrm/5.26

5.26

Merge pull request #17261 from seamuslee001/dev_core_1681

dev/core#1681 Add in deprecation notice for Systems using MySQL versions before 5.7 and require 5.5 for install

Merge pull request #17277 from mattwire/fatalerrorhandler

Trigger fatalErrorHandler for PEAR errors and for logs with severity ERROR or higher

Merge pull request #17307 from agh1/privacy-flag-conditional

Comm prefs: add condition param for {privacyFlag}

Merge pull request #17310 from agh1/autorenew-active-icon

Member tab: auto-renew status was hidden from screen readers

Merge pull request #17326 from demeritcowboy/ts-params

Remove unneeded parameters in {ts} on membership form

Merge pull request #17320 from totten/master-url-type

(NFC) CRM_Utils_Url - Fix declared type

prevent someone thinking this is wrong

unneeded parameters

Merge pull request #17322 from eileenmcnaughton/ids

(NFC) Remove some cvs docblock help

Merge pull request #16992 from colemanw/customSerialize

Add serialize column to civicrm_custom_field, remove "Multi-Select" html type

Merge pull request #17314 from jitendrapurohit/dev-1758

dev/core#1758 - Fix check number and trxn id values on record payment…

Merge pull request #17285 from agh1/no-rendered-icons-in-data-source

No rendered icons in data source

Merge pull request #17303 from agh1/no-icon-crmbutton

crmButton: support icon=0 for no icon

dev/core#1762 Don't log subscription_history

Remove some cvs docblock help

The $ pattern turns out to be cvs support. We agreed the whole block should only
stay if it adds extra info

(NFC) CRM_Utils_Url - Fix declared type

Before
------

References a non-existent type `\GuzzleHttp\Psr7\UriInterface`.

After
-----

References an existent type `\Psr\Http\Message\UriInterface`.

Merge pull request #17306 from colemanw/priceFieldValue

Cleanup PriceFieldValue BAO

Set minimum install MySQL version to be 5.5 and a recommended minimum of 5.7

Fix to use min install version

Add in deprecation notice for MySQL/MariaDB versions below 5.7

Merge pull request #17078 from herbdool/ui-18

Ui 18

Merge pull request #17116 from lcdservices/dev-core-1721

dev/core#1721 send email to case follow-up activity assignees

Fix setting profile defaults

Fix dedupe merger handling of serialized custom fields

Update contact import parser to validate serialized custom fields

Custom Field serialization test fixes

Cleanup references to custom field html_type

Update CustomField form with 'serialize' option

Eliminate Multi-Select field types

APIv4 - Update custom field spec for serialize column

Rely on serialize property when checking if custom field is multivalued

APIv3 - Legacy handling for old way of naming serialized fields

Add serialize column to civicrm_custom_field

Don't send bare markup to formLink, support icon link attrib instead

Merge pull request #17311 from KarinG/clarify_amount

Clarify definition of amount field in civicrm_contribution_recur table.

Merge pull request #17313 from civicrm/5.26

5.26

PHPUnit test for icon construction

icon helpers - allow specification of HTML attributes

Update icons with aria-hidden

Common.js Icons should be aria-hidden

Merge pull request #17316 from agh1/ab-test-flask-icon

A/B test: icon should be Erlenmeyer flask, not bar chart

A/B test: icon should be Erlenmeyer flask, not bar chart

Switch docblock code to use markdown syntax.

We are moving toward markdown in docblock comments. This is a step in that direction.

dev/core#1758 - Fix check number and trxn id values on record payment form

Merge pull request #17265 from MegaphoneJon/mime-error-rc

[regression] Fix error on no MIME type

Merge pull request #17309 from colemanw/MailingAB

[REF] Cleanup MailingAB BAO

Clarify definition of amount field in civicrm_contribution_recur table.

Merge pull request #17293 from agh1/sr-wrenches

Screen reader text for wrenches

Merge pull request #17294 from agh1/sr-rel-perms

sr-only text for relationship permissions

Member tab: auto-renew status was aria-hidden

Cleanup MailingAB BAO

- Use writeRecord function instead of boilerplate.
- Use api defaults for domain_id instead of setting in BAO:
  - APIv3 default domain is set via _civicrm_api3_mailing_a_b_create_spec() function.
  - APIv4 FieldDomainIdSpecProvider will set it automatically based on the required property in the schema.

CRM.utils.formatIcon: updated function name and made condition optional

Comm prefs: add condition param for {privacyFlag}

Cleanup PriceFieldValue BAO

- Use writeRecord() helper function
- Remove $ids param from add()
- Make sense of previously unused $priceFieldID variable

crmButton: support icon=0 for no icon

Update CRM/Core/Form/Renderer.php

Adopting @colemanw's suggestion since it's more of a concern that the label be blank than undefined.

Co-authored-by: colemanw <coleman@civicrm.org>

Merge pull request #17301 from eileenmcnaughton/recur

dev/core#1728 Ensure that custom fields that extend specified financi…

Merge pull request #17302 from eileenmcnaughton/mem

[REF] Minor tidy up

Merge pull request #17246 from agh1/review-contrib

Proceeding to confirmation page should be a "next" button not a check

[REF] Minor tidy up

Removes a few old constructs like the value() function

Alternate to #17156

Merge pull request #17299 from eileenmcnaughton/recur

Add ContributionRecur crud api

Merge pull request #17295 from agh1/fa-more

Replaced more> icon with Font Awesome

Add ContributionRecur crud api

Removed last TreePlus.gif and TreeMinus.gif spots

Event aggregate custom search: Remove archaic form patterns

Get rid of unused CRM_Core_ShowHideBlocks::links()