moving around issues between bugs and features

4.7.24 release notes: boilerplate added

4.7.24 release notes: raw from script

Merge pull request #10948 from colemanw/CRM-20941

CRM-20941 - Display correct version number in upgrade message

CRM-20941 - Add punctuation to the end of sentences.

CRM-20941 - Display correct version number in upgrade message

Merge pull request #10932 from jitendrapurohit/CRM-21135-rcfix

CRM-21135 - Activity filter preference are not remembered when enable…

CRM-21135 - Activity filter preference are not remembered when enabled in display settings

Merge pull request #10895 from seamuslee001/CRM-21100-rc

CRM-21100 Hotfix to ensure that testGroup list is still the same as p…

CRM-21100 Hotfix to ensure that testGroup list is still the same as previous, need to work on converting to AJAX list

Merge pull request #10880 from seamuslee001/CRM-21070

CRM-21070 Remove reference to outdated template url file

CRM-21070 Remove reference to outdated template url file

Merge pull request #10869 from MegaphoneJon/4.7.24-rc

CRM-21076 - ACL Role Assignments aren't browseable

Merge pull request #10875 from civicrm/4.7.23-rc

Bring 4.7.23 Release notes into 4.7.24

Merge pull request #10872 from seamuslee001/CRM-20941-rc

CRM-20941 Port changes to push minimum version of PHP to 5.6 with new message for php 5.3 and php5.4

CRM-20941 Add in Link to blog post on bumping PHP Version

CRM-20941 Bump Minimum PHP Version to 5.6, Add in pre-upgrade messages specific to 5.4 and 5.3 and alter Status Check message accordingly

CRM-21076 - ACL Role Assignments aren't browseable

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 #10818 from eileenmcnaughton/recur_date

CRM-21027 Fix next recurring payment not accurately calculated

test build failure fix

Merge pull request #10820 from JMAConsulting/CRM-21029

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

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

concatenate check_number and trxn_id in contribution

additional fixes

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

added unit test and other changes

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

Merge pull request #10810 from totten/4.7.23-rc-notes

(NFC) Update 4.7.23 release notes

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

Update 4.7.23.md

Update 4.7.23.md

Update 4.7.23.md

Update 4.7.23.md - Capitalization

Merge pull request #10703 from agh1/4.7.23-release-notes

4.7.23 release notes

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

4.7.23 release notes: contributors

4.7.23-release-notes: issues done, on to the contributors

Add link to release notes

4.7.23 release notes: more bugs, plus added version summary grid

add metadata to trxn_id

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…

4.7.23 release notes: features

4.7.22 release notes: added credits for 4.7.22

4.7.23 release notes: add item to main list

4.7.23 release notes: new script run

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'

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)