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-20987, added batch transaction column

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

Merge pull request #10726 from JMAConsulting/CRM-20946

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

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

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

CRM-20972 Attempt to handle new type of Execption thrown in php7.1 when function call has too few variables

Merge pull request #10771 from civicrm/4.7.23-rc

4.7.23 rc

Merge pull request #10770 from jitendrapurohit/CRM-18177-rc

CRM-18177 RC Fix for param updated in previous PR #10173

Merge pull request #10768 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function

CRM-20952 Refreshed event tab on payment made to contribution

----------------------------------------
* CRM-20952: Refresh membership tab on edits to membership payments
  https://issues.civicrm.org/jira/browse/CRM-20952

Upgrade changes, weight and qill fix

CRM-20952 Bug fixes

----------------------------------------
* CRM-20952: Refresh membership tab on edits to membership payments
  https://issues.civicrm.org/jira/browse/CRM-20952

Fix updatePledgeStatus execution to use pledge_status og values

Add separate option group for pledge status

CRM-18177 RC Fix for param updated in previous PR #10173

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

Fix permissions on CRM/Contribute/BAO/Contribution.php

CRM-20947, fixed notice error for deprecated function

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

CRM-20947, fixed notice error for deprecated function

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

Merge pull request #10764 from civicrm/4.7.23-rc

4.7.23 rc

Merge pull request #10750 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function on new Bulk ent…

Merge pull request #10762 from seanmadsen/premium-images

Fix placement of call to simplifyURL() function

Merge pull request #10742 from eileenmcnaughton/dead

CRM-20953 Importing contacts with deceased_date not setting is_deceased

Merge pull request #10752 from seamuslee001/CRM-20961

CRM-20961 Fix issue where test on php7.1 was reporting that a non num…

Merge pull request #10761 from monishdeb/CRM-20821_CRM-20938

CRM-20821 & CRM-20938 : minor fix

Fix placement of call to simplifyURL() function

Before this commit, the URL simplification was performed within the
$params array used as the *default* values, so these values got
over-written by the submitted values with array_merge().

This is a follow-up to changes made in #10720 bb80d2f

CRM-20821 & CRM-20938 : minor fix

CRM-20965: Add PR template

Merge pull request #10751 from seamuslee001/CRM-20960

CRM-20960 Upgrade dompdf to 0.8 for php7.1 compatability

Merge pull request #10747 from samuelsov/CRM-20954

CRM-20954 Fixing localized financial status problems

CRM-20961 Fix issue where test on php7.1 was reporting that a non numeric was being passed

CRM-20960 Upgrade dompdf to 0.8 for php7.1 compatability

CRM-20947, fixed notice error for deprecated function on new Bulk entry Batch

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

Merge pull request #10720 from seanmadsen/CRM-20821_CRM-20938

CRM-20821 & CRM-20938 - fix bugs in managing premium product images

Merge pull request #10723 from seamuslee001/CRM-20945

CRM-20945 Remove underline on active pager step

Merge pull request #10743 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function on Bulk entry B…

Make the button a bit more prominiant

Merge pull request #10748 from seamuslee001/CRM-20943-2

CRM-20943 Fix subsequent issue found by Monish and Steven

CRM-20954 Initiate paid status id only once

CRM-20943 Fix subsequent issue found by Monish and Steven

Merge pull request #10686 from totten/master-sql

CRM_Utils_SQL_Select - Allow fluent query execution

CRM-20954 Fixing localized finantial status problems

Merge pull request #10653 from JKingsnorth/CRM-20863

CRM-20863 Improve report list ordering

CRM-20947, fixed notice error for deprecated function on Bulk entry Batch listing page

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

CRM-20953 Importing contacts with deceased_date not setting is_deceased

CRM-20952 Refresh membership tab if contribution is membership payment

----------------------------------------
* CRM-20952: Refresh membership tab on edits to membership payments
  https://issues.civicrm.org/jira/browse/CRM-20952

additional fix

Merge pull request #10732 from JMAConsulting/CRM-20947

CRM-20947, fixed warning for deprecated function while generating inv…

Merge pull request #10739 from civicrm/4.7.23-rc

4.7.23 rc merge to master

CRM-20759 (follow up fix) Primary address is 'distributed' over several ID

This wasn't picked up originally in the test as it was only using one field of the primary type.

The old code was indexing by 'addressCnt' (checking that the location type id was different each time). I
have switched to index by location_type_id - we should only have one address per location type per contact.

I also removed the code that sets 'is_primary' if we are importing the first address for a contact.
That has been handled by the BAO for some time

Don't use array dereferencing (for PHP 5.3 compat)

CRM-20938 - Maintain aspect ratio in resizeImage()

Make CRM_Utils_File::resizeImage() preserve image aspect ratio by
default.

CRM-20821 - Use smarter logic to adjust img URL

This change is the crux of CRM-20821.

Before this change, editing a premium product would automatically
break the image URL. After this change, premium products can be edited
while retaining the image URL. Functionality is preserved which changes
the URL to a local URL when possible.

This commit adds two new Utils functions (along with tests) to handle
the logic of changing a supplied URL to a local URL.

CRM-20821 - Improve ManagePremiums formRule()

- Improve CRM_Contribute_Form_ManagePremiums::formRule()
- Require an image file, if "upload" is chosen
- Remove the commented-out code for a rule which required images to be
  within a certain size
- Minor refactoring to improve code clarity

CRM-20821 - Refactor ManagePremiums postProcess()

- Refactor CRM_Contribute_Form_ManagePremiums::postProcess()
- No major changes to functionality
- Eliminate deeply nested code (addressing "FIXME" comment)
- Split code into multiple functions
- Add more comments

CRM-20821 - Refactor PremiumProduct BAO add()

- Refactor CRM_Contribute_BAO_ManagePremiums::add
- Don't change any functionality
- Improve docblock
- Use a $defaults array

CRM-20821 - Refactor resizeImage()

- Clean up CRM_Utils_File::resizeImage()
- No major functionality changes
- Better docblock
- Better error handling
- Better variable names

CRM-20821 - Move resizeImage() to Utils class

- Move resizeImage() from CRM_Contribute_Form_ManagePremiums to
  CRM_Utils_File
- Make it public static
- Don't change anything about how it works

Merge pull request #10736 from eileenmcnaughton/reg

CRM-20950 Prt 2 Contact import mapping to wrong location type