I just recently upgraded to version 6 of Akimet, after upgrading drupal as well to version 6, ran update.php as well. Now when I try to submit spam on any page, it goes to a 404 Page not found error. The same if I try to Unpublish a comment.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Ares78’s picture

Category: bug » support

moved to the correct area, my apologies... I should clarify, when marking a posted comment as spam, it will go to the 404 Page not found error, I'm not actually submitting spam myself :P

coofercat’s picture

For what it's worth, I get the same problem (after a similar upgrade experience). Actually, I'm finding most Akismet based comment management is problematic :-(

keathmilligan’s picture

Same issue. So far, Akismet module does not appear to be working at all.

tcblack’s picture

Ditto on the same issue.
When I click "Submit Spam" or "Submit Ham" in a commentI get a 404.

alpinejag’s picture

same here

Napzilla’s picture

Component: User interface » Code
Category: support » bug

Akismet causes me to receive the same 404 message when I'm trying to post any content at all, not just when marking a comment as spam.

Drupalone’s picture

I get the same error. But I think this modul is not maintained anymore...

peterpoe’s picture

Version: 6.x-1.0-beta1 » 6.x-1.x-dev
Status: Active » Needs review
FileSize
2.84 KB

Try applying this patch, which should solve this problem. It also fixes a bug (which you could not even see, since it the 404 error blocked everything) where the "submit ham" link on node and comments would not work.

patricksettle’s picture

Dang beat me to the punch, was going to work on this today.

Patch mostly works.

Had to change the function call on line 302 form:

    if (!$map[2] = comment_load($map[2])) {

over to :

    if (!$map[2] = _comment_load($map[2])) {

I'm working on a clean patch with both changes as I type, I'll upload soon.

EDIT:
Oops. Also forgot, there's several l() calls that are not formated for drupal 6.

patricksettle’s picture

FileSize
7.24 KB

Ok, so there's several things here that were causing the akismet module to not work at all for me. I've several of which peterpoe knocked out in the patch in #8

I had also found a few more that were still causing some problems. Which include multiple cases of the incorrect use of the l() function. In drupal 6 the extra attributes are in an array, not as separate arguments. And a couple more instances where the akismet_content_load function was being fed the whole object and not just the id.

I've rolled them all into a patch, which (fingers crossed) should take care of the basics. This doesn't address the admin screens layout or any other issues, but does get the links working completely again.

spookypld’s picture

@@ -269,15 +269,13 @@ function akismet_menu() {
         'file' => 'akismet.admin.inc',
       );
     }
-  }
-  else {
     $item = array(
       'title' => 'switch content status',
       'page callback' => 'akismet_page',
       'page arguments' => array(0, 1, 2, 3),
       'load arguments' => array('%map', '%index'),
       'access callback' => 'akismet_access_callback',
-      'access argument' => array(0, 1, 2, 3),
+      'access arguments' => array(0, 1, 2, 3),
     );
     foreach (array('publish', 'unpublish', 'submit-spam', 'submit-ham') as $op) {
       $items['akismet/%akismet/%/'. $op] = $item;

What does it mean ?

I count to 17 rows? I've found it a little bit messy :/

peterpoe’s picture

@spookypld: It's a patch, see http://drupal.org/node/60108

apaderno’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

I am closing this issue, since it's for a Drupal version not supported.