Last modified: 2011-03-17 19:33:40 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 T28268, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 26268 - Make TopFiveReviewers configurable
Make TopFiveReviewers configurable
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
FlaggedRevs (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 25775 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-12-06 18:26 UTC by Umherirrender
Modified: 2011-03-17 19:33 UTC (History)
5 users (show)

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


Attachments

Description Umherirrender 2010-12-06 18:26:33 UTC
It is possible to make the TopFiveReviewers on Special:ValidationStatistics configurable? When there is a config it is possible to increase or decrease the numbers auf users. Maybe allow a value less or equal to 0 to disable this list and do not show the relevant section on ValidationStatistics.

Thanks.
Comment 1 Aaron Schulz 2010-12-06 20:17:27 UTC
*** Bug 25775 has been marked as a duplicate of this bug. ***
Comment 2 Rob Lanphier 2010-12-06 21:12:45 UTC
Save for after the Pending Changes fork
Comment 3 MZMcBride 2010-12-07 06:30:10 UTC
(In reply to comment #2)
> Save for after the Pending Changes fork

There's going to be a Pending Changes fork?
Comment 4 Chad H. 2010-12-07 12:27:16 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > Save for after the Pending Changes fork
> 
> There's going to be a Pending Changes fork?

Yes, and it was mentioned back over the summer (I'm pretty sure I've talked about it on IRC before).

FlaggedRevs is really two extensions: traditional FlaggedRevs and Pending Changes. A good portion of the code is disjoint. From a code maintenance perspective, it makes the most sense in the long run to split off the Pending Changes feature into its own extension.
Comment 5 Rob Lanphier 2010-12-07 17:56:24 UTC
(In reply to comment #3)
> There's going to be a Pending Changes fork?

Latest version of the roadmap talks about this some more:
http://www.mediawiki.org/wiki/Pending_Changes_enwiki_trial/Roadmap

The fork is contingent on Pending Changes being kept on enwiki.  If English Wikipedia drops it, we'll probably drop development on the configuration and focus any dev activity on the traditional FlaggedRevs config.
Comment 6 Aaron Schulz 2011-02-25 10:37:24 UTC
(In reply to comment #0)
> It is possible to make the TopFiveReviewers on Special:ValidationStatistics
> configurable? When there is a config it is possible to increase or decrease the
> numbers auf users. Maybe allow a value less or equal to 0 to disable this list
> and do not show the relevant section on ValidationStatistics.
> 
> Thanks.

Is the motivation here to increase or decrease/disable it? Maybe it could be hard coded to 10 (or something) if it's the former (increase)?

Perhaps a CSS class could be used to allow for disabling it without needed more config cruft.
Comment 7 p858snake 2011-02-25 10:43:01 UTC
(In reply to comment #6)
> Perhaps a CSS class could be used to allow for disabling it without needed more
> config cruft.
CSS to "disable" stuff is bad, it can easily be reshown then, if they want it disabled, it should be able to be done.
Comment 8 Aaron Schulz 2011-02-25 10:51:10 UTC
I lean towards either keeping it fixed or getting rid of it. I'm tired of all these silly config vars running around.
Comment 9 Aaron Schulz 2011-03-17 19:33:40 UTC
Done in r84198.

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


Navigation
Links