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

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

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

Merge pull request #10699 from JMAConsulting/CRM-20321-1

[re-submitted and unchanged] CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity

Merge pull request #10383 from MegaphoneJon/unsupervised-reserved-threshold

CRM-20607 - Dedupe - Allow saving reserved groups

generated data

Merge pull request #10782 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function on batch form

Merge pull request #10784 from totten/master-prtpl

CRM-20965 - PULL_REQUEST_TEMPLATE - Change header notation

CRM-20965 - PULL_REQUEST_TEMPLATE - Change header notation

If you use the `#` notation for headers, the Github CLI (`hub pull-request`)
would filter out those lines as comments.

Using the `---` notation should provide that.

Merge pull request #10741 from JMAConsulting/CRM-20952

[ready for review] CRM-20952 Refresh membership tab if contribution is membership payment

Merge pull request #10698 from jitendrapurohit/CRM-20910

CRM-20910: Check permission param while retrieving participants from api

Merge pull request #10348 from eileenmcnaughton/token

CRM-20577 - When creating an activity per-contact when sending letters, store the version with rendered tokens

CRM-20947, fixed notice error for deprecated function on batch form

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

CRM-20577 Fix activities to record actual html if activity per contact

(At least, this covers contributions at this stage).

I have also tried to move towards clearer inputs & outpus on createActivities, with the
weirdness around contactIds being moved up to the forms themselves and
not being passed in. Would prefer not to see form passed in either but, next time.

Unit tests cover emails & activity create (for the first time)

Towards CRM-20577 preliminary form tidy up

[nfc] Add Unit test for group by in pdf letters.

In addition to the unit test there is a minor change to not do the early return before assigning variables
if smarty is not set. This doesn't really have any performance advantage, as they are already calculated,
blocks the unit test and also blocks using these variables from outside extensions

Merge pull request #10758 from mickadoo/CRM-20965_add-pull-request-template

CRM-20965: Add PR template

Merge pull request #10753 from seamuslee001/CRM-20962

CRM-20962 Fix issue where we were passing through a string when it sh…

Merge pull request #10701 from JMAConsulting/CRM-20915

CRM-20915: Creating credit card registration for event stores payment method as check