Merge pull request #15439 from eileenmcnaughton/ex_bug

dev/core#1293 Fix regression on  export filtering for postal address only

Merge pull request #15440 from eileenmcnaughton/ex_zero

dev/core#1293 Still output csv file on export, even if no rows are in it

dev/core#1293 Fix regression on  export filtering for postal address only

Still output csv file, even if no rows are in it

Merge pull request #15428 from greenpeace-cee/fix-status-check

Fix status check not rendering before 5.19 migrations

Merge pull request #15420 from eileenmcnaughton/live_notice

dev/core#1281 fix e-notice on isLiveMode

Fix status check not rendering before 5.19 migrations

This fixes an issue where the status check page does not render when
migrations to 5.19 are pending because the is_active column hasn't been
added yet.

The rationale for this change is to avoid scenarios where users forget
to run upgrades because there's no status check warning.

Set version to 5.19.beta1

dev/core#1281 fix e-notice on isLiveMode

Looks like a regression from https://github.com/civicrm/civicrm-core/commit/752cee47213094c29b8e6ae5d0b20f0800520d80
which hit in 5.15 - seems good to merge to rc / about to cut to rc

Merge pull request #15417 from eileenmcnaughton/matt

Add trxn_id as a parameter on Payment.get

Merge pull request #15414 from eileenmcnaughton/fix_log_text

dev/core#1283 fix inability to export more than 255 chars

Merge pull request #15416 from civicrm/eileenmcnaughton-patch-1

Update fix - should not be localisable

Accept trxn_id to Payment.get

Alternative to #15166 (incorporates #15229 )

Unit test for #15166 fix

Merge pull request #15381 from eileenmcnaughton/w_test

Add test for uF group one bug

Update fix - should not be localisable

dev/core#1283 fix inability to export more than 255 chars

Merge pull request #15409 from eileenmcnaughton/status

dev/core#1295 Add is_active field to civicrm_status_preference

Merge pull request #15413 from eileenmcnaughton/default_sub_type

[REF] don't handle non-array, pass by reference

Heed is_active value if set to zero

Note this is done BEFORE doing checks in case it is for performance reasons

Add test for uF group one bug

[REF] don't handle non-array, pass by reference

This is only called from one place & it is definitely coming in as an array & using the return values.

Merge pull request #15352 from colemanw/extCompat

Improve obsolete extension management

Merge pull request #15404 from seamuslee001/dev_core_861

dev/core#861 Ensure that when processing mailings that no emails are …

Merge pull request #15384 from mattwire/membership_updaterecur

Remove call to updateRecurMembership which is handled by Membership::create BAO

Merge pull request #15406 from mattwire/morecontributionstatuslabel

Fix checking of isSubscriptionCancelled when label is changed

Merge pull request #15405 from MikeyMJCO/nomoreyahoo

Remove legacy Yahoo references from the Mapping/Geocoding settings/code.

Fix check for isSubscriptionCancelled in contribute

Explicitly retrieve label for contributionStatus (no change in behaviour)

Add is_active to status pref

Merge pull request #15403 from smaen123/patch-1

Fix height of select2 box (appears squished)

Remove call to updateRecurMembership which is handled by Membership::create BAO

extension-compatibility - Update DB state of force-uninstalled extensions

Note:
1. This is more advisory than functional - the functionality is really in the runtime bits
   in the previous commits.
2. Technically, this is more of a disable, but we should keep the record around in
   `civicrm_extension` so that we know the old schema revision.

dev/core#861 Ensure that when processing mailings that no emails are sent to deceased contacts or those with emails now on hold

Update ConfigTaskList to remove Yahoo reference and add OSM

Remove Yahoo reference from the Mapping settings.

Merge remote-tracking branch 'MikeyMJCO/patch-2' into patch-1

Remove conditional for Yahoo mapping provider

The Yahoo mapping provider is no longer in core and therefore this conditional logic is no longer necessary.

Merge pull request #14184 from mattwire/relationshipcolumns

Allow columns in relationship table to be modified by searchColumns hook

Remove Yahoo references

Remove references to Yahoo Geocoder which is no longer included core.

Merge pull request #15397 from mattwire/contributionstatuslabel

REF Explicitly specify params so we can find problems with comparing translated labels

Merge pull request #14577 from mattwire/type_consolidation

REF Remove some duplication in CRM_Utils_Type::escape/validate

Merge pull request #15402 from eileenmcnaughton/nfc

[NFC] code formatting & exceptions in doc blocks

Merge pull request #15401 from seamuslee001/master

Master / 5.19 version of #15392

Fix height of select2 box (appears squished)

Add exceptions to phpdoc

[NFC] code formatting

fix typo resulting in fatal error

Merge pull request #15395 from seamuslee001/5_18_1_release_notes

5 18 1 release notes

Merge pull request #15398 from eileenmcnaughton/adv

dev/core#1294 Remove getFormValues function

Merge pull request #14897 from mattwire/membership_payment2

Don't use random parameters to detect if we should create MembershipPayment

Merge pull request #15385 from demeritcowboy/test-case-type-edit-1

[TEST] dev/core#1046 - tests for case type edit screen

dev/core#1294 Remove getFormValues function

This is overiding the parent function which gives an empty result

Explicitly specify params so we can find problems with comparing translated labels

Update 5.18.1.md

Update release notes for the export fix

Merge pull request #15389 from eileenmcnaughton/cleanup_query

Extract the code to get the field name.

Update 5.18.1.md

Add in 5.18.1 release notes

Merge pull request #15376 from mattwire/ts_membershippending

Use name instead of label to check contribution status

Merge pull request #15378 from demeritcowboy/deanonymize-on-the-fly

[REF] Pull out anonymous javascript function to make it testable

pull out function for testing

Merge pull request #15329 from mattwire/contributepagetab_cleanupstatic

Remove static caching of static on Contribution page tab

Allow columns in relationship table to be modified by searchColumns hook

Merge pull request #15345 from yashodha/CRM-21677

CRM-21677 - report clean up for contact detail report

Extract the code to get the field name.

Ideally we can share this across searches (@monishdeb is hitting some things in case search that are solved
in contribution search so trying to cleanup these bits  - this just pulls out one small piece.

Once this is passing tests (which could be affected by my new deprecations
I think the other fields - fieldspec & 'qillname' if still required can get the same treatment.
in the next step.

Getting the fieldSpec & adding the qill is what I specifically want to genericise.

Merge pull request #15388 from MegaphoneJon/financial-71

financial#71 - allow emailing partially paid invoices

Merge pull request #15379 from eileenmcnaughton/cleanup_form_values

[REF] Consolidate getFormValues on contribution search

Merge pull request #15312 from seamuslee001/export_hook_test

Extend Export test to cover altering sqlColumns and headerRows via ho…

Merge pull request #15387 from eileenmcnaughton/merge

Merge 5.18 to master

Merge branch '5.18' of https://github.com/civicrm/civicrm-core into merge

Consolidate getFormValues on contribution search

This consolidates 2 places where formValues are determined on contribution search. In one place
force is handled in the other retrieving from the form is handled. It seems Ok to handle both in
one place & call that. This might help iron on some glitches Monish is hitting on #15370

We should probably add saved search retrieval in too

To confirm this works use this url
civicrm/contribute/search?reset=1&reset=1&sort_name=p&receive_date_high=20180101&force=1

- it should filter. If you add another criteria to the form afterwards the url criteria should
be present on the form from the url & the new one should also be respected. If you actively remove the url
criteria on the form it is ignored in the search

tests for case type edit screen

Set version to 5.18.0

Merge pull request #15382 from totten/5.18-note-event

(NFC) release-notes/5.18.0.md - Add dev/event#15

Merge pull request #15380 from eileenmcnaughton/cust_uf

(dev/event#15) Fix regression in handling of post profiles in email

Event::sendMail() - Don't assume $postProfileID has content

The unit-test suite includes cases in which the value is actually NULL.

Event::sendEmail() - Fix displaying of heading. Use same key.

The message templates use this loop:

```
{foreach from=$customPost item=customPos key=j}
   <tr> <th {$headerStyle}>{$customPost_grouptitle.$j}</th></tr>
```

Note that the keys in `$customPost` and `$customPost_grouptitle` need to
match.

release-notes/5.18.0.md - Add dev/event#15

Event::sendEmail() - Fix profile ID for realz this time

Fix regression in handling of post profiles in email

Merge pull request #15377 from demeritcowboy/relTypes-not-relTypes

[NFC] Rename local variable

rename local variable

Merge pull request #15365 from samuelsov/userdashboard

Removing 'Array' from user dashboard contributions

extension-compatibility - Add 'force-uninstall' mode for api4 transition

There is a compatibility problem when upgrading a system which  has an old
copy of api4: the top level `api4.php` indiscriminately declares the
function `civicrm_api4()`, which eventually provokes a conflict with the
copy that is now in `civicrm-core`.

Using the normal disable/uninstall actions doesn't resolve this because the
error arises too early during boot (before the upgrader gets a chance to
remove the extension).

The 'force-uninstall' option will make the system behave as if the extension
is uninstalled, regardless of what files exist and what state may be stored
in the `civicrm_extension` table.

This commit technically does ~3 things:

1. Changes the policy for `org.civicrm.api4` to use `force-uninstall`
2. Makes the extension-cache version-dependent. During an upgrade, you
   might have a stale cache that claims that the old extension is
   active. This ensures that (as soon as you drop in new code)
   it begins working with a fresh cache.
3. Update any spots which query the table `civicrm_extensions` for
   extension status. Have it consult `extension-compatibility.json`
   for `force-uninstall`ed items.

Completely uninstall obsolete extensions during upgrade

Do not allow obsolete extensions to be enabled

Use name instead of label to check contribution status

financial#71 - allow emailing partially paid invoices

Merge pull request #15361 from seamuslee001/index_geo_code_fields

Add in an index on geo_code_1 and geo_code_2 to assist with distance …

Merge pull request #15368 from seamuslee001/deprecated_getvalue_mailing_prefernces

[REF] Remove call to depreacated getValue call in Mailing Preferences…

Merge pull request #15372 from civicrm/5.18

5.18

Merge pull request #15371 from agh1/5.18.0-releasenotes-final

5.18.0 release notes final edits

5.18.0 release notes: some final edits

5.18.0 release notes: added late changes

[REF] Remove call to depreacated getValue call in Mailing Preferences settings form

Merge pull request #15366 from demeritcowboy/radio-date-gmt

dev/core#1279 - crmMailingRadioDateSpec.js always has a failure if west of GMT

GMT fix

NFC - removing 'Array' from user dashboard contributions

Merge pull request #15360 from seamuslee001/ids_path_fix

[REF] Fix path to htmlpurifier in IDS file