Update comment in Category.php

Per code review I gave on r72547.  I'm actually not sure how to rewrite
this so it works with all DBs -- a replace() variant that called
nextSequenceValue() if the row didn't already exist, and reused the
existing id otherwise, would perhaps be ideal.  (Speaking of which,
replace() needs to be properly factored out, with the DBMS-independent
version in DatabaseBase and the MySQL one in DatabaseMysql.)
This commit is contained in:
Aryeh Gregor 2010-09-08 18:11:28 +00:00
parent 7fb452c2a7
commit 27d1e4ae23

View file

@ -256,8 +256,11 @@ class Category {
$dbw = wfGetDB( DB_MASTER );
$dbw->begin();
# Let's be sure that the row exists in the table. We don't need to
# do this if we got the row from the table in initialization!
# Insert the row if it doesn't exist yet (e.g., this is being run via
# update.php from a pre-1.16 schema). TODO: This will cause lots and
# lots of gaps on some non-MySQL DBMSes if you run populateCategory.php
# repeatedly. Plus it's an extra query that's unneeded almost all the
# time. This should be rewritten somehow, probably.
$seqVal = $dbw->nextSequenceValue( 'category_cat_id_seq' );
$dbw->insert(
'category',