Currently I have hundreds of SPAM trackbacks I want to delete.
I see the "check all" button is there -- thank you for implementing that. It does indeed select all.
However when I select the delete option in the select list, and click the "Update Trackbacks" button ... the page reloads, but nothing is deleted. The same is true if I individually click on one of the checkboxes and click on the Update Trackbacks button.
If, however, I click on the Delete link to the side of one of the items, and go through the delete steps, then the trackback is deleted. But that's not the point, really, because it's such a circuitous route to follow to delete trackbacks.
Comment | File | Size | Author |
---|---|---|---|
#12 | trackback_10.patch | 3.9 KB | matt@antinomia |
#7 | trackback-69942.patch2.txt | 3.64 KB | beginner |
#5 | spam-delete.diff.txt | 750 bytes | beginner |
#2 | trackback-69942.patch.txt | 2.98 KB | zorac |
Comments
Comment #1
kindafun CreditAttribution: kindafun commentedI have had the exact same experience.
Comment #2
zorac CreditAttribution: zorac commentedI fix this issue.
Comment #3
sKanD CreditAttribution: sKanD commentedThank you for the patch!
I also have hundred of spams and that patch spared me from flushing the database in phpmyadmin as i used to.
Comment #4
beginner CreditAttribution: beginner commentedthe patch can only work if you have the spam module enabled. It removes an important check.
Comment #5
beginner CreditAttribution: beginner commentedIt was a simple bug that affected people who do NOT have the spam module.
Tested: works.
Comment #6
beginner CreditAttribution: beginner commentedThe patch just above must be applicable to HEAD too.
It fixes the immediate problem, but I notice that there is a lot of code redundancy between the function spam_delete_trackback and the function trackback_delete.
The code can be optimized in many places.
Comment #7
beginner CreditAttribution: beginner commentedForget my previous patch. My apologies to zorac: not only his patch fixes the bug, but it also removes the code redundancy I was speaking about. This new patch is the same as his, but with one more little code optimization (don't call
module_exist('spam')
twice in a row, but use$spam_module_exists
that was set the first time.)this patch is good to go.
Comment #8
zorac CreditAttribution: zorac commentedNever mind :-)
I seem trackback module is a patchwork now.
I wish to rewrite it but ...
Comment #9
beginner CreditAttribution: beginner commentedRewrite it?
I don't think it's such a good idea:
http://www.joelonsoftware.com/articles/fog0000000069.html
Now, if you wish to improve it, clean it, etc... try to get cvs access to this module, so that you can commit your own patches.
And if you need help, contact me: the issue queue is not that long, but needs some love.
Comment #10
zorac CreditAttribution: zorac commentedYou are right. I don't wish to rewrite it but i wish to clean it :-)
Thanks for your proposal. I'll think about it.
Comment #11
dgrant CreditAttribution: dgrant commentedI used beginner's most recent patch above and it works now. Thanks.
Comment #12
matt@antinomia CreditAttribution: matt@antinomia commentedThe patch works well for me.
Additionally, there was an error in the drupal_goto() in trackback_received_delete() that didn't handle the URL fragment (#trackbacks) correctly. The attached patch fixes this as well.
Comment #13
Christoph C. Cemper CreditAttribution: Christoph C. Cemper commentedHey,
none of these patches work
when I try to applay the last one trackback_10 I get
missing header for unified diff at line 2 of patch
Assertion failed: p_base <= file_pos, file ../patch-2.5.9-src/pch.c, line 771
can you attach your recent 4.7 trackback module or just commit it there?
(btw I could commit it too as I plan to implement akismet stuff as well)
Comment #14
Christoph C. Cemper CreditAttribution: Christoph C. Cemper commentedI could apply this stuff and fixed in in the last 4.7.x version with my akismet integration
merge to the 5.x version still pending, but closing this now as the 4.7.x tree works