Author: Thorncrag
Description:
When a user saves and triggers a 'warn, disallow' filter and when system message is specified for that filter, the selected message is displayed, but if the user clicks save again, it then switches and displays the default abuse filter message. Clicking save again causes it to switch back to the selected message, and so forth.
I've tested this on both of my installations, unfortunately I don't have another I can test it on at this time.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Allow selecting custom disallow message | mediawiki/extensions/AbuseFilter | master | +200 -84 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Daimona | T27086 Warning alternates between default and specified warning | |||
| Invalid | None | T207456 Warning alternates between default and specified warning do not work |
needs no testing, still happening. Toggles between warning and disallow state, -> between (custom) warnmessage and default disallow msg. Thing is, only for warning an admin can specify a custom system message.
I'm currently working on a new feature, which would solve this usecase/problem. soon on gerrit(TM) ;-)
Change 164987 had a related patch set uploaded by Se4598:
Allow selecting custom disallow message
Change 164987 had a related patch set uploaded (by Qgil):
Allow selecting custom disallow message
So I didn't find this task and just filed another duplicate (T138435), with a duplicate patch (https://gerrit.wikimedia.org/r/295596). It seems pretty similar to the one here (https://gerrit.wikimedia.org/r/164987), but with noticeably fewer changes – I'm not sure if I missed things, or if @Se4598's patch has additional changes apart from fixing this bug.
@Iniquity Yes, there's a patch under review since 2014 which I picked up some time ago.
Since I am not active since a long time ago but this is still assigned to me I reassign it to Daimona, who is more active and has taken over the original patch.
Hope you finish it, @Daimona :)
@Se4598 Sure, I just converted it to use OOUI and made some tests to ensure it works :-)
Change 164987 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Allow selecting custom disallow message
@matmarex Thanks :-) As for Tech News, maybe
Abuse filters that prevent edits can now use a custom error message, instead of using a predefined one.
could be even better, and a bit more realistic.
Thanks. I simplified the language a bit further for en-1 and en-2 speakers:
When you create an abuse filter that prevents edits you can now write a specific error message for it. This is instead of using the already existing error message.
OK?
@Johan I'd say yes. The only point we could improve is the last sentence, to clearly state that, before this patch, there was a single fixed message for every filter (abusefilter-disallowed).