Last modified: 2014-03-10 12:31:28 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 T54968, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52968 - Increase $mChangeLimit from 100 to 500 in AbuseFilterViewTestBatch.php (often not matching any filters currently)
Increase $mChangeLimit from 100 to 500 in AbuseFilterViewTestBatch.php (often...
Status: RESOLVED WONTFIX
Product: MediaWiki extensions
Classification: Unclassified
AbuseFilter (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
https://github.com/wikimedia/mediawik...
: easy, performance
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-17 14:27 UTC by Helder
Modified: 2014-03-10 12:31 UTC (History)
4 users (show)

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


Attachments

Description Helder 2013-08-17 14:27:47 UTC
The current value is too low to be useful. Most of the time I see no recent changes matching a given filter in the last 100 edits.
Comment 1 Gerrit Notification Bot 2013-09-02 10:07:58 UTC
Change 82229 had a related patch set uploaded by Helder.wiki:
Increase $mChangeLimit from 100 to 500

https://gerrit.wikimedia.org/r/82229
Comment 2 Gerrit Notification Bot 2014-01-25 07:48:16 UTC
Change 109461 had a related patch set uploaded by Arav93:
Increased $mChangeLimit from 100 to 500

https://gerrit.wikimedia.org/r/109461
Comment 3 Gerrit Notification Bot 2014-03-10 01:42:48 UTC
Change 109461 abandoned by Hoo man:
Increased $mChangeLimit from 100 to 500

Reason:
Duplicate of: https://gerrit.wikimedia.org/r/82229

https://gerrit.wikimedia.org/r/109461
Comment 4 Marius Hoch 2014-03-10 01:45:04 UTC
I don't think we can test (much) more than 100 changes here (performance wise) without running into timeouts or going out of memory. It's already possible to make this fail with 100 revisions, that's why I wouldn't allow more with the current state of the extension.
Comment 5 Helder 2014-03-10 12:30:40 UTC
We have being using an AJAX workaround for the relevant part of this anyway...
https://pt.wikibooks.org/wiki/User:Helder.wiki/Tools/AbuseFilterRegressionTest.js
Comment 6 Gerrit Notification Bot 2014-03-10 12:31:28 UTC
Change 82229 abandoned by Helder.wiki:
Increase $mChangeLimit from 100 to 500

Reason:
We have being using an AJAX workaround for the relevant part of this anyway...
https://pt.wikibooks.org/wiki/User:Helder.wiki/Tools/AbuseFilterRegressionTest.js

https://gerrit.wikimedia.org/r/82229

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


Navigation
Links