node_repeat checks on node update to see if the date has changed and removes it from the sequence when it has, but it does this check when the node hasn't been created yet. Of course, if the old date doesn't exist, it tries to remove it from the sequence. Since the node hasn't been created, it doesn't remove it, but it does spam the user with messages saying it has, which is confusing and annoying. This patch just skips the check if the node doesn't exist yet so these messages don't get sent.

CommentFileSizeAuthor
#5 node_repeat-865130-01.patch1.06 KBilluminatedgeek
node_repeat-unspam.patch1.05 KBthrnio

Comments

MasterChief’s picture

MasterChief commented

It would be great if somebody can maintain this module to commit patch like this and add other feature or just fix issues.

If you read this and want to be this person, go here : http://drupal.org/node/845068

gagarine’s picture

gagarine commented
Status: Needs review » Fixed

Look like this bug is fixed for me.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

illuminatedgeek’s picture

illuminatedgeek commented
Status: Closed (fixed) » Needs review

The bug is not fixed in the module, but the patch posted does work. It just needs to be merged into the branch.

illuminatedgeek’s picture

illuminatedgeek commented
StatusFileSize
new node_repeat-865130-01.patch1.06 KB

New rerolled patch to remove 1 line offset (for drush make)