When subscriptions is enabled with storylink of voting up/down module, and when a user subscribes for notifications on the storylink node, an email is sent each time a user votes up or down on that module.

This should be activated only for comments and not for every vote.

Not sure if this should be posted here or in storylink module.

Comments

dziemecki’s picture

dziemecki commented
Title: When subsciptions is enabled with storylink of voting up/down » When subscriptions is enabled with storylink of voting up/down

Subscriptions is reacting to triggers for node updates and comment inserts. My guess is a that a vote sends the same even as a node update. My initial reaction here is that Subscriptions is doing what it is supposed to. Not sure how I could tell a vote from a regular node update. I'm disinclined to make module specific decision brackets, especially for contributed modules. Seems like a messy road to go down.

drupalzack’s picture

drupalzack commented

Agreed. First of all thanks for the quick reponse! Really appreciate it. Secondly, subscriptions is a wonderful module. Works perfectly everytime.

I'm trying to understand how this works. Let me know if this is correct:

Each time a node is updated or a comment is updated, that nodetype sends out a trigger to the subscriptions module? Now is this trigger in the form of a table update (if so which table) or a hook (if so where should I look for in code).

Thanks again and I understand this is not subscriptions problem, but appreciate any info you can give on this.

dziemecki’s picture

dziemecki commented

When a node is updated or commented, hook_nodeapi() and hook_comment() are called. These, in turn, call every registered module with a local implementation of those hooks (e.g. subscriptions_nodeapi() and subscriptions_comment()).

If you know a little PHP, and it sounds like you do, you can probably identify your storylink posts by looking at $node->type.

drupalzack’s picture

drupalzack commented

Thanks...I was reading about the hook system. Looks like I'm on the right track.

dziemecki’s picture

dziemecki commented
Status: Active » Closed (won't fix)

Looks like this was addressed another way.