Merge pull request #10920 from eileenmcnaughton/pi

CRM-20750: Incorrect financial trxn entries when payment instrument i…

Merge pull request #10976 from eileenmcnaughton/4.7.25-rc

4.7.25 rc

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

Merge pull request #10975 from eileenmcnaughton/4.7.25-rc

CRM-21175 fix fatal error on multiple custom field import.

CRM-21175 fix fatal error on multiple custom field import.

Additional test is tangental....

Merge pull request #10439 from jitendrapurohit/CRM-20657

CRM-20657: get all participant ids if multiple contacts register from…

Merge pull request #10838 from jmcclelland/CRM-16964

CRM-16964: ensure FILL works on import with custom fields.

Merge pull request #10952 from civicrm/michaelmcandrew-patch-1

Fixing type (missing function call) in example for hook_civicrm_alterAngular

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

Update Hook.php

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-20750 follow up.

Change logic for determinig paymentInstrumentChange to look at whether
['payment_instrument_id'] isset. If not it has not been passed by the calling
function and by definition the calling function is not attempting to change it

CRM-20750 follow up fix, reversal should be against original account.

On trying to investigate why a test change was required in the original PR I came to the following conclusions:

1) the test function was causing problems by being hard to read. As a result it was too easy to see it as
'accounts magic' - I started a cleanup here
2) Once I understood what was being tested & failing I concluded the change in the PR was in fact wrong
as it was putting a negative entry against the new financial account rather than the original one.

CRM-20750: Incorrect financial trxn entries when payment instrument is changed on backoffice Contribution edit form

added unit test

CRM-20750, fixed db entries

----------------------------------------
* CRM-20750: Incorrect financial trxn entries when payment instrument is changed on backoffice Contribution edit form
  https://issues.civicrm.org/jira/browse/CRM-20750

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

Update Hook.php

Update Hook.php

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

Fixing type (missing function call) in example for hook_civicrm_alterAngular

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