Merge pull request #10435 from GinkgoFJG/CRM-20653

CRM-20653: Allow non-truthy defaults for retrieval of request parameters.

Merge pull request #11011 from eileenmcnaughton/comment_add

CRM-20226 [NFC] add comment to warn coders about performance impact

Merge pull request #11027 from artfulrobot/master

CRM-21221 fix logic precedence error when finding contribution record

Merge pull request #10670 from cividesk/CRM-20881

CRM-20881 Fixed Membership Pending issue, it was checking contribution status u…

Merge pull request #11025 from colemanw/CRM-21195

CRM-21195 - Add icon picker when editing menu item

Merge pull request #10902 from MiyaNoctem/CRM-12167-price-field-value-visibility

CRM-12167: Price Field Value Visibility

CRM-12167: Fix Geocode Test

IconPicker fixes - add crm-i class and display icon in jstree

CRM-12167: Finish Implementing Visibility per Price Option

Added validations so that:
- A public price field has at least one public option
- An admin price field has all admin options

Implemented removal for Select and Checkbox field types. Also added condition
to remove Admin values only if logged user doen't have appropriate
permissions.

Implemented different permissions check to see if admin options should be
deleted or not if the price set is being used for a contribution page or an
event page.

Implemented functionality so that if a user chooses Admin visibility for the
price field, all options are forced to have Admin visibility as well.

Added default visibility as public on creation, in case it's not set, and
protected access to unexisting visibility_id array key.

Moved SQL to add new visibility field to PHP upgrader for v4.7.26.

CRM-12167: Add Tests for Price Field Option Visibility

Added tests to verify the field is added to the DAO and validations work on
Field creation form and Option create/edit form.

CRM-12167: Start Implementation of Visibility per Price Option

template updates
php updates
adding column to upgrader and xml
some improvements from comparing to current price field visibility id approach

Merge pull request #10834 from jitendrapurohit/CRM-20999

CRM-20999 - Fix multiple auto_renew id on contribution live page

CRM-21221 fix logic precedence error when finding contribution record

CRM-21195 - Add icon picker when editing menu item

Merge pull request #10996 from omarabuhussein/CRM-21195-Adding-icons-to-navigation-menu

CRM-21195: Adding the ability to add and show icons on top navigation menu items

Merge pull request #11024 from jackrabbithanna/CRM-21220

spacing fix CRM-21220

spacing fix CRM-21220

Merge pull request #11023 from civicrm/4.7.25-rc

4.7.25 rc

Merge pull request #10977 from jitendrapurohit/CRM-21160

CRM-21160 - Make event_type_id available in event message templates

Merge pull request #11020 from mattwire/CRM-21216_getMembershipTypesByOrg_to_api

CRM-21216 Replace member_BAO_membershiptype_getMembershipTypesByOrg with API eq…

Merge pull request #11003 from JMAConsulting/CRM-21199

CRM-21199, removed JS to hide invoice page setting

Merge pull request #11013 from davejenx/CRM-16836

CRM-16836 - make Basic Search form group select respect ACLs

Merge pull request #11008 from lcdservices/CRM-21205

CRM-21205 use proper currency in batch list

Replace member_BAO_membershiptype_getMembershipTypesByOrg with API equivalent

CRM-16836 - make Basic Search form group select respect ACLs

CRM-20226 [NFC] add comment to warn coders about performance impact

CRM-21205 use proper currency in batch list

Merge pull request #10990 from seamuslee001/CRM-21006

CRM-21006 Ensure that Buttons are correctly displayed

Merge pull request #11002 from seanmadsen/CRM-21022

CRM-21022 - Parameterize variables in SQL query

Merge pull request #10993 from seamuslee001/CRM-21123

CRM-21123 Ensure that the selectedChild on message templates is one o…

Merge pull request #10983 from JMAConsulting/CRM-21189

CRM-21189 - Add permission for Close and reopen Batch

CRM-21123 Ensure that the selectedChild on message templates is one of allowed types

CRM-21123 Put the array into the if statement

Add preUpgrade message

CRM-21189, updated code to respect new permission on batches

----------------------------------------
* CRM-21189: Add permission for Close and reopen Batch
  https://issues.civicrm.org/jira/browse/CRM-21189

CRM-21189, added new permissions

----------------------------------------
* CRM-21189: Add permission for Close and reopen Batch
  https://issues.civicrm.org/jira/browse/CRM-21189

Merge pull request #10994 from seamuslee001/CRM-20906

CRM-20906 Validate the extension key is of general standard

CRM-21170 - Provide option to filter by contact id & external id (#10966)

* CRM-21170 - Provide option to filter by contact id & external id

CRM-21006 - Escape all title and alt attributes

This commit opportunistically adds HTML output encoding to all Smarty
variables any time they appear within an HTML `title` or `alt`
attribute. Why? Because this helps us prevent XSS and is very very
unlikely to cause any unwanted side effects.

Code locations found by searching the `templates` directory for:

(title|alt)=(['"])((?!\2).)*\{\$((?!(\|(escape|crmDate))|\}).)+\}((?!\2).)*\2

CRM-21006 Ensure that Buttons are correctly displayed

Merge pull request #10989 from seamuslee001/CRM-21014

CRM-21014 Ensure that preimum name is EOU

CRM-21199, removed JS to hide invoice page setting

----------------------------------------
* CRM-21199: Remove dependancy for  'Default invoice payment page'
  https://issues.civicrm.org/jira/browse/CRM-21199

CRM-21014 only modify the alt tags

Move to using CRM_Utils_Rule and CRM_Utils_Type::validate as per Sean's comments

CRM-20906 Add in unit test of Extenion check

CRM-20906 - Fix regex for validating extension key

CRM-20906 Validate the extension key is of general standard

Merge pull request #10992 from seamuslee001/CRM-20907

CRM-20907 Ensure that contact_type is valid in deduperules

Merge pull request #10974 from highfalutin/CRM-21174-upgrade-menu-module_data

CRM-21174: Do not try to write civicrm_menu.module_data if it doesn't exist yet

CRM-21022 - Parameterize variables in SQL query

Merge pull request #11001 from seanmadsen/CRM-20895

CRM-20895 - Encode display of search criteria

CRM-20895 - Encode display of search criteria

CRM-21195: Fixing the code style

CRM-21195: a little improvment

Merge pull request #10935 from eileenmcnaughton/dd

CRM-21134 fix enotices in payment processor code assignment.

Merge pull request #10997 from francescbassas/CRM-21196

CRM-21196 - Event Income Details and Income Count Summary report localization

CRM-21195: Using DAO instead of direct SQL when building navigation menu

CRM-21196 - Event reports localization

CRM-21195: Adding the ability to add icons to menu items

Merge pull request #10978 from jitendrapurohit/CRM-21117

CRM-21117: Line item not shown when paying later using membership priceset

Merge pull request #10991 from seamuslee001/CRM-21011

CRM-21011 Sort out tag descriptions when displaying better

Merge pull request #10982 from eileenmcnaughton/trxn_currency

CRM-21187 Fix handling of currency when updating a contribution to be…

CRM-20907 only throw Execption if contactType is set in the Ruquest

Merge pull request #10857 from adixon/CRM-21063

CRM-21063 Add date to survey/details report

CRM-20907 Ensure that contact_type is valid in deduperules

CRM-21011 Sort out tag descriptions when displaying better

CRM-21014 Ensure that preimum name is EOU

CRM-21134 fix enotices in payment processor code assignment.

This is generally a code tidy up to push out the principles we have been implementing (ie.
the idea that the processors define their fields. The code can cause e-notices in extensions as is
because it is prescriptive about fields.

I removed the ANDing in the tpl because actually we are not looking to match numbers like 2
2 is the number we have hard-coded for direct debits (we should be using an option group but that
is another story

CRM-21187 Fix handling of currency when updating a contribution to be completed.

This is replicable via the api & potentially other flows. The currency id
of NULL is not accepted in financialTrxn BAO without it replacing with a
default, which is not appropriate on an update.

Merge pull request #10984 from colemanw/CRM-21127

CRM-21127 - Fix return properties in api.Event.get

Merge pull request #10981 from jitendrapurohit/CRM-21183

CRM-21183: Updating Partially paid contribution to Completed doesn't …

Merge pull request #10985 from colemanw/CRM-21178

CRM-21178 - Make custom field of type Link clickable

CRM-21178 - Make custom field of type Link clickable

CRM-21127 - Fix return properties in api.Event.get

Merge pull request #10980 from eileenmcnaughton/qa

CRM-20750 follow up fix from QA

Merge pull request #10964 from yashodha/CRM-21169

CRM-21169 - Fix broken inline edit for profiles

Merge pull request #10979 from colemanw/CRM-21182

CRM-21182 - Activity API: Fetch case fields via the join syntax

CRM-21182 - Add test for fetching case fields in activity.get

Minor fixes

CRM-21183: Updating Partially paid contribution to Completed doesn't update membership

Merge pull request #10680 from JMAConsulting/CRM-20889

CRM-20889 : Toggle check_number field on backoffice form as payment form fields

Merge pull request #10970 from JMAConsulting/CRM-20276

CRM-20276, fixed code to store correct value in financial item table …

CRM-20750 follow up fix from QA

In pr #10920 @sunilpawar spotted this error

Merge pull request #10920 from eileenmcnaughton/pi

CRM-20750: Incorrect financial trxn entries when payment instrument i…

CRM-21182 - Activity API: Fetch case fields via the join syntax

add unit test

CRM-21117: Enable useForMember for pending membership signup

add unit test

fixed jenkins style

CRM-20276, fixed code to store correct value in financial item table for amount when contribution amount is changed

----------------------------------------
* CRM-20276: When editing a contribution the value in civicrm_financial_item_amount is not updated
  https://issues.civicrm.org/jira/browse/CRM-20276

fix payment type label

additional fix to load selected payment instrument field on edit

CRM-20889: Toggle check_number field on backoffice form as payment form fields

CRM-20869, removed cruft code for check number

----------------------------------------
* CRM-20869: Remove cruft code for check number
  https://issues.civicrm.org/jira/browse/CRM-20869

Merge pull request #10976 from eileenmcnaughton/4.7.25-rc

4.7.25 rc

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

Merge pull request #10975 from eileenmcnaughton/4.7.25-rc

CRM-21175 fix fatal error on multiple custom field import.

CRM-21175 fix fatal error on multiple custom field import.

Additional test is tangental....

Merge pull request #10439 from jitendrapurohit/CRM-20657

CRM-20657: get all participant ids if multiple contacts register from…

Store menu item fields other than module_data

Do not try to write civicrm_menu.module_data if it doesn't exist yet