Last modified: 2014-07-03 09:22:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T47272, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 45272 - Creating new section makes summary inconsistent between testing and real abuse filter application
Creating new section makes summary inconsistent between testing and real abus...
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
AbuseFilter (Other open bugs)
unspecified
All All
: Low normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-02-22 12:15 UTC by Mormegil
Modified: 2014-07-03 09:22 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Mormegil 2013-02-22 12:15:47 UTC
When an abuse filter is tested on past changes from RC, the edit summary (the “summary” AF variable) is filled with the contents of rc_comment (which is e.g. “/* Section name */ new section”). However, when the filter is really applied during an edit which adds a new section (using action=edit&section=new), the hook is called with $summary without any of the section syntax, e.g. just “Section name”.

1) The inconsistency is wrong by itself, because it makes filter development and testing quite difficult.
2) I would _really_ like to be able to recognize edits to sections (and creating new sections) because of spambot filtering. (E.g. not many humans create new userpages as action=edit&section=new, but many bots do exactly that.)

Not sure about the proper way to fix that, though. This seems more like a lower-level inconsistency in MediaWiki (the EditFilterMerged[Content] hook versus what gets stored in the RC table). Not sure if the hook behavior could change, even in such a small aspect...

On second thought, this might be really just a bug in EditPage: When you _edit_ an already existing section, the hook is called with the whole $summary = '/* Section name */ user comment', but when you create a _new_ section, the hook is called with just $summary = 'Section name'. This does not seem to be logical.
Comment 1 Danny B. 2013-02-22 15:35:55 UTC
More generic bug 41691 related.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links