Last modified: 2013-04-22 16:16:17 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 T41942, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 39942 - Remove Patrol from Fi-wiki
Remove Patrol from Fi-wiki
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Site requests (Other open bugs)
unspecified
All All
: Unprioritized enhancement (vote)
: ---
Assigned To: Dereckson
: shell
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-03 14:08 UTC by Pxos
Modified: 2013-04-22 16:16 UTC (History)
9 users (show)

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


Attachments

Description Pxos 2012-09-03 14:08:02 UTC
We at fi.wikipedia would like to get rid of the old Patrol-feature altogether as we have implemented the Flagged Revisions. Link to the discussion where this received unanimous support can be found here: 

http://fi.wikipedia.org/wiki/Wikipedia:Kahvihuone_%28k%C3%A4yt%C3%A4nn%C3%B6t%29/Arkisto_68#Mielipidetiedustelu_muutosten_tarkastus_-lis.C3.A4osasta_luopumisesta

The question was as follows: If the FlaggedRevisions are fully implemented on fi.wiki, can we get rid of the old patrolled edits? 15 votes supporting. The FR was made a permanent feature on 31 May 2012 so the condition has been met.

Pxos
Comment 1 Alex Monk 2012-09-03 14:13:48 UTC
+shell as a shell user will be needed to deploy a configuration change.
Comment 2 Niklas Laxström 2012-09-03 14:19:07 UTC
I can confirm the vote but looks like you are confused on what you are actually voting about.

I gather you are talking about recent changes patrolling (including new page patrolling?) governed by wgUseRCPatrol variable. This is core feature and not an extension as stated in the vote.

This line gives me a nostalgic feeling:
'fiwiki' => true, // By request of Nikerabbit in #wikimedia-tech on 2005-12-20 -ævar (source http://noc.wikimedia.org/conf/InitialiseSettings.php.txt ).
Comment 3 Pxos 2012-09-03 14:34:21 UTC
The vote may have called the feature an extension, but the point is (as Nikerabbit is able to read in Finnish) that the community does not need the core feature at all.

If the core feature cannot be disabled, an alternative proposal (mine only) is to enable sysops to give or remove the "patrol" and "autopatrol" bits so that all patroller-rights could be removed from all users. This will lead to the same result as far as I know (or do not know).
Comment 4 Dereckson 2012-09-06 20:57:16 UTC
Thank you for the clarification.

Taking this bug.
Comment 5 Dereckson 2012-09-06 21:01:37 UTC
Gerrit change #22999
Comment 6 Nemo 2012-09-06 21:18:32 UTC
Actually I don't see any clarification, did the survey ask removal of RC patrol, new pages patrol or both? This question wasn't answered.
Comment 7 Dereckson 2012-09-06 21:22:50 UTC
I've blocked the Gerrit change to mitigate the shell policy.
Comment 8 Dereckson 2012-09-07 07:09:51 UTC
I tried to read the consultation and see "luovuttaisiin (...) tarkastus-lisäosan (Patrolled Edits)". This explicitely requests the removal of the ''patrolled edits'' feature (they declare it as extension).

In a second paragraph, they describe the feature and the rationale.

No element in the consultation refered to RC patrol or to new pages patrol. We could see:
* the RC patrol feature is a duplicate of flagged revs ;
* pxos analyses it at every patrol feature removal ;
* the bug #1405 split the patrol into RC / new pages, before this bug, they were enabled by the same setting ;
* 4 wikis disabled the new pages patrol: hu. (bug #19241), sq. (bug #25822), ru. (bug #31650) and uk. (bug #33273).

I so think the conclusion is the consultation technical interpretation is they wish to remove both flags.
Comment 9 Dereckson 2012-09-07 07:12:40 UTC
(In reply to comment #8)
> * 4 wikis disabled the new pages patrol: hu. (bug #19241), sq. (bug #25822),
> ru. (bug #31650) and uk. (bug #33273).
hu., sq., ru. (see bug #15525 for the FlaggedRevs reference) and uk. all requested to install flagged revisions or to disable the new pages patrolling with explicit reference to flagged revisions.
Comment 10 kimmo.virtanen 2012-09-08 08:55:25 UTC
Hi, i think that it is ok to remove both. New pages patrolling functionality is now duplicated by flagged revisions so it is not needed.
Comment 11 kimmo.virtanen 2012-09-08 08:59:31 UTC
For clarity my Wikipedia nick is Zache and wrote the query for enabling the flagged revision at wiki to bugzilla. I started also the proposal for the disabling the new page patrolling which was linked by pxos in the first comment.
Comment 12 Dereckson 2012-09-08 19:08:26 UTC
Thank you for your clarification.

Keywords update: -shellpolicy +shell +patch-in-gerrit
Comment 13 Antoine "hashar" Musso (WMF) 2012-09-18 19:45:59 UTC
Deployed in live site. Thanks Dereckson!

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


Navigation
Links