Extension talk:FlaggedRevs/archive 1

Initial thoughts
Here are my initial thoughts.

The contribution list for a user could include the reviews done.

The text of the box on a non-reviewed page when there is a reviewed revision could be:

This version has not been reviewed. The latest reviewed version is available here.

There could also be no mention of any review notes on a non-reviewed page.

When I log in, I get the latest version. I would normally like to see the reviewed version. Perhaps there should be a user preference option to say which version is initially displayed.


 * Thats on the wishlist - see the specs.txt file --Jhb 19:00, 2 April 2007 (UTC)

I protected the main page but there was no message indicating that it was protected.

Also, after protection, it would not be possible for many users to edit the page which goes against the text

The current revision is editable

These comments are pretty cosmetic.

I found no bugs. Well done. Eiler7 21:58, 30 March 2007 (UTC)

Hmm. Just found a bug.

If you visit the main page, you get one version. Click on Permanent Link you get another. Eiler7 22:42, 30 March 2007 (UTC)
 * This was fixed by a small change to trunk but brion reverted it. Ideally, there will be a Special:Stableversions page with an oldid= param to parse it as a stable page, as even giving the right perma-link Id can still be a tad misleading (old rev, current images and templates). Voice of All 00:36, 31 March 2007 (UTC)

I have noticed something else. If you do a change when not logged in, you arrived at the reviewed page, not the version you just created. This is unnatural. In my view, neither of these issues should hold up deployment on the English wikipedia.

I wonder if we need a requirements document. Certainly, the current interface meets my main requirement. Namely, it allows people to avoid seeing a vandalised version of a page, which happened with the Tony Blair page in an external review. See this link  Eiler7 18:57, 31 March 2007 (UTC)
 * When editing a page that has a stable version, the tag should probably be clear about this and suggesting the use of preview. Voice of All 23:45, 1 April 2007 (UTC)

Another thought. If reviewed versions use the current images, then this might not be a good thing. Perhaps reviewing a page should cascade to reviewing all images and templates which have no reviewed version. Eiler7 22:29, 31 March 2007 (UTC)
 * Reviewed pages clone the images and place them in the /stable directory.Voice of All 23:02, 31 March 2007 (UTC)
 * OK, that is no longer the case, they can be called directly. Aaron 19:14, 24 June 2007 (UTC)

BLP concerns
Will the threshold for displaying versions (i.e., non-vandalized, checked for accuracy, etc.) be a universal namespace setting, or can it be set for individual articles and/or classes of articles? It will be useful if this feature could support a "living person" tag for biographies of living people, and all articles with that tag have a higher threshold for public display than simply "non-vandalized". Will it be able to work that way? -- ragesoss


 * I understand what you are saying. I suggest that BLP pages have a different definition of vandalised. So, if the George Bush page were changed to say that he was involved in the Kennedy assassination without proof, that would count as vandalism in my book. Eiler7 11:24, 1 April 2007 (UTC)


 * The main issue is that it may be desirable to have a more select group of editors who can review BsLP (and things like the main page, if it becomes open to editing) than those who can review other articles. --ragesoss
 * I've set up two review groups, editor and review. By configuring what tags they can set to what level and what tags/levels are needed for page overriding (by the latest stable version), this can be dealt with. Voice of All 23:44, 1 April 2007 (UTC)

No reviewed revisions
I just created a page Eric Clapton and reviewed it. However, clicking on the page now, we find it says "There are no reviewed revisions of this page". This is a bad bug. Eiler7 12:14, 1 April 2007 (UTC)
 * Not sure why that is, but that site is still running an older version. Voice of All 17:04, 1 April 2007 (UTC)

Latest revision
I have tried the test wiki again. The Clapton page now displays with correct information. However, I think the wiki should show the reviewed version by default. Currently, it does not. The same will, I hope, apply to en.wikipedia.org when the changes are applied there. The logic is that the average viewer and the researcher will both want to do a non-vandalised version. So, this will give a better impression of the quality available through the wiki process.Eiler7 20:11, 5 April 2007 (UTC)
 * That only happens if the flagged rev meets "stable" critera. Depending on how it is set, "draft level" may not be good enough. This useful if one revision is featured, and one newer one is somehow "draft level", in which case we would likely want the feauted one.Voice of All 06:32, 6 April 2007 (UTC)

Well, currently the Clapton page shows the latest revision when there is a draft revision available. I think that the draft revision (read unvandalised) should be shown. Is that a setting change on the Clapton page or is this a bug? Eiler7 13:17, 6 April 2007 (UTC)

Baach.de test wiki
I have just been playing around with the test wiki again and it seems that the Main Page link in the navigation box takes one to the page Wikiversity:Main_Page. Was this intended? Eiler7 20:30, 18 April 2007 (UTC)
 * That is an unrelated bug. Aaron 07:39, 16 June 2007 (UTC)

Multiple flagged revisions
I haven't had the chance to play around with the extension, but I have several questions:


 * 1) Will the extension allow for several types of revision flags? (E.g. "vandalized", "reviewed", "assessed", "featured", etc...)
 * Yes. Voice of All 06:34, 4 May 2007 (UTC)
 * 1) Will the extension allow flag sub-types (e.g. "assessed: quality: B-Class", "assessed: importance: Top", "featured: FAC: promoted", etc.)
 * No pretty way to do this. Voice of All 06:34, 4 May 2007 (UTC)
 * How about a couple of null fields on the table? Titoxd (?!?) 07:13, 4 May 2007 (UTC)
 * 1) Is this essentially stable versions?
 * I guess. Voice of All 06:34, 4 May 2007 (UTC)
 * 1) Will the extension produce a way to query articles with certain properties? (e.g. w:en:WP:1.0/I)
 * The stable revisions and their overall level (stable,quality,featured) can be found at Special:Stableversions. Voice of All 06:34, 4 May 2007 (UTC)
 * 1) Will revisions be able to be tagged retroactively?
 * Yes. Voice of All 06:34, 4 May 2007 (UTC)
 * 1) Will revisions be able to hold more than one tag simultaneously?
 * Yes. Voice of All 06:34, 4 May 2007 (UTC)
 * 1) Will pages be able to hold more than one revision tagged in a particular way?
 * Yes. Voice of All 06:34, 4 May 2007 (UTC)

Titoxd (?!?) 03:16, 3 May 2007 (UTC)

Changes submitted to trunk?
I think this is a great feature! Just 2 questions: --Y.combarnous 16:48, 23 May 2007 (UTC)
 * 1) When do you think the changes will be submitted to trunk, so that this extension can be used without using a forked codebase ?
 * 2) Are there chances this feature makes it for MW 1.11 ?
 * Another dev will merge in FileRepo work, I'll reconcile with that and submit the revision delete code along with the parser hook stuff needed for this extension to work. This will definitetly be done by the time 1.11 is out. Aaron 02:14, 25 May 2007 (UTC)


 * Great news, thanks for the quick answer.--Y.combarnous 07:25, 25 May 2007 (UTC)

Questions

 * 1) What exactly are the differences between "editor" and "reviewer" privileges? One can be given automatically and one needs a bureaucrat so I assume they are significant differences.
 * Reviewers are more trusted and can rate revisions higher. Aaron 00:32, 1 June 2007 (UTC)
 * So in a really basic sense, an editor could rate a revision as "stable" but only a reviewer could rate one as "featured"? Mr.Z-man 06:48, 1 June 2007 (UTC)
 * 1) What exactly are all the "tagging levels"?
 * Customized, by default, there are four. I'll probably get an up to date demo site up soon. Aaron 00:32, 1 June 2007 (UTC)
 * 1) Which version will be the version seen by search engines? Mr.Z-man 22:29, 31 May 2007 (UTC)
 * The stable version. Aaron 00:32, 1 June 2007 (UTC)

Templates and images
How would one go about updating a template or image transcluded into many articles? Would the new version of the template have to be approved for each article? Audacity 02:14, 1 June 2007 (UTC)
 * Currently, each page would have to be re-reviewed. Updating images/templates in masse could case problems such as the template params/format change and break some pages or the new image version makes the caption no longer make sense (say it says "picture of east Munich, and the new version if from the west). Nevertheless, the way the schema is set up, it is possible to run an UPDATE query WHERE tmp_rev_id=x and replace it with Y, same with images, a feature that could perhaps be added later. Aaron 16:49, 1 June 2007 (UTC)
 * It would require an extra INDEX on the tables. Aaron 03:42, 4 June 2007 (UTC)

Error with namespaces

 * Another question: I would like to cut out the links in description above in the print version, because this makes it less good viewable. But I would like to have the revision number there. Will you change this? --Wissenslogistiker 15:42, 5 June 2007 (UTC)
 * Hmm...what are you using it for? What wiki is this and where is it? . Aaron 16:19, 5 June 2007 (UTC)
 * I've removed tags from printable versions altogether. Aaron 16:37, 5 June 2007 (UTC)
 * Thank you, this is much better. I use wiki as organisational memory system and for creating docs for the company where i am working in. Thatswhy approved article versions have a high interest. ;-) --Wissenslogistiker 08:01, 6 June 2007 (UTC)

First I have to say that this is great stuff and an absolutly 'must have extension'. :-) Then I think that I found a bug:
 * When you have an article in the standard namespace - at my wiki it is called 'Meta:Test' and it is the same as - the rating fields are not shown below the article (error 1).
 * When you have an article with no namespace the rating fields are shown. If you set it to a higher review level all is fine, but if you set it not for publishing (=lowest review level) I get: "Internal error. This action can not be used on this page.(error 2)". If you first set it to a higher and later to the lowest I get the following error message: "Fatal error: Call to a member function getNamespace on a non-object in meta\includes\WatchedItem.php on line 26. (error 3)"

The cause can probably be that I have a wiki which is viewable only by members. --Wissenslogistiker 13:09, 5 June 2007 (UTC)
 * Is the meta namespace in $wgContentNamespaces? Aaron 15:10, 5 June 2007 (UTC)
 * OK, the getNamespace error was fixed. Aaron 15:15, 5 June 2007 (UTC)
 * I reloaded Extension via SVN, did update.php. For error 1: Yes, meta namespace is as default in Defaultsettings.php part of it: '$wgContentNamespaces = array( NS_MAIN );' and '$wgSitename        = "Meta";'. I did not change $wgContentNamespaces in Localsettings.php. The other ones (error 1 and 2) are solved, but I have now a different behavior: When I create the page, then approve this version, make an edit and set the prove to the lowest level it says "Internal error. This action can not be used on this page."  --Wissenslogistiker 15:57, 5 June 2007 (UTC)
 * I can't repeat any of these errors. If you reviewed the first revision, then made a new one, you can't "unreview" the new one, only the one that was reviewed. If you try to unreview a revision that was not reviewed, it gives the error message. Aaron 22:27, 5 June 2007 (UTC)
 * I think this this ok, but still error 1 is there. I found out that it shows register 'project page' in Meta:Test and register 'article' in Test (=the article without namespace). Perhaps knowing these differences helps? In Special:Allpages I can select 'Meta' namespace and '(Main)' so i think namespace has been created correctly. In namespace 'Help' it does not work either. --Wissenslogistiker 08:01, 6 June 2007 (UTC)
 * Do you have any news about error 1? --Wissenslogistiker 11:31, 11 June 2007 (UTC)
 * If "meta" and "(main)" are options, then Meta is not the mainspace, and probably is not in the content namespaces array. Aaron 05:03, 13 June 2007 (UTC)
 * I understood now what you meant and set $wgContentNamespaces = array('Meta'); in LocalSettings.php. But the problem stays the same. I uploaded a screenshot of article Test without Namespace, Special:Allpages and article Meta:Test here. Isn't it possible to set a flag for project namespaces to have this feature too? Or do you have another suggestion what I could try to get this to work? --Wissenslogistiker 12:37, 14 June 2007 (UTC)
 * Haha, $wgContentNamespaces should be an array of numbers. For example NS_MAIN is a constant variable with value '0'. You need the number that corresponds to Meta. Aaron 07:37, 16 June 2007 (UTC)
 * Cool, that worked. Thank you for your assistance. Perhaps you can build a special page to manage the namespaces for revision in the future. :-) --Wissenslogistiker 10:44, 21 June 2007 (UTC)

Blocked users
If a user with editor rights is blocked, would editor rights be automatically stripped or would this have to be done as a separate action? Mr.Z-man 19:38, 8 June 2007 (UTC)
 * It would have to be manually removed, however, blocked users cannot review revisions while they are blocked, so their flag is useless while blocked. Aaron 23:17, 8 June 2007 (UTC)

1.11
Where can I get MW 1.11 to make this work? 66.31.209.239 00:25, 12 June 2007 (UTC)
 * You will have to use the current development version from SVN, read here: Download from SVN --Wissenslogistiker 09:16, 12 June 2007 (UTC)

Test wiki
Where is the test wiki for this feature? Thanks in advance for a link. Dovi 06:19, 14 June 2007 (UTC)
 * There is one here. Aaron 07:38, 16 June 2007 (UTC)

List of reviewed pages with changes
If articles have been reviewed but there are newer and unreviewed revisions, is there a way to list these articles - similar to the page Special:Unreviewedpages? --217.5.218.98 13:42, 9 July 2007 (UTC)
 * Note that by default, patrolled edits are enabled, and reviewed revision count as patrolled, so if you are watching the pages, you can get a good idea of this information (a watch checkbox appears on the review form). Aaron 11:47, 10 July 2007 (UTC)

Doesn't work
Hi,

Error message :

Warning: require(/home/xxx/xxx/extensions/FlaggedRevs/../ExtensionFunctions.php) [function.require]: failed to open stream: No such file or directory in /home/xxx/xxx/extensions/FlaggedRevs/FlaggedRevs.php on line 13

Fatal error: require [function.require]: Failed opening required '/home/xxx/xxx/extensions/FlaggedRevs/../ExtensionFunctions.php' (include_path='/home/xxx/xxx:/home/xxx/xxx/includes:/home/xxx/xxx/languages:.:/usr/lib/php:/usr/local/lib/php') in /home/xxx/xxx/extensions/FlaggedRevs/FlaggedRevs.php on line 13

And it could be good to explain for beginners how to Run maintenance/archives/populateSha1.php.

Thanks. 81.53.23.145 13:58, 2 September 2007 (UTC)


 * Download ExtensionFunctions.php from SVN and place it under extensions/ - simple. Running maintenance/archives/populateSha1.php is just like that: you open your SSH client (like PuTTY), type in  and then type  . Very simple. -- Sayuri  14:06, 2 September 2007 (UTC)
 * Thanks. But I'm sorry to tell that for beginners it's very hard to find the good informations. That's not simple at all... 81.53.23.145 14:27, 2 September 2007 (UTC)
 * Lastly, it is still in testing, so things may change. I wouldn't yet download for production use. But the instructions above are fairly standard for most extensions around here. Aaron 14:35, 2 September 2007 (UTC)
 * Ok, Works fine, thanks for your help ! 81.53.23.145 14:44, 2 September 2007 (UTC)

Any news on implementation?
Now that Wikimedia 1.11 is out, is this on the way to actually being implemented at the Wikimedia projects? Any news or updates on this? Dovi 11:29, 11 September 2007 (UTC)
 * Brion may do code review this week and put it up on a hidden (to be publicized site). Aaron 12:51, 12 September 2007 (UTC)


 * I see that this extension is still listed as experimental. I'm about to attempt implementation for an internal wiki were we need to review all changes made to articles before they go live for versioning purposes.  Is this now considered stable, or what will it take to go from experimental to stable?  Would you recommend implementing it as an extension, or once with Wiki code goes live?  And if the second, any idea when that will be?  Thanks in advance.


 * -Marc 10/02/07 06:35PM EST.

Showing empty pages insteat of not reviewed page
I upgraded my wiki from 1.9.3 to 1.11, which means that all my articles ain't review yet. Now, when I load an article, a message is shown, that the article ain't reviewed yet but is shown anyway. I wanna hide the content of any not reviewed article exept for the editors or reviewers. --213.214.18.64 06:06, 28 September 2007 (UTC)
 * Why? Either your editors are trusted, and thus non-reviewed content should be decent, or they are not all trusted, in which case they need review...however how can you submit edits to a page you can't see? Aaron 14:05, 28 September 2007 (UTC)

Adding a parameter to only show current version of an article to editor/reviewer classes
We have a private wiki, so all users have to log-in to get into it. We would like them to only see the current stable version of an article, and that the editor/reviewer see the latest revision. Would it be possible to add a parameter to do so?

For instance a permission could do the trick:

barns 14:05, 02 October 2007 (UTC)


 * It's funny you should mention this because this is exactly what I need as well. For the exact same purpose too.  I hope this is possible.  BTW, great extension.  This is exactly the type of thing I needed.  However, rather than go through 3 different flags, I just need an "Approved" and the ability to add comments.  What would I remove with disturbing the whole extension.  - Marc 10/02/07  07:43PM EST.
 * By default, logged in users always see the current revision, whereas outsiders see the stable one. The number of tags can be reduced to just one, like "accuracy". Aaron 00:52, 3 October 2007 (UTC)
 * Actually, I will change the code a tad so that the tag array can be set to "array" (blank). Aaron 01:01, 3 October 2007 (UTC)