X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fdb%2Flib-mysql3.php;h=bcdf041e3668e2a9bd7d81f7ff254fc03f3de647;hp=2f060bbe726e76ff69069e2b4b44d800c8785bc0;hb=bcabdff3613bbc285c600ac5588dcc85356e6d66;hpb=320431167332e4212bbba7a442349393cab3e047

diff --git a/inc/db/lib-mysql3.php b/inc/db/lib-mysql3.php
index 2f060bbe72..bcdf041e36 100644
--- a/inc/db/lib-mysql3.php
+++ b/inc/db/lib-mysql3.php
@@ -38,59 +38,62 @@ if (!defined('__SECURITY')) {
 }
 
 // SQL queries
-function SQL_QUERY($sql_string, $F, $L) {
-	global $link, $CSS, $_CONFIG, $OK;
+function SQL_QUERY ($sql_string, $F, $L) {
+	// Link is up?
+	if (!SQL_IS_LINK_UP()) return false;
 
 	// Remove \t, \n and \r from queries they may confuse some MySQL version I have heard
 	$sql_string = str_replace("\t", " ", str_replace("\n", " ", str_replace("\r", " ", $sql_string)));
 
+	// Replace {!_MYSQL_PREFIX!} with constant, closes #84. Thanks to profi-concept
+	$sql_string = str_replace("{!_MYSQL_PREFIX!}", constant('_MYSQL_PREFIX'), $sql_string);
+
 	// Starting time
 	$querytimeBefore = array_sum(explode(' ', microtime()));
 
 	// Run SQL command
 	//* DEBUG: */ echo $sql_string."<br />\n";
-	$result = @mysql_query($sql_string, $link)
-	 or ADD_FATAL($F." (".$L."):".mysql_error()."<br />
+	$result = mysql_query($sql_string, SQL_GET_LINK())
+	 or addFatalMessage($F." (".$L."):".mysql_error()."<br />
 Query string:<br />
 ".$sql_string);
 
-	// Save last successfull query
-	$_CONFIG['db_last_query'] = $sql_string;
-
 	// Ending time
 	$querytimeAfter = array_sum(explode(' ', microtime()));
 
 	// Calculate query time
 	$queryTime = $querytimeAfter - $querytimeBefore;
 
+	// Save last successfull query
+	setConfigEntry('db_last_query', $sql_string);
+
 	// Count this query
-	if (!isset($_CONFIG['sql_count'])) $_CONFIG['sql_count'] = 0;
-	$_CONFIG['sql_count']++;
+	incrementConfigEntry('sql_count');
 
 	// Debug output
-	//* DEBUG: */ print "Query=<pre>".$sql_string."</pre>, affected=<b>".SQL_AFFECTEDROWS()."</b>, numrows=<b>".SQL_NUMROWS($result)."</b><br />\n";
+	//* DEBUG: */ print "Query=<pre>".$sql_string."</pre>, affected=<strong>".SQL_AFFECTEDROWS()."</strong>, numrows=<strong>".SQL_NUMROWS($result)."</strong><br />\n";
 
-	if (($CSS != "1") && ($CSS != "-1") && (isBooleanConstantAndTrue('DEBUG_MODE')) && (isBooleanConstantAndTrue('DEBUG_SQL'))) {
+	if (($GLOBALS['output_mode'] != "1") && ($GLOBALS['output_mode'] != "-1") && (isBooleanConstantAndTrue('DEBUG_MODE')) && (isBooleanConstantAndTrue('DEBUG_SQL'))) {
 		//
 		// Debugging stuff...
 		//
-		$fp = @fopen(PATH."inc/cache/mysql.log", 'a') or mxchange_die("Cannot write mysql.log!");
-		if (!isset($OK)) {
+		$fp = fopen(constant('PATH')."inc/cache/mysql.log", 'a') or mxchange_die("Cannot write mysql.log!");
+		if (!isset($GLOBALS['sql_first_entry'])) {
 			// Write first entry
 			fwrite($fp, "Module=".$GLOBALS['module']."\n");
-			$OK = true;
+			$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', " ", $sql_string))."\n");
 		fclose($fp);
 	} // END - if
 
 	// Count DB hits
-	if (!isset($_CONFIG['db_hits_run'])) {
+	if (!isConfigEntrySet('db_hits_run')) {
 		// Count in dummy variable
-		$_CONFIG['db_hits_run'] = 1;
+		setConfigEntry('db_hits_run', 1);
 	} else {
 		// Count to config array
-		$_CONFIG['db_hits_run']++;
+		incrementConfigEntry('db_hits_run');
 	}
 
 	// Return the result
@@ -98,11 +101,11 @@ Query string:<br />
 }
 
 // SQL num rows
-function SQL_NUMROWS($result) {
+function SQL_NUMROWS ($result) {
 	// Is the result a valid resource?
 	if (is_resource($result)) {
 		// Get the count of rows from database
-		$lines = @mysql_num_rows($result);
+		$lines = mysql_num_rows($result);
 
 		// Is the result empty? Then we have an error!
 		if (empty($lines)) $lines = 0;
@@ -115,13 +118,11 @@ function SQL_NUMROWS($result) {
 
 // SQL affected rows
 function SQL_AFFECTEDROWS() {
-	global $link;
-
 	// Valid link resource?
-	if (!is_resource($link)) return false;
+	if (!SQL_IS_LINK_UP()) return false;
 
 	// Get affected rows
-	$lines = @mysql_affected_rows($link);
+	$lines = mysql_affected_rows(SQL_GET_LINK());
 
 	// Return it
 	return $lines;
@@ -135,7 +136,7 @@ function SQL_FETCHROW($result) {
 	// Is a result resource set?
 	if (!is_resource($result)) return false;
 
-	$DATA = @mysql_fetch_row($result);
+	$DATA = mysql_fetch_row($result);
 	return $DATA;
 }
 
@@ -148,7 +149,7 @@ function SQL_FETCHARRAY($res, $nr=0, $remove_numerical=true) {
 	$row = array();
 
 	// Load row from database
-	$row = @mysql_fetch_array($res);
+	$row = mysql_fetch_array($res);
 
 	// Return only arrays here
 	if (is_array($row)) {
@@ -159,8 +160,8 @@ function SQL_FETCHARRAY($res, $nr=0, $remove_numerical=true) {
 			for ($idx = 0; $idx < ($max / 2); $idx++) {
 				// Remove entry
 				unset($row[$idx]);
-			}
-		}
+			} // END - for
+		} // END - if
 
 		// Return row
 		return $row;
@@ -171,59 +172,74 @@ function SQL_FETCHARRAY($res, $nr=0, $remove_numerical=true) {
 }
 
 // SQL result
-function SQL_RESULT($res, $row, $field) {
-	$result = @mysql_result($res, $row, $field);
+function SQL_RESULT ($res, $row, $field) {
+	$result = mysql_result($res, $row, $field);
 	return $result;
 }
+
 // SQL connect
-function SQL_CONNECT($host, $login, $password, $F, $L) {
-	$connect = @mysql_connect($host, $login, $password) or ADD_FATAL($F." (".$L."):".mysql_error());
-	return $connect;
+function SQL_CONNECT ($host, $login, $password, $F, $L) {
+	// Try to connect
+	$connect = mysql_connect($host, $login, $password) or addFatalMessage($F." (".$L."):".mysql_error());
+
+	// Set the link resource
+	SQL_SET_LINK($connect);
 }
+
 // SQL select database
-function SQL_SELECT_DB($dbName, $link, $F, $L) {
-	$select = false;
-	if (is_resource($link)) {
-		$select = @mysql_select_db($dbName, $link) or ADD_FATAL($F." (".$L."):".mysql_error());
-	}
-	return $select;
+function SQL_SELECT_DB ($dbName, $F, $L) {
+	// Is there still a valid link? If not, skip it.
+	if (!SQL_IS_LINK_UP()) return false;
+
+	// Return the result
+	return mysql_select_db($dbName, SQL_GET_LINK()) or addFatalMessage($F." (".$L."):".mysql_error());
 }
-// SQL close link
-function SQL_CLOSE(&$link, $F, $L) {
-	global $_CONFIG, $cacheInstance, $cacheArray;
 
-	// Is there still a valid link?
-	if (!is_resource($link)) {
+// SQL close link
+function SQL_CLOSE ($F, $L) {
+	if (!SQL_IS_LINK_UP()) {
 		// Skip double close
 		return false;
 	} // END - if
-	//* DEBUG: */ echo "DB=".$_CONFIG['db_hits'].",CACHE=".$_CONFIG['cache_hits']."<br />\n";
-	if ((GET_EXT_VERSION("cache") >= "0.0.7") && (isset($_CONFIG['db_hits'])) && (isset($_CONFIG['cache_hits'])) && (is_object($cacheInstance))) {
+
+	// Do we need to update cache/db counter?
+	//* DEBUG: */ echo "DB=".getConfig('db_hits').",CACHE=".getConfig('cache_hits')."<br />\n";
+	if ((GET_EXT_VERSION("cache") >= "0.0.7") && (getConfig('db_hits') > 0) && (getConfig('cache_hits') > 0) && (is_object($GLOBALS['cache_instance']))) {
 		// Add new hits
-		$_CONFIG['db_hits'] += $_CONFIG['db_hits_run'];
+		incrementConfigEntry('db_hits', getConfig('db_hits_run'));
 
 		// Update counter for db/cache
-		UPDATE_CONFIG(array("db_hits", "cache_hits"), array(bigintval($_CONFIG['db_hits']), bigintval($_CONFIG['cache_hits'])));
+		UPDATE_CONFIG(array("db_hits", "cache_hits"), array(bigintval(getConfig('db_hits')), bigintval(getConfig('cache_hits'))));
 	} // END - if
 
 	// Close database link and forget the link
-	$close = @mysql_close($link) or ADD_FATAL($F." (".$L."):".mysql_error());
-	$link = null;
+	$close = mysql_close(SQL_GET_LINK()) or addFatalMessage($F." (".$L."):".mysql_error());
+
+	// Close link
+	SQL_SET_LINK(null);
+
+	// Return the result
 	return $close;
 }
+
 // SQL free result
-function SQL_FREERESULT($result) {
+function SQL_FREERESULT ($result) {
 	if (!is_resource($result)) {
 		// Abort here
 		return false;
 	} // END - if
 
-	$res = @mysql_free_result($result);
+	$res = mysql_free_result($result);
 	return $res;
 }
+
 // SQL string escaping
-function SQL_QUERY_ESC($qstring, $data, $file, $line, $run=true, $strip=true) {
-	global $link;
+function SQL_QUERY_ESC ($qstring, $data, $F, $L, $run=true, $strip=true) {
+	// Link is there?
+	if (!SQL_IS_LINK_UP()) return false;
+
+	// Init variable
+	$query = "failed";
 
 	if ($strip) {
 		$strip = "true";
@@ -231,7 +247,6 @@ function SQL_QUERY_ESC($qstring, $data, $file, $line, $run=true, $strip=true) {
 		$strip = "false";
 	}
 
-	$query = "";
 	$eval = "\$query = sprintf(\"".$qstring."\"";
 	foreach ($data as $var) {
 		if ((!empty($var)) || ($var === 0)) {
@@ -239,74 +254,94 @@ function SQL_QUERY_ESC($qstring, $data, $file, $line, $run=true, $strip=true) {
 		} else {
 			$eval .= ", ''";
 		}
-	}
+	} // END - foreach
 	$eval .= ");";
 	//
 	// Debugging
 	//
-	//$fp = fopen(PATH."inc/cache/escape_debug.log", 'a') or mxchange_die("Cannot write debug.log!");
-	//fwrite($fp, $file."(".$line."): ".str_replace("\r", "", str_replace("\n", " ", $eval))."\n");
-	//fclose($fp);
-	@eval($eval);
+	//* DEBUG: */ $fp = fopen(constant('PATH')."inc/cache/escape_debug.log", 'a') or mxchange_die("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 (empty($query)) {
+	if ($query == "failed") {
 		// Something went wrong?
-		print "eval=".htmlentities($eval)."<pre>";
-		debug_print_backtrace();
-		die("</pre>");
+		debug_report_bug("eval={$eval}");
 	} // END - if
 
-	if ($run) {
+	if ($run === true) {
 		// Run SQL query (default)
-		return SQL_QUERY($query, $file, $line);
+		return SQL_QUERY($query, $F, $L);
 	} else {
 		// Return secured string
 		return $query;
 	}
 }
+
 // Get ID from last INSERT command
-function SQL_INSERTID() {
-	return @mysql_insert_id();
+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) {
-	global $link;
 
+// Escape a string for the database
+function SQL_ESCAPE ($str, $secureString=true,$strip=true) {
 	// Secure string first? (which is the default behaviour!)
 	if ($secureString) {
 		// Then do it here
 		$str = secureString($str, $strip);
 	} // END - if
 
-	if (!is_resource($link)) {
-		// Fall-back to addslashes() when there is no link
-		return addslashes($str);
-	} // END - if
-
-	if (function_exists('mysql_real_escape_string')) {
+	if (!SQL_IS_LINK_UP()) {
+		// Fall-back to smartAddSlashes() when there is no link
+		return smartAddSlashes($str);
+	} elseif (function_exists('mysql_real_escape_string')) {
 		// The new and improved version
 		//* DEBUG: */ print __FUNCTION__."(<font color=\"#0000aa\">".__LINE__."</font>):str={$str}<br />\n";
-		return mysql_real_escape_string($str, $link);
+		return mysql_real_escape_string($str, SQL_GET_LINK());
 	} elseif (function_exists('mysql_escape_string')) {
-		// The obsulete function
-		return mysql_escape_string($str, $link);
+		// The obsolete function
+		return mysql_escape_string($str, SQL_GET_LINK());
 	} else {
-		// If nothing else works
-		return addslashes($str);
+		// If nothing else works, fall back to smartAddSlashes()
+		return smartAddSlashes($str);
 	}
 }
+
 // SELECT query string from table, columns and so on... ;-)
 function SQL_RESULT_FROM_ARRAY ($table, $columns, $idRow, $id, $F, $L) {
+	// Is columns an array?
+	if (!is_array($columns)) {
+		// No array
+		trigger_error(sprintf("columns is not array. %s!=array", gettype($columns)));
+	} // END  - if
+
 	// Prepare the SQL statement
-	$SQL = "SELECT ".implode(", ", $columns)." FROM "._MYSQL_PREFIX."_".$table." WHERE ".$idRow."=%s LIMIT 1";
+	$SQL = "SELECT `".implode("`, `", $columns)."` FROM `{!_MYSQL_PREFIX!}_%s` WHERE ``='%s' LIMIT 1";
 
 	// Return the result
-	return SQL_QUERY_ESC($SQL, array(bigintval($id)), $F, $L);
+	return SQL_QUERY_ESC($SQL,
+		array(
+			bigintval($id),
+			$table,
+			$idRow
+		), $F, $L);
 }
+
 // ALTER TABLE wrapper function
-function SQL_ALTER_TABLE($sql, $F, $L) {
-	// Shall we add?
-	if (eregi("ADD", $sql) > 0) {
+function SQL_ALTER_TABLE ($sql, $F, $L) {
+	// This is the default result...
+	$result = false;
+
+	// Determine index/fulltext/unique word
+	//         12     3             3         2    2     3                3         2    2     3              3         21
+	$noIndex = ((eregi("INDEX", $sql) == false) && (eregi("FULLTEXT", $sql) == false) && (eregi("UNIQUE", $sql) == false));
+
+	// Shall we add/drop?
+	if (((eregi("ADD", $sql) > 0) || (eregi("DROP", $sql) > 0)) && ($noIndex)) {
 		// Extract table name
 		$tableArray = explode(" ", $sql);
 		$tableName = str_replace("`", "", $tableArray[2]);
@@ -318,15 +353,56 @@ function SQL_ALTER_TABLE($sql, $F, $L) {
 		$result = SQL_QUERY_ESC("SHOW COLUMNS FROM %s LIKE '%s'",
 			array($tableName, $columnName), $F, $L);
 
-		// Do we have no entry?
-		if (SQL_NUMROWS($result) == 0) {
+		// 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) && (eregi("ADD", $sql) > 0)) || ((SQL_NUMROWS($result) == 1) && (eregi("DROP", $sql) > 0))) {
 			// Do the query
-			return SQL_QUERY($sql, $F, $L, false);
+			$result = SQL_QUERY($sql, $F, $L, false);
 		} // END - if
 	} else {
 		// Send it to the SQL_QUERY() function
-		return SQL_QUERY($sql, $F, $L, false);
+		$result = SQL_QUERY($sql, $F, $L, false);
 	}
+
+	// Return result
+	return $result;
 }
+
+// Getter for SQL link
+function SQL_GET_LINK () {
+	// Init link
+	$link = null;
+
+	// Is it in the globals?
+	if (isset($GLOBALS['sql_link'])) {
+		// Then take it
+		$link = $GLOBALS['sql_link'];
+	} // END - if
+
+	// Return it
+	return $link;
+}
+
+// Setter for link
+function SQL_SET_LINK ($link) {
+	// Is this a resource or null?
+	if ((!is_resource($link)) && (!is_null($link))) {
+		// This should never happen!
+		trigger_error(sprintf("link is not resource or null. Type: %s", gettype($link)));
+	} // END - if
+
+	// Set it
+	$GLOBALS['sql_link'] = $link;
+}
+
+// Checks if the link is up
+function SQL_IS_LINK_UP () {
+	// Get the link
+	$link = SQL_GET_LINK();
+
+	// Return the result
+	return (is_resource($link));
+}
+
 //
 ?>