In privatemsg.api.php, the documentation mentions hook_privatemsg_message_delete.
This doesn't appear to exist anywhere (or be called anywhere).
It would be good to see this functionality (e.g. to integrate with activity module, and so track when a message is deleted).
As deleted messages can be restored, it would be good to also have an undelete hook: hook_privatemsg_message_undelete.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | add_delete_hook_7.patch | 4.01 KB | berdir |
| #7 | add_delete_hook2.patch | 3.22 KB | berdir |
| #6 | add_delete_hook.patch | 3.22 KB | berdir |
| #1 | 750566_add_delete_undelete_hooks.patch | 3.04 KB | manarth |
Comments
Comment #1
manarth commentedThe attached patch adds delete and undelete hooks, and adds additional documentation for both.
Comment #3
berdirWe decided to remove the hook because modules don't need to act when a message is deleted but when it is flushed and we forgot to update the documentation. But I can see that it could be of use.
However, I think this needs to be done against 2.x because new features are not allowed in 1.x.
Comment #4
berdir#1: 750566_add_delete_undelete_hooks.patch queued for re-testing.
Comment #5
berdirSee title. And see #1054082: Allow modules to respond to a status change..
Comment #6
berdirAttaching a patch.
Comment #7
berdirFixed a wrongly named variable in the patch. This passes all tests now locally, the test bot currently doesn't like Privatemsg so much.
Comment #8
berdirCommited.
Comment #9
berdirAttaching a patch for 7.x to run the tests on it..
Comment #10
berdirCommited.