Last modified: 2012-05-05 18:44:55 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 T22310, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 20310 - Invalid values of added_lines, removed_lines, edit_diff
Invalid values of added_lines, removed_lines, edit_diff
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
AbuseFilter (Other open bugs)
unspecified
All All
: Low major with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://pl.wikipedia.org/w/index.php?t...
:
: 20851 23304 26433 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-19 11:54 UTC by Beau
Modified: 2012-05-05 18:44 UTC (History)
7 users (show)

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


Attachments

Description Beau 2009-08-19 11:54:28 UTC
There is something wrong with diff function for abusefilter. In the provided URL bot added one interwiki link (one line), but from abusefilter point of view it looks like the whole content of the page was replaced.
Comment 1 Andrew Garrett 2009-09-18 09:54:41 UTC
Looks fine to me.
Comment 2 Beau 2009-09-18 10:32:15 UTC
The diff shown at the top of the page and the value of edit_diff do not match.
Comment 3 AlexSm 2009-10-07 15:30:54 UTC
Sometimes added_lines and removed_lines both contain one unchanged line from above. (As a result, edit_diff variable is wrong as well.) 

Example: [[Special:AbuseFilter/examine/327603336]]; diff: http://en.wikipedia.org/w/index.php?diff=318457689

This actually happens quite a lot (at least on ru.wp), leading to false positives and forcing me to make extra checks on old_wikitext. Bumping severity to major.

I don't have access to URL in beau's post, but I suspect it's the same behaviour.
Comment 4 AlexSm 2009-10-07 16:28:42 UTC
Update: looks like the unchanged line above is always included into added_lines and removed_lines whenever user makes changes to the end of the page. 

Although technically AbuseFilter might be right (linebreak \n is indeed added to or removed from the "unchanged line"), this is clearly not the behaviour expected by filters maintainers. 

bug 20851 looks similar to this one.
Comment 5 seth 2011-07-25 19:48:56 UTC
seems to be the same as bug 19716
Comment 6 Beau 2011-08-20 20:12:12 UTC

*** This bug has been marked as a duplicate of bug 19716 ***
Comment 7 Beau 2012-04-13 15:17:51 UTC
Bug 19716 is about a different issue.
Comment 8 Beau 2012-04-13 15:18:46 UTC
*** Bug 20851 has been marked as a duplicate of this bug. ***
Comment 9 Beau 2012-04-13 15:19:44 UTC
*** Bug 26433 has been marked as a duplicate of this bug. ***
Comment 10 Beau 2012-04-29 09:04:54 UTC
I have submitted Gerrit change #6090 for review.
Comment 11 db [inactive,noenotif] 2012-05-05 18:44:55 UTC
*** Bug 23304 has been marked as a duplicate of this bug. ***

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


Navigation
Links