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

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

Merge pull request #10812 from kainukempowerment/keikelboom-civicoop

Klaas Eikelboom (Kainuk Empowerment) is member of CiviCooP

Merge pull request #10813 from JMAConsulting/CRM-21019

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

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

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

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

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

Merge pull request #10787 from JMAConsulting/CRM-20984

CRM-20984: Move placement of payment processor field

CRM-20987, added batch transaction column

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

CRM-20947, fixed notice error on Activity tab

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

Merge pull request #10726 from JMAConsulting/CRM-20946

CRM-20946, use line total of line item to store in financial item table

CRM-20984: Move placement of payment processor field

Merge pull request #10765 from seamuslee001/report_fixes

CRM-20969 Fix issues in order by clauses of reports showing errors in 7.1

added unit test

CRM-20946, use line total of line itme to store in financial item table

----------------------------------------
* CRM-20946: Wrong entries in financial table when contribution is cancelled
  https://issues.civicrm.org/jira/browse/CRM-20946

Remove orderBy function following Eileen's testing

Use the orderByArray and groupByArray variables

CRM-20969 alternate approach to resolving php 7.1 errors

I am proposing this as an alternate approach for fixing the 7.1 errors.

Note this only covers one report - I think we should move that function which is actually
generic to the parent class, however I think the one in contribute.summary is
a bit more nuanced.

Note this is also the approach taken by the extendedreport
extension (which takes it a little further). I will remove the declaration of
groupByArray from that extension if we add it here

I do have a medium term plan to move a lot of the helpers from extended report back into core.

Change-Id: I377d33a7417ff93169f12ccbda77f3662da5bac1

Fix issues in order by clauses of reports showing errors in 7.1

Merge pull request #10737 from jitendrapurohit/CRM-20913

CRM-20913 - Add separate option group for pledge status