From: Andrew Dolgov <fox@fakecake.org>
Date: Wed, 17 Apr 2013 17:40:04 +0000 (+0400)
Subject: revert OO-db access in dbupdater, create DbUpdater() correctly
X-Git-Tag: 1.7.9~25^2~155^2~1
X-Git-Url: https://git.wh0rd.org/?a=commitdiff_plain;h=0630a100d5f863ab7a3b04f7be7dcce5cafbaf8a;p=tt-rss.git

revert OO-db access in dbupdater, create DbUpdater() correctly
---

diff --git a/classes/dbupdater.php b/classes/dbupdater.php
index f157342d..a319da03 100644
--- a/classes/dbupdater.php
+++ b/classes/dbupdater.php
@@ -2,18 +2,18 @@
 class DbUpdater {
 
 	private $dbh;
-	private $$this->dbh->type;
+	private $db_type;
 	private $need_version;
 
-	function __construct($dbh, $$this->dbh->type, $need_version) {
+	function __construct($dbh, $db_type, $need_version) {
 		$this->dbh = $dbh;
-		$this->$this->dbh->type = $db_type;
+		$this->db_type = $db_type;
 		$this->need_version = (int) $need_version;
 	}
 
 	function getSchemaVersion() {
-		$result = $this->dbh->query("SELECT schema_version FROM ttrss_version");
-		return (int) $this->dbh->fetch_result($result, 0, "schema_version");
+		$result = db_query("SELECT schema_version FROM ttrss_version");
+		return (int) db_fetch_result($result, 0, "schema_version");
 	}
 
 	function isUpdateRequired() {
@@ -21,7 +21,7 @@ class DbUpdater {
 	}
 
 	function getSchemaLines($version) {
-		$filename = "schema/versions/".$this->$this->dbh->type."/$version.sql";
+		$filename = "schema/versions/".$this->db_type."/$version.sql";
 
 		if (file_exists($filename)) {
 			return explode(";", preg_replace("/[\r\n]/", "", file_get_contents($filename)));
@@ -37,21 +37,21 @@ class DbUpdater {
 
 			if (is_array($lines)) {
 
-				$this->dbh->query("BEGIN");
+				db_query("BEGIN");
 
 				foreach ($lines as $line) {
 					if (strpos($line, "--") !== 0 && $line) {
-						$this->dbh->query($line);
+						db_query($line);
 					}
 				}
 
-				$$this->dbh->version = $this->getSchemaVersion();
+				$db_version = $this->getSchemaVersion();
 
-				if ($$this->dbh->version == $version) {
-					$this->dbh->query("COMMIT");
+				if ($db_version == $version) {
+					db_query("COMMIT");
 					return true;
 				} else {
-					$this->dbh->query("ROLLBACK");
+					db_query("ROLLBACK");
 					return false;
 				}
 			} else {
diff --git a/classes/handler/public.php b/classes/handler/public.php
index 439cc7d4..16d628c8 100644
--- a/classes/handler/public.php
+++ b/classes/handler/public.php
@@ -869,7 +869,7 @@ class Handler_Public extends Handler {
 
 			<?php
 				@$op = $_REQUEST["subop"];
-				$updater = new DbUpdater(DB_TYPE, SCHEMA_VERSION);
+				$updater = new DbUpdater(Db::get(), DB_TYPE, SCHEMA_VERSION);
 
 				if ($op == "performupdate") {
 					if ($updater->isUpdateRequired()) {
diff --git a/update.php b/update.php
index 1e653311..e13abafc 100755
--- a/update.php
+++ b/update.php
@@ -320,7 +320,7 @@
 	if (isset($options["update-schema"])) {
 		_debug("checking for updates (" . DB_TYPE . ")...");
 
-		$updater = new DbUpdater( DB_TYPE, SCHEMA_VERSION);
+		$updater = new DbUpdater(Db::get(), DB_TYPE, SCHEMA_VERSION);
 
 		if ($updater->isUpdateRequired()) {
 			_debug("schema update required, version " . $updater->getSchemaVersion() . " to " . SCHEMA_VERSION);