Make WordPress Core

Opened 16 years ago

Closed 16 years ago

#380 closed enhancement (fixed)

Mark all for delete (instead of "delete all marked for defer")

Reported by: Jenn99 Owned by: matt
Milestone: Priority: normal
Severity: normal Version:
Component: Administration Keywords:
Focuses: Cc:


I noticed that there is now a checkbox that says "delete all comments that are marked as defer"

I had come up with a hack for 1.2 that set all the radio boxes to delete:
http://www.scriptygoddess.com/archives/2004/06/30/comment-moderation-select-all-for-delete/ I assume that what is currently in 1.3 is a take off on that idea, but (not trying to be obnoxious) I think my way of doing it is better, and this is why:

My way:
-allows you to STILL "defer" some comments.
-Clearly indicates WHICH comments will be deleted - and WHICH will not.
ie. you can click that "mark all for delete" and then go back in and select some back to "defer" etc. - and what is marked for DELETE will be DELETED. what is marked for DEFER will be DEFERED

current 1.3 way:
-forces you to either approve or delete all comments currently in moderation. (no option to leave some in "defer" status)
-makes things confusing... it's marked for DEFER but it will actually be DELETED.... ??

So attched is a diff file (if I'm doing this correctly) of the link that says "Mark all for delete", the javascript that does it, and the "delete those marked for defer" removed. As well as my revised version of moderation.php

Please let me know if there's another way I should be submitting these changes/suggestions.

Attachments (2)

0000380-markAllForDeleteDiff.txt (780 bytes) - added by Jenn99 16 years ago.
0000380-moderation.php (6.0 KB) - added by Jenn99 16 years ago.

Download all attachments as: .zip

Change History (4)

#2 @anonymousbugger
16 years ago

Note that as of 11/15 CVS, this feature doesn't work in the first place (and it hasn't for a while). I wish it did...

#3 @matt
16 years ago

  • fixed_in_version set to 1.3
  • Owner changed from anonymous to matt
  • Resolution changed from 10 to 20
  • Status changed from new to closed


Note: See TracTickets for help on using tickets.