Merge pull request #6839 from yashodha/CRM-17302

CRM-17302: Fix component for contribution activities

Merge pull request #6743 from jitendrapurohit/CRM-17182

CRM-17182 -- Renaming a price option / fee level breaks event partici…

Merge pull request #6836 from yashodha/CRM-12675

CRM-12675: Contribution activities shown to users without permission

Merge pull request #6840 from monishdeb/test-master

CRM-17287 fix for XML Exports

CRM-17287 fix for XML Exports

----------------------------------------
* CRM-17287: Broken Unit Tests on master
  https://issues.civicrm.org/jira/browse/CRM-17287

CRM-17302: Fix component for contribution activities

----------------------------------------
* CRM-17302: Fix component for contribution activities
  https://issues.civicrm.org/jira/browse/CRM-17302

Merge pull request #6815 from eileenmcnaughton/CRM-17289

CRM-17289 limit membership_type entity reference to active types

Merge pull request #6831 from eileenmcnaughton/CRM-17176-api-add

Crm 17176 api add navigation

CRM-17289 add unite test for membership_type.getlist

test class tidy-up

Let's not over-do the failure tests! We brought them from v2 but they don't add much

CRM-17289 limit membership_type entity reference to active types

CRM-17176 add Navigation api

Adding this was mostly to make the syntax conformance happy since adding a navigation.reset action:

CRM-17176 add titles to navigation xml

Merge pull request #6833 from eileenmcnaughton/email-test-fix

enable Single Value Alter test for email

enable Single Value Alter test for email

Conflicts:
tests/phpunit/api/v3/SyntaxConformanceTest.php

CRM-12675: Contribution activities shown to users without permission

----------------------------------------
* CRM-12675: Contribution activities shown to users without permission
  https://issues.civicrm.org/jira/browse/CRM-12675

Merge pull request #6829 from davecivicrm/CRM-16086

CRM-16086 - schema corrections.

CRM-16086 - schema corrections.

----------------------------------------
* CRM-16086: schema differences between upgrades and install
  https://issues.civicrm.org/jira/browse/CRM-16086

Merge pull request #6828 from monishdeb/test-master

CRM-17287 Test fix for contribution note

format fix

Merge forward miss

CRM-17287 Test fix for contribution note

----------------------------------------
* CRM-17287: Broken Unit Tests on master
  https://issues.civicrm.org/jira/browse/CRM-17287

Merge pull request #6820 from eileenmcnaughton/master

enotice fix

Merge pull request #6825 from jitendrapurohit/CRM-17169reg

CRM-17169 -- Fix upgrade conflict

Merge pull request #6821 from davecivicrm/CRM-16274

CRM-16274 - Change permission for Add Contacts to Groups in reports t…

Merge pull request #6823 from colemanw/CRM-16353

CRM-16353 - For tables hidden by collapsed accordions, wait

Merge pull request #6827 from monishdeb/test-master

CRM-17287 Legacy support for group and tag

format fixes

legacy support for group and tag

CRM-17169 -- Fix upgrade conflict

CRM-16353 - For tables hidden by collapsed accordions, wait

CRM-16274 - Change permission for Add Contacts to Groups in reports to be the same as for that task in search results.

----------------------------------------
* CRM-16274: Change the permission required for 'Add Contacts to Group' in reports to be consistent with other workflows
  https://issues.civicrm.org/jira/browse/CRM-16274

enotice fix

Merge pull request #6819 from monishdeb/test-master

 CRM-17287 Test fixes

Merge pull request #6818 from totten/master-deadfunc

CRM_Contact_BAO_Contact::getRelativePath - Remove dead code

master Test fixes

CRM_Contact_BAO_Contact::getRelativePath - Remove dead code

Merge pull request #6816 from colemanw/CRM-16643

CRM-16643 - Fix enableDisable when using api setvalue

CRM-16643 - Fix enableDisable when using api setvalue

Merge pull request #6810 from davecivicrm/CRM-16643

CRM-16643 - Revert to setvalue for inline editing of the entities who…

Merge pull request #6808 from totten/master-config-tests

CRM-16373 - CRM_Core_Config - Fix test regressions

Merge pull request #6813 from davecivicrm/CRM-17228

CRM-17228 - Improve help text for Profile form snippets.

CRM-17228 - Improve help text for Profile form snippets.

----------------------------------------
* CRM-17228: HTML Form snippets from profiles should use absolute URLs
  https://issues.civicrm.org/jira/browse/CRM-17228

CRM-16643 - Revert to setvalue for inline editing of the entities whose APIs do not handle create as update properly.

----------------------------------------
* CRM-16643: Deprecate & remove setvalue api
  https://issues.civicrm.org/jira/browse/CRM-16643

Merge pull request #6809 from davecivicrm/CRM-17269

CRM-17269 - Simplify retrieval of relationship types for query, and s…

Merge pull request #6749 from eileenmcnaughton/CRM-17144

CRM-17144 add ability for permissioned user to see report sql

CRM_Report_Form_Contribute_DetailTest, CRM_Report_Form_TestCaseTest - Spaces

It appears that the content of the report CSV has changed slightly -- in certain
cases with undefined/missing info, it had returned a literal space (' ') but
now returns a empty string (''). Empty string seems a bit better, so updating
the unit-tests to match.

CRM-17269 - Simplify retrieval of relationship types for query, and support relationship types on contact subtypes properly.

----------------------------------------
* CRM-17269: advanced search with custom relationship types fails
  https://issues.civicrm.org/jira/browse/CRM-17269

CRM_Core_Config::$keyDisable - Restore property

Merge pull request #6687 from totten/master-sched-ref

CRM-16355, CRM-13244 - Scheduled Reminders

CRM-16373 - Fix error on $fiscalYearStart

Error: "Indirect modification of overloaded property CRM_Core_Config::$fiscalYearStart has no effect"

CRM-16373 - CRM_Core_BAO_ConfigSetting - Fix enableComponent

Merge pull request #6802 from seamuslee001/CRM-17283

CRM-17283 Allow for searching on partcipant registration date

CRM-13422 - ActionScheduleTest - Loosen domain token handling

Discussed with a few people -- neither the contact nor the scheduled
reminder has a domain_id, so it seems hard to define a clear handling of
domain tokens.

CRM-17038, CRM-13422 - Tokens - Adjust greeting evaluation order

Following f551f7d32, replaceGreetingTokens tramples other tokens, so it has
to run at the end.

CRM-17028 - RecipientBuilder - Forward port 3fc9bc80ee

CRM-13422 - Add AbstractMappingTest. Fix soft-credit targeting.

CRM_Core_Error::debug_query_result - Don't dump hidden fields

CRM_Utils_SQL_Select - Better error for malformed numbers

CRM-13422 - ActionMappings - getRecipientTypes(), getRecipientListing()

Changes:
 * Declare getRecipientTypes() before getRecipientListing() because it's easier to read in that order.
 * Tweak docblocks
 * Split combined/legacy functions (Civi\ActionSchedule\Mapping) into simpler per-entity functions (CRM_Activity_ActionMapping, CRM_Event_ActionMapping).
 * For "Contribution Type" reminders, allow targetting to soft creditors.
 * For "Contribution Page" reminders, don't bother with soft-creditors. If someone paid through a self-service form, then I'm not sure we have soft-creditors.

CRM-13422 - Form/ScheduleReminders - Allow role filters on custom mappings

Previously, the 'Recipient Role' was only displayed if the results were
limited to 'participant_role'.  Now, the field will be displayed if the
mapping's getRecipientListing($recipientType) returns any data.

CRM-13422 - MappingInterface::getRecipientTypes - Remove $noThanksJustKidding

This is a weird option that owes to a quirk in the UX -- where activity
roles are presented differently than other roles.  That UX quirk probably
shouldn't exist, but given that it does...  we'll push the quirk up into the
form layer (rather than in our API).

CRM-13422 - ActionMapping - Remove gratuitous `entity_recipient`

This just functions an alternate-key for `entity`. It's only used
internally by the legacy `Mapping` class and does not appear in
the public interface.

CRM-13422 - ActionSchedule - Kill getSelection() and getSelection1()

These are silly functions whose job is to wrap up data in a bundle using
mysterious names so that it can get immediately unwrapped.

CRM-13422 - Consolidate extra rules for contact reminders

CRM-13422 - Add MappingInterface::validateSchedule().

Also: In CRM_Admin_Form_ScheduleReminders, extract parseActionSchedule()
from postProcess().  Now we can call parseActionSchedule() from both
postProcess() and formRule().

CRM_Core_BAO_ActionSchedule::getSelection - Make it easier to trace dataflow

TokenRow - Fix docs

CRM_Utils_Array::collectMethod() and ::toKeyValueRows()

CRM-13422 - CRM_Contribute_ActionMapping_ByType

CRM-13422 - CRM_Contribute_ActionMapping_ByPage

CRM-13244 - civicrm_action_schedule.mapping_id - Change int to varchar

When modules define new mappings, it's easier if they can makeup their own
symbolic names rather than picking a number.

CRM-13244 - MappingInterface - Document/cleanup getRecipient* stuff

CRM-13244 - Split prepareMailingQuery() into entity-specific pieces.

CRM-13244 - ActionScheduleTmp - Move entity-specific parts to separate classes

CRM-13244 - RecipientBuilder::prepareStartDateClauses - Less stringiness

CRM-13244 - ActionMapping - Strictly use MappingInterface

CRM-13244 - RecipientBuilder - Move entity-specific bits to ActionMappings

CRM-13244 - Move buildRecipientContacts into new class. Split into smaller functions.

Note: This was originally written as a series of smaller changes, but they
were squashed.  For more detailed history, check totten's repo for
alternative history leading up to, eg, 9e4d69e.

CRM-13244 - prepareMailingQuery - Use CRM_Utils_SQL_Select

CRM-13244 - ActionSchedule\Mapping - Privatize various fields

These fields are kind of stupid and are mostly only used within the Mapping
class.

The main external usage was for looking up a mapping record by
`entity_value`.  However, this can be made simpler now that mapping IDs are
constant.

CRM-13244 - ActionSchedule - Externalize getMappings

CRM-13244 - ActionSchedule - Extract entity-specific logic for tokens and dropdowns.

The `ActionSchedule` BAO had a ton of duplicate, helter-skelter code for
looking up specific details about specific entities and mappings.  The
mapping class (`CRM_Core_DAO_ActionMapping`) had an anemic model.  The
mapping records could be modified in the DB, but in practice this was
pointless because they depended on so many hard-coded lookups.  And there
were some confusing variable names (sel1, sel2, sel3, etc) and
variable-shuffling.

This revision moves several entity-specific bits into new classes
(`Civi\ActionSchedule\Mapping` and `CRM_Core_ActionScheduleTmp`), abandons
the pretense of DB editing, centralizes mapping-related lookups, and replaces
various references to `CRM_Core_DAO_ActionMapping`.

This is envisioned as an intermediate step; the two classes are still a bit
heavy (mixing together logic for unrelated entities) and should be split
further.

CRM_Core_BAO_ActionScheduleTest - Fix racy test

CRM-13244 - CRM_Core_BAO_ActionScheduleTest - Add some event examples

CRM-13244 - CRM_Core_BAO_ActionSchedule - Simplify `sendMailing()`

CRM-13244 - CRM_Core_BAO_ActionSchedule - Use TokenProcessor instead of replaceComponentTokens

CRM-13244 - CRM_Core_BAO_ActionSchedule - Simplify `sendReminder`

The old `sendReminder()` had too many responsibilities (loading token data
and processing token data and sending emails and sending SMS's), contained
lots of duplicate code, and did pointless variable shuffling.  The Smarty
and HTML parameters were enforced inconsistently, and the "From" address had
weird side-effects, ad nauseum.

This commit splits `sendReminder()` into three pieces:
 * `TokenCompatSubscriber` for filling in token data.
 * `sendReminderSms()` for sending SMS.
 * `sendReminderEmail()` for sending email.

CRM-13244 - Civi\Token - Add new class for processing messages with tokens.

CRM-13244 - CRM_Core_BAO_ActionSchedule - Comment about seemingly dead code

CRM-13244 - ActionSchedule - Normalize static references

Some code uses "self::" and other use "CRM_Core_BAO_ActionSchedule::". It'll
be easier to reorganize if the notation is consistent.

CRM-13244 - ActionSchedule - Extract methods from sendMailings

New methods
 * listMailingTokens()
 * prepareMailingQuery()
 * prepareMailingTokens()
 * createMailingActivity()

CRM-13244 - ActionScheduleTest - Use some tokens

CRM-16355 - bgm addon to change CMS locale - make it temporary

CRM-16355 - bgm addon to change CMS locale - cleanup

CRM-16355 - bgm addon to change CMS locale

CRM-16355 - checkstyle typos

CRM-16355 - checkstyle typos

CRM-16355 - communication language choices order + help text fixes

CRM-16355 - cleanup, use of CRM_Core_I18n::language instead of a new function