r3793 getDatabaseVersion(): move to upgrade.php, where it is only used
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 10 Jun 2010 13:36:47 +0000 (13:36 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 10 Jun 2010 13:36:47 +0000 (13:36 +0000)
inc/database.php
upgrade.php

index 5a6c555..942aa52 100644 (file)
@@ -2262,30 +2262,6 @@ function storeConfigVar ($varname = NULL, $varvalue = NULL)
        );
 }
 
-// Database version detector. Should behave corretly on any
-// working dataset a user might have.
-function getDatabaseVersion ()
-{
-       $result = usePreparedSelectBlade ('SELECT varvalue FROM Config WHERE varname = "DB_VERSION" and vartype = "string"');
-       if ($result == NULL)
-       {
-               global $dbxlink;
-               $errorInfo = $dbxlink->errorInfo();
-               if ($errorInfo[0] == '42S02') // ER_NO_SUCH_TABLE
-                       return '0.14.4';
-               die (__FUNCTION__ . ': SQL query #1 failed with error ' . $errorInfo[2]);
-       }
-       $rows = $result->fetchAll (PDO::FETCH_NUM);
-       if (count ($rows) != 1 || !strlen ($rows[0][0]))
-       {
-               $result->closeCursor();
-               die (__FUNCTION__ . ': Cannot guess database version. Config table is present, but DB_VERSION is missing or invalid. Giving up.');
-       }
-       $ret = $rows[0][0];
-       $result->closeCursor();
-       return $ret;
-}
-
 // Return an array of virtual services. For each of them list real server pools
 // with their load balancers and other stats.
 function getSLBSummary ()
index 4d621e7..33b940a 100644 (file)
@@ -786,7 +786,6 @@ function showFailure ($info = '', $location = 'N/A')
 }
 
 require_once 'inc/config.php'; // for CODE_VERSION
-require_once 'inc/database.php'; // for getDatabaseVersion()
 require_once 'inc/dictionary.php';
 // Enforce default value for now, releases prior to 0.17.0 didn't support 'httpd' auth source.
 $user_auth_src = 'database';
@@ -820,6 +819,28 @@ function authenticate_admin ($username, $password)
        return $rows[0][0] == 1;
 }
 
+function getDatabaseVersion ()
+{
+       global $dbxlink;
+       $result = $dbxlink->query ('SELECT varvalue FROM Config WHERE varname = "DB_VERSION" and vartype = "string"');
+       if ($result == NULL)
+       {
+               $errorInfo = $dbxlink->errorInfo();
+               if ($errorInfo[0] == '42S02') // ER_NO_SUCH_TABLE
+                       return '0.14.4';
+               die (__FUNCTION__ . ': SQL query #1 failed with error ' . $errorInfo[2]);
+       }
+       $rows = $result->fetchAll (PDO::FETCH_NUM);
+       if (count ($rows) != 1 || !strlen ($rows[0][0]))
+       {
+               $result->closeCursor();
+               die (__FUNCTION__ . ': Cannot guess database version. Config table is present, but DB_VERSION is missing or invalid. Giving up.');
+       }
+       $ret = $rows[0][0];
+       $result->closeCursor();
+       return $ret;
+}
+
 switch ($user_auth_src)
 {
        case 'database':