Here is the patch.

CommentFileSizeAuthor
#2 nodeapi_delete_ok.patch991 bytessavioret
nodeapi_delete.patch470 bytessavioret

Comments

solipsist’s picture

solipsist commented
Status: Patch (to be ported) » Needs work

Thanks for submitting. I'll fix this in the next release.

If you could rewrite the patch in a way that separates the DB query from the hook implementation, I could use it as it is. Look at how Watcher is written for some examples. Note the include file with the DB functions.

Also, please do not mark your own patches as "to be ported". A patch should be reviewed by at least one other person before it's considered ready for porting or commit.

savioret’s picture

savioret commented
Title: watched nodes are not cleared when a node is deleted » Corrected version
Status: Needs work » Needs review
StatusFileSize
new nodeapi_delete_ok.patch991 bytes

Thanks for your advising. Here it is the requested version.

solipsist’s picture

solipsist commented
Status: Needs review » Reviewed & tested by the community

Cheers! It's queued for committing!

solipsist’s picture

solipsist commented
Status: Reviewed & tested by the community » Closed (fixed)

Committed.