From a263b7c9f05cbee2e8e53df017eafe04a073314b Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Sun, 24 Oct 2004 22:02:02 +0000 Subject: [PATCH] Let Database do the quoting of the 'group' table, so that prefixed tables will work. --- includes/Database.php | 3 +++ includes/DatabasePostgreSQL.php | 5 +++-- includes/Group.php | 12 ++++++------ includes/SpecialUserlevels.php | 3 ++- 4 files changed, 14 insertions(+), 9 deletions(-) diff --git a/includes/Database.php b/includes/Database.php index 0fcb4e9fa3..75440d64fb 100644 --- a/includes/Database.php +++ b/includes/Database.php @@ -951,6 +951,9 @@ class Database { if ( isset( $wgSharedDB ) && 'user' == $name ) { $name = $wgSharedDB . '.' . $name; } + if( $name == 'group' ) { + $name = '`' . $name . '`'; + } return $name; } diff --git a/includes/DatabasePostgreSQL.php b/includes/DatabasePostgreSQL.php index acaf7e2242..e0257612ba 100644 --- a/includes/DatabasePostgreSQL.php +++ b/includes/DatabasePostgreSQL.php @@ -247,9 +247,10 @@ class DatabasePgsql extends Database { # Now quote PG reserved keywords switch( $name ) { case 'user': - return '"user"'; case 'old': - return '"old"'; + case 'group': + return '"' . $name . '"'; + default: return $name; } diff --git a/includes/Group.php b/includes/Group.php index 6c02ecd42b..e9d8da005f 100644 --- a/includes/Group.php +++ b/includes/Group.php @@ -49,7 +49,7 @@ class Group { if($this->id) { $dbr =& wfGetDB( DB_SLAVE ); - $r = $dbr->selectRow('`group`', array('group_id', 'group_name', 'group_description', 'group_rights'), array( 'group_id' => $this->id ), $fname ); + $r = $dbr->selectRow('group', array('group_id', 'group_name', 'group_description', 'group_rights'), array( 'group_id' => $this->id ), $fname ); $this->id = $r->group_id; $this->name = $r->group_name; $this->description = $r->group_description; @@ -57,7 +57,7 @@ class Group { $this->dataLoaded = true; } else { $dbr =& wfGetDB( DB_SLAVE ); - $r = $dbr->selectRow('`group`', array('group_id', 'group_name', 'group_description', 'group_rights'), array( 'group_name' => $this->name ), $fname ); + $r = $dbr->selectRow('group', array('group_id', 'group_name', 'group_description', 'group_rights'), array( 'group_name' => $this->name ), $fname ); $this->id = $r->group_id; $this->name = $r->group_name; $this->description = $r->group_description; @@ -70,7 +70,7 @@ class Group { function addToDatabase() { $fname = 'Group::addToDatabase'; $dbw =& wfGetDB( DB_MASTER ); - $dbw->insert( '`group`', + $dbw->insert( 'group', array( 'group_name' => $this->name, 'group_description' => $this->description, @@ -87,7 +87,7 @@ class Group { $dbw =& wfGetDB( DB_MASTER ); - $dbw->update( '`group`', + $dbw->update( 'group', array( /* SET */ 'group_name' => $this->name, 'group_description' => $this->description, @@ -131,7 +131,7 @@ class Group { function nameFromId($id) { $fname = 'Group::nameFromId'; $dbr =& wfGetDB( DB_SLAVE ); - $r = $dbr->selectRow( '`group`', array( 'group_name' ), array( 'group_id' => $id ), $fname ); + $r = $dbr->selectRow( 'group', array( 'group_name' ), array( 'group_id' => $id ), $fname ); if($r === false) { return ''; @@ -147,7 +147,7 @@ class Group { function idFromName($name) { $fname = 'Group::idFromName'; $dbr =& wfGetDB( DB_SLAVE ); - $r = $dbr->selectRow( '`group`', array( 'group_id' ), array( 'group_name' => $name ), $fname ); + $r = $dbr->selectRow( 'group', array( 'group_id' ), array( 'group_name' => $name ), $fname ); if($r === false) { return 0; diff --git a/includes/SpecialUserlevels.php b/includes/SpecialUserlevels.php index f68b2813af..c4e0a082d1 100644 --- a/includes/SpecialUserlevels.php +++ b/includes/SpecialUserlevels.php @@ -249,7 +249,8 @@ class UserlevelsForm extends HTMLForm { function HTMLSelectGroups($selectname, $selected=array(), $multiple=false, $size=6, $reverse=false) { $selectname = $this->mName.'-'.$selectname; $dbr =& wfGetDB( DB_SLAVE ); - $sql = 'SELECT group_id, group_name FROM `group`'; + $group = $dbr->tableName( 'group' ); + $sql = 'SELECT group_id, group_name FROM $group'; $res = $dbr->query($sql,'wfSpecialAdmin'); $out = wfMsg($selectname); -- 2.20.1