Posted by Ares78 on July 22, 2008 at 3:27am
| Project: | Akismet |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
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.
Comments
#1
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
#2
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 :-(
#3
Same issue. So far, Akismet module does not appear to be working at all.
#4
Ditto on the same issue.
When I click "Submit Spam" or "Submit Ham" in a commentI get a 404.
#5
same here
#6
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.
#7
I get the same error. But I think this modul is not maintained anymore...
#8
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.
#9
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.
#10
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.
#12
@@ -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 :/
#13
@spookypld: It's a patch, see http://drupal.org/node/60108