From 8b9f1698fdec9d70a63b8a83962ccb2646fd00e3 Mon Sep 17 00:00:00 2001 From: latham <latham@ispconfig3> Date: Wed, 04 May 2011 15:55:24 -0400 Subject: [PATCH] Fix bugtracker issue 1610 --- interface/lib/classes/db_mysql.inc.php | 146 +++++++++++++++++++++++++----------------------- 1 files changed, 75 insertions(+), 71 deletions(-) diff --git a/interface/lib/classes/db_mysql.inc.php b/interface/lib/classes/db_mysql.inc.php index 6caf0c9..5e38e3b 100644 --- a/interface/lib/classes/db_mysql.inc.php +++ b/interface/lib/classes/db_mysql.inc.php @@ -1,14 +1,6 @@ <?php -/** - * mySQL Database class - * - * @author Till Brehm - * @copyright 2005, Till Brehm, projektfarm Gmbh - * @version 0.2 - * @package ISPConfig - */ /* -Copyright (c) 2005, Till Brehm, projektfarm Gmbh +Copyright (c) 2007, Till Brehm, projektfarm Gmbh All rights reserved. Redistribution and use in source and binary forms, with or without modification, @@ -35,17 +27,16 @@ EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -class db -{ +class db { private $dbHost = ''; // hostname of the MySQL server private $dbName = ''; // logical database name on that server private $dbUser = ''; // database authorized user private $dbPass = ''; // user's password - private $dbCharset = ""; // what charset comes and goes to mysql: utf8 / latin1 + private $dbCharset = ''; // what charset comes and goes to mysql: utf8 / latin1 private $linkId = 0; // last result of mysql_connect() private $queryId = 0; // last result of mysql_query() private $record = array(); // last record fetched - private $autoCommit = 1; // Autocommit Transactions + private $autoCommit = 1; // Autocommit Transactions private $currentRow; // current row number private $errorNumber = 0; // last error number public $errorMessage = ''; // last error message @@ -76,7 +67,7 @@ } public function connect() - { + { if($this->linkId == 0){ $this->linkId = mysql_connect($this->dbHost, $this->dbUser, $this->dbPass); if(!$this->linkId){ @@ -84,12 +75,13 @@ return false; } $this->queryId = @mysql_query('SET NAMES '.$this->dbCharset, $this->linkId); + $this->queryId = @mysql_query("SET character_set_results = '".$this->dbCharset."', character_set_client = '".$this->dbCharset."', character_set_connection = '".$this->dbCharset."', character_set_database = '".$this->dbCharset."', character_set_server = '".$this->dbCharset."'", $this->linkId); } return true; } public function query($queryString) - { + { if(!$this->connect()){ return false; } @@ -108,7 +100,7 @@ /** Returns all records as an array */ public function queryAllRecords($queryString) - { + { if(!$this->query($queryString)){ return false; } @@ -121,7 +113,7 @@ /** Returns one row as an array */ public function queryOneRecord($queryString) - { + { if(!$this->query($queryString) || $this->numRows() == 0){ return false; } @@ -130,8 +122,8 @@ /** Returns the next record as an array */ public function nextRecord() - { - $this->record = mysql_fetch_assoc($this->queryId); + { + $this->record = mysql_fetch_assoc($this->queryId); $this->updateError('DB::nextRecord()<br />mysql_fetch_array'); if(!$this->record || !is_array($this->record)){ return false; @@ -153,7 +145,7 @@ /** Returns the last mySQL insert_id() */ public function insertID() - { + { return mysql_insert_id($this->linkId); } @@ -224,24 +216,15 @@ } } */ - - //** Function to fill the datalog with a full differential record. - public function datalogSave($db_table, $action, $primary_field, $primary_id, $record_old, $record_new) { - global $app,$conf; - // Insert backticks only for incomplete table names. - if(stristr($db_table,'.')) { - $escape = ''; - } else { - $escape = '`'; - } - + public function diffrec($record_old, $record_new) { $diffrec_full = array(); $diff_num = 0; if(is_array($record_old) && count($record_old) > 0) { foreach($record_old as $key => $val) { - if(!isset($record_new[$key]) || $record_new[$key] != $val) { + // if(!isset($record_new[$key]) || $record_new[$key] != $val) { + if($record_new[$key] != $val) { // Record has changed $diffrec_full['old'][$key] = $val; $diffrec_full['new'][$key] = $record_new[$key]; @@ -265,15 +248,37 @@ } } + return array('diff_num' => $diff_num, 'diff_rec' => $diffrec_full); + + } + + //** Function to fill the datalog with a full differential record. + public function datalogSave($db_table, $action, $primary_field, $primary_id, $record_old, $record_new) { + global $app,$conf; + + // Insert backticks only for incomplete table names. + if(stristr($db_table,'.')) { + $escape = ''; + } else { + $escape = '`'; + } + + $tmp = $this->diffrec($record_old, $record_new); + $diffrec_full = $tmp['diff_rec']; + $diff_num = $tmp['diff_num']; + unset($tmp); + // Insert the server_id, if the record has a server_id - $server_id = (isset($record_old["server_id"]) && $record_old["server_id"] > 0)?$record_old["server_id"]:0; - if(isset($record_new["server_id"])) $server_id = $record_new["server_id"]; + $server_id = (isset($record_old['server_id']) && $record_old['server_id'] > 0)?$record_old['server_id']:0; + if(isset($record_new['server_id'])) $server_id = $record_new['server_id']; if($diff_num > 0) { + //print_r($diff_num); + //print_r($diffrec_full); $diffstr = $app->db->quote(serialize($diffrec_full)); - $username = $app->db->quote($_SESSION["s"]["user"]["username"]); - $dbidx = $primary_field.":".$primary_id; + $username = $app->db->quote($_SESSION['s']['user']['username']); + $dbidx = $primary_field.':'.$primary_id; if($action == 'INSERT') $action = 'i'; if($action == 'UPDATE') $action = 'u'; @@ -321,8 +326,8 @@ return true; } - - + + public function closeConn() { @@ -332,7 +337,7 @@ return true; } else { return false; } } - + public function freeResult($query) { if(mysql_free_result($query)) @@ -400,10 +405,10 @@ if(isset($col['option']) && $col['option'] == 'primary'){ $index .= 'PRIMARY KEY ('.$col['name'].'),'; } if(isset($col['option']) && $col['option'] == 'index'){ $index .= 'INDEX ('.$col['name'].'),'; } if(isset($col['option']) && $col['option'] == 'unique'){ $index .= 'UNIQUE ('.$col['name'].'),'; } - } + } $sql .= $index; $sql = substr($sql,0,-1); - $sql .= ')'; + $sql .= ')'; $this->query($sql); return true; } @@ -422,36 +427,36 @@ */ public function alterTable($table_name,$columns) { - $index = ''; - $sql = "ALTER TABLE $table_name "; - foreach($columns as $col){ + $index = ''; + $sql = "ALTER TABLE $table_name "; + foreach($columns as $col){ if($col['action'] == 'add'){ - $sql .= 'ADD '.$col['name'].' '.$this->mapType($col['type'], $col['typeValue']).' '; + $sql .= 'ADD '.$col['name'].' '.$this->mapType($col['type'],$col['typeValue']).' '; }elseif($col['action'] == 'alter') { $sql .= 'CHANGE '.$col['name'].' '.$col['name_new'].' '.$this->mapType($col['type'],$col['typeValue']).' '; }elseif($col['action'] == 'drop') { $sql .= 'DROP '.$col['name'].' '; } - if($col["action"] != 'drop') { - if($col["defaultValue"] != "") $sql .= "DEFAULT '".$col["defaultValue"]."' "; - if($col["notNull"] == true) { - $sql .= "NOT NULL "; + if($col['action'] != 'drop') { + if($col['defaultValue'] != '') $sql .= "DEFAULT '".$col['defaultValue']."' "; + if($col['notNull'] == true) { + $sql .= 'NOT NULL '; } else { - $sql .= "NULL "; + $sql .= 'NULL '; } - if($col["autoInc"] == true) $sql .= "auto_increment "; - $sql.= ","; + if($col['autoInc'] == true) $sql .= 'auto_increment '; + $sql.= ','; //* Index definitions if($col['option'] == 'primary') $index .= 'PRIMARY KEY ('.$col['name'].'),'; if($col['option'] == 'index') $index .= 'INDEX ('.$col['name'].'),'; if($col['option'] == 'unique') $index .= 'UNIQUE ('.$col['name'].'),'; } - } - $sql .= $index; - $sql = substr($sql,0,-1); - //die($sql); - $this->query($sql); - return true; + } + $sql .= $index; + $sql = substr($sql,0,-1); + //die($sql); + $this->query($sql); + return true; } public function dropTable($table_name) @@ -477,16 +482,15 @@ public function tableInfo($table_name) { - global $go_api,$go_info; //* Tabellenfelder einlesen ? - if($rows = $go_api->db->queryAllRecords("SHOW FIELDS FROM $table_name")){ + if($rows = $this->queryAllRecords('SHOW FIELDS FROM '.$table_name)){ foreach($rows as $row) { - $name = $row[0]; - $default = $row[4]; - $key = $row[3]; - $extra = $row[5]; - $isnull = $row[2]; - $type = $row[1]; + $name = $row['Field']; + $default = $row['Default']; + $key = $row['Key']; + $extra = $row['Extra']; + $isnull = $row['Null']; + $type = $row['Type']; $column = array('name' => $name, 'defaultValue' => $default); //$column["type"] = $type; @@ -507,9 +511,9 @@ $tmp_typeValue = explode('(',$type); $column['typeValue'] = substr($tmp_typeValue[1], 0, -1); } - if(stristr($type, 'text')) $metaType = 'text'; - if(stristr($type, 'double')) $metaType = 'double'; - if(stristr($type, 'blob')) $metaType = 'blob'; + if(stristr($type,'text')) $metaType = 'text'; + if(stristr($type,'double')) $metaType = 'double'; + if(stristr($type,'blob')) $metaType = 'blob'; $column['type'] = $metaType; $columns[] = $column; @@ -543,7 +547,7 @@ return 'blob'; } } - + } -?> \ No newline at end of file +?> -- Gitblit v1.9.1