Opened 15 years ago
Closed 15 years ago
#11432 closed defect (bug) (fixed)
Dashboard Spam link uses wrong action
Reported by: | caesarsgrunt | Owned by: | |
---|---|---|---|
Milestone: | 2.9 | Priority: | normal |
Severity: | normal | Version: | 2.9 |
Component: | Trash | Keywords: | has-patch |
Focuses: | Cc: |
Description
The Spam link on the dashboard still uses action=deletecomment&dt=spam
, rather than the new action=spamcomment
.
Although this does spam the comment, it means that the previous status isn't stored and so it isn't correctly unspammed.
Attachments (2)
Change History (5)
@
15 years ago
Better patch (also combines deletecomment into the same function as all the other delete actions)
#1
@
15 years ago
- Keywords has-patch added; needs-patch removed
The first patch just fixes the bug by changing the spam link to use the spamcomment
action.
The second patch does the same, but also moves the deletecomment
action into the same function as trashcomment
, untrashcomment
, spamcomment
, and unspamcomment
, since almost all code for it was duplicated. I should have merged this when I merged the others before....
Note: See
TracTickets for help on using
tickets.
Simple fix (changes action in spam link)