Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I am wondering if somebody has wrote something to integrate the Privatemsg module and Activity module. Any help would be great.
Comment | File | Size | Author |
---|---|---|---|
#2 | privatemsg_activity.zip | 1.65 KB | neurovation.kiwi |
Comments
Comment #1
Michsk CreditAttribution: Michsk commented+1
Comment #2
neurovation.kiwi CreditAttribution: neurovation.kiwi commentedhi folks
i just implemented it. please give your 2 cents!
cu
kiwi
Comment #3
BerdirInteresting, just a few first thoughts...
- The .info file seems to be missing...
- The module seems to integrate with token.module and trigger.module too. Is that necessary for activity or why are you doing that? Note that we are working on token integration for Drupal 7 here: #511796: Support tokens in private messages. Maybe that should be splitted up into several modules (so that you could use token integration without activity.module). Actually, token integration could live directly in privatemsg.module IMHO as these are only hook_implementations.
- Please always use two spaces instead of tabs. It is currently a mix and that totally messes up the display when viewing it with an editor that makes tabs 4 spaces wide.
- I have never used activity before but a few things seem overly complex. For example, you're defining a new hook and call that inside hook_privatemsg_message_sent(). Why don't you do that trigger stuff directly in that hook?
- I don't understand what the grants hooks are for but the query that loads the mid's of a user could return a rather big result set...
- Not yet necessary, but if you can upload the module as a patch (Creating patches with new files is tricky but possible with tools like cvsdo. See http//drupal.org/patch/create), I can give you a more details review.
Comment #4
neurovation.kiwi CreditAttribution: neurovation.kiwi commentedad info - file:
since this is no finished module yet - i didn't give one.
ad tabs:
big oopsi on my side. corrected it
ad patch:
not yet tested enough to really give a useable version - i will post a patch in a few weeks.
ad other stuff:
well - that's how activity works :(
ad token:
i'm fine when it would be integrated.
cu
kiwi
Comment #5
sirkitree CreditAttribution: sirkitree commentedWas this for Activity 2.x?
Comment #6
ptmkenny CreditAttribution: ptmkenny commentedComment #7
oadaeh CreditAttribution: oadaeh as a volunteer commentedThis 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.