#23800 closed enhancement (fixed)
Filter Comment Blacklist, to delete duplicated terms
Reported by: | shadowhck | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Comments | Keywords: | good-first-bug has-patch |
Focuses: | Cc: |
Description
Hi,
In Comment Blacklist, when I save terms, duplicates are not eliminated.
Would be good if the function array_unique would remove these duplicate terms.
Attachments (1)
Change History (7)
#3
@
10 years ago
The solution might be similar to [22255] (explode()
, array_filter()
& array_unique()
, implode()
).
This should be applied to "Comment Moderation" list as well for consistency.
#4
@
10 years ago
- Keywords has-patch added; needs-patch removed
I added 23800.diff patch that should take care of this by explode()
, array_filter()
, array_unique()
and finally implode()
so pretty much the same way it was done in #22255.
Note: See
TracTickets for help on using
tickets.
The comment blacklist is currently saved as a string, not an array, so this would be somewhat more involved than might first appear.