Merge pull request #10837 from civicrm/4.7.24-rc

4.7.24 rc

Merge pull request #10814 from JMAConsulting/CRM-20610-3-rc

CRM-20610: Handle payment instrument change on Payment Edit form

Merge pull request #10833 from JMAConsulting/CRM-20800-rc

CRM-20800 : User Cannot Cancel Recurring Payment With Paypal

CRM-20800 : User Cannot Cancel Recurring Payment With Paypal

Merge pull request #10793 from eileenmcnaughton/tpl

CRM-20990 add contributionStatus to online template

Merge pull request #10427 from mattwire/report_bounce_type

CRM-20643 Fix invalid bounce type filter for bounce report

Merge pull request #10824 from seamuslee001/CRM-21026

CRM-21026 fix issue with ContributionCount not including disabled fin…

Merge pull request #10827 from mattwire/CRM-17157_support_multiple_decimalplaces_formrule_money

CRM-17157 support multiple decimalplaces formrule money

Merge pull request #10817 from JMAConsulting/CRM-21005

CRM-21005, "Record Payment" dialogue lacks required field and is clunky

Minor comment fixes and further reduction of passing by reference with changing to new wrapper functions

Merge pull request #10818 from eileenmcnaughton/recur_date

CRM-21027 Fix next recurring payment not accurately calculated

Checkstyle

Add unit test

Support multiple decimal places in CRM_Utils_Rule::money for currencies such as bitcoin, also helps towards CRM-20772 which requires higher precision decimals to be stored in the database to calculate tax inclusive amounts correctly

additional fix

Code updates as per Eileen

Attempt to get rid of pasing by reference and change variable as needed

CRM-21026 fix issue with ContributionCount not including disabled financialtypes

Fix references and move flushing of cache to all financialTypes for the safe side

Merge pull request #10823 from mattwire/CRM-21031_financialtypedisplay

CRM-21031 Fix for id instead of financialtype name

Fix for id instead of financialtype name

Merge pull request #10801 from JMAConsulting/CRM-13309

CRM-13309: Participant Role on Event's Schedule Reminder form does not populate

test build failure fix

Merge pull request #10812 from kainukempowerment/keikelboom-civicoop

Klaas Eikelboom (Kainuk Empowerment) is member of CiviCooP

Merge pull request #10820 from JMAConsulting/CRM-21029

CRM-21029 - Report - Fix Activity Report SQL Syntax Error

Merge pull request #10813 from JMAConsulting/CRM-21019

CRM-21019, Changed column labels on batch transaction listing page

CRM-21029 - Report - Fix Activity Report SQL Syntax Error

Merge pull request #10819 from tschuettler/CRM-21029

CRM-21029 - Report - Fix Activity Report SQL Syntax Error

Merge pull request #10744 from JMAConsulting/CRM-20942

CRM-20942: Email location type must be 'primary' in profile or user creation won't work for event

label change

concatenate check_number and trxn_id in contribution

additional fixes

CRM-21029 - Report - Fix Activity Report SQL Syntax Error

CRM-21027 Fix next recurring payment not accurately calculated when effective date passed in

CRM-21005, mark payment instrument as required

----------------------------------------
* CRM-21005: "Record Payment" dialogue lacks required field and is clunky
  https://issues.civicrm.org/jira/browse/CRM-21005

CRM-21005, set amount to default owed

----------------------------------------
* CRM-21005: "Record Payment" dialogue lacks required field and is clunky
  https://issues.civicrm.org/jira/browse/CRM-21005

Merge pull request #10816 from mattwire/nfc_activity_pseudoconstant

CRM-21023 Replace deprecated code and hardcoded IDs with pseudoconstants in Activity BAO

Merge pull request #10709 from totten/master-ids-cleanbreak

CRM-20926 - Allow extensions to flag PHPIDS html/json fields

CRM-20926 - Resolve "exception"/"exceptions" discrepency. Improve test.

The PHPIDS config format uses the property:

```
exceptions => array('field_1)`
```

But our XML format uses the tag:

```
<exception>field_1</exception>
```

Grammatically, those both make sense. But they were getting confused.

This modifies the XML parser to map the tags `<exception>` to property
`exceptions`, and it updates the unit-test to ensure that the data
(from the route and the default config) are correctly merged.

CRM-20926 - CRM_Core_Menu - Parse XML element <ids_arguments>

CRM-20926 - CRM_Core_MenuTest - Add basic test for XML parsing

CRM-20926 - Kill Config.IDS.ini

Background
==========

Previously, the IDS configuration was written to a temp file and then read from the tempfile.

Since we no longer read the tempfile, we don't need to write to it either.

Before
======

System autocreates the `Config.IDS.ini` file.

After
=====

System does not create `Config.IDS.ini`.

CRM-20926 - CRM_Core_IDS - Generate configuration from an array

Background
==========

PHPIDS scans all inputs and uses heuristic checks to identify malicious
content.  It relies on a configuration with data about the fields we'll
process.

Before
======

The function `createConfigFile()` produces the standard configuration as an
INI file.  The configuration was represented as an unalterable string.

After
=====

 * The configuration is represented as an array.
 * The configuration is generated via `createStandardConfig()`
 * The function `createConfigFile()` only handles file I/O.

CRM-20927 - civicrm_menu - Add test for reading/writing `module_data`

CRM-20927 - civicrm_menu - Update examples in `civicrm_generated.mysql`

CRM-20927 - civicrm_menu - Add generic column for other data

Background
==========

We need a place to store per-page information about IDS configuration.

More generally, as we continue adding features through extensions, we should
anticipate that the route metadata will expand in ways that we can't
anticipate.  Adding a general-purpose serialized field will help us define
new behaviors through extensions.

Before
======

All metadata in the routing table (`civicrm_menu`) had to stored in a
separate SQL column.

After
=====

Any unrecognized fields will be stored and loaded from the column
`module_data`.

CRM_Core_BAO_Navigation::getMenus() - Remove unused function

 * This function does not appear to be called anywhere in `universe`.
 * It references the table/DAO for `civicrm_menu`, which seems like a bad idea.
 * This function goes back v3.0 (when the navbar and router were decoupled) -- but even then, it wasn't used!
   I'm guessing it was an interim and forgotten artifact written while decoupling navbar and router.

Merge pull request #10679 from yashodha/4.7.CRM-20733

CRM-20733 - Relationship report improvements

Merge pull request #10802 from lcdservices/CRM-21004

CRM-21004 only remove html snippet task if setting enabled

Merge pull request #10798 from jitendrapurohit/CRM-21000Fix

CRM-21000 : Fix DB error on Advance Search Mailings

Replace deprecated code and hardcoded IDs with pseudoconstants.

Merge pull request #10712 from davidjosephhayes/master

Change "Last Action Date" filter variable name CRM-20930

Merge pull request #10794 from tschuettler/CRM-20993

CRM-20993 - API - Extention get - Fix filtering by ID

Merge pull request #10781 from eileenmcnaughton/postal

Unit test for CRM-20970

added unit test and other changes

CRM-20610: Handle payment instrument change on Payment Edit form

Allow filtering when return options are set

CRM-20993 - API - Extention get - Fix filtering by ID

----------------------------------------
* CRM-20993: API - Extension get - Cannot filter by ID anymore
  https://issues.civicrm.org/jira/browse/CRM-20993

Merge pull request #10808 from jmcclelland/CRM-20790

CRM-20790 - ensure relationships are created on import

Klaas Eikelboom (Kainuk Empowerment) is member of CiviCooP

CRM-21019, Changed column labels

----------------------------------------
* CRM-21019: Change column labels
  https://issues.civicrm.org/jira/browse/CRM-21019

changed labels

fixed labels

Set version to 4.7.25

Merge pull request #10811 from jitendrapurohit/CRM-20913-sync-inprogress

CRM-20913: Fix inprogress value to match with contribution status

CRM-20913: Fix inprogress value to match with contribution status

Merge pull request #10807 from mattwire/link_to_release_notes

CRM-21012 Add link to release notes

Merge pull request #10803 from jitendrapurohit/CRM-20913-sync-ov

CRM-20913 use pledge option group for pledge payments

Merge pull request #10806 from JMAConsulting/CRM-20610-2

CRM-20610: 'Payment Edit' form improvements

CRM-20790 - ensure relationships are created on import

Add link to release notes

add metadata to trxn_id

CRM-21004: change conditions under which we remove html snippet action

CRM-20610: 'Payment Edit' form improvements

Merge pull request #10677 from fuzionnz/CRM-20885-suppress_fakefile_error

Suppress error if fakeFile cannot be created.

sync value for overdue with contribution status for safety

CRM-20913 use pledge option group for pledge payments

Merge pull request #10303 from seamuslee001/CRM-20521

CRM-20521 Convert returning groups and mailings for CiviMail new mail…

CRM-21004 only remove html snippet task if setting enabled

Merge pull request #10799 from JMAConsulting/CRM-21002

CRM-21002: Credit card type icons are incorrectly populated on backoffice live mode

CRM-13309: Participant Role on Event's Schedule Reminder form does not populate

CRM-20521 Ensure that only Mailing List groups appear in the receipients box

CRM-20521 Ensure that Disabled groups are also excluded

CRM-20521 Convert returning groups and mailings for CiviMail new mailing screen into an API request rather than loading in the DOM

CRM-21002: Credit card type icons are incorrectly populated on backoffice live mode

Merge pull request #10796 from tschuettler/CRM-20995

CRM-20995 - API - Extension get - Filter by full_name

Change new parameter to default to 'keys'

CRM-21000 : Fix DB error on Advance Search Mailings

Merge pull request #10340 from Stoob/master

CRM-20563 change fatal error message

Merge pull request #10795 from jitendrapurohit/CRM-20994

CRM-20994 : Fix hardcoded relationship type value

Merge pull request #10792 from seamuslee001/CRM-20989

CRM-20989 Fix Active provider count to know about multisite

Merge pull request #10789 from JMAConsulting/CRM-20987

CRM-20987, added batch transaction column

Fix coding standard

Fix undefined index in UnitTests

CRM-20995 - API - Extension get - Filter by full_name

----------------------------------------
* CRM-20995: API - Extension get - Ignores parameter full_name as created by API explorer
  https://issues.civicrm.org/jira/browse/CRM-20995

CRM-20994 : Fix hardcoded relationship type value

Merge pull request #10788 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error on Activity tab

CRM-20987, fixed unit test

----------------------------------------
* CRM-20987: Add transaction date field to listings of transactions
  https://issues.civicrm.org/jira/browse/CRM-20987

fixed indexes

CRM-20990 add  to online template

Add Unit test to provde domain filtering is working with activeProviderCount

CRM-20989 Fix Active provider count to know about multistie and return a proper count

CRM-20947, fixed notice error

----------------------------------------
* CRM-20947: Remove Deprecation Notice for Option group
  https://issues.civicrm.org/jira/browse/CRM-20947

fixed fieldname

Merge pull request #10763 from JMAConsulting/CRM-20967

CRM-20967, use is_payment = TRUE to retrieve payments