Redirects are removed by undoing the edit that created them or restoring a previous edit.
This does not trigger the EditFilterHook in Wikidata!!!!
See also
T72715: AbuseFilter doesn't log creating redirects in mainspace
Redirects are removed by undoing the edit that created them or restoring a previous edit.
This does not trigger the EditFilterHook in Wikidata!!!!
See also
T72715: AbuseFilter doesn't log creating redirects in mainspace
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
Run EditFilterHooks on undo undoafter & restore | mediawiki/extensions/Wikibase | master | +58 -6 |
Change 209274 had a related patch set uploaded (by Addshore):
Run EditFilterHooks on undo undoafter & restore
I think this is actually intended: undo/restore should not be filtered like manual edits. They should not require a captcha, etc. A revert should be possible even if it violates an AbuseFilter rule.
That is probably true in core, I think Wikidata has a slightly different case.
The ONLY way that redirect can be removed is by undoing the edit that created the redirect or by restoring a previous version (Though the UI anyway).
Which is something that users have been doing, and it is near impossible to spot and stop.
Being able to filter edits like this would solve the issue.
Thoughts?
Hmm @matej_suchanek just said there are also issues with not logging merging of item. Is this related? It seems to have changed in September 2014. See the following links:
@Addshore: I think it's actually true especially for wikidata, where a revert *cannot* be edited.
It seems to me that using AbuseFilter for logging is kind of an abuse in and of itself. Perhaps this would be better? T73236: Automatically tag edits that make a redirect, or convert a redirected page to a normal page, or move a page across namespaces, etc.
@Lydia_Pintscher that looks like a separate issue and should likely have another task created for it.
Removing this from the sprint, since it seems to need more discussion. Maybe admin reverts should not trigger abuse filter, but undos should?...
I think the tagging solution in core would defiantly be the cleanest, I guess we just wait and see how long that takes!
Change 209274 abandoned by Addshore:
Run EditFilterHooks on undo undoafter & restore
Reason:
per comments etc.
This thing in core would be better