Update Hook.php

Update Hook.php

Update Hook.php

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

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

Merge pull request #10831 from MiyaNoctem/CRM-20858-fix-merge-of-unselected-custom-fields

CRM-20858: Fix Merging of Unselected Custom Fields

additional fixes

CRM-18231 Added settings on developer page

Merge pull request #10923 from jitendrapurohit/CRM-21126

CRM-21126: fix default selection of member_is_primary input

Merge pull request #10921 from eileenmcnaughton/cachekey

CRM-21124 fix error on cacheKey when too long

Merge pull request #10731 from eileenmcnaughton/pref_comm

CRM-19933 proposed fix for import wiping out preferred comm method (possibly CRM-20035 too)

Merge pull request #10890 from eileenmcnaughton/dedupe

Towards CRM-20155 clean up form code in order to consolidate function…

CRM-20858: Remove Uneeded Variable

CRM-19933 proposed fix for import wiping out preferred comm method

modify check to 0 or 1

CRM-21121 - Fix crash with sql_mode is only_full_group_by

Merge pull request #10924 from MegaphoneJon/CRM-21129

CRM-21129 - CLI API CSV import doesn't allow single-column CSVs

Merge pull request #10757 from jitendrapurohit/CRM-20964

CRM-20964: Include queue id while retrieving rows from Event Queue

CRM-21129 - CLI API CSV import doesn't allow single-column CSVs

CRM-21126: fix default selection of member_is_primary input

CRM-21124 fix error on cacheKey when too long

Merge pull request #10734 from jitendrapurohit/CRM-20949

CRM-20949 - Fix payment processor assignment

Merge pull request #10886 from eileenmcnaughton/task

NFC remove some unused vars, fix comment

Merge pull request #10906 from MegaphoneJon/CRM-21108-2

CRM-21108 - Greatly improve performance of adding contacts

Merge pull request #10916 from eileenmcnaughton/permission_problem_address_get

CRM-20027 add test for locBlock permission issue

Merge pull request #10917 from mattwire/CRM-21118_messagetemplate_buttons_at_top

CRM-21118 Add action links at top of page on message template overview

Merge pull request #10585 from JMAConsulting/CRM-17748

[ready for core team review] CRM-17748: Expose options to CRM_Core_DAO

CRM-20027 add test for locBlock permission issue

Merge pull request #9973 from lcdservices/CRM-19469

CRM-19469 support returning payment params after mapping

Add action links at top of page

Drop Table before returning from function

Include word temporary hopefully appeases jenkins

Merge pull request #10780 from jitendrapurohit/CRM-20979

CRM-20979 -  expose pre help for price fields to the UI

Merge pull request #10874 from totten/master-ts-warn

CRM-21079, CRM-9683 - Display advisories about DATETIME/TIMESTAMP columns

Merge pull request #10730 from eileenmcnaughton/test_name

Renamed test class.

CRM-21079, CRM-9683 - CRM_Utils_Check - Expand advice

CRM-21108 - handle single or multi-dimensional contact detail arrays

Merge pull request #10888 from mattwire/CRM-21092_financial_batch_fixes

Fix PHP Warnings. Replace fatal with statusBounce. Mark appendBreadCr…

Merge pull request #10860 from mattwire/CRM-21065_optiongroup_to_pseudoconstant

CRM-21065 Replace some CRM_Core_OptionGroup::values with CRM_Activity_BAO_Activ…

Merge pull request #10865 from mattwire/CRM-21069_check_eventid

CRM-21069 Fix crash when event_id is not set

Add warning log message about bad data

Update based on comments from @eileenmcnaughton

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

CRM-21110 Do not run same/similar query twice when generating Relationships Tab in CiviCRM

Fix comments on new unit test

CRM-21108 - Refactor greeting token replacement code to avoid calls to expensive queries in most situations

Formatting fixes

Merge pull request #10863 from rtobias81/lastModified-graceful

CRM-21068 - getLastModified fails more gracefully now.

Merge pull request #10882 from jitendrapurohit/CRM-21087

CRM-21087: Payment popup goes weird if you click 'adjust payment amount'

Merge pull request #10903 from colemanw/CRM-20680

CRM-20680 - Only spin the logo in the menubar