Description: When marking comment spam on a blog post as "not spam" via the admin console, or via the link on the comment itself, it fails to successfully marked it as not spam.

Expected Result: When marking comment spam on a blog post, it is marked as not spam.

Impact: The impact is that the administration of the comments is harder effecting user adoption, and the spam is not marked correctly which effects use of the blog comments and causes the bayesian filter not to filter correctly.

Steps to Reproduce (Scenario 1):
- As logged in as an admin, go to the admin section, and click the "Spam" link under the "Content Management" group.
- Select the comment in the table at the bottom of the page by clicking the tick box next to it.
- In the "Update options" drop down (above the table) select "Mark as not spam" and click "Update".
- The page will refresh and state a message that "The update has been performed.", however the comment is still marked as spam and the score is still 99.
- The database logs will also show the following two error messages:
"Missing argument 3 for spam_operations_callback() in //modules/spam/spam.module on line 1489."
"Invalid argument supplied for foreach() in //modules/spam/spam.module on line 1490"

Steps to Reproduce (Scenario 2):
- As logged in as an admin, go to the blog post with the comment spam and navigate to the comment.
- Click the link at the bottom of the comment labeled "mark as not spam".
- The page will refresh with the Drupal 404 error.
- The database logs will also show a warning for page not found: "spam/comment//notspam"

Comments

gnassar’s picture

Status: Active » Fixed

Fixed in newest dev snapshot.

Status: Fixed » Closed (fixed)

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