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

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-21196 - Event reports localization

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

Merge pull request #10838 from jmcclelland/CRM-16964

CRM-16964: ensure FILL works on import with custom fields.

Merge pull request #10952 from civicrm/michaelmcandrew-patch-1

Fixing type (missing function call) in example for hook_civicrm_alterAngular

Merge pull request #10973 from civicrm/4.7.25-rc

4.7.25 rc

Merge pull request #10963 from eileenmcnaughton/cleanup

[NFC] comments clean up in test classes.

Merge pull request #10972 from totten/4.7.25-rc-tpl

CRM-18231 - Cleanup `environment` example in `civicrm.settings.php.template`

CRM-18231 - civicrm.settings.php.template - Remove extra whitespace

CRM-18231 - civicrm.settings.php.template - Move `global` declaration

When using the example snippets in the settings file, it's easy to
accidentally leave `global $civicrm_setting` commented out -- which is
fairly confusing/mysterious way to have your settings ignored.

There's no cost to declaring `global $civicrm_setting` by default in new
config files.

CRM-18231 - civicrm.settings.php.template - Fix broken `environment` example

The expression `CRM_Core_BAO_Setting::DEVELOPER_PREFERENCES_NAME` does not
work within `civicrm.settings.php` because the classloader has not been
initialized yet.  This file is just supposed to store the data needed for
bootstrap.

Merge pull request #10922 from eileenmcnaughton/tpl

CRM-21125 allow class assignment on report row links

Merge pull request #10969 from yashodha/CRM-21172

CRM-21172 - Fix 'Edit Contact Information' link on contact dashboard

CRM-21172 - Fix 'Edit Contact Information' link on contact dashboard

Update Hook.php

Merge pull request #10968 from civicrm/4.7.25-rc

4.7.25 rc

Merge pull request #10936 from seamuslee001/CRM-18231

CRM-18231 Remove need for define statement and replace with new setti…

CRM-18231 Remove need for define statement and replace with new setting not settable in UI and always use settings to get information

Switch to getMandatory function as per Tim's comment

Merge pull request #10953 from seamuslee001/CRM-20892

CRM-20892 Add in modified_date column to civicrm_mailing to assist in…

CRM-20892 Fix issue where saving to the database wasn't creating a new modified_date

CRM-21169 - Fix broken inline edit for profiles

CRM-20750 follow up.

Change logic for determinig paymentInstrumentChange to look at whether
['payment_instrument_id'] isset. If not it has not been passed by the calling
function and by definition the calling function is not attempting to change it

CRM-20750 follow up fix, reversal should be against original account.

On trying to investigate why a test change was required in the original PR I came to the following conclusions:

1) the test function was causing problems by being hard to read. As a result it was too easy to see it as
'accounts magic' - I started a cleanup here
2) Once I understood what was being tested & failing I concluded the change in the PR was in fact wrong
as it was putting a negative entry against the new financial account rather than the original one.