CRM-20368 fix code style issues

CRM-20368 case detail report - add sort option

CRM-20368 case detail report

CRM-20368 mailing summary report

CRM-20368 contact relationship report

CRM-20368 time spent report

CRM-20368 case summary report

CRM-20368 case demographic report

Merge pull request #10058 from colemanw/CRM-20345

CRM-20345 - Case API - Sort by client & addtimeline action

CRM-20345 - CRM_Utils_SQL_Select::orderBy() (#4)

* CRM-20345 - CRM_Utils_Array::crmArraySortByField - Add test. Allow multiple fields.

* CRM-20345 - CRM_Utils_SQL_Select::orderBy - Use more deterministic ordering

The technique of computing default `$weight = count($this->orderBys)`
addresses a valid point: we need to preserve ordering for existing callers
who don't specify weights -- while also allowing weights.

However, it feels weird in my gut. Not sure why -- maybe it's something like this:

```php
// A1: Non-deterministic ordering
$select->orderBy('alpha', 1);
$select->orderBy('beta');
$select->orderBy('delta', 2);
$select->orderBy('gamma', 3);

// A2: Deterministic ordering
$select->orderBy('alpha', 10);
$select->orderBy('beta');
$select->orderBy('delta', 20);
$select->orderBy('gamma', 30);

// B1: Deterministic ordering
$select->orderBy('alpha');
$select->orderBy('beta');
$select->orderBy('delta');
$select->orderBy('gamma');

// B2: Non-deterministic ordering
$select->orderBy('alpha', 1);
$select->orderBy('beta', 1);
$select->orderBy('delta', 1);
$select->orderBy('gamma', 1);
```

As a reader, I would expect A1/A2 to be the same, and I would expect B1/B2
to be the same.  But they're not.  If there's a collision in the `weight`s,
the ordering becomes non-deterministic (depending on obscure details or
happenstance of the PHP runtime).

Of course, there's no right answer: in A1/A2, you can plausibly put `beta`
before `alpha` or after `alpha` or after `gamma`.  But it should be
determinstic so that it always winds up in the same place.

Merge pull request #10086 from JMAConsulting/CRM-20367

CRM-20367: Add wrapper function for adding and dropping Index in Upgrder

CRM-20367: Add wrapper function for adding and dropping Index in Upgrader

Merge pull request #10083 from JMAConsulting/CRM-19715-2

CRM-19715, added upgrade script to delete current_period_opening_bala…

CRM-19715, added upgrade script to delete current_period_opening_balance and opening_balance if they are <> 0

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

Merge pull request #10006 from JMAConsulting/CRM-20022

CRM-20022 Extended function to support multiple batch labels

Merge pull request #10055 from jitendrapurohit/CRM-20344

CRM-20344: Make 'Member Since' displayed on membership tabs for Inac…

CRM-20345 - Don't be tripped up by deleted contacts or inactive relationships

Merge pull request #10041 from mlutfy/master-crm20336

CRM-20336: Failed contributions should be set as failed, not left as pending

Merge pull request #10065 from colemanw/CRM-19778

CRM-19778 - Improve layout of caseType config UI

Merge pull request #9974 from JMAConsulting/CRM-19715-2

CRM-19715 (IIDA-96), removed opening_balance and current_period_openi…

Merge pull request #10071 from eileenmcnaughton/dedup_4

CRM-20357 remove unused Finder::dupesOfContact function.

CRM-20336: Failed contributions should be set as failed, not left as pending (cleanup)

CRM-19715 (IIDA-96), removed opening_balance and current_period_opening_balance field from civicrm_financial_account table and code

----------------------------------------
* CRM-19715:
  https://issues.civicrm.org/jira/browse/CRM-19715

CRM-19715, added DAO changes

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

CRM-19715, removed upgrade code to drop fields from civicrm_financial_account table

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

Merge pull request #9985 from JMAConsulting/CRM-19715-4

CRM-19715 (IIDA-96), Removed Close Accounting Period Activity Type

CRM-20336: Failed contributions should be set as failed, not left as pending (more test fixes).

Merge pull request #10077 from JMAConsulting/CRM-19715-5

CRM-19715, fixed api test so that its not affected by any deletion of…

Merge pull request #10078 from colemanw/Icons

Icons for attachments

Merge pull request #10069 from eileenmcnaughton/dedup_2

Towards CRM-20328 remove another call to the duplicate code

Merge pull request #10070 from eileenmcnaughton/dedup_3

CRM-20328 code rationalisation on calling dedupe code

Merge pull request #10046 from JMAConsulting/CRM-20332

CRM-20332 Added support for third column of actions list to be modifi…

Merge pull request #8525 from twomice/CRM-18251b

CRM-18251 - Domain stats and VersionCheck

CRM-20336: Failed contributions should be set as failed, not left as pending (fix the test).

Icons for attachments

Merge pull request #9729 from eileenmcnaughton/cust_date

CRM-20012, CRM-19490 (now merged & closed), CRM-18387(merged& closed), CRM-20011 (in separate pr), CRM-15948(resolved by arlready-merged-part), CRM-19911 profile date fixes & code improvement

CRM-19715, Removed Close Accounting Period Activity Type

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

CRM-19715, updated sql to delete activity type if not present

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

CRM-19715, removed cruft

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

CRM-19715, fixed api test so that its not affected by any deletion of activity types from core

----------------------------------------
* CRM-19715: Remove Close Accounting Period code
  https://issues.civicrm.org/jira/browse/CRM-19715

Merge pull request #10064 from eileenmcnaughton/dedup_tidy

Towards CRM-20328 remove duplicate code, towards taking extension fro…

CRM-20328 further fix on deprected duplicate code.

Remove parameters only passed in in unit tests. This deprecated function is called from
only one place, so reduce parameters & testing to reflect. Remove test
that duplicates other test aside from deprecated paramter

Merge pull request #10076 from eileenmcnaughton/master

Merge to master from 4.7.18rc

Merge branch '4.7.18-rc' of https://github.com/civicrm/civicrm-core

Empty merge commit to mark 4.7.18-rc merged.

Somehow we lost the merged marker with a cherry pick.

Have used merge -s ours d46d3ea to grab these as empty-but-merged

Merge pull request #10061 from eileenmcnaughton/fatal

CRM-20347 Fatal error on delete participant record with related contr…

Merge from 4.7.18rc to master (#10075)

* Minor test tidy up.

(this is actually because we need a commit to do a merge to master after the same commit was merged to both).

* CRM-20342: add 'view' Api action

* CRM-19385 revert changes to order by id on prevnext_cache.

After PR #10019 was merged @bhahumanists reported on #9801 that smart group ordering was broken. The
report is lacking detail and I could not replicate it, but am reverting the search changes for safety.

After this revert sites can still experiment with dropping/altering the primary key on
-civicrm_cache
-civicrm_group_contact_cache
-civicrm_acl_cache

But not civicrm_prevnext_cache.

Although I could not replicate the search issue it seems to me that our Ubuntu test suite
is failing where the other one is not on search-related items, suggesting that
for some reason Ubuntu does not default to the natural search order, due to
some config or other difference

Merge pull request #10073 from eileenmcnaughton/4.7.18-rc

CRM-19385 revert changes to order by id on prevnext_cache.

CRM-20347 fix for fatal on view records from participant

Merge pull request #10074 from eileenmcnaughton/master

Merge from 4.7.18rc

Fix merge head on master.

Master is treating some 4.7.18-rc commits as unmerged. This just marks as merged : Merge commit '58025217985f3c8cf'

CRM-19385 revert changes to order by id on prevnext_cache.

After PR #10019 was merged @bhahumanists reported on #9801 that smart group ordering was broken. The
report is lacking detail and I could not replicate it, but am reverting the search changes for safety.

After this revert sites can still experiment with dropping/altering the primary key on
-civicrm_cache
-civicrm_group_contact_cache
-civicrm_acl_cache

But not civicrm_prevnext_cache.

Although I could not replicate the search issue it seems to me that our Ubuntu test suite
is failing where the other one is not on search-related items, suggesting that
for some reason Ubuntu does not default to the natural search order, due to
some config or other difference

CRM-20345 - Allow weight to be specified for an orderBy

Functions wishing to extend & override an api sort properties can use the placeholder (1)
to maintain sort param order.

CRM-20345 - Fix retrieval of primary client

CRM-20345 - Remove undefined variable

CRM-20355 - Add Case.addtimeline action

CRM-20345 - Add integration point for extending api.Case.get

This could be a good addition every api.get function.

CRM-20345 - Case API - Sort by client

CRM-20357 remove unused Finder::dupesOfContact function.

I can find no evidence this function is used anywhere

Towards CRM-20328 remove another call to the duplicate code

CRM-20328 code rationalisation on calling dedupe code

Towards CRM-20328 remove over-loading from checkRegistration function

Towards CRM-20328 extract dedupe code to function

Towards CRM-20328 remove duplicate code, towards taking extension from core

Merge pull request #10067 from twomice/CRM-20350_dont_escape_quotes

CRM-20350: Don't escape double-quotes in icalendar.

Merge pull request #9987 from JMAConsulting/CRM-20273

CRM-20273 Added credit card type field for contribution details report

CRM-20350: Don't escape double-quotes in icalendar.

CRM-19778 - Improve layout of caseType config UI

CRM-20332 Modified function to prevent tampering with original menu

----------------------------------------
* CRM-20332: Refactor summaryActions hook to allow user-defined actions in third column
  https://issues.civicrm.org/jira/browse/CRM-20332

CRM-20332 Added support for third column of actions list to be modified by hook summaryActions

----------------------------------------
* CRM-20332: Refactor summaryActions hook to allow user-defined actions in third column
  https://issues.civicrm.org/jira/browse/CRM-20332

CRM-20273 Fixed report to read from card type rather than credit card type

CRM-20273 Added credit card type field for contribution details report

----------------------------------------
* CRM-20273: Add credit card type field for Contribution Details Report
  https://issues.civicrm.org/jira/browse/CRM-20273

CRM-19997 Added Credit Card Type field for Contribution Details report

----------------------------------------
* CRM-19997: Add Credit card type fields in contribution related reports
  https://issues.civicrm.org/jira/browse/CRM-19997

Merge pull request #10063 from eileenmcnaughton/master

Merge to Master from 4.7.18-rc

CRM-20342: add 'view' Api action

CRM-20012 alternate fix to setting null dates

This is part of the work started in CRM-19490 to switch profile dates to use the date picker,
as a solution to multiple problems

Merge pull request #10060 from jitendrapurohit/add-api-action

CRM-20342: add 'view' Api action

CRM-20342: add 'view' Api action

Merge pull request #10059 from fuzionnz/CRM-20346-remove_encryptdb

CRM-20346. Remove encryptDB.php

CRM-20346. Remove encryptDB.php

Merge pull request #10053 from eileenmcnaughton/dedup_tidy

Towards CRM-20155 remove duplicate code, towards taking extension fro…

Merge pull request #10054 from jitendrapurohit/CRM-20311-unit-test

Add unit test to check CRM-20311

Merge pull request #9694 from systopia/CRM-19892

Proposed fix for CRM-19892

Add unit test to check CRM-20311

CRM-20344 - Make 'Member Since' displayed on membership tabs for Inactive Memberships.

CRM-20022 Added batch title as pseudoconstant, refactored label for batch id

----------------------------------------
* CRM-20022: Add Batch Title to Columns, Grouping tabs on Contribution Summary report
  https://issues.civicrm.org/jira/browse/CRM-20022

Merge pull request #10024 from rnao/fix-mem-cancel-activity

CRM-20311 - Membership cancelation via contribution doesn't create me…

CRM-20022 Extended function to support multiple batch labels

----------------------------------------
* CRM-20022: Add Batch Title to Columns, Grouping tabs on Contribution Summary report
  https://issues.civicrm.org/jira/browse/CRM-20022

Merge pull request #9991 from JMAConsulting/CRM-20278

CRM-20278 Added credit card type field for top donors report

Towards CRM-20155 remove duplicate code, towards taking extension from core

Merge pull request #9989 from JMAConsulting/CRM-20275

CRM-20275 Added credit card type field for contributions by organization report

Merge pull request #9990 from JMAConsulting/CRM-20277

CRM-20277 Added credit card type field for contributions by household report

Merge pull request #9992 from JMAConsulting/CRM-20279

CRM-20279 Added credit card type field for SYBUNT report

Merge pull request #9993 from JMAConsulting/CRM-20280

CRM-20280 Added credit card type field for LYBUNT report

Merge pull request #9994 from JMAConsulting/CRM-20281

CRM-20281 Added credit card type field for Soft Credit report

Merge pull request #9997 from JMAConsulting/CRM-20284

CRM-20284 Added credit card type field for Contribution Aggregate by Relationship Report

Merge pull request #10048 from totten/master-gcs

(NFC) Update stale GenCode checksums

Merge pull request #10047 from yashodha/CRM-20340

CRM-20340 - translate string

Merge pull request #10001 from JMAConsulting/CRM-20285

CRM-20285 Added credit card type field to recurring contributions report

Merge pull request #10003 from eileenmcnaughton/cust_date2

CRM-20011 fix loss of data for yyyy-mm custom field

(NFC) Update stale GenCode checksums

The checksums in `CRM/Activity/DAO/Activity.php` and `CRM/Core/DAO/AllCoreTables.data.php` are
stale (although the logic is current).

Probable explanation: there was a recent whitespace cleanup to
`xml/schema/Activity/Activity.xml` (165a7fc03b86).  This would affect the
checksum of anything derived from there -- but it wouldn't cause a
substantive change.

CRM-20340 - translate string

Merge pull request #10025 from civicrm/JoeMurray-patch-1

CRM-20313 whitespace cleanup

CRM-20275 Added credit card type field for contributions by organization report