Ticket #351 (closed defect: fixed)

Opened 7 months ago

Last modified 3 months ago

Mass Delete missing from Comments screen.

Reported by: chrisjdavis Owned by: arthus
Priority: minor Milestone: 0.6
Component: Habari Core Software Version: SVN
Keywords: spam mass delete has_patch Cc:

Description

We need to put the mass delete option back in to the comments admin screens. I have eleventy-billion spam comments that I would like to get rid of, and doing them 20 at a time is no fun.

Attachments

selecteverything.patch (9.0 kB) - added by arthus 3 months ago.

Change History

Changed 7 months ago by ringmaster

The "mass delete" should be achieved in a different way. Here's what I think the vision was:

When you click the "Select All" checkbox, you should have the option of somehow clicking it again to select all of the items that match the current search. This idea is similar in concept to how GMail allows you to select all of the visible items in a search, and then click a link to select all of the items that qualify for a search. (Looking at how GMail does this is really helpful for illustration purposes.)

Then with all of the search results selected, using the regular delete button would delete truly everything. (Yikes.)

Changed 7 months ago by bjohnson

I like ringmaster's suggestion.

Changed 6 months ago by tinyau

+1 for the suggestion of ringmaster.

Changed 6 months ago by dmondark

I am +1. Maybe when you do a select all, a link will show next to the n selected that will do its magic to select all of the applicable results. As a workaround, now you can do the search, expand the loupe width in the timeline to cover up to 191 items ,use the 'select all' checkbox to delete them.

Changed 6 months ago by Heilemann

The way dmondark describes it is the way I had it in mind. But then I'm not sure I'd thought about what would happen in the case that you ahve 25.000 comments (like I do), so... Let's just say I'm +1 on ringmaster :)

Changed 3 months ago by Heilemann

  • milestone changed from 0.5 to 0.6

+1 as well.

Changed 3 months ago by arthus

  • owner set to arthus
  • status changed from new to assigned

Changed 3 months ago by arthus

Changed 3 months ago by arthus

  • keywords has_patch added

I have implemented this for everything which uses itemManage – posts, comments, and logs. It also works nicely with paging, so even if you move the timeline after selecting everything it will still show everything as selected. The select all link is placed to the right of the super-checkbox.

Changed 3 months ago by chrismeller

  • status changed from assigned to closed
  • resolution set to fixed

This was committed in r2455.

Note: See TracTickets for help on using tickets.