This module only considers the first term of a node when sending notifications. Instead, It should iterate over all terms, and send notifications as needed. Take care though not to email same user twice. How about

foreach ($node->taxonomy as $tid)

Also, one could argue that we should not email when updating a node. This might save some useless notifications when an admin is just reclassifying a node quietly. I personally prefer undernotifying to overnotifying.

-moshe

Comments

Anonymous’s picture

I agree with this request and will implement with my next update.

As for the second part, that is a separate issue and should be posted separately.

dziemecki’s picture

Title: Consider all terms, not just the first one » Dropping anonymity

Pardon the lack of ID. THat last post is mine.

dziemecki’s picture

Title: Dropping anonymity » Notify all taxa
Assigned: Unassigned » dziemecki

Correcting title, adding assignment.

dziemecki’s picture

These changes were made to the head branch.

Anonymous’s picture