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).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

scor’s picture

Status: Active » Needs review
FileSize
2.43 KB

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.

scor’s picture

minor grammar fix.

greggles’s picture

Status: Needs review » Needs work

Thanks scor!

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

greggles’s picture

removing spam tag

greggles’s picture

Status: Needs work » Needs review
FileSize
1.49 KB

I think these are right now.

greggles’s picture

Status: Fixed » Closed (fixed)

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