Corrected version
birwel - June 14, 2009 - 20:08
| Project: | Watcher |
| Version: | 6.x-1.2 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed |
Description
Here is the patch.
| Attachment | Size |
|---|---|
| nodeapi_delete.patch | 470 bytes |
#1
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.
#2
Thanks for your advising. Here it is the requested version.
#3
Cheers! It's queued for committing!
#4
Committed.