The two rules actions 'Send a message' and 'Send a message to a role' both create a new thread.

I think it would be useful if the actions provided the newly created thread_id as a variable for further processing by rules actions.

CommentFileSizeAuthor
#1 privatemsg-1387740-1.patch2.03 KBarnested
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arnested’s picture

arnested CreditAttribution: arnested at Reload commented
Status: Active » Needs review
FileSize
privatemsg-1387740-1.patch2.03 KB

Patch

ptmkenny’s picture

ptmkenny CreditAttribution: ptmkenny commented

#1: privatemsg-1387740-1.patch queued for re-testing.

ptmkenny’s picture

ptmkenny CreditAttribution: ptmkenny commented
Status: Needs review » Reviewed & tested by the community

I've confirmed that the thread ID variable is properly provided and can be used in Rules actions with the latest dev version (2013-Jun-08). Though the thread ID variable is not always necessary, having it available could be very helpful in some use cases.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed and pushed to 7.x-2.x and 7.x-1.x, does not apply to 6.x. Also fixed a comment that did not follow the coding standard.

@ptmkenny: Remember to also check coding standards (the parts you know) of patches before you rtbc them.

ptmkenny’s picture

ptmkenny CreditAttribution: ptmkenny commented

Sorry I missed that comment. I will be more aware of this in the future.

ptmkenny’s picture

ptmkenny CreditAttribution: ptmkenny commented
Issue summary: View changes

Patch supplied

oadaeh’s picture

oadaeh CreditAttribution: oadaeh as a volunteer commented
Issue summary: View changes
Status: Patch (to be ported) » Closed (won't fix)

This issue is being closed because it is against a branch for a version of Drupal that is no longer supported.
If you feel that this issue is still valid, feel free to re-open and update it (and any possible patch) to work with the 7.x-1.x branch (bug fixes only) or the 7.x-2.x branch.
Thank you.