First WIP for AJAX cleanup.

Merge pull request #8254 from eileenmcnaughton/4.7.7-rc

CRM-17984 getTree regression

CRM-17984 - CustomGroupTest - Fix style errors

There were problems in this line:
 * The missing comma (required by Drupal style)
 * The use of literal CtrlA causes phpcs to produce invalid output

Further custom tree regression fix, where the label does not match the name the name is the key to use

CRM-18469, CRM-17984 - getTree regression on multiple integers separated by the cnrtl char

Merge pull request #8245 from colemanw/CRM-18006-b

Last-minute upgrade fixes

Merge pull request #8243 from totten/477rc-gettree-a

CRM-18469, CRM-17984 - getTree - No more pass by reference

CRM-16846 - Fix missing upgrade steps

Merge pull request #8232 from eileenmcnaughton/4.7.7-rc

CRM-18193 logging schema: fix mismatch between comment & code convention

CRM-18006 - civicrm_install_canary - Ensure ENGINE is InnoDB

This should resolve warnings on systems where MyISAM was default engine.

CRM-18006 - Specify db engine for civicrm_install_canary table

CRM-18469, CRM-17984 - getTree - No more pass by reference

CRM-18193 logging schema: fix mismatch between comment & code convention

Comment suggests 'c_' to identify a non-CRM connection + timestamp to the hour + connection_id but code used 'con_'

I think the shorter version in comment is better as there is more real information space

Merge pull request #8049 from eileenmcnaughton/revert-api

CRM-18332 - Logging.revert API

CRM-18193 add default log_conn values for when interacting with mysql outside of CiviCRM.

In this instand @uniqueID will not be set. We had a go at setting it within the trigger but that was unreliable. Eventual decision was a string 'con_' to identify a system connection + a timestamp to hourly granularity + the connection id. Proposed further work is to specifically block the revert from the UI for these and add a higher permission or a 'yes I really really mean it' param to the revert api for these

Merge pull request #8216 from totten/xurizaemon-CRM-17983

 CRM-17983 - Improve params for getContactRelationships

CRM-17983, CRM-18401 - Swap loose/strict escaping

The old names suggested that loose checking (`MysqlColumnName`) should be
the norm, and strict checking (`MysqlColumnNameStrict`) the exception.
Rather, strict should be the norm.

This also has a functional affect switching the new
`defaultSortAndPagerParams()` to use strict behavior.

CRM-18193 - CRM_Logging_Differ - Improve interval checking

a.  This validates the interval in the constructor.  Previously, there were
two different functions which consumed the interval, but only one of them
validated.

b.  My brain does not understand what it means to addslashes for an interval
expression. This validation is based on the docs from mysql.com.

Sanitise ->interval

Spec function for revert

Logging schema update, fix connection_id

Test fix - test was relying on running slowly enough for the second to increment. Use longer gap

fixes for tests when they run in suite, cache clearing etc

CRM-18404 Rebuild triggers not including all tables

This is an alternate fix to the one proposed here https://github.com/civicrm/civicrm-core/pull/8017 - the test part of itis leveraging tests in the revert-api PR so I have added it into the PR

CRM-18214 Add link to detail report for contact merge activities, with oid if applicable

After merging all my other changes the only thing needed for the dedupe merge report was it to be linked from the summary report and
to redirect after merge to the comparison of the 2 contacts, covered in this commit

CRM-18215 Fix grouping so that Merge activities are not sometimes lost

Change-Id: I729703c51768955b02e8d85c1297fdf10c90cbe1

CRM-18193 do not pass log_date_id to detail report if the log_conn_id is sufficient

Prior to CRM-18193 the connection was estimated using the a log_date_range + log_conn_id. After CRM-18193 the connection_id is unique
and the log_date is not required (and could make the result less accurate

Change-Id: I72c8cf254720654003154702b45e7f0a0b8aabe5

CRM-18193 do not require log_date

Log_date should be an optional report parameter reflecting the changes in CRM-18193 to make the connection_id unique

CRM-18366 Fix contact logging detail to show all related changes & revert all related changes

This report switches from a hard-coded list of contact related tables to considering all tables with an FK to civicrm_contact, including entity_tag
group_subscription, batch, financial_item etc. The revert makes the same switch.

Note that with this change a full subset may still not be retrieved - in order to get that you need to enable the changes from CRM-18193 & apply all patches related to that issue (one will build on this)

CRM-18193 remove arbitrary setting of log_date when not provided.

In order to be able to ignore the log_conn_date where it is not relevant the logging detail reports need to stop making up a log_conn_date where one is not provided. The code that does that was part of the inital report commit and on removing it I could not replicate any issues - ie. accessing the detail reports without criteria worked fine

CRM-18193 Make log_date optional for revert / retrieving changes where uniqueID is in play

CRM-18360 Expose logging data through GET api

CRM-18332 add first cut of api for revert actions

Later I expect to add more parameters & deal with situation when log_date is not passed

Update api call, group param is deprecated

CRM-18193 add routine for converting log tables to support new log_conn_id format & hook spec

i

CRM-18193 create all new log tables with log_conn_id = varchar(17)

This provides support for the unique id rather than the non-unique connection id

Change-Id: Ia2ce2f1854d1f6f90095585dbb3924f704be38b5

CRM-18193 adapt reports to expect a 17 char string rather than 24 char varchar for log_conn_id

This is in anticipation of us converting to unique IDs for these fields

Merge pull request #8215 from colemanw/CRM-17985

CRM-17985 - Fix javascript error

CRM-17985 - Fix javascript error

Merge pull request #8205 from eileenmcnaughton/break_fix

(Followup) CRM-17984 - Regression in getTree()

CRM-17984 - CRM_Core_BAO_CustomGroup - Extract whereListHas()

This seems to have been a rather fiddly bit of SQL that was duplicated in the past.

Merge pull request #8210 from JKingsnorth/patch-6

(NFC) ActivityTest minor tidy

Merge pull request #8209 from jitendrapurohit/webtest

CRM-18454: webtest fixes

Merge pull request #8208 from rohankatkar/WebtestFixes-master

Webtests Fixes: CRM-18454

Merge pull request #8207 from rohankatkar/CRM-18050-webtest

CRM-18050: Webtest

ActivityTest minor tidy (NFC)

Webtests Fixes

CRM-18454: webtest fixes

CRM-18050: Webtest

Fix Strict warning discovered in testing

CRM-5507 CRM-17984 fix regression when subType var is not leaking

Fix the RC regrsision

CRM-17984 test to demonstrate breakage

CRM-17983, CRM-18401. Input validation.

Merge pull request #8187 from kenwest/CRM-18275

CRM-18275 sort contact's Activity list by descending date order

Minimal check to validate relationship params.

Merge pull request #8197 from jitendrapurohit/webtest

CRM-18454: webtest fixes

Merge pull request #8034 from eileenmcnaughton/CRM-18310

CRM-18310 add tests for debug logging and fix test-identified error

Merge pull request #7988 from eileenmcnaughton/CRM-18193

CRM-18193, CRM-18104, CRM-18180 logging definition enhancements

CRM-18275 make consistent with CRM-18173

Add tests for logging functionality to ensure the tables are still creating well

Change-Id: I6ef4d48d16e25d232b0f46e7ce3cf5d569e7053c

CRM-18180 allow non-standard table names in logging reports

CRM-18104 Hook for hook for defining log tables.

Allows defining ENGINE and any indexes

Merge pull request #8199 from colemanw/CRM-18456

CRM-18456 - Catch api case permission exceptions

Merge pull request #8048 from eileenmcnaughton/revert

CRM-18332 - Logging report - Extract methods storeDB() & revert()

CRM-18456 - Catch api case permission exceptions

CRM-18454: webtest fixes

Merge pull request #8185 from yashodha/CRM-18438

CRM-18438: fix currency for payment info

Merge pull request #8184 from rohankatkar/Webtest-CRM-18438

Webtest for CRM-18438

Merge pull request #8194 from rohankatkar/CRM-18421-QA-Fix

CRM-18421: QA warning fix

CRM-18421: QA warning fix

Merge pull request #8149 from JKingsnorth/CRM-18421

CRM-18421: Fix getContactMailings() notice

Merge pull request #8182 from jitendrapurohit/CRM-18434

CRM-18434 - First and last names for billing not displayed in confirm…

CRM-18310 add tests for debug logging, fix caching error

CRM-18332 - Logging report - Extract methods storeDB() & revert()

Merge pull request #8193 from colemanw/scardinius-issue-CRM-16911

CiviMail: distinct events in API Mailing stats

CRM-16911 - Add title to api field

----------------------------------------
* CRM-16911: AB reporting takes the total number of open instead of the unique
  https://issues.civicrm.org/jira/browse/CRM-16911

Merge pull request #8191 from jitendrapurohit/webtest

webtest fixes

Merge pull request #8127 from lcdservices/CRM-18402

CRM-18402 ensure admin statuses are not used

Merge pull request #8186 from kenwest/CRM-18445

CRM-18445 don't 'Assign Case Role' to client

Merge pull request #8188 from kenwest/CRM-18447

CRM-18447 Can't create Survey if Events have 'is required' custom data

webtest fixes

CRM-18438: Webtest

Minor changes

Warning fix

CRM-18447 Cannot create Survey when Events have 'is required' custom data

CRM-18275 sort contact's Activity list by descending date order

extend webtest to check CRM-18434

CRM-18445 don't 'Assign Case Role' to client

CRM-18438: fix currency for payment info

----------------------------------------
* CRM-18438: Default currency shown in contribution selector
  https://issues.civicrm.org/jira/browse/CRM-18438

Merge pull request #8175 from yashodha/CRM-18438

CRM-18438: Default currency shown in contribution selector

CRM-18434 - First and last names for billing not displayed in confirmation screen

Merge pull request #8174 from jitendrapurohit/CRM-18420

CRM-18420 : Details field of a scheduled reminder contains raw HTML

Merge pull request #8171 from totten/master-trigger-log2

CRM-18212 - Allow triggers to be managed offline

Merge pull request #8178 from monishdeb/CRM-18436

CRM-18436: Non-mandatory custom rich text fields are mandatory in advanced search

CRM-18436: Non-mandatory custom rich text fields are mandatory in advanced search

CRM-18438: Default currency shown in contribution selector

----------------------------------------
* CRM-18438: Default currency shown in contribution selector
  https://issues.civicrm.org/jira/browse/CRM-18438

CRM-18420 : Details field of a scheduled reminder contains raw HTML

Ornery test fix - more flushing of statics (#8172)

Merge pull request #8161 from eileenmcnaughton/CRM-18303-follow-up

CRM-18303 follow up fix

Merge pull request #8164 from jitendrapurohit/CRM-18411

CRM-18411 fix for missing argument

CRM-18411 fix for missing argument