Merge pull request #12945 from seamuslee001/lab_core_448

dev/core#448 Fix issue where when building mailings with smart groups, removed members of the smart group were being included

smart groups as mailing lists were not taking unsubscribed users into account

dev/core#448 Add in a Unit test to demonstrate the problem with smart groups and contats removed from smart groups not being properly checked

Merge pull request #12937 from colemanw/d7toolbar

Intelligently adjust menubar for D7 toolbar toggle

Intelligently adjust menubar for D7 toolbar toggle

Merge pull request #12693 from eileenmcnaughton/membership_submit

Membership form submit status message refactor

Merge pull request #12934 from scardinius/core-443

dev/core#443 Fix Non-static method calls at CRM_Utils_Mail_Incoming

Merge pull request #12936 from agh1/patch-1

Status page: hush menu should be on top of other headers

Status page: hush menu should be on top of other headers

Also tempered a bit of z-index overkill on the `<li>` elements in the menu

dev/core#443 Fix Non-static method calls at CRM_Utils_Mail_Incoming class

Merge pull request #12931 from mecachisenros/master

Correct inconsistent fileID reference

Merge pull request #12926 from eileenmcnaughton/processor_zero

dev/financial#23  Do not display e-notice for zero dollar transactions

Move setting of status to be with related code.

Next round we will integrated this with the actual building of updates

correct inconsistent fileID reference

Merge pull request #12306 from JO0st/dev-core-166

dev-core-166/lessen memory impact

i before e except after c

Merge pull request #12927 from davejenx/job_process_memberships_test_inherit

Added further tests for process_membership job: check status for inherited memberships

Added further checks for process_membership job: check that status is updated correctly for inherited membership. In function testProcessMembershipUpdateStatus.

Merge pull request #12922 from yashodha/report-fixes

CRM-21677 - more report clean up

Merge pull request #12758 from samuelsov/DraftMailing

Adding [CiviMail Draft] on test mailing (for mosaico)

Do not display e-notice for zero dollar transactions

Merge pull request #12810 from eileenmcnaughton/annual

Extract query to generate annual totals and add test

Merge pull request #12920 from JKingsnorth/patch-11

(NFC) Fix misleading comment for dupesInGroup()

Merge pull request #12883 from twomice/CRM-19751_search_on_hold-2

Toward CRM-19751: change db schema so that on_hold is INT instead of BOOLEAN.

Merge pull request #12919 from eileenmcnaughton/setting

Translate various settings information

Merge pull request #12765 from eileenmcnaughton/sms

Fix order by on is_primary.

Move status setting back to main function.

It actually doesn't have much in common with the other portion of that column
and we can move the setting to where it belongs

Stop returning createdMemberships.

This parameter is built up purely to set the status message, which I argue should be
set 'as she goes' saving us building up complex variables. Removing this
var will help us simplify the earlier loops

Extract query to generate annual totals and add test

Merge pull request #12495 from eileenmcnaughton/dragon_slayer

Slay dragons

CRM-21677 - more report clean up

Merge pull request #12695 from eileenmcnaughton/mem2

Membership form - tidy up receipt code

Fix order by on is_primary.

As pointed out in https://lab.civicrm.org/dev/mail/issues/26 the order by is not the
correct format for order_by and has no effect. I tested to make sure it was not
some magic.

I opted for (implict) ASC as the order by for non sms is ASC and further down
processing seems to overwrite each row as it happens so primary later
would overwrite earlier

Toward CRM-19751: change db scmema so on_hold is INT instead of BOOLEAN.

Fix misleading comment for dupesInGroup()

The function does also work with smart groups, so remove the comment that says otherwise.

Merge pull request #12828 from seamuslee001/lab_core_357

dev/core#357 Fix issue where user email addresses were keyed on email…

Translate various settings information

Move customValue retrieval to calling function.

After digging on CRM_Batch_Form_Entry I couldn't find any evidence that the groupTree property would ever be set

Wrap membership receipt in an outer function.

This function is neither static nor shared & we work to replace the
other / put the parts that really are shared somewhere sensible

Merge pull request #12918 from davejenx/job_process_memberships_tests

Added more unit tests for process_membership job: testProcessMembershipUpdateStatus

Retain preferred array format while searching Money values.

The preferred BAO_Query format is ['BETWEEN' => [12.40 AND 13.50]] but this is currently being munged back
into legacy format, creating a test fail

Note that the search doesn't work properly with international commas, with, or without
this patch :-(

Slay dragons

Merge pull request #12906 from eileenmcnaughton/setting

[NFC] Convert some fields on admin display preferences to use metdata

Merge pull request #12747 from mattwire/tabheader_extract_and_events

Extract selectedChild tabheader functionality and enable for Manage Events

Added more unit tests for process_membership job, checking that statuses are updated correctly in various cases where they should be and left alone in cases where they shouldn't. Adds new function testProcessMembershipUpdateStatus.

Merge pull request #12916 from yashodha/report-fixes

CRM-21677 - report clean up

Convert Preserve activity tab

Extract selectedChild tabheader functionality and enable for Manage Events

Convert Contact Search to metadata driven

Add metadata for editing contacts

Convert 'Viewing Smart Groups' on admin display preferences to use metadata.

This has no functional impact - cleanup only

Convert 'Viewing Smart Groups' on admin display preferences to use metadata.

This has no functional impact - cleanup only

Merge pull request #12910 from monishdeb/dev-core-394

(dev/core#394) : Wildcards are ignored in some smart group criteria, when the smart group is directly generated for a mailing

Merge pull request #12890 from dereklewis123/CW-905-2

core/384 CiviSMS does not fall back to non-primary mobile number

CRM-21677 - report clean up

Merge pull request #12784 from eileenmcnaughton/cache

Do not treat an empty array as not set in caches.

Merge pull request #12914 from eileenmcnaughton/record

Fix payment hangs when registering a user for a paid event without recording payment

Merge pull request #12864 from eileenmcnaughton/export

Export - small cleanup

(dev/core#394) : Wildcards are ignored in some smart group criteria, when the smart group is directly generated for a mailing

Merge pull request #12913 from wmortada/shorcuts

Fix typo in hook that enables links to be added to a block

Merge pull request #12819 from mattwire/contributionrecur_contactname

Show contact name on recurring contribution view and improve consistency with Contribution/Membership view

Fix payment hangs when registering a user for a paid event without recording payment

This is an alternative to https://github.com/civicrm/civicrm-core/pull/12888
I wasn't actually able to reproduce the issue but this solution, proposed by
Monish, seems safe & sensible

Merge pull request #12841 from colemanw/richText

Skip html encoding rich-text fields

Merge pull request #12912 from civicrm/totten-patch-1

Fix spelling error ("sript" vs "script")

Fix typo in hook that enables links to be added to a block

Merge pull request #12904 from yashodha/report-fixes

On reports make current employer available as a field for display

Merge pull request #12891 from jmcclelland/issue418

ensure all valid relationship types are available

Update Drupal.php

If my username is going to be used for a grammatical edit, then we should get it right!

Fix spelling error ' script' rather than 'sript'

Merge pull request #12824 from jmcclelland/issue389

Ensure relative dates are preserved for custom fields in smart group

Merge pull request #12911 from civicrm/5.7

5.7 to master

Merge pull request #12909 from eileenmcnaughton/57reg

Ensure hard-coded date ranges are preserved in smart groups

Merge pull request #11821 from michaelmcandrew/500-http-response-code

Send an 500 http response code when an exception is encountered.

Merge pull request #12873 from totten/master-short-long

Register "short" and "long" cache services

ensure date ranges are preserved in smart groups

See https://lab.civicrm.org/dev/core/issues/396

Merge pull request #12903 from kainuk/drupal8-codegen-path

Added a path to the drupal8 path candidates to find civicrm.config.php

Merge pull request #12908 from xurizaemon/GL425-warning_on_redirectToSSL

GL-425: Log a message if CiviCRM thinks SSL is not enabled

Merge pull request #12907 from civicrm/5.7

5.7

GL-425: Log a message if CiviCRM thinks SSL is not enabled

The enableSSL setting doesn't reliably detect the presence of SSL (especially
when behind SSL terminating proxies). This log message will allow people
affected by this behaviour to more readily debug it, because CiviCRM
will give them a clue to work with.

See https://lab.civicrm.org/dev/core/issues/425

Merge pull request #12896 from seamuslee001/dev_core_421

dev/core#421 Fix issue where creating user driven message templates w…

dev/core#421 Fix issue where creating user driven message templates was requireing the ssystem workflow message template permission as well

Wrap permission checking in the check_permissions param

Move Permission checking to BAO level from API

Allow for the fact that edit message templates permission should still be able to work on both sets of templates without the granular permissions. Also reduce duplication of code abit

Merge pull request #12905 from totten/master-gitignore

Add `ext/` folder to .gitignore

Merge pull request #12731 from eileenmcnaughton/setting_form

Setting form - start to sync preferences with Setting, using trait

Make checkboxes save

Add `ext/` folder to .gitignore

This folder is populated in various build processes (civibuild's
`drupal-demo`, `wp-demo`, etc; as well as `distmaker`), and sometimes
I like to use it for personal things. Moreover, there is no committed
content inside here.

Merge pull request #12902 from seamuslee001/towards_500

Move towards allowing sending 500 http errors back by switching civiE…

expose current employer in fields and display in results as well

Added a path to the drupal8 path candidates to find civicrm.config.php.drupal. So civbuild can find it

Move towards allowing sending 500 http errors back by switching civiExit(1) to be just civiExit()

Merge pull request #12898 from kenwest/dev_core_423

dev/core#423 MySQL 5.7 complains about 4.7.19 upgrade script

Merge pull request #12829 from eileenmcnaughton/validate

Add validate api action for ContributionPage.submit

Merge pull request #12900 from eileenmcnaughton/recur_tidy

[NFC] Minor cleanup, return preferred parameter

Merge pull request #12869 from dereklewis123/CW-892

Added missing field name in custom-field deletion dialog

Add validate api action for ContributionPage.submit

Move setting defaults to form

Allow Preferences form to work from settings

[NFC] Minor cleanup, return preferred parameter

The result parameter is the same as $recurring (save()) returns $this. But
it's more correct to return $recurring than result as
a) it's defined as a Contribute_BAO_ContributionRecur object which is correct.
b) it's the object passed through hooks

Merge pull request #12899 from eileenmcnaughton/recur_tpl

Recur tpl

Extract rows section of recur template to separate tpl, for readability

Merge pull request #12821 from mattwire/updatesubscription_fix

Show recurring contribution links based on payment processor capabilities