Revision: 48483
Author:   aaron
Date:     2009-03-17 09:18:46 +0000 (Tue, 17 Mar 2009)

Log Message:
-----------
add queue comment

Modified Paths:
--------------
    trunk/extensions/Renameuser/SpecialRenameuser_body.php

Modified: trunk/extensions/Renameuser/SpecialRenameuser_body.php
===================================================================
--- trunk/extensions/Renameuser/SpecialRenameuser_body.php      2009-03-17 
09:02:17 UTC (rev 48482)
+++ trunk/extensions/Renameuser/SpecialRenameuser_body.php      2009-03-17 
09:18:46 UTC (rev 48483)
@@ -455,6 +455,12 @@
                        );
                }
                // Construct jobqueue updates...
+               // FIXME: if a bureaucrat renames a user in error, he/she
+               // must be careful to wait until the rename finishes before
+               // renaming back. This is due to the fact the the job "queue"
+               // is not really FIFO, so we might end up with a bunch of edits
+               // randomly mixed between the two new names. Some sort of rename
+               // lock might be in order...
                foreach( $this->tablesJob as $table => $params ) {
                        $userTextC = $params[0]; // some *_user_text column
                        $userIDC = $params[1]; // some *_user column



_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to