Posted by indytechcook on December 10, 2009 at 1:51am
16 followers
| Project: | Mollom |
| Version: | 6.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | major |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
This little patch creates 2 actions. These are basically a copy of the submit function in the comment moderation form in an actions format. This is very useful for Views Bulk Operations compatibility and in use in a comment moderation view.
- mollom_unpublish_comment
- mollom_delete_comment
Cheers,
Neil
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| mollom-action.patch | 2.91 KB | Idle | FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch mollom-action.patch. Unable to apply patch. See the log in the details link for more information. | View details | Re-test |
Comments
#1
Also see #152603: Revise comment admin interface like content admin, add hook_comment_operations() since the Mollom module technically already provides these actions, but has to hack them into the current comment listing interface.
#2
Taking into account #717212: Remove "report to Mollom" links and integrate with entity delete confirmation forms instead, we should only provide actions to report a post.
Although I'm not sure how useful such an action is.
#3
Thanks for taking the time to report this issue.
However, marking as duplicate of #245689: Integrate actions with reporting functionality on delete confirmation forms.
You can follow up on that issue to track its status instead. If any information from this issue is missing in the other issue, please make sure you provide it over there.
#4
I don't agree with marking this as a dupe. This is a small, useful feature that can be used in Views Bulk Operations and elsewhere. Making it dependant on a confirm form issue is needlessly blocking it. It has already blocked for months. If the admins disagree, feel free to set this back to dupe.
Instead of a drupal_set_message(), a watchdog() would be more consistent with core. See the action below from core. The drupal_set_message calls would look upgly after deleting a hundred spams using views bulk operations.
I'll make the above change if the committers think this patch can proceed.
#5
subscribing.
I agree this is a great idea to add. I agree holding it back based on a relatively unrelated issue is unfortunate.
Ultimately this means that mollom.com will get less information about spam comments. I'm building a view to delete a few hundred spam comments on g.d.o and now instead of reporting them to mollom I will simply delete them. That is valuable information that mollom.com loses out on.
#6
Is there anything that can be done to hurry this along to a conclusion? I could test the patch if that helps (although - 2009? - is this going to work with the current dev release?).
I'm not sure how much longer I can use Mollom if I can't integrate it into Views Bulk Actions and Rules. Viewing individual comments and reporting them is just too time-consuming, so, like greggles, I often find my only option is to delete them and deny mollom.com the spam information.
#7
Subscribe
#8
subscribe
#9
I'm going to bump the priority here. Otherwise, what can I do to help :) ?
#10
I was so surprised to find that mollom doesn't support a link in views to mark as spam or a views bulk operation!
#11
I marked #1441924: Provide hook to Views Bulk Operations as a duplicate.
#12
Updated patch for the latest version of Mollom
#13
#14
The last submitted patch, 0001-Issue-655846-by-indytechcook-gordon-Added-Mollom-Act.patch, failed testing.
#15
Fix up issue in mollom_action_info().
#16
Here is a view which I use with great success to remove spam quickly from a site.
#17
Found a small bug when going to 2.x
#18
Fix up version
#19