Hook on case change client and case merge cases

[jaapjansma]

Hey all,

Is there a hook available for when a case is reassigned to another client? And in addition is there a hook when a case is merged with another case?

I need such hooks because I have developed a document solutions for civicrm and I want to store (link) documents to a case but as soon as a case is reassigned the documents disappear (because the old case is deleted). The same is true on a merge case.

Thanks,
Jaap

[Coleman Watts]

If hook_civicrm_post doesn't do it, maybe we need to add hooks for this.

[jaapjansma]

The hook post is called but it doesn't give a clue wether this was a save on a new case, edit existing one, or a merge or a reassign...

I am happy to implement such a hook. Shall I register an issue for this and start working on it?

[Coleman Watts]

Sounds good to me.

[jaapjansma]

I have added an issue to the issue trakcer: https://issues.civicrm.org/jira/browse/CRM-14409
And created a pull request (https://github.com/civicrm/civicrm-core/pull/2782)

This PR contains two hooks
a civicrm_pre_case_merge and a civicrm_post_case_merge both hooks have the same parameters as the mergecases function in the CRM_Case_BAO_Case class.

[Erik Hommel]

Do you want to do this as separate hooks? I think it would make more sense to me to have it in the civicrm_post hook, with object Case and $op 'merge' or 'reassign'?

[jaapjansma]

That would definitely make sense if the hook pre and the hook post had an ability to use two objects (two cases). But they haven't so I had to use this solution. Also the way a case is reassigned could be refactored so that on a reassign it would not do a merge of the original case and a new one (and then deleting the old one).

[Erik Hommel]

Once more proof that the way CiviCase works should be overhauled a bit....but I guess we will have to find some funding first 

[Eileen]

I just added a new hook to the reference - http://wiki.civicrm.org/confluence/display/CRMDOC/Hook+Reference - it seemed confusion so I added (from 4.5) in the title so it would show on the page of contents - not sure if there is a better way

[jaapjansma]

Hey Eileen,

I don't see the documentation of the hook....
I think you have replied on the wrong topic..

[Eileen]

sorry - I meant - I just added another hook to that reference - I am assuming you are about to add these new ones to it & might hit the question I did - how to make it clear in the main hook reference page that this hook is only available from version x.

- I put (from 4.5) in the title of the new hook - so it would show up in the table of contents. Not sure if that is a good approach

[jaapjansma]

Yes, I was not sure about the documentation yet. I thought I would definitly update the docs as soon as 4.4.5 comes out. But this would save me some work by then.