Merge pull request #10973 from civicrm/4.7.25-rc

4.7.25 rc

Merge pull request #10963 from eileenmcnaughton/cleanup

[NFC] comments clean up in test classes.

Merge pull request #10972 from totten/4.7.25-rc-tpl

CRM-18231 - Cleanup `environment` example in `civicrm.settings.php.template`

CRM-18231 - civicrm.settings.php.template - Remove extra whitespace

CRM-18231 - civicrm.settings.php.template - Move `global` declaration

When using the example snippets in the settings file, it's easy to
accidentally leave `global $civicrm_setting` commented out -- which is
fairly confusing/mysterious way to have your settings ignored.

There's no cost to declaring `global $civicrm_setting` by default in new
config files.

CRM-18231 - civicrm.settings.php.template - Fix broken `environment` example

The expression `CRM_Core_BAO_Setting::DEVELOPER_PREFERENCES_NAME` does not
work within `civicrm.settings.php` because the classloader has not been
initialized yet.  This file is just supposed to store the data needed for
bootstrap.

Merge pull request #10922 from eileenmcnaughton/tpl

CRM-21125 allow class assignment on report row links

Merge pull request #10969 from yashodha/CRM-21172

CRM-21172 - Fix 'Edit Contact Information' link on contact dashboard

CRM-21172 - Fix 'Edit Contact Information' link on contact dashboard

Merge pull request #10968 from civicrm/4.7.25-rc

4.7.25 rc

Merge pull request #10936 from seamuslee001/CRM-18231

CRM-18231 Remove need for define statement and replace with new setti…

CRM-18231 Remove need for define statement and replace with new setting not settable in UI and always use settings to get information

Switch to getMandatory function as per Tim's comment

Merge pull request #10953 from seamuslee001/CRM-20892

CRM-20892 Add in modified_date column to civicrm_mailing to assist in…

CRM-20892 Fix issue where saving to the database wasn't creating a new modified_date

CRM-20892 Convert Created_date to default NULL and add in trigger to set it

Remove duplicate was

Move field around in table definition as per Eileen's comment

CRM-20892 Add in modified_date column to civicrm_mailing to assist in preventing cross edit of mailings

Merge pull request #10954 from seamuslee001/CRM-21157

CRM-21157 Convert civicrm_subscription_history.date column to be time…

[NFC] comments clean up in test classes.

Also remove a few unused variables

Merge pull request #10947 from JMAConsulting/CRM-21133

CRM-21133: Price set error with NULL financial types

Merge pull request #10957 from coldrunKacper/CRM-20630-change-activity-form-to-accept-url-search-parameters

CRM-20630: Modifying CRM_Activity_Form_Search to use URL parameters as search form values when passing force=1.

Merge pull request #10959 from mlutfy/cleanup-pcp

templates/CRM/PCP/Form/Campaign.tpl: remove comment (not relevant).

templates/CRM/PCP/Form/Campaign.tpl: remove comment (not relevant).

CRM-20630: Fixing indentations.

CRM-20630: Modifying CRM_Activity_Form_Search to use URL parameters as search form values when passing force=1.

Merge pull request #10826 from jitendrapurohit/CRM-21038

CRM-21038 - select default processor if displayed by amount selection.

CRM-21157 date column information into array of converted columns in System Check

CRM-21157 Convert civicrm_subscription_history.date column to be timestamp rather than datetime to help with timezone issues

Merge pull request #10883 from mickadoo/CRM-21086-allow-inline-file-view

CRM-21086: Allow Inline Download of Files

Merge pull request #10911 from lcdservices/CRM-21113

CRM-21113: find case - search by id, subject

Merge pull request #10904 from seamuslee001/CRM-21106

CRM-21106 Add Warnings about new required extension to support Financ…

Merge pull request #10955 from fuzionnz/typofix_in_deprecation

Correct spelling.

Set version to 4.7.26

Merge pull request #10956 from civicrm/4.7.24-rc

4.7.24 rc

Merge pull request #10897 from agh1/4.7.24-releasenotes

4.7.24 release notes

Correct spelling.

4.7.24 Release notes: contributors

Release notes 4.7.24: edits and reorganization

Remove unnecessary line

Merge pull request #3 from alifrumin/4.7.24-releasenotes

4.7.24 releasenotes edits

CRM-21106 Add Warnings about new required extension to support Financial Type ACLs in reports

CRM-21106 Make language less urgent indicating it will in the futre need the extension

cleaning up and dealing with documentation of bigger changes

moving around issues between bugs and features

Merge pull request #10754 from totten/master-actcase-ts

CRM-20958 - Track creation+modification times for activities+cases

CRM-20958 - api_v3_CaseTest::testCaseUpdate - Style tweak

CRM-20958 - TimestampTriggers - Update docblock

CRM-20958 - api_v3_CaseTest::testCaseUpdate - Ignore creation/modification date.

Checking the `modified_date` is a bit racy -- depending on sub-second
performance/alignment, the original `Case` creation and the subsequent
`Case` update may have the same `modified_date` or may have different
`modified_date`.

CRM-20958 - Health check - If NULL timestamps are still around, show link to DoctorWhen

CRM-20958 - Case, Activity BAOs - Watch out for saving stale timestamps

There appears to be some application logic which follows a process like this:

 1. Read the case
 2. Tweak the data
 3. Save updated case

The problem is comes if step #4 resaves a timestamp loaded in step #1, which
is fairly likely to happen if you read+save the same record.

This was specifically observed on the "Manage Case" screen when editing
activities, but the data-flow is pretty common, so make a general fix to the
BAO.

CRM-20958 - api_v3_CaseTest - Add test for timestamp management

CRM-20958 - api_v3_ActivityTest::testActivityCreateExample - Ignore creation/modification date.

The technique of using hard-coded example record doesn't work with this data.

CRM-20958 - Create case-activity triggers (to update case when activity changes)

CRM-20958 - Create activity and case triggers

CRM-20958 - Add helper Civi\Core\SqlTrigger\StaticTriggers

CRM-20958 - Extract helper Civi\Core\SqlTrigger\TimestampTriggers

== Before ==

 * SQL triggers to populate `civicrm_contact.created_date` and  `civicrm_contact.modified_date` are
   generate via `CRM_Contact_BAO_Contact::triggerInfo($info, $tableName)`

== After ==

 * `CRM_Contact_BAO_Contact::triggerInfo` calls a helper `TimestampTriggers`
 * The helper `TimestampTriggers` accepts arguments describing the names of the tables/columns
   which needed for the timestamp triggers.

== Comments ==

To test, I used this command to update and dump the schema:

```
cv api system.flush triggers=1 && mysqldump --triggers ...
```

The schema was identical before and after.  (Notably, by alternately hacking
the code, I was able to validate the test was capable of revealing
discrepencies.)

CRM-20958 - On upgrade, declare created/modified timestamps for case+activity

CRM-20958 - Declare created/modified timestamps for case+activity

CRM-21113: find cases -- use tag multi-select widget

4.7.24 release notes: boilerplate added

4.7.24 release notes: raw from script

additional fix

CRM-21133: Price set error with NULL financial types

Merge pull request #10908 from jitendrapurohit/CRM-20533

CRM-20533 - Drop false indices directly from update indices button

Merge pull request #10950 from totten/master-setstk-doc

(NFC) SettingsStack - Finish incomplete docblock

(NFC) SettingsStack - Finish incomplete docblock

Merge pull request #10949 from civicrm/4.7.24-rc

4.7.24 rc merge

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 #10907 from seamuslee001/CRM-21110

CRM-21110 Do not run same/similar query twice when generating Relatio…

Merge pull request #10944 from mattwire/CRM-21152_membership_status_spec

CRM-21152 Add spec for MembershipStatus.calc API function

Merge pull request #9783 from mattwire/back_to_single_lang

CRM-19968 SQL tweaks to make it more likely that reverting to single language w…

Merge pull request #10945 from mattwire/nfc_changes

CRM-21153 NFC Fix mispelled variable

Fix mispelled variable

Add spec for MembershipStatus.calc API function

Merge pull request #10940 from seamuslee001/CRM-20769-fields

CRM-20769 Add in columns to support customising profile buttons

CRM-20769 Add in columns to support customising profile buttons

Merge pull request #10926 from MegaphoneJon/CRM-21121

CRM-21121 - Fix crash with sql_mode is only_full_group_by

Merge pull request #10938 from eileenmcnaughton/deltest

Reinstate accidentally deleted test.

Merge pull request #10939 from MegaphoneJon/xml-install

CRM-21142 - CiviCRM installer - Check for XML module before install

Better labeling of XML requirement

Merge pull request #10719 from JKingsnorth/CRM-20935

CRM-20935 Remove related profile links when event deleted

Check for XML module before install

CRM-20935 Remove related profile links when event deleted

Reinstate accidentally deleted test.

This got deleted in renaming a file, re-adding

Merge pull request #10915 from eileenmcnaughton/primary

CRM-20855: Disabling "Search Primary Details Only" causes partial CiviMail delivery failure

CRM-20855: Disabling "Search Primary Details Only" causes partial CiviMail delivery failure

CRM-20855: Adjust variables for clarity

I'm concerned about the use of $apiEntity to mean 'do something unconnected' so suggest
a specific paramteter.

Merge pull request #10785 from kenwest/CRM-20982

CRM-20982 Configure the SMTP mailer to identify itself to the SMTP se…

Merge pull request #10937 from colemanw/CRM-21139

CRM-21139 - Add crm-i.css to common/fatal.tpl

CRM-21139 - Add crm-i.css to common/fatal.tpl

Merge pull request #10929 from seamuslee001/CRM-20237

CRM-20237 Don't add in extra copy of ids and only use most recent id …

Merge pull request #8724 from JMAConsulting/CRM-18231

CRM-18231 Environment variables

Merge pull request #10933 from civicrm/4.7.24-rc

4.7.24 rc

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

QA fixes

CRM-21130 - Search builder throws error if location type has space (#10928)

* CRM-21130 - Search builder throws error if location type has space

* minor fixes

Update tests following changes by Eileen

CRM-20237 keep some of original name in name field

CRM-21128 Personal Campaign Page selection with force=1 in Find Contributions (#8997)

* Personal Campaign can now be passed in force mode

Passing pcpid=n with force=1 alloow to search on corresponding Personal
Campaign id

* Improves parameter passing

Now cope with both single value form (pcpid=n) and multivalue form
(pcpid[]=n&pcpid[]=m)

* Removed useless ++$n

* Fixed checkstyle warnings

* Operator spacing and comment indentation

CRM-20237 Don't add in extra copy of ids and only use most recent id copied from in title and don't put copy of details in name for pricesets