Merge pull request #12876 from colemanw/Don't

Don't mess up fragment links in nav menu

Merge pull request #12879 from totten/master-contact-settings

(DX) Civi::contactSettings - Add a facade for working with the logged-in user's settings

Merge pull request #12878 from colemanw/menu-icons

Add icons to top-level navigation menu

Merge pull request #12877 from colemanw/MailingComponentDAO

Regenerate MailingComponent DAO file to reflect recent changes

Regenerate sql for Add icons to top-level navigation menu (#7)

Merge pull request #12790 from alifrumin/core362

dev/core#362 fix search functionality of contact reference fields in On Behalf Profiles

Merge pull request #12874 from seamuslee001/lab_core_410

dev/core#410 Ensure that Email related fields in Scheduled Reminders …

Update activity_tab_filter setting code to use new fascade

Merge pull request #12867 from mattwire/entityformajaxcustomdatafreeze

Freeze custom data elements when viewing an entity

(DX) Civi::contactSettings - Add a facade for working with the logged-in user's settings

Before
------

To read/write a setting for the logged-in user, you need a snippet like this:

```php
$cid = CRM_Core_Session::getLoggedInContactID();
$myFilter = Civi::service('settings_manager')
  ->getBagByContact(NULL, $cid)
  ->get('activity_tab_filter');
```

After
-----

To read/write a setting for the logged-in user, you can use a snippet like this:

```php
$myFilter = Civi::contactSettings()->get('activity_tab_filter');
```

Technical Details
-----------------

* A convenience function like this is liable to be used in lazy circumstances where
  the developer is unlikely to check their conditions carefully. Therefore, the
  errors are reported as exceptions so that mistakes are easiliy revealed.
* This is technically a small contract change to `SettingsManager::getBagByContact()`
  because it now interprets `$contactId===NULL` as meaning "the logged-in user".
  However, I don't think NULL was a sensible value before, and this interface
  isn't widely known/used.

Regenerate MailingComponent DAO file to reflect recent changes

Fix bad menu name

Add icons to nav menu

dev/core#362 fixing contact reference fields on On Behalf Profiles search and select functionality

Don't mess up fragment links in nav menu

dev/core#410 Ensure that Email related fields in Scheduled Reminders are not required when creating an SMS reminder and ensure that SMS Text Message field is required when creating an SMS reminder

Merge pull request #12871 from civicrm/5.6

Merge 5.6 to master

Freeze custom data elements when viewing an entity

Merge pull request #12855 from colemanw/dev/core#391

dev/core#391 - Fix saving tags in profile

dev/core#391 - Fix saving tags in profile

Merge pull request #12865 from civicrm/5.6

5.6

Merge pull request #12833 from mattwire/relationshipform_fix

dev/399 Fix saving of relationship permissions when relationship is b_a on form

Merge pull request #12728 from pradpnayak/MailingComponent

Renamed Mailing Component entity

Merge pull request #12863 from civicrm/5.6

5.6 to master

Merge pull request #12672 from eileenmcnaughton/fix_location_export_labels

Export code improvement

Rename MailingComponent BAO

Further fix to permission fields on relationship form

Merge pull request #12861 from eileenmcnaughton/locked

Fix typo in 5.5 upgrade script

Fix typo in 5.5 upgrade script

Merge pull request #12847 from mmyriam/patch-1

Add span to amount label in Confirm.tpl

Merge pull request #12858 from eileenmcnaughton/this

Fix typo

Fix typo

https://lab.civicrm.org/dev/core/issues/402

Merge pull request #12856 from seamuslee001/dev_core_281

dev/core#281 Fix invoice number in message template

Merge pull request #12800 from mattwire/relationshiptab_format

Improve display of relationships tab

Merge pull request #12854 from mattwire/cc_contribution_clears_email2

When submitting a standalone credit card contribution the contact email address should not be deleted

dev/core#281 Fix invoice number in message template

shift upgrade to 5.7.alpha1 and Get passing unit tests where template didn't exist before 4.6.alpha1

Merge pull request #12773 from eileenmcnaughton/perf_conts

Use cached function for financialAccount retrieval.

Remove redundant code to get email address which we've already retrieved

When submitting a standalone credit card contribution the contact email address should not be deleted

Merge pull request #12853 from seamuslee001/nfc_case_dashboad

NFC Fix overindenting in caseDashboard template

NFC Fix overindenting in caseDashboard template

Merge pull request #12849 from seamuslee001/case_dashboard_indent_fix

(NFC) Fix indenting of Case DashBoard.tpl

Merge pull request #12748 from compucorp/370-no-new-tasks-and-documents-get-added-to-workflow

dev/core#370: No new tasks and documents get added to workflow

Merge pull request #12846 from eileenmcnaughton/activitytest

[NFC] Add unit test to cover component ACLs.

Merge pull request #12852 from eileenmcnaughton/master

5.6 to Master

Merge branch '5.6' of https://github.com/civicrm/civicrm-core

Change-Id: I2dd0a9d8f83fe13b7f731db0b959a67e3dd0ae9f

Merge pull request #12844 from totten/5.6-asg-opt-group

(dev/core#107) Fix regression which prevents editing CiviCase configuration

Merge pull request #12850 from civicrm/5.6

5.6

Merge pull request #12845 from eileenmcnaughton/activity56

Partial revert of pull/12559

(NFC) Fix indenting of Case DashBoard.tpl

Merge pull request #12848 from gah242s/patch-5

Remove Stray Extra Div in Case Dashboard TPL

(dev/core#107) civicrm_data.tpl - Fix typo

Update DashBoard.tpl

Looks like there is an extra </div> tag at the bottom of this file.  The WordPress footer was coming up into the middle of the page.  Discussion threads on MM:
https://chat.civicrm.org/civicrm/pl/9eywqazzt3yb5e49jg9yh71rfh
https://chat.civicrm.org/civicrm/pl/taebbfneutyrfdh7ed9srhoiwy

Update Confirm.tpl

Added span to amount label.

Remove some unused variables

Add unit test to cover component ACLs.

This adds a unit test and does a very minor cleanup making the return a little clearer

Partial revert of pull/12559

https://github.com/civicrm/civicrm-core/pull/12559/files is currently in RC.

While we find that this change improves performance and we are running it in production
we have identified that performance is very poor in the specific scenario where only
email is linked to 39000 assignees - in this case it was an old experimental civimail
activity. In the interests of prudence I am reverting the parts that switch function
in the rc.

(dev/core#107) Upgrader - Re-run addActivityDefaultAssigneeOptions() for 5.4/5.5 sites

(dev/core#107) Update civircrm_generated.mysql

Based on a call to `bin/regen.sh`

(dev/core#107) Add the required option group to the default data template

Set version to 5.6.beta2

Cache payment instrument financial ids.

I don't really see any value in this calling a shared function as it fails to simplify

Use cached function for financialAccount retrieval.

In performance testing I find that when inserting 100 contributions this change
saves 198 queries by using an alternative that is cached

Merge pull request #12774 from colemanw/pledgePicker

Cleanup pledge form to use datepicker

Merge pull request #12840 from eileenmcnaughton/master

5.6 to Master

Merge branch '5.6' of https://github.com/civicrm/civicrm-core

Merge pull request #12837 from colemanw/basicTpl

Fix html wrapper on basic summary block

Merge pull request #12838 from civicrm/JoeMurray-patch-1

NFC: fix grammar/punctuation in help

NFC: fix grammar/punctuation in help

Fix html wrapper on basic summary block

Merge pull request #12830 from colemanw/emptyProfileTags

Allow profile.submit api for tags

Allow profile.submit api to remove all tags from a contact

Fix api profile.submit to work with tag & note fields

Merge pull request #12835 from civicrm/5.6

5.6

Merge pull request #12832 from colemanw/groupContact

groupContact BAO - standardize add function

Merge pull request #12690 from eileenmcnaughton/member_dates

Convert membership date fields to datepicker & form to entity form

groupContact BAO - standardize add function

Merge pull request #12827 from colemanw/note

Better title for note field

Fix saving of relationship permissions when relationship is b_a on form

Merge pull request #12734 from mattwire/membershiprenew_duplicatepayprocfield

Remove duplicate payment processor select on Membership Renewal form

Merge pull request #12825 from colemanw/profileTagNote

Fix api profile.submit to work with tag & note fields

Better title for note field

Fix api profile.submit to work with tag & note fields

Merge pull request #12820 from mukeshcompucorp/SHOR-10-regression-issues-find-contacts

SHOR-10: Fixed wrapper

SHOR-10: Fixed wrapper

Merge pull request #12768 from mattwire/register_participant_permissions

Don't show 'Register Participant' on the manage events page if the logged in user does not have 'Edit Event Participants' permission

Merge pull request #12817 from MegaphoneJon/zero-option-values

Closes core#393 - allow 0 as a valid option value when option group d…

Merge pull request #12762 from mattwire/payproc_supportsX

Add new paymentprocessor supportsX methods to replace remaining isSupported methods

Merge pull request #12776 from mattwire/eventinfo_datepicker

Convert "Info and Settings" tab on ManageEvent to use datepicker

Merge pull request #12772 from pradpnayak/DetailContactReport

Fixes to Contact Detail report

linting fixes

Closes core#393 - allow 0 as a valid option value when option group data_type is Integer

Add new paymentprocessor supportsX methods to replace remaining isSupported methods

Event configuration - Simplify labels for start/end date

Convert EventInfo to datepicker

NFC Formatting for EventInfo.tpl

Fixed styling and included Contact detail report in SelectWhereClause test

only include the tables that will be in from clause

Fixed indentation and styling and removed looping

Merge pull request #12815 from seamuslee001/test_fix_report_activity_test

Alter Test to ensure that order of city is output doesn't mater in th…