Download & Extend
Comment notify » Issues

if user's don't have the ability to post nodes don't confuse them about a node-notify default and better db insert/update code

Posted by greggles on March 29, 2009 at 6:36pm
Project:Comment notify
Version:5.x-2.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Yep.

Login or register to post comments

Comments

#1

Posted by greggles on June 14, 2009 at 7:33pm
Title:if user's don't have the abilit to post nodes don't confuse them about a node-notify default» if user's don't have the abilit to post nodes don't confuse them about a node-notify default and better db insert/update code
Status:active» needs review

I'm somewhat concerned that the test for "permission to create nodes" might be too strict, but the only reasonable way to figure that out is to get feedback from people with non-standard nodes.

I also took this opportunity to use a slightly more efficient way of updating/storing the data.

AttachmentSize
417542_comment_notify_no_node_setting_for_underprivileged_users.patch 3.41 KB

#2

Posted by greggles on October 20, 2009 at 10:28pm
Title:if user's don't have the abilit to post nodes don't confuse them about a node-notify default and better db insert/update code» if user's don't have the ability to post nodes don't confuse them about a node-notify default and better db insert/update code
Version:7.x-1.x-dev» 5.x-2.x-dev
Status:needs review» patch (to be ported)

Re-rolled the patch. And committed it. http://drupal.org/cvs?commit=277496

AttachmentSize
417542_comment_notify_no_node_setting_for_underprivileged_users_2.patch 4.07 KB

#3

Posted by greggles on March 11, 2010 at 12:01am
Status:patch (to be ported)» closed (fixed)

This module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.

nobody click here