Optimise proximity custom search, by reducing addGeocodingData fn call

Merge pull request #18046 from eileenmcnaughton/trans

[REF] Remove transaction from completeOrder signature

Merge pull request #17966 from pradpnayak/divwrap

Wrap multi record custom field inside a div

Merge pull request #18045 from eileenmcnaughton/sleep

[Test fix] We might need this to ensure really quick test runs don't fail

[REF] Remove transaction from completeOrder signature

It is no longer passed in

Merge pull request #17706 from demeritcowboy/mysql-ssl-alt

dev/core#1137 - Allow ssl connection to mysql by specifying in DSN

Merge pull request #18038 from eileenmcnaughton/perm_test

Test - attempt to replicate #17852

Merge pull request #18042 from eileenmcnaughton/transact

[REF] Remove transaction from BaseIPN completeTransaction call

Merge pull request #17700 from ixiam/dev-#1843

Remove url-tracking in mass sms. dev/core#1843

Merge pull request #17972 from eileenmcnaughton/matt_another

Do not overwrite values saved from the repeatContribution routine

Merge pull request #18043 from demeritcowboy/liar-fields

dev/core#1916 - Fix naming of case export fields / remove ones that aren't true

Merge pull request #17824 from colemanw/checks

SystemCheck: add ability to efficiently run only specified checks

[Test fix] We might need this to ensure really quick test runs don't fail

Fix naming of fields /remove ones that aren't true

[REF] Remove transaction from BaseIPN completeTransaction call

This is now never passed in

Fix code to not overwrite source contribution - unit test for 17455

Merge pull request #18028 from eileenmcnaughton/anet

Add testing to Authorize.net and remove the lines that are repeated

Merge pull request #18037 from eileenmcnaughton/status_check

Add test on status calculation

Merge pull request #18041 from demeritcowboy/status-type-whats-the-diff

Fix qill typo

fix qill typo

Merge pull request #18015 from demeritcowboy/resubscribe-missing

dev/core#1919 - Missing resubscribe url in text/plain version of unsubscribe confirmation email

Merge pull request #18029 from eileenmcnaughton/sendmail

[Ref] Simplify is_email_receipt in sendMail

Merge pull request #17912 from eileenmcnaughton/flex2

Update flexmailer release information

Merge pull request #18019 from eileenmcnaughton/event_clean

[REF] [Test] Minor simplification on test

Test - attempt to replicate #17852

Add testing to Authorize.net and remove the lines that are repeated

In Authorize.netIPN updates are made to the contributionRecur that are later made in ContributionRecur::updateOnNewPayment

This adds test cover to ensure no change and then removes them.

With these lines gone it's apparent that the transaction is not required here

Merge pull request #17943 from jitendrapurohit/core-1906

dev/core#1906 - Allow payment create api to record payment on Failed …

Merge pull request #18036 from eileenmcnaughton/exp_test

Fix for failing test

Add test on status calculation

Fix for failing test

BY ensuring join_date is in the past we get away from situations where there is no valid status

Merge pull request #18031 from mattwire/crmmoney

Remove invalid use of crmMoney formatter

[REF] [Test] Minor simplification on test

This is towards standardising on one way to reference created entities -ie

->ids['Participant'][key] = 4

missing resubscribe url

Remove invalid use of crmMoney

Merge pull request #18010 from eileenmcnaughton/pex

Remove main PaymentExpress class

Merge pull request #18026 from eileenmcnaughton/pptrans

[Ref] Remove transaction instantiation in PaypalPro

[Ref] Simplify is_email_receipt in sendMail

This function has evolved through refactoring so there was a time when this code needed to decide whether to
send emails out or not. However, the function is now called more deliberately -ie
1) from completeTransaction - where it is only called if the decision to send an email is made
2) from the search action send Email receipts - where a user has made the decision to send (or pdf)
receipts and unless it's being called with returnMessageText = TRUE (to construct a pdf) then
send makes sense

Ergo we should send unless returnMessageText = TRUE

[Ref] Remove transaction instantiation in PaypalPro

The transaction is an optional param & we are not doing any saves so let's take the option to not pass it in...

Merge pull request #18020 from eileenmcnaughton/pptrans

[REF] Stop instantiating transaction in PaypalIPN

Merge pull request #18021 from eileenmcnaughton/ids_billing

Remove unused parameter ids['billing']

Merge pull request #18023 from civicrm/5.28

5.28 to master

Merge pull request #18017 from seamuslee001/dev_mail_72

dev/mail#72 Remove call to custom fatal error handler from CRM_Core_E…

Merge pull request #17999 from eileenmcnaughton/sql_cache

Cache loader - remove legacy handling, handle null result from setting

Remove unused parameter ids['billing']

This is no longer used in later functions and a quick grep of the code finds only one deprecated
function still refers to ids-billing after this

[REF] Stop instantiating transaction in PaypalIPN

Merge pull request #18001 from mattwire/inform-icon

Change inform-icon to fa-info-circle

Merge pull request #17984 from eileenmcnaughton/sendref

[REF] Remove pass-by-reference & always empty param

Merge pull request #17990 from totten/master-bao-cache-adapter

CIVICRM_BAO_CACHE_ADAPTER - Remove obsolete option

Merge pull request #18012 from demeritcowboy/utf8mb4-temptable

SQL temp table not using utf8mb4 if server default already set to utf8mb4

Merge pull request #17996 from demeritcowboy/wrong-link-pcp-domain

Wrong link to admin page in error message about FROM address on PCP page

Merge pull request #17923 from eileenmcnaughton/dedupe_return

[REF] Tighten up function signature for dedupePair

dev/mail#72 Remove call to custom fatal error handler from CRM_Core_Error::debug_log_message

Merge pull request #17982 from eileenmcnaughton/isreceipt

[Ref] Move noisily deprecate  BaseIPN->sendMail, call api from it rather than BAO function

Merge pull request #17396 from seamuslee001/use_util_mail_events

[REF] Use CRM_Utils_Mail::send for sending emails for confirming unsu…

CIVICRM_BAO_CACHE_ADAPTER - Remove obsolete option

Overview
--------

The constant `CIVICRM_BAO_CACHE_ADAPTER` was a low-visibility transitional
option added during early 5.x.  It facilitated the deprecation of
`CRM_Core_BAO_Cache` (in favor of `Civi::cache()`).  The option allowed
certain builds/configurations to be tested in a "purer" configuration where
`Civi::cache()` handled all storage.

However, following #17989, the deprecated functions no longer exist.  Hence,
there's no need for transitional mechanism to support them.  The
replacements are now mainline.

Before
-------

There's some obsolete stuff hanging around.

After
-----

The obsolete stuff is taking a long walk to the moon.

Merge pull request #17989 from eileenmcnaughton/cache_deprecate

CRM_Core_BAO_Cache - Remove functions deprecated a year ago

Merge pull request #18011 from totten/5.28-tbl

FiveTwentyEight - Provide concrete details about civicrm.files

Merge pull request #18013 from demeritcowboy/fix-fix-test-test

[Test framework] re-re-fix test and add test for test

re-re-fix test

temptable not using utf8mb4 if server default already set

Change inform-icon to fa-info-circle

FiveTwentyEight - Provide concrete details about civicrm.files

This hopefully makes it easier to decide what to do without needing a
scavenger hunt.

Merge pull request #18009 from eileenmcnaughton/test

Re-fix test

Remove main PaymentExpress class

I recently identified that this processor had been broken for a long time & removed the broken IPN class

At that time I didn't remove the main class as I was unsure how to deal with any legacy transactions related to it.

However, @mattwire pointed out he had done work earlier to make CiviCRM cope with not-present processor classes and,
on testing, I was unable to trigger any errors after setting up a contribution page & an event page & making a payment
and then deleting the class. I couldn't edit the contribution, but with appropriate in app messaging and no fatal
error.

Re-fix test

The strtotime calculation adds 4 months before setting the day of month. However

July 31 + 4 months is 1 Dec - ie the month is 12 not 11 due to there being only 30 days. So to
get 27 Nov we need to get the July month (7) and add 4 and voila 11, not 12

Merge pull request #18004 from colemanw/saveDoc

[NFC] Improve docs for APIv4 Save action

Merge pull request #18006 from demeritcowboy/missing-transactioninfo

dev/core#1909 - E_NOTICE on contribution edit

Merge pull request #18007 from demeritcowboy/strpos-revisited

dev/core#1918 - Remove dubious qfkey checking code that never runs

Merge pull request #18008 from civicrm/5.28

5.28

Merge pull request #17983 from kcristiano/5.28-wp-notices

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Merge pull request #17987 from eileenmcnaughton/pseudo_mem

[REF] Reduce calls to CRM_Member_PseudoConstant::membershipType

Merge pull request #18002 from demeritcowboy/missing-soft-credit

dev/core#1915 - E_NOTICE when making pcp contribution

Merge pull request #18003 from demeritcowboy/membership-fail

[Test framework] - Update failing test

remove dubious code that never runs

notice on contribution edit

Improve docs for APIv4 Save action

Merge pull request #17975 from eileenmcnaughton/matt_yet_another

Fix repeattransaction api to use custom data from the template contribution

update failing test

Merge pull request #17777 from agh1/dontchangereceivedate

dev/financial#139 contribution receive_date shouldn't change when payments come in

Cache loader - remove legacy handling, handle null result from setting

We are seeing this symptom in our tests in some cases
https://civicrm.stackexchange.com/questions/28897/error-when-merging-accounts

I'm not quite sure why the setting would be empty - but treating empty the same as 'default'
seems better than a hard fail and we know that settings have a track history of not all being
loaded at the right times.

This also removes some legacy handling

E_NOTICE when pcp contribution

Merge pull request #17997 from civicrm/5.28

5.28

Merge pull request #17868 from seamuslee001/remove_civicrm_files_override

[REF] Remove civicrm.files override for WordPress to fix issues with…

Merge pull request #17970 from eileenmcnaughton/case_export

Fix case activity field set to allow long details to be exported

wrong link to admin page

Merge pull request #17986 from seamuslee001/dev_core_1913_alternate

dev/core#1913 Allow for schemas to be added by extensions if they are…

Merge pull request #17707 from seamuslee001/ref_permission_clause_search

[REF] Use Standard function cacheClause to re-use contact acl cache t…

Merge pull request #17978 from kainuk/issue-1914

Fixed filling default values for tagssets in the advanced search form

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Signed-off-by: Kevin Cristiano <kcristiano@kcristiano.com>

Merge pull request #17985 from demeritcowboy/sms-apiurl

dev/core#1909 - Avoid E_Notice on SMS provider form when no default url

Fix fieldSpec when exporting cases using certain (default) fields

Merge pull request #17988 from eileenmcnaughton/cache_dup

Remove duplicate cache flush

Remove functions deprecated a year ago

Reduce calls to CRM_Member_PseudoConstant::membershipType

This function defaults to not using the cached version so that's a pretty good argument for making a more
active effort to deprecate it. The createMembership function flushes it so we don't need to do that
so much in tests now either. I also included swapping status over in the test

Merge pull request #17968 from eileenmcnaughton/cont_page

[REF] Extract setUserContext on contribution form & cleanup on backend add membership form

Remove duplicate cache flush

As the comment points out CRM_ACL_BAO_Cache::resetCache is also called in CRM_Utils_System::flushCache
so we are running a query that is inclined to lock tables an extra time for no reason

dev/core#1913 Allow for schemas to be added by extensions if they are in the format of <entityname>Model

Update test doc blocks

Call api from BaseIPN->sendMail

This function has been deprecated since 2015 and it's tempting just to remove it but at this stage
it is being noisily deprecated & by-passed

avoid notice when no default url

Merge pull request #17305 from mlutfy/core1755

dev/core#1755 Fix reCaptcha on Mailing Subscribe