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

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-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

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

Merge pull request #9660 from colemanw/CRM-19769

CRM-19769 - Add colorContrast js function

CRM-19853 set minimum value to zero

CRM-19853 number attributes

Merge pull request #9653 from jmcclelland/CRM-19847

Crm 19847

CRM-19853 replace interval controls with numeric

CRM-19847: Style fixes

CRM-19847: ensure we get all parts of the query, not just the values

CRM-19769 - Add colorContrast js function

Merge pull request #9503 from agh1/same-match-honoree

CRM-19725 CiviContribute: exclude donor from potential honoree dedupe matches

Merge pull request #9659 from seamuslee001/php-comment-fix

Comment Fixes

Merge pull request #9658 from eileenmcnaughton/comments

Comments

Fix return for contribution api call

Remove duplicated doc bloc comments

Merge pull request #9657 from totten/master-docblocks

(NFC) Civi/*, CRM_Utils_SQL_Select - Fix various warnings

Comment Fixes

A few more comment fixes. Remove unused var rather than document it

Clarify return val, fix casing on a couple of function calls

Further comment fixes

Further comment fixes

Add comment blocks

Merge pull request #9656 from eileenmcnaughton/comments

Add comment blocks

Civi/Payment - Fix various PHPStorm warnings

Civi/Token - Fix various PHPStorm warnings

Civi/Test - Fix various PHPStorm warnings

Civi/Core - Fix various PHPStorm warnings

Civi/CCase - Fix various PHPStorm warnings