CRM-19830 minor refactor on ORDER BY to facilitate further CRM-19815 fixed.

The larger part of this is simply the removal of an IF not obsolete due to the early return clause

Set version to 4.7.17

Merge pull request #9727 from eileenmcnaughton/BAO_Updates

CRM-19925 Bao updates

CRM-19925 Updated DAO files

Merge pull request #9722 from eileenmcnaughton/comments

Comment fixes plus fix inconsistent unused return bool

CRM-19925 Add Entity data and date format data to fields array on  DAO.

    Per ticket this is intended to reduce in-code calculations by effectively
    caching some commonly used data into the DAO. In many cases civicrm
    compiles a large fields array & then twists itself up trying to distinguish the
    fields from each other. This makes the entity info readily available.

    Also moved to metadata is the format of core date fields.

Merge pull request #9720 from eileenmcnaughton/new_tests

[nfc] Updated test generated examples

Merge pull request #9721 from eileenmcnaughton/cont4

Remove unused var, comments & clean up & refactor caching function

Merge pull request #9705 from colemanw/CRM-19778

CRM-19778 - Case statuses per case type & caseType config ui improvements

CRM-19723 - CaseType Config UI improvements: icons, popups, labels

When adding a new role or activity type, use a popup to allow more robust configuration.
Show activty type icons.
Show activity type labels instead of machine names.

CRM-19778 - Limit statuses in 'change case status' activity

Cleanup redundant caching

CRM-19778 - Limit statuses in 'open case' activity

Case form code cleanup

CRM-19778 - Configuration ui for case statuses per case-type

Comment fixes plus fix inconsistent unused return bool

Towards CRM-19815 refactor cached clause to use Civi:Statics

Towards CRM-19815 Comment fixes & test tidy-up

Remove unused var

Update test examples.

This is not an exhaustive update. It covers a good chunk of the common changes but a lot of copy & paste means the examples are being overwritten by other tests. I have removed some of that

Merge pull request #9678 from GinkgoFJG/CRM-19877-joomla-setting-create

CRM-19877: Blacklisted params extraneously passed by Joomla to the settings API.

Merge pull request #9708 from litespeedmarc/CRM-19906

CRM-19906 - allow @ on drupal users

Merge pull request #9713 from jitendrapurohit/CRM-19895

CRM-19895: membership renewal page style collapsed if email disabled

Merge pull request #9703 from colemanw/CRM-19897

CRM-19897 - Remove reference to nonexistant permission

Merge pull request #9714 from colemanw/CRM-19829

CRM-19829 - Attachment API: return icons

Merge pull request #9717 from eileenmcnaughton/comments

[nfc] Comment fixes, remove one unused var, declare visibility

Comment fixes

Fix visibility on legacy functions

Remove unused variable

Merge pull request #9716 from eileenmcnaughton/nfc-cont

[NFC] add comments & extract function in contribution search

CRM-19829 - Move icon classes to data file

CRM-19829 - Attachment API: return icons

Comment fixes

CRM-19815 tidy test & lock in return of financial_type

CRM-19830 add test to lock in test contract

CRM-19815 extract soft credit fields for readability

CRM-19895: membership renewal page style collapsed if email disabled

Merge pull request #9699 from jitendrapurohit/CRM-19003

CRM-19003: Profile listing doesn't produce expected results when rang…

Merge pull request #9579 from omarabuhussein/CRM-19801-fix-relationship-saving

CRM-19801: Fixing changing a relationship type from A to B or vice versa

Merge pull request #9709 from seamuslee001/CRM-19907

CRM-19907 Fix error where show events setting is NULL

Merge pull request #9707 from colemanw/CRM-19723

CRM-19723 iconPicker style tweak

CRM-19907 Fix error where show events setting is NULL

CRM-19906 - allow @ on drupal users

CRM-19723 iconPicker style tweak

Merge pull request #9706 from colemanw/iconPicker

iconPicker style improvements

iconPicker style improvements

Merge pull request #9698 from jitendrapurohit/CRM-19842

CRM-19842-Can't delete a dedupe rule

Merge pull request #9695 from jitendrapurohit/CRM-19875

CRM-19875: Search builder: Fatal error when searching custom field of…

Merge pull request #9697 from jitendrapurohit/CRM-19794

CRM-19794: Wrong fields displayed when setting a price set for a cont…

Merge pull request #9700 from jitendrapurohit/tagfix

typo fix in New Tag Form.

CRM-19897 - Remove reference to nonexistant permission

Merge pull request #9701 from fuzionnz/docfix-case_mgmt_setup_url

Docs fix: update for changed URL of Case Management > Setup page.

Docs fix: update for changed URL of Case Management > Setup page.

typo fix in Manage Tag page

CRM-19003: Profile listing doesn't produce expected results when range-searching custom date field

CRM-19801: Fix inablitiy of changing (current employer) checkbox of relationship

CRM-19842-Can't delete a dedupe rule

CRM-19794: Wrong fields displayed when setting a price set for a contribution page

Merge pull request #9696 from colemanw/CRM-19893

CRM-19893 - Tweak optionValue template to show grouping field if present

CRM-19893 - Tweak optionValue template to show grouping field if present

CRM-19875: Search builder: Fatal error when searching custom field of type country

Merge pull request #9693 from totten/master-browser-cache

CRM_Extension_Browser - Allow different cache files for different feeds

Merge pull request #9689 from totten/master-tmpl

CRM-19690 - Task_AdhocMailing - Use highest-priority template type

Merge pull request #9691 from jmcclelland/CRM-19890

CRM-19890 - ensure we are only listing the active case manager

CRM_Extension_Browser - Allow different cache files for different feeds

The motivation here is to support these two different CLI commands:

```
cv dl myextension
cv dl myextension --dev
```

The two commands differ in that they pull from different feeds -- e.g.  the
stable feed `https://civicrm.org/extdir/ver={ver}|cms={uf}` vs the
developmental feed `https://civicrm.org/extdir/ver={ver}|cms={uf}|status=`.
However, this creates the real possibility that the user might go back/forth
among different feeds (omitting/enabling the `--dev` option per whim).

CRM-19890 - ensure we are only listing the active case manager

Merge pull request #9687 from yashodha/CRM-19883

CRM-19883 - Activity Priority Field is Not Included in Advanced Searc…

CRM-19690 - Task_AdhocMailing - Use highest-priority template type

If you perform a contact search and create a new mailing, it would use
`template_type=traditional`, even if another template type had greater
priority. With this patch,  it respects the priority.

Note: I considered changing the default in Mailing.create API to always
match most-preferred template-type.  However, that would break some existing
API consumers (e.g.  headless consumers or ones who define their own UI).
For external API-based integrations, we should preserve the default
semantics of `body_text`/`body_html` by defaulting to
`template_type=traditional`.

The preference in CRM-19690 is that any use-case based on the screen
`civicrm/a/#/mailing/{id}` should have its default determined by weight.

minor fix

CRM-19883 - Activity Priority Field is Not Included in Advanced Search or Activity Reports

Merge pull request #9661 from monishdeb/CRM-18141

CRM-8526, CRM-19786, CRM-18141, CRM-19757 : Fix Schedule Reminder tokens

Merge pull request #9672 from colemanw/CRM-17663

CRM-17663 repair null dashlet names

Merge pull request #9681 from jitendrapurohit/CRM-19884

CRM-19884: DB Error on Activity Details Report

Merge pull request #9669 from jitendrapurohit/CRM-19172

CRM-19172: Fix onbehalf contact id on membership signup

CRM-19884: DB Error on Activity Details Report

Merge pull request #9675 from agileware/CRM-19874

CRM-19874: Allow overriding sort in the getlist API without clobbering pager

CRM-19877: Blacklisted params extraneously passed by Joomla to the settings API.

extended unit test to asset contact custom field token

Merge pull request #9677 from yashodha/CRM-19863

CRM-19863 - 'item_name' unnecessarily truncated when passed to PayPal

greeeting token and handle multi-value custom field fixes

CRM-19863 - 'item_name' unnecessarily truncated when passed to PayPal

CRM-19874: Allow overriding sort in the getlist API without clobbering pager

----------------------------------------
* CRM-19874: Allow overriding sort in the getlist API without clobbering pager
  https://issues.civicrm.org/jira/browse/CRM-19874

CRM-19707 CiviContribute Assigns Payment Processor ID correctly (#9481)

* CRM-19707 CiviContribute Assigns Payment Processor ID correctly

Original code used Payment Processor Type ID for PayPal.  New code
looks up actual Payment Processor ID, instead.

* Corrected coding standards

Removed white space

* Revert "Corrected coding standards"

This reverts commit 6f8109650736ccb20ee08328f208c3852742b71e.

* Revert "CRM-19707 CiviContribute Assigns Payment Processor ID correctly"

This reverts commit 597e639e632ae793c422a2f5ce6c41e44c70b29e.

* CRM-19707

Created function to get Payment Processing ID and called it in the two
locations needed

* Corrected Coding Standards

I’m still learning

* Corrected function call

Misnamed Processor as Processing

* Forgot this

It’s been a while,  Forgot the this-> to call the proper function.

* Still trying removed this

Attempting to get through the errors

* self, not this

Still learning OO PHP.  This should finally fix it.

Merge pull request #9673 from seamuslee001/CRM-19831

CRM-19831 Fix the loading of child groups

CRM-19831 Fix the loading of child groups

Merge pull request #8916 from cividesk/CRM-19256-4.7

CRM-19256: Add permission hook

CRM-17663 repair null dashlet names

See http://civicrm.stackexchange.com/questions/16644

alternative way to handle custom field token

add unit test

fix for greeting token

CRM-18141: Scheduled reminders for activities not rendering custom field tokens

CRM-19172: Fix onbehalf contact id on membership signup

Merge pull request #9667 from mlutfy/master-crm19860

CRM-19860: Event participant batch update: respect "options per line" setting for radio buttons

Merge pull request #9633 from jaapjansma/CRM-19831

Fixed CRM-19831

CRM-19827 Avoid unnecessary core_resources processing during ajax calls (#9623)

* CRM-19827 Avoid unnecessary core_resources processing during ajax calls

* CRM-19827 - Add unit test for isAjaxMode

* CRM-19827 - Test fix

Merge pull request #9651 from fuzionnz/CRM-19844-hook_only_when_success

Fire hook_civicrm_emailProcessor('activity') if Activity,create successful

CRM-19860: Event participant batch update: respect "options per line" setting for radio buttons

Merge pull request #9595 from totten/master-extbatch-env

Civi\API\ExternalBatch - Verify that $_ENV is working

Merge pull request #9664 from konadave/CRM-19853

CRM-19853 replace interval controls with numeric

Merge pull request #9637 from omarabuhussein/CRM-19832-searchtask-hook

CRM-19832: Ensure that searchTasks hook get invoked once