SQL fix
[mailer.git] / inc / db / lib-mysql3.php
index a0f4499b147e583f014001c6f179f54888486c07..7a33983a1ec2d59cb71e9ae08a518dc84280d4c5 100644 (file)
@@ -1,23 +1,22 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 08/29/2004 *
- * ===============                              Last change: 08/29/2004 *
+ * Mailer v0.2.1-FINAL                                Start: 08/29/2004 *
+ * ===================                          Last change: 08/29/2004 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : lib-mysql3.php                                   *
  * -------------------------------------------------------------------- *
- * Short description : Database layer for MySQL +3.x server             *
+ * Short description : Database layer for MySQL 3/4/5 server            *
  * -------------------------------------------------------------------- *
- * Kurzbeschreibung  : Datenbankschicht fuer MySQL +3.x Server          *
+ * Kurzbeschreibung  : Datenbankschicht fuer MySQL 3/4/5 Server         *
  * -------------------------------------------------------------------- *
  * $Revision::                                                        $ *
  * $Date::                                                            $ *
  * $Tag:: 0.2.1-FINAL                                                 $ *
  * $Author::                                                          $ *
- * Needs to be in all Files and every File needs "svn propset           *
- * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2009 by Roland Haeder                           *
+ * Copyright (c) 2009 - 2011 by Mailer Developer Team                   *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -42,60 +41,67 @@ if (!defined('__SECURITY')) {
 } // END - if
 
 // SQL queries
-function SQL_QUERY ($sqlString, $F, $L) {
-       // Trim SQL string
-       $sqlString = trim($sqlString);
+function SQL_QUERY ($sqlString, $F, $L, $enableCodes = true) {
+       // Do we have cache?
+       if (!isset($GLOBALS[__FUNCTION__][$sqlString])) {
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Called: ' . $sqlString);
+
+               // Trim SQL string
+               $sqlStringModified = trim($sqlString);
+
+               // Empty query string or link is not up?
+               if (empty($sqlStringModified)) {
+                       // Empty SQL string!
+                       debug_report_bug(__FUNCTION__, __LINE__, sprintf("SQL string is empty. Please fix this. file=%s, line=%s",
+                               basename($F),
+                               $L
+                       ));
+               } elseif (!SQL_IS_LINK_UP()) {
+                       // We should not quietly ignore this
+                       debug_report_bug(__FUNCTION__, __LINE__, sprintf("Cannot query database: sqlString=%s,file=%s,line=%s",
+                               $sqlStringModified,
+                               basename($F),
+                               $L
+                       ));
+               }
 
-       // Link is up?
-       if (!SQL_IS_LINK_UP()) {
-               // We should not quietly ignore this!
-               debug_report_bug(sprintf("Cannot query database: sqlString=%s,file=%s,line=%s",
-                       $sqlString,
-                       basename($F),
-                       $L
-               ));
+               // Remove \t, \n and \r from queries they may confuse some MySQL versions
+               $sqlStringModified = str_replace("\t", ' ', str_replace("\n", ' ', str_replace("\r", ' ', $sqlStringModified)));
 
-               // Return 'false' because it has failed
-               return false;
-       } elseif (empty($sqlString)) {
-               // Empty SQL string!
-               debug_report_bug(sprintf("SQL string is empty. Please fix this. file=%s, line=%s",
-                       basename($F),
-                       $L
-               ));
+               // Compile config entries out
+               $sqlStringModified = SQL_PREPARE_SQL_STRING($sqlStringModified, $enableCodes);
 
-               // This is invalid, of course
-               return false;
-       }
+               // Cache it and remember as last SQL query
+               $GLOBALS[__FUNCTION__][$sqlString] = $sqlStringModified;
+               $GLOBALS['last_sql'] = $sqlStringModified;
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Stored cache: ' . $sqlStringModified);
+       }  else {
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Cache used: ' . $sqlString);
 
-       // Remove \t, \n and \r from queries they may confuse some MySQL version I have heard
-       $sqlString = str_replace("\t", ' ', str_replace("\n", ' ', str_replace("\r", ' ', $sqlString)));
+               // Use cache (to save a lot function calls
+               $GLOBALS['last_sql'] = $GLOBALS[__FUNCTION__][$sqlString];
 
-       // Compile config out
-       $sqlString = FILTER_COMPILE_CONFIG($sqlString);
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Cache is: ' . $sqlString);
+       }
 
        // Starting time
-       $querytimeBefore = array_sum(explode(' ', microtime()));
+       $querytimeBefore = microtime(true);
 
        // Run SQL command
-       //* DEBUG: */ print($sqlString . '<br />');
-       $result = mysql_query($sqlString, SQL_GET_LINK())
-               or addFatalMessage(__FUNCTION__, __LINE__, $F . ' (' . $L . '):' . mysql_error() . '<br />
-Query string:<br />
-' . $sqlString);
-       //* DEBUG: */ print(__LINE__ . ': numRows=' . SQL_NUMROWS($result) . ',affected=' . SQL_AFFECTEDROWS() . "<br />");
-
-       // Ending time
-       $querytimeAfter = array_sum(explode(' ', microtime()));
+       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'F=' . basename($F) . ',L=' . $L . ',sql=' . $GLOBALS['last_sql']);
+       $result = mysql_query($GLOBALS['last_sql'], SQL_GET_LINK())
+               or debug_report_bug($F, $L, 'file='. basename($F) . ',line=' . $L . ':mysql_error()=' . mysql_error() . "\n".
+'Query string:' . $GLOBALS['last_sql']);
+       //* DEBUG: */ logDebugMessage($F, $L, 'sql=' . $GLOBALS['last_sql'] . ',affected=' . SQL_AFFECTEDROWS() . ',numRows='.(is_resource($result) ? SQL_NUMROWS($result) : gettype($result)));
 
        // Calculate query time
-       $queryTime = $querytimeAfter - $querytimeBefore;
+       $queryTime = microtime(true) - $querytimeBefore;
 
        // Add this query to array including timing
-       addSqlToDebug($result, $sqlString, $queryTime, $F, $L);
+       addSqlToDebug($result, $GLOBALS['last_sql'], $queryTime, $F, $L);
 
        // Save last successfull query
-       setConfigEntry('db_last_query', $sqlString);
+       setConfigEntry('db_last_query', $GLOBALS['last_sql']);
 
        // Count all query times
        incrementConfigEntry('sql_time', $queryTime);
@@ -104,18 +110,16 @@ Query string:<br />
        incrementConfigEntry('sql_count');
 
        // Debug output
-       if ((getOutputMode() != 1) && (isDebugModeEnabled()) && (isConfigEntrySet('DEBUG_SQL')) && (getConfig('DEBUG_SQL') == 'Y')) {
-               //
-               // Debugging stuff...
-               //
-               $fp = fopen(getConfig('CACHE_PATH') . 'mysql.log', 'a') or app_die(__FILE__, __LINE__, "Cannot write mysql.log!");
+       if ((!isCssOutputMode()) && (isDebugModeEnabled()) && (isSqlDebuggingEnabled())) {
+               // Is this the first call?
                if (!isset($GLOBALS['sql_first_entry'])) {
                        // Write first entry
-                       fwrite($fp, 'Module=' . getModule() . "\n");
+                       appendLineToFile(getCachePath() . 'mysql.log', 'Module=' . getModule());
                        $GLOBALS['sql_first_entry'] = true;
                } // END - if
-               fwrite($fp, $F . '(LINE=' . $L . '|NUM=' . SQL_NUMROWS($result) . '|AFFECTED=' . SQL_AFFECTEDROWS() . '|QUERYTIME:' . $queryTime . '): ' . str_replace("\r", '', str_replace("\n", ' ', $sqlString)) . "\n");
-               fclose($fp);
+
+               // Append debug line
+               appendLineToFile(getCachePath() . 'mysql.log', $F . '(LINE=' . $L . '|NUM=' . SQL_NUMROWS($result) . '|AFFECTED=' . SQL_AFFECTEDROWS() . '|QUERYTIME:' . $queryTime . '): ' . str_replace("\r", '', str_replace("\n", ' ', $GLOBALS['last_sql'])));
        } // END - if
 
        // Count DB hits
@@ -132,20 +136,26 @@ Query string:<br />
 }
 
 // SQL num rows
-function SQL_NUMROWS ($result) {
+function SQL_NUMROWS ($resource) {
+       // Valid link resource?
+       if (!SQL_IS_LINK_UP()) return false;
+
        // Link is not up, no rows by default
        $lines = false;
 
        // Is the result a valid resource?
-       if (is_resource($result)) {
+       if (isset($GLOBALS['sql_numrows'][$resource])) {
+               // Use cache
+               $lines = $GLOBALS['sql_numrows'][intval($resource)];
+       } elseif (is_resource($resource)) {
                // Get the count of rows from database
-               $lines = mysql_num_rows($result);
+               $lines = mysql_num_rows($resource);
 
-               // Is the result empty? Then we have an error!
-               if (empty($lines)) $lines = 0;
-       } elseif (SQL_IS_LINK_UP()) {
-               // No resource given, no lines found!
-               $lines = 0;
+               // Remember it in cache
+               $GLOBALS['sql_numrows'][intval($resource)] = $lines;
+       } else {
+               // No resource given, please fix this
+               debug_report_bug(__FUNCTION__, __LINE__, 'No resource given! result[]=' . gettype($resource) . ',last_sql=' .  $GLOBALS['last_sql']);
        }
 
        // Return lines
@@ -165,65 +175,51 @@ function SQL_AFFECTEDROWS() {
 }
 
 // SQL fetch row
-function SQL_FETCHROW ($result) {
+function SQL_FETCHROW ($resource) {
        // Is a result resource set?
-       if ((!is_resource($result)) || (!SQL_IS_LINK_UP())) return false;
+       if ((!is_resource($resource)) || (!SQL_IS_LINK_UP())) return false;
 
        // Fetch the data and return it
-       return mysql_fetch_row($result);
+       return mysql_fetch_row($resource);
 }
 
 // SQL fetch array
-function SQL_FETCHARRAY ($res, $nr=0, $remove_numerical=true) {
+function SQL_FETCHARRAY ($res) {
        // Is a result resource set?
        if ((!is_resource($res)) || (!SQL_IS_LINK_UP())) return false;
 
-       // Initialize array
-       $row = array();
-
        // Load row from database
-       $row = mysql_fetch_array($res);
+       $row = mysql_fetch_assoc($res);
 
        // Return only arrays here
        if (is_array($row)) {
-               // Shall we remove numerical data here automatically?
-               if ($remove_numerical) {
-                       // So let's remove all numerical elements to save memory!
-                       $max = count($row);
-                       for ($idx = 0; $idx < ($max / 2); $idx++) {
-                               // Remove entry
-                               unset($row[$idx]);
-                       } // END - for
-               } // END - if
-
                // Return row
                return $row;
        } else {
-               // Return a false here...
+               // Return a false, else some loops would go endless...
                return false;
        }
 }
 
 // SQL result
-function SQL_RESULT ($res, $row, $field = 0) {
-       // Is $res valid?
-       if ((!is_resource($res)) || (!SQL_IS_LINK_UP())) return false;
+function SQL_RESULT ($resource, $row, $field = '0') {
+       // Is $resource valid?
+       if ((!is_resource($resource)) || (!SQL_IS_LINK_UP())) return false;
+
+       // Run the result command
+       $result = mysql_result($resource, $row, $field);
 
-       // Run the result command and return the result
-       $result = mysql_result($res, $row, $field);
+       // ... and return the result
        return $result;
 }
 
 // SQL connect
 function SQL_CONNECT ($host, $login, $password, $F, $L) {
        // Try to connect
-       $connect = mysql_connect($host, $login, $password) or addFatalMessage(__FUNCTION__, __LINE__, $F." (".$L."):".mysql_error());
+       $linkResource = mysql_connect($host, $login, $password) or addFatalMessage(__FUNCTION__, __LINE__, $F . ' (' . $L . '):' . mysql_error());
 
        // Set the link resource
-       SQL_SET_LINK($connect);
-
-       // Destroy cache
-       unset($GLOBALS['sql_link_res']);
+       SQL_SET_LINK($linkResource);
 }
 
 // SQL select database
@@ -232,7 +228,7 @@ function SQL_SELECT_DB ($dbName, $F, $L) {
        if (!SQL_IS_LINK_UP()) return false;
 
        // Return the result
-       return mysql_select_db($dbName, SQL_GET_LINK()) or addFatalMessage(__FUNCTION__, __LINE__, $F." (".$L."):".mysql_error());
+       return mysql_select_db($dbName, SQL_GET_LINK()) or addFatalMessage(__FUNCTION__, __LINE__, $F . ' (' . $L . '):' . mysql_error());
 }
 
 // SQL close link
@@ -249,82 +245,37 @@ function SQL_CLOSE ($F, $L) {
        // Close link
        SQL_SET_LINK(null);
 
-       // Destroy cache
-       unset($GLOBALS['sql_link_res']);
-
        // Return the result
        return $close;
 }
 
 // SQL free result
-function SQL_FREERESULT ($result) {
-       if ((!is_resource($result)) || (!SQL_IS_LINK_UP())) {
+function SQL_FREERESULT ($resource) {
+       if ((!is_resource($resource)) || (!SQL_IS_LINK_UP())) {
                // Abort here
                return false;
        } // END - if
 
-       $res = mysql_free_result($result);
+       // Free result
+       $res = mysql_free_result($resource);
+
+       // And return that result of freeing it...
        return $res;
 }
 
 // SQL string escaping
-function SQL_QUERY_ESC ($qstring, $data, $F, $L, $run=true, $strip=true, $secure=true) {
+function SQL_QUERY_ESC ($sqlString, $data, $F, $L, $run = true, $strip = true, $secure = true) {
        // Link is there?
        if ((!SQL_IS_LINK_UP()) || (!is_array($data))) return false;
 
-       /* Other way */
        // Escape all data
-       $dataSecured['__sql_string'] = $qstring;
+       $dataSecured['__sql_string'] = $sqlString;
        foreach ($data as $key => $value) {
                $dataSecured[$key] = SQL_ESCAPE($value, $secure, $strip);
        } // END - foreach
 
        // Generate query
        $query = call_user_func_array('sprintf', $dataSecured);
-       /*
-       /* Old eval() way... *
-       // Init variable
-       $query = 'failed';
-
-       if ($strip === true) {
-               $strip = 'true';
-       } else {
-               $strip = 'false';
-       }
-
-       if ($secure === true) {
-               $secure = 'true';
-       } else {
-               $secure = 'false';
-       }
-
-       $eval = "\$query = sprintf(\"" . $qstring . "\"";
-       foreach ($data as $var) {
-               if ((!empty($var)) || ($var === 0)) {
-                       $eval .= ", SQL_ESCAPE('" . $var . "', " . $secure . ', ' . $strip . ')';
-               } else {
-                       $eval .= ", ''";
-               }
-       } // END - foreach
-       $eval .= ');';
-       **/
-
-       // Debugging
-       //
-       //* DEBUG: */ $fp = fopen(getConfig('CACHE_PATH') . 'escape_debug.log', 'a') or app_die(__FILE__, __LINE__, "Cannot write debug.log!");
-       //* DEBUG: */ fwrite($fp, $F.'('.$L."): ".str_replace("\r", '', str_replace("\n", " ", $eval))."\n");
-       //* DEBUG: */ fclose($fp);
-
-       // Run the code
-       /**
-       eval($eval);
-
-       // Was the eval() command fine?
-       if ($query == 'failed') {
-               // Something went wrong?
-               debug_report_bug('eval=' . $eval);
-       } // END - if
-       **/
 
        if ($run === true) {
                // Run SQL query (default)
@@ -335,14 +286,14 @@ function SQL_QUERY_ESC ($qstring, $data, $F, $L, $run=true, $strip=true, $secure
        }
 }
 
-// Get ID from last INSERT command
+// Get id from last INSERT command
 function SQL_INSERTID () {
        if (!SQL_IS_LINK_UP()) return false;
        return mysql_insert_id();
 }
 
 // Escape a string for the database
-function SQL_ESCAPE ($str, $secureString=true, $strip=true) {
+function SQL_ESCAPE ($str, $secureString = true, $strip = true) {
        // Do we have cache?
        if (!isset($GLOBALS['sql_escapes'][''.$str.''])) {
                // Secure string first? (which is the default behaviour!)
@@ -352,18 +303,17 @@ function SQL_ESCAPE ($str, $secureString=true, $strip=true) {
                } // END - if
 
                if (!SQL_IS_LINK_UP()) {
-                       // Fall-back to smartAddSlashes() when there is no link
-                       $ret = smartAddSlashes($str);
+                       // Fall-back to escapeQuotes() when there is no link
+                       $ret = escapeQuotes($str);
                } elseif (function_exists('mysql_real_escape_string')) {
                        // The new and improved version
-                       //* DEBUG: */ print(__FUNCTION__."(<font color=\"#0000aa\">".__LINE__."</font>):str={$str}<br />");
                        $ret = mysql_real_escape_string($str, SQL_GET_LINK());
                } elseif (function_exists('mysql_escape_string')) {
                        // The obsolete function
                        $ret = mysql_escape_string($str, SQL_GET_LINK());
                } else {
-                       // If nothing else works, fall back to smartAddSlashes() again
-                       $ret = smartAddSlashes($str);
+                       // If nothing else works, fall back to escapeQuotes() again
+                       $ret = escapeQuotes($str);
                }
 
                // Cache result
@@ -379,7 +329,7 @@ function SQL_RESULT_FROM_ARRAY ($table, $columns, $idRow, $id, $F, $L) {
        // Is columns an array?
        if (!is_array($columns)) {
                // No array
-               debug_report_bug(sprintf("columns is not an array. %s != array, file=%s, line=%s",
+               debug_report_bug(__FUNCTION__, __LINE__, sprintf("columns is not an array. %s != array, file=%s, line=%s",
                        gettype($columns),
                        basename($F),
                        $L
@@ -403,7 +353,7 @@ function SQL_RESULT_FROM_ARRAY ($table, $columns, $idRow, $id, $F, $L) {
 }
 
 // ALTER TABLE wrapper function
-function SQL_ALTER_TABLE ($sql, $F, $L) {
+function SQL_ALTER_TABLE ($sql, $F, $L, $enableCodes = true) {
        // Abort if link is down
        if (!SQL_IS_LINK_UP()) return false;
 
@@ -414,6 +364,8 @@ function SQL_ALTER_TABLE ($sql, $F, $L) {
        $noIndex = (
        (
                strpos($sql, 'INDEX') === false
+       ) && (
+               strpos($sql, 'KEY') === false
        ) && (
                strpos($sql, 'FULLTEXT') === false
        ) && (
@@ -422,80 +374,95 @@ function SQL_ALTER_TABLE ($sql, $F, $L) {
        );
 
        // Extract table name
-       $tableArray = explode(" ", $sql);
+       $tableArray = explode(' ', $sql);
        $tableName = str_replace('`', '', $tableArray[2]);
 
+       // Debug log
+       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'sql=' . $sql . ',tableName=' . $tableName);
+
        // Shall we add/drop?
-       if (((strpos($sql, 'ADD') !== false) || (strpos($sql, 'DROP') !== false)) && ($noIndex === true)) {
-               // And column name as well
-               $columnName = str_replace('`', '', $tableArray[4]);
-
-               // Get column information
-               $result = SQL_QUERY_ESC("SHOW COLUMNS FROM `%s` LIKE '%s'",
-                       array($tableName, $columnName), __FILE__, __LINE__);
-
-               // Do we have no entry on ADD or an entry on DROP?
-               // 123           4       4     3    3      4           4          32    23           4       4     3    3      4            4          321
-               if (((SQL_NUMROWS($result) == 0) && (strpos($sql, 'ADD') !== false)) || ((SQL_NUMROWS($result) == 1) && (strpos($sql, 'DROP') !== false))) {
-                       // Do the query
-                       //* DEBUG: */ print(__LINE__.':'.$sql."<br />");
-                       $result = SQL_QUERY($sql, $F, $L, false);
-               } // END - if
-       } elseif ((getConfig('_TABLE_TYPE') == 'InnoDB') && (strpos($sql, 'FULLTEXT') !== false)) {
+       if (((strpos($sql, 'ADD') !== false) || (strpos($sql, 'DROP') !== false) || (strpos($sql, 'CHANGE') !== false)) && ($noIndex === true)) {
+               // Try two columns, one should fix
+               foreach (array(4,5) as $idx) {
+                       // If an entry is not set, abort here
+                       if (!isset($tableArray[$idx])) {
+                               // Debug log this
+                               logDebugMessage(__FUNCTION__, __LINE__, 'columnName=' . $columnName . ',idx=' . $idx . ',sql=' . $sql . ' is missing!');
+                               break;
+                       } // END - if
+
+                       // And column name as well
+                       $columnName = str_replace('`', '', $tableArray[$idx]);
+
+                       // Get column information
+                       $result = SQL_QUERY_ESC("SHOW COLUMNS FROM `%s` LIKE '%s'",
+                               array($tableName, $columnName), __FUNCTION__, __LINE__);
+
+                       // Debug log
+                       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'columnName=' . $columnName . ',idx=' . $idx . ',sql=' . $sql . ',hasZeroNums=' . intval(SQL_HASZERONUMS($result)));
+
+                       // Do we have no entry on ADD or an entry on DROP/CHANGE?
+                       // 123               4       43    3      4           4          32    23           4       4     3    3      4            4          32    23                4       43    3      4              4          3    3         321
+                       if (((SQL_HASZERONUMS($result)) && (strpos($sql, 'ADD') !== false)) || ((!SQL_HASZERONUMS($result)) && (strpos($sql, 'DROP') !== false)) || ((!SQL_HASZERONUMS($result)) && (strpos($sql, 'CHANGE') !== false) && ($idx == 4))) {
+                               // Do the query
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Executing: ' . $sql);
+                               $result = SQL_QUERY($sql, $F, $L, false);
+
+                               // Skip further attempt(s)
+                               break;
+                       //       1234                5       54    4      5           5          43    34                5       54    4      5            5          43    3      4              4          32    2                    21
+                       } elseif ((((!SQL_HASZERONUMS($result)) && (strpos($sql, 'ADD') !== false)) || ((!SQL_HASZERONUMS($result)) && (strpos($sql, 'DROP') !== false)) || (strpos($sql, 'CHANGE') !== false)) && ($columnName != 'KEY')) {
+                               // Abort here because it is alreay there
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Skipped: sql=' . $sql . ',columnName=' . $columnName . ',idx=' . $idx);
+                               break;
+                       } elseif ((SQL_HASZERONUMS($result)) && (strpos($sql, 'DROP') !== false)) {
+                               // Abort here because we tried to drop a column which is not there (never created maybe)
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'No drop: ' . $sql);
+                               break;
+                       } elseif ($columnName != 'KEY') {
+                               // Something didn't fit, we better log it
+                               logDebugMessage(__FUNCTION__, __LINE__, 'Possible problem: ' . $sql . ',hasZeroNums=' . intval(SQL_HASZERONUMS($result)) . '');
+                       }
+               } // END - foreach
+       } elseif ((getTableType() == 'InnoDB') && (strpos($sql, 'FULLTEXT') !== false)) {
                // Skip this query silently
-               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, sprintf("Skipped FULLTEXT: sql=%s,file=%s,line=%s", $sql, $F, $L));
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, sprintf("Skipped FULLTEXT: sql=%s,tableName=%s,hasZeroNums=%d,file=%s,line=%s", $sql, $tableName, intval((is_bool($result)) ? 0 : SQL_HASZERONUMS($result)), $F, $L));
        } elseif ($noIndex === false) {
                // And column name as well
-               $columnName = str_replace('`', '', $tableArray[4]);
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'tableArray=<pre>' . print_r($tableArray, true) . '</pre>');
+               $keyName = str_replace('`', '', $tableArray[5]);
 
                // Is this "UNIQUE" or so? FULLTEXT has been handled the elseif() block above
-               if (in_array(strtoupper($columnName), array('INDEX', 'UNIQUE', 'KEY', 'FULLTEXT'))) {
+               if (in_array(strtoupper($keyName), array('INDEX', 'UNIQUE', 'KEY', 'FULLTEXT'))) {
                        // Init loop
-                       $begin = 1; $columnName = ',';
-                       while (strpos($columnName, ',') !== false) {
+                       $begin = 1; $keyName = ',';
+                       while (strpos($keyName, ',') !== false) {
                                // Use last
-                               $columnName = str_replace('`', '', $tableArray[count($tableArray) - $begin]);
-                               //* DEBUG: */ print(__LINE__.':'.$columnName."----------------".$begin."<br />");
+                               $keyName = str_replace('`', '', $tableArray[count($tableArray) - $begin]);
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, $keyName . '----------------' . $begin);
 
                                // Remove brackes
-                               $columnName = str_replace('(', '', str_replace(')', '', $columnName));
-                               //* DEBUG: */ print(__LINE__.':'.$columnName."----------------".$begin."<br />");
+                               $keyName = str_replace('(', '', str_replace(')', '', $keyName));
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, $keyName . '----------------' . $begin);
 
                                // Continue
                                $begin++;
                        } // END while
                } // END - if
 
-               // Show indexes
-               $result = SQL_QUERY_ESC("SHOW INDEX FROM `%s`",
-                       array($tableName), __FILE__, __LINE__);
-
-               // Walk through all
-               $skip = false;
-               while ($content = SQL_FETCHARRAY($result)) {
-                       // Is it found?
-                       //* DEBUG: */ print("<pre>".print_r($content, true)."</pre>");
-                       if (($content['Column_name'] == $columnName) || ($content['Key_name'] == $columnName)) {
-                               // Skip this query!
-                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, sprintf("Skiped: %s", $sql));
-                               $skip = true;
-                               break;
-                       } // END - if
-               } // END - while
-
-               // Free result
-               SQL_FREERESULT($result);
-
                // Shall we run it?
-               if ($skip === false) {
+               if (!isSqlTableIndexAdded($tableName, $tableArray[3])) {
                        // Send it to the SQL_QUERY() function
-                       //* DEBUG: */ print(__LINE__.':'.$sql."<br />");
-                       $result = SQL_QUERY($sql, $F, $L, false);
-               } // END - if
+                       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, $sql);
+                       $result = SQL_QUERY($sql, $F, $L, $enableCodes);
+               } else {
+                       // Not executed
+                       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Not executed: ' . $sql);
+               }
        } else {
                // Other ALTER TABLE query
-               //* DEBUG: */ print(__LINE__.':'.$sql."<br />");
-               $result = SQL_QUERY($sql, $F, $L, false);
+               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, $sql);
+               $result = SQL_QUERY($sql, $F, $L, $enableCodes);
        }
 
        // Return result
@@ -520,13 +487,19 @@ function SQL_GET_LINK () {
 // Setter for link
 function SQL_SET_LINK ($link) {
        // Is this a resource or null?
-       if ((!is_resource($link)) && (!is_null($link))) {
+       if ((ifFatalErrorsDetected()) && (isInstallationPhase())) {
+               // This may happen in installation phase
+               return;
+       } elseif ((!is_resource($link)) && (!is_null($link))) {
                // This should never happen!
-               debug_report_bug(sprintf("link is not resource or null. Type: %s", gettype($link)));
+               debug_report_bug(__FUNCTION__, __LINE__, sprintf("Type of link is not resource or null, type=%s", gettype($link)));
        } // END - if
 
        // Set it
        $GLOBALS['sql_link'] = $link;
+
+       // Re-init cache
+       $GLOBALS['is_sql_link_up'] = is_resource($link);
 }
 
 // Checks if the link is up
@@ -535,20 +508,106 @@ function SQL_IS_LINK_UP () {
        $linkUp = false;
 
        // Do we have cached this?
-       if (isset($GLOBALS['sql_link_res'])) {
+       if (isset($GLOBALS['is_sql_link_up'])) {
                // Then use this
-               $linkUp = $GLOBALS['sql_link_res'];
+               $linkUp = $GLOBALS['is_sql_link_up'];
        } else {
                // Get it
                $linkUp = is_resource(SQL_GET_LINK());
 
                // And cache it
-               $GLOBALS['sql_link_res'] = $linkUp;
+               $GLOBALS['is_sql_link_up'] = $linkUp;
        }
 
        // Return the result
        return $linkUp;
 }
 
+// Wrapper function to make code more readable
+function SQL_HASZERONUMS ($result) {
+       // Just pass it through
+       return (SQL_NUMROWS($result) === 0);
+}
+
+// Wrapper function to make code more readable
+function SQL_HASZEROAFFECTED () {
+       // Just pass it through
+       return (SQL_AFFECTEDROWS() === 0);
+}
+
+// Private function to prepare the SQL query string
+function SQL_PREPARE_SQL_STRING ($sqlString, $enableCodes = true) {
+       // Is it already cached?
+       if (!isset($GLOBALS['sql_strings'][$sqlString])) {
+               // Compile config+expression code
+               $sqlString2 = FILTER_COMPILE_EXPRESSION_CODE(FILTER_COMPILE_CONFIG($sqlString));
+
+               // Do final compilation
+               $GLOBALS['sql_strings'][$sqlString] = doFinalCompilation($sqlString2, false, $enableCodes);
+               //die($sqlString.'<br />'.$sqlString2.'<br />'.$GLOBALS['sql_strings'][$sqlString]);
+       } // END - if
+
+       // Return it
+       return $GLOBALS['sql_strings'][$sqlString];
+}
+
+// Check if there is a SQL table created
+function isSqlTableCreated ($table) {
+       // Do we have cache?
+       if (!isset($GLOBALS[__FUNCTION__][$table])) {
+               // Check if the table is there
+               $result = SQL_QUERY_ESC("SHOW TABLES FROM `{?__DB_NAME?}` WHERE `Tables_in_{?__DB_NAME?}`='{?_MYSQL_PREFIX?}_%s'",
+                       array($table), __FILE__, __LINE__);
+
+               // Is there an entry?
+               $GLOBALS[__FUNCTION__][$table] = (SQL_NUMROWS($result) == 1);
+       } // END - if
+
+       // Return cache
+       return $GLOBALS[__FUNCTION__][$table];
+}
+
+// Checks depending on the mode if the table is there
+function isSqlTableIndexAdded ($tableName, $mode) {
+       // Do we have cache?
+       if (!isset($GLOBALS[__FUNCTION__][$tableName][$mode])) {
+               // Show indexes
+               $result = SQL_QUERY_ESC("SHOW INDEX FROM `%s`", array($tableName), __FUNCTION__, __LINE__);
+
+               // Non-skipping is default for ADD
+               $GLOBALS[__FUNCTION__][$tableName][$mode] = false;
+
+               // But should we DROP?
+               if ($mode == 'DROP') {
+                       // Then skip if nothing found
+                       $GLOBALS[__FUNCTION__][$tableName][$mode] = true;
+                       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Going to drop key ' . $keyName);
+               } // END - if
+
+               // Walk through all
+               while ($content = SQL_FETCHARRAY($result)) {
+                       // Is it found?
+                       //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'columnName='.$keyName.',content=<pre>' . print_r($content, true) . '</pre>');
+                       if (($content['Key_name'] == $keyName) && ($mode == 'ADD')) {
+                               // Skip this query!
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, sprintf("ADD: Skiped: %s", $sql));
+                               $GLOBALS[__FUNCTION__][$tableName][$mode] = true;
+                               break;
+                       } elseif (($content['Key_name'] == $keyName) && ($mode == 'DROP')) {
+                               // Don't skip this!
+                               //* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, sprintf("DROP: Not skiped: %s", $sql));
+                               $GLOBALS[__FUNCTION__][$tableName][$mode] = false;
+                               break;
+                       }
+               } // END - while
+
+               // Free result
+               SQL_FREERESULT($result);
+       } // END - if
+
+       // Return cache
+       return $GLOBALS[__FUNCTION__][$tableName][$mode];
+}
+
 // [EOF]
 ?>