comment_notify_schema() claims that the notify column in the comment_notify table is 'A boolean indicator for whether or not they subscribed: 1 means subscribed, 0 means not subscribed.'. This is inaccurate because it can sometimes contain 2 (COMMENT_NOTIFY_COMMENT).

Files: 
CommentFileSizeAuthor
#6 958670_6_comment_notify_docs.patch1.49 KBgreggles
#2 958670_2_comment_notify_docs.patch2.47 KBscor
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 958670_2_comment_notify_docs.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 958670_1_comment_notify_docs.patch2.43 KBscor
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 958670_1_comment_notify_docs.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new2.43 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 958670_1_comment_notify_docs.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Proposed wording:

A integer indicating the type of subscription: 0 means not subscribed, 1 means subscribed to all comments, and 2 means only subscribed to replies of this comment.

also includes some indentation fix in the same file.

StatusFileSize
new2.47 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 958670_2_comment_notify_docs.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

minor grammar fix.

Status:Needs review» Needs work

Thanks scor!

As long as we're fixing this how about some descriptions on node_notify and comment_notify?

removing spam tag

Status:Needs work» Needs review
StatusFileSize
new1.49 KB

I think these are right now.

Status:Fixed» Closed (fixed)

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