Merge pull request #10073 from eileenmcnaughton/4.7.18-rc

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

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 #10060 from jitendrapurohit/add-api-action

CRM-20342: add 'view' Api action

CRM-20342: add 'view' Api action

Merge pull request #10044 from eileenmcnaughton/4.7.18rc

Minor test tidy up.

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

Merge pull request #10042 from eileenmcnaughton/4.7.18rc

CRM-20334, some times payment object is not set and is protected vari…

CRM-20334, some times payment object is not set and is protected variable

----------------------------------------
* CRM-20334: Fatal Error on Submit Credit Card Form
  https://issues.civicrm.org/jira/browse/CRM-20334

Merge pull request #10032 from eileenmcnaughton/enotice

Enotice fix

Enotice fix

Merge pull request #9970 from eileenmcnaughton/front_end_recur

CRM-20251 Make it easier to overwrite front end form help text regard…

Merge pull request #10005 from eileenmcnaughton/membership_date

CRM-20294 Update membership dates in profiles to use datepicker

CRM-20251 Make it easier to overwrite front end form help text regarding recurring

Merge pull request #10029 from eileenmcnaughton/test

Fix up test to create line_items & memberships accurately

Fix up test to create line_items & memberships accurately

Merge pull request #10017 from colemanw/CRM-20302

CRM-20302 - Support search by/return tags in case api

Merge pull request #9788 from fliespl/patch-1

CRM-19914 - civicrmHooks.php issues on windows

Merge pull request #10016 from ErichBSchulz/refaction_entity_action_permissions

extract two core API permission functions

CRM-20102 - Fix case role lookup by name instead of label

Api3 - Fix metadata for getsingle

CRM-20302 - Support search by/return tags in case api

Activity API - Remove redundant input checks

Validate integer is performed by the api wrapper.

Merge pull request #10020 from jitendrapurohit/CRM-20216

CRM-20216 - Dedupe rules should be prevented from being saved if Weig…

Merge pull request #10019 from eileenmcnaughton/cacheid

CRM-19385 - remove dependence on ids in cache tables.

CRM-20216 - Dedupe rules should be prevented from being saved if Weight Threshold is null or zero

CRM-19385 - remove dependence on ids in cache tables.

This removes the use of the primary id keys in the cache tables. The upgrade part of this
has gone stale a few times but I feel sure that this part is tested & confirmed
and committing this part will make testing easier

fix comment to comply with style

extract two core API permission functions

This will enable a bit of discoverability for the entity x action
matrix.

This is pretty much a cut and paste job so should be no functional
change.

Merge pull request #8980 from ergonlogic/dev/CRM-19308

Fixes CRM-19308: Disallow false-y values for paths purged during cach…

Merge pull request #10014 from totten/master-email-recip

CRM-20299 - CiviMail UI - Cleanup "Location Type" and "Selection Method"

Merge pull request #9789 from fliespl/patch-2

CRM-19979 - Too big amount saved despite error

Merge pull request #9971 from eileenmcnaughton/delayed_recur

CRM-20252 Improve text when processing possibly-delayed-payments

Merge pull request #10013 from seanmadsen/GenCodeChecksum

New GenCodeChecksum for auto-generated DAO files

Merge pull request #9751 from eileenmcnaughton/report

Minor comment fixes & function extraction.

Fix mis-cased class ref

CRM-20299 - CiviMail - "Location Type" requirement feels wonky

If you select a "Location Type" and then deselect it, the field changes
to "Automatic". This is the default value (which is acceptable), but the UI
flags it as erroneous.

The "Selection Method" also defaults to "Automatic".  These fields are oddly
a bit different (e.g.  "Automatic" translates to `location_type_id==null`
and `email_selection_method=automatic`), so the fields are a little
different, but this revision brings them closer.

CRM-20299 - CiviMail UI - Refresh recipient list after changing advanced options

When composing a new mailing, there are advanced options for targetting
recipients by location type (`location_type_id`, `email_selection_method`).
This ensures that the estimate recipient count updates to match the
selection.

Comment fixes

Extract report functions for readability & overidabiity

New GenCodeChecksum for auto-generated DAO files
@totten recommended that I submit this PR
http://civicrm.stackexchange.com/questions/17722/why-do-i-have-gencodechecksum-uncommitted-changes-after-a-fresh-civibuild-create/17724#17724

CRM-20252 Improve text when processing possibly-delayed-payments

Merge pull request #10011 from eileenmcnaughton/test

Fix test to use name that will not cause intermittant errors

Merge pull request #9999 from JMAConsulting/CRM-20286

CRM-20286, added card type field on contribution search form

Fix test to use name that will not cause intermittant errors

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

Correct description of order api

Correct description

There are no order records as there is no order table.

CRM-20286, added card type form fields

----------------------------------------
* CRM-20286: Add card type field on search form
  https://issues.civicrm.org/jira/browse/CRM-20286

CRM-20286, code cleanup and defined $_wheretables and $_tables correctly for batch and is_payment fields

----------------------------------------
* CRM-20286: Add card type field on search form
  https://issues.civicrm.org/jira/browse/CRM-20286

CRM-20286, added where clause for card type

----------------------------------------
* CRM-20286: Add card type field on search form
  https://issues.civicrm.org/jira/browse/CRM-20286

CRM-20286, added unit test

----------------------------------------
* CRM-20286: Add card type field on search form
  https://issues.civicrm.org/jira/browse/CRM-20286

Use metadata based method to add field to form

CRM-20286 Minor changes

----------------------------------------
* CRM-20286: Add card type field on search form
  https://issues.civicrm.org/jira/browse/CRM-20286

Merge pull request #9995 from JMAConsulting/CRM-20282

CRM-20282 Added credit card type field to bookkeeping report

Merge pull request #10004 from eileenmcnaughton/selector

CRM-20208 rename 'In Selector' to 'Results Column' on Profile admin s…

CRM-20294 Update membership dates in profiles to use datepicker

Merge pull request #10002 from colemanw/News

News dashlet - don't show count when it's zero

CRM-20208 rename 'In Selector' to 'Results Column' on Profile admin screens.

THis is based on a recommendatation in the JIRA suggesting this would be less confusing & inline with the documentation

News dashlet - don't show count when it's zero

Merge pull request #9979 from mattwire/disable_payment_processor2

CRM-19900: Disable/enable both processors directly in BAO create function

Merge pull request #9421 from eileenmcnaughton/recur_icon

CRM-19674 Add ability to see at a glance if a contribution is part of…

Merge pull request #10000 from eileenmcnaughton/agile

CRM-19710 - Preserve is_email_receipt parameter through to email sent

Merge pull request #9986 from JMAConsulting/CRM-20272

CRM-20272 Added credit card type field for contribution summary report

Merge pull request #9980 from mickadoo/CRM-20254-cache-busting-custom-css

CRM-20254: add cache buster string for custom CSS

Merge pull request #9802 from eileenmcnaughton/lower

 CRM-19980, CRM-19811 - Fix slow queries due to LOWER on contact name…

CRM-20282 Added credit card type field to bookkeeping report

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

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

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

----------------------------------------
* CRM-20282: Add credit card type field for Bookkeeping report
  https://issues.civicrm.org/jira/browse/CRM-20282

CRM-20272 Removed line break

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

CRM-20272 Generalised function to add labels

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

CRM-20272 Generalised function to add labels

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

CRM-19674 Add ability to see at a glance if a contribution is part of a recurring series

Note this is a subset of https://github.com/civicrm/civicrm-core/pull/9421/commits/c6ba3ef7253b4b27ac69f70d71b6a78d3475b635

It re-instates the display of the word recurring, but leaves out the extra column
for recurring & sortability based on that. This is mostly because I wanted to get the
PR resolved & merged & hence left out the stuff that did not
seem like I had a consensus on. However, I do think the loss
of sortability on recurring is a shame & hence kept a link to the
commit if anyone wants to revisit

CRM-19710 - Preserve is_email_receipt parameter through to email sent

CRM-20272 Added Credit Card Type field for Contribution Summary report

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

CRM-20272 Added credit card type field for contribution summary report

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

CRM-20272 Fixed function to read from card type rather than credit card type

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

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

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

CRM-20272 Removed entity financial trxn column in favour of hardcoded alias

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

Merge pull request #9602 from eileenmcnaughton/triggers

CRM-19051 improve respect for the offline trigger setting.

Merge pull request #9998 from eileenmcnaughton/part_fee

CRM-19273 Fix in the usage of insertedLines and added test

CRM-19273 Fix in the usage of insertedLines and added test

Merge pull request #9983 from JMAConsulting/CRM-19715-3

CRM-19715 (IIDA-96), removed settings related to close accounting period and mo…

CRM-20254: code style changes

CRM-20254: simplify adding cache code logic. remove useless test case

CRM-19715, removed settings related to close accounting period and moved to an extension

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

Merge pull request #9982 from totten/master-zetamail

CRM-20270 - composer.lock - Fix multibyte warnings by updating ezcMail

CRM-20270 - composer.lock - Fix multibyte warnings by updating ezcMail

On many systems, the test suite outputs a long message which begins with:

```
PHP Warning: declare(encoding=...) ignored because Zend multibyte feature is turned off by settings in /path/to/sites/all/modules/civicrm/vendor/zetacomponents/mail/src/transports/variable/var_set.php on line 2
```

This patch addresses the warning by updating the version of ezcMail (aka `zetacomponents/mail`).  See also:

 * https://github.com/civicrm/zetacomponents-mail/pull/1
 * http://civicrm.stackexchange.com/questions/16314/does-civicrm-have-a-recommended-setting-for-zend-multibyte

Merge pull request #9940 from colemanw/CRM-20102

CRM-20102 - Case Activity api improvements

CRM-20102 - make case_type_id required

Merge pull request #9776 from jitendrapurohit/CRM-19741

CRM-19741: Price set and price set value label fields are inconsistan…

CRM-20254: set cache buster code for tests

CRM-20254: add cache buster string for custom CSS and improve query string generation (duplicated from original PR which was to the wrong branch)

Coding style

Disable/enable both processors directly in BAO create function

Merge pull request #9978 from yashodha/CRM-20255

CRM-20255 - Add developer tabs to more reports

CRM-20255 - Add developer tabs to more reports

Merge pull request #9420 from eileenmcnaughton/settings

Settings

Fix double-negative in function signature

CRM-20102 - Tests

More efficient loading of case type definition

Don't re-fetch info from the database if we already have it.

CRM-20102 - Retrieve more case role info from api

CRM-20102 - Return case_id from activity api

CRM-20102 - Better filters for case contact_id field

Use strict type checking in in_array

This php function has a bug where it incorrectly returns true if the first param is 0.

Merge pull request #9966 from eileenmcnaughton/is_recur_template

CRM-20247 Add test & ensure $is_recur is assigned to the message_template.

Merge pull request #9910 from seamuslee001/CRM-19835

CRM-19835 Use host rather than server and put in port if supplied

CRM-19672 - Adapt TokenCompatSubscriber to new getDomain() signature

A recent commit for CRM-19672 changed the signature for `getDomain()`.  It
now returns a regular object (not a reference), but this means that you
cannot pass the domain directly through to `replaceDomainTokens()`.

This manifested as a test error when running `flexmailer` on my local PHP
5.6/7.0 system.

Merge pull request #9969 from totten/master-extbatch-cond

Civi\API\ExternalBatch - Improve test for variables_order/$_ENV

Merge pull request #9965 from eileenmcnaughton/activity_report

CRM-20242 - Add developer Tab to Activity Detail report

Civi\API\ExternalBatch - Improve test for variables_order/$_ENV

The recent revision #9595 updated `ExternalBatch` to produce a more
informative error message.  However, in the test environment used by
`flexmailer`, there was exactly 1 value in `$_ENV` (even if
`variables_order` was misconfigured).

This should make the test a bit harder to trip-up by affirmatively checking
for the most common environment variable.

Review note: In the known universe, the only direct references to
`ExternalBatch` are in `CiviUnitTestCase`, so this shouldn't impact any
runtime logic.

Merge pull request #9967 from eileenmcnaughton/dummy

CRM-20248 correct hard coded payment processor type id for Dummy proc…

CRM-20248 correct hard coded payment processor type id for Dummy processor

CRM-20247 further removal of ignored is_recur parameter.

This variable was part of a legacy flow via the BaseIPN. Removing the variable is a good tidy up step