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

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

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

Unit test for CRM-20970

CRM-20962 Add deprication notice in regards to settings with no metadata

Merge pull request #10769 from mfb/CRM-20970

CRM-20970 - avoid MySQL error caused by long numeric postal codes

CRM-20962 Fix issue where we were passing through a string when it should be an array causing error in php7.1

Merge pull request #10721 from seamuslee001/CRM-20939

CRM-20939 CRM-19941 Fix issue where $_recent variable is initialised as a strin…

Add option_group unit test

Merge pull request #10778 from civicrm/4.7.23-rc

4.7.23 rc

Merge pull request #10738 from eileenmcnaughton/prim_address

CRM-20759 (follow up fix) Primary address is 'distributed' over sever…

Merge pull request #10777 from eileenmcnaughton/rdate

Code cleanup towards CRM-20610.

Merge pull request #10773 from dsnopek/crm-20973

CRM-20973: Fixes "Call to undefined function Civi\API\civicrm_api3_create_error() …

Merge pull request #10772 from seamuslee001/CRM-20971

CRM-20972 Attempt to handle new type of Exception thrown in php7.1 wh…

Code cleanup towards CRM-20610.

We have code repetition of the receipt_date construct in the tpl. Once for
credit card payments & once not, with no substantive difference.

This commit consolidates them and moves them out of the payment block,
which going forwards will better distinguish between the contribution and
the payments towards it.

Add in code comment about it being a PHP7.1 issue

Merge pull request #10774 from eileenmcnaughton/payment_form

CRM-20610 add a form for editing payment details

Merge pull request #10775 from eileenmcnaughton/bounce

CRM-20976 Improve error handling when someone tries to make a credit …

CRM-20976 Improve error handling when someone tries to make a credit card contribution and no processor is configured

CRM-20610 additional code comments.

I have added comments to some things I'd really like to see tidied up (but could
not reasonably extend the scope here to include)

CRM-20610 add a form for editing payment details

This commit adds a link to the left of the payment listing for editing
payments. Provided the payments do not have an attached
payment processor it is possible to edit the check number,
or card_type_id and pan_truncation, along with the
trxn_date

CRM-20973: Fixes "Call to undefined function Civi\API\civicrm_api3_create_error() in Civi/API/Kernel.php on line 413"

Merge pull request #10766 from seamuslee001/contribute_perm_fix

(NFC) Fix permissions on CRM/Contribute/BAO/Contribution.php