detain / db_abstraction

src/Mysqli/Db.php

Spacing +17 added lines, -17 removed lines

@@ -71,7 +71,7 @@ 
 			if ($this->connectionAttempt > 1)
 				error_log("MySQLi Connection Attempt #{$this->connectionAttempt}/{$this->maxConnectErrors}");
 			if ($this->connectionAttempt >= $this->maxConnectErrors) {
-				$this->halt("connect($host, $user, \$password) failed. " . $mysqli->connect_error);
+				$this->halt("connect($host, $user, \$password) failed. ".$mysqli->connect_error);
 				return 0;
 			}
 			//$this->linkId = new mysqli($host, $user, $password, $database);
@@ -87,7 +87,7 @@ 
 			* }
 			*/
 			if ($this->linkId->connect_errno) {
-				$this->halt("connect($host, $user, \$password) failed. " . $mysqli->connect_error);
+				$this->halt("connect($host, $user, \$password) failed. ".$mysqli->connect_error);
 				return 0;
 			}
 			/*if ($this->characterSet != '') {
@@ -255,7 +255,7 @@ 
 			$this->Errno = @mysqli_errno($this->linkId);
 			$this->Error = @mysqli_error($this->linkId);
 			if ($try == 1 && (null === $this->queryId || $this->queryId === FALSE)) {
-				$email = "MySQLi Error<br>\n".'Query: '.$queryString . "<br>\n".'Error #'.$this->Errno.': '.$this->Error . "<br>\n".'Line: '.$line . "<br>\n".'File: '.$file . "<br>\n" . (isset($GLOBALS['tf']) ? 'User: '.$GLOBALS['tf']->session->account_id . "<br>\n" : '');
+				$email = "MySQLi Error<br>\n".'Query: '.$queryString."<br>\n".'Error #'.$this->Errno.': '.$this->Error."<br>\n".'Line: '.$line."<br>\n".'File: '.$file."<br>\n".(isset($GLOBALS['tf']) ? 'User: '.$GLOBALS['tf']->session->account_id."<br>\n" : '');
 				$email .= '<br><br>Request Variables:<br>'.print_r($_REQUEST, TRUE);
 				$email .= '<br><br>Server Variables:<br>'.print_r($_SERVER, TRUE);
 				$subject = $_SERVER['HOSTNAME'].' MySQLi Error';
@@ -322,7 +322,7 @@ 
 		if ($status) {
 			$this->Row = $pos;
 		} else {
-			$this->halt("seek($pos) failed: result has " . $this->num_rows().' rows');
+			$this->halt("seek($pos) failed: result has ".$this->num_rows().' rows');
 			/* half assed attempt to save the day,
 			* but do not consider this documented or even
 			* desirable behaviour.
@@ -380,7 +380,7 @@ 
 	 */
 	public function getLastInsertId($table, $field) {
 		if (!isset($table) || $table == '' || !isset($field) || $field == '')
-			return - 1;
+			return -1;
 
 		return @mysqli_insert_id($this->linkId);
 	}
@@ -514,22 +514,22 @@ 
  * @param int|string $field
  * @return bool
  */
-function mysqli_result($result, $row, $field=0) {
-	if ($result===false) return FALSE;
-	if ($row>=mysqli_num_rows($result)) return FALSE;
+function mysqli_result($result, $row, $field = 0) {
+	if ($result === false) return FALSE;
+	if ($row >= mysqli_num_rows($result)) return FALSE;
 	if (is_string($field) && !(mb_strpos($field, '.') === FALSE)) {
-		$tField=explode('.', $field);
-		$field=-1;
-		$tFields=mysqli_fetch_fields($result);
-		for ($id=0, $idMax = mysqli_num_fields($result);$id<$idMax;$id++) {
-			if ($tFields[$id]->table==$tField[0] && $tFields[$id]->name==$tField[1]) {
-				$field=$id;
+		$tField = explode('.', $field);
+		$field = -1;
+		$tFields = mysqli_fetch_fields($result);
+		for ($id = 0, $idMax = mysqli_num_fields($result); $id < $idMax; $id++) {
+			if ($tFields[$id]->table == $tField[0] && $tFields[$id]->name == $tField[1]) {
+				$field = $id;
 				break;
 			}
 		}
-		if ($field==-1) return FALSE;
+		if ($field == -1) return FALSE;
 	}
 	mysqli_data_seek($result, $row);
-	$line=mysqli_fetch_array($result);
-	return isset($line[$field])?$line[$field]:false;
+	$line = mysqli_fetch_array($result);
+	return isset($line[$field]) ? $line[$field] : false;
 }

src/Pdo/Db.php

Spacing +6 added lines, -6 removed lines

@@ -198,18 +198,18 @@ 
 		$this->queryId = $this->linkId->prepare($queryString);
 		$success = $this->queryId->execute();
 		$this->Rows = $this->queryId->fetchAll();
-		$this->log("PDO Query $queryString (S:$success) - " . count($this->Rows).' Rows', __LINE__, __FILE__);
+		$this->log("PDO Query $queryString (S:$success) - ".count($this->Rows).' Rows', __LINE__, __FILE__);
 		$this->Row = 0;
 		if ($success === FALSE) {
-			$email = "MySQL Error<br>\n".'Query: '.$queryString . "<br>\n".'Error #'.print_r($this->queryId->errorInfo(), TRUE) . "<br>\n".'Line: '.$line . "<br>\n".'File: '.$file . "<br>\n" . (isset($GLOBALS['tf']) ? 'User: '.$GLOBALS['tf']->session->account_id . "<br>\n" : '');
+			$email = "MySQL Error<br>\n".'Query: '.$queryString."<br>\n".'Error #'.print_r($this->queryId->errorInfo(), TRUE)."<br>\n".'Line: '.$line."<br>\n".'File: '.$file."<br>\n".(isset($GLOBALS['tf']) ? 'User: '.$GLOBALS['tf']->session->account_id."<br>\n" : '');
 
 			$email .= '<br><br>Request Variables:<br>';
 			foreach ($_REQUEST as $key => $value)
-				$email .= $key.': '.$value . "<br>\n";
+				$email .= $key.': '.$value."<br>\n";
 
 			$email .= '<br><br>Server Variables:<br>';
 			foreach ($_SERVER as $key => $value)
-				$email .= $key.': '.$value . "<br>\n";
+				$email .= $key.': '.$value."<br>\n";
 			$subject = $_SERVER['HOSTNAME'].' MySQLi Error';
 			$headers = '';
 			$headers .= 'MIME-Version: 1.0'.PHP_EOL;
@@ -259,7 +259,7 @@ 
 		if (isset($this->Rows[$pos])) {
 			$this->Row = $pos;
 		} else {
-			$this->halt("seek($pos) failed: result has " . count($this->Rows).' rows');
+			$this->halt("seek($pos) failed: result has ".count($this->Rows).' rows');
 			/* half assed attempt to save the day,
 			* but do not consider this documented or even
 			* desirable behaviour.
@@ -301,7 +301,7 @@ 
 	 */
 	public function getLastInsertId($table, $field) {
 		if (!isset($table) || $table == '' || !isset($field) || $field == '')
-			return - 1;
+			return -1;
 		return $this->linkId->lastInsertId();
 	}
 

src/Generic.php

Spacing +15 added lines, -15 removed lines

@@ -164,9 +164,9 @@ 
 	public function limit($start = 0) {
 		echo '<b>Warning: limit() is no longer used, use limitQuery()</b>';
 		if ($start == 0) {
-			$s = 'limit '.(int)$this->maxMatches;
+			$s = 'limit '.(int) $this->maxMatches;
 		} else {
-			$s = 'limit '.(int)$start.','.(int)$this->maxMatches;
+			$s = 'limit '.(int) $start.','.(int) $this->maxMatches;
 		}
 		return $s;
 	}
@@ -185,9 +185,9 @@ 
 		if (!$numRows)
 			$numRows = $this->maxMatches;
 		if ($offset == 0) {
-			$queryString .= ' LIMIT '.(int)$numRows;
+			$queryString .= ' LIMIT '.(int) $numRows;
 		} else {
-			$queryString .= ' LIMIT '.(int)$offset.','.(int)$numRows;
+			$queryString .= ' LIMIT '.(int) $offset.','.(int) $numRows;
 		}
 
 		if ($this->Debug)
@@ -258,26 +258,26 @@ 
 	 * @return mixed|void
 	 */
 	public function logBackTrace($msg, $line = '', $file = '') {
-		$backtrace=(function_exists('debug_backtrace') ? debug_backtrace() : []);
+		$backtrace = (function_exists('debug_backtrace') ? debug_backtrace() : []);
 		$this->log(
 			('' !== getenv('REQUEST_URI') ? ' '.getenv('REQUEST_URI') : '').
 			((isset($_POST) && count($_POST)) ? ' POST='.json_encode($_POST) : '').
 			((isset($_GET) && count($_GET)) ? ' GET='.json_encode($_GET) : '').
 			((isset($_FILES) && count($_FILES)) ? ' FILES='.json_encode($_FILES) : '').
 			('' !== getenv('HTTP_USER_AGENT') ? ' AGENT="'.getenv('HTTP_USER_AGENT').'"' : '').
-			(isset($_SERVER[ 'REQUEST_METHOD' ]) ?' METHOD="'. $_SERVER['REQUEST_METHOD']. '"'.
-				($_SERVER['REQUEST_METHOD'] === 'POST' ? ' POST="'. json_encode($_POST). '"' : '') : ''), $line, $file, 'error');
-		for($level=1, $levelMax = count($backtrace);$level < $levelMax;$level++) {
-			$message=(isset($backtrace[$level]['file']) ? 'File: '. $backtrace[$level]['file'] : '').
-				(isset($backtrace[$level]['line']) ? ' Line: '. $backtrace[$level]['line'] : '').
-				' Function: '.(isset($backtrace[$level] ['class']) ? '(class '. $backtrace[$level] ['class'].') ' : '') .
+			(isset($_SERVER['REQUEST_METHOD']) ? ' METHOD="'.$_SERVER['REQUEST_METHOD'].'"'.
+				($_SERVER['REQUEST_METHOD'] === 'POST' ? ' POST="'.json_encode($_POST).'"' : '') : ''), $line, $file, 'error');
+		for ($level = 1, $levelMax = count($backtrace); $level < $levelMax; $level++) {
+			$message = (isset($backtrace[$level]['file']) ? 'File: '.$backtrace[$level]['file'] : '').
+				(isset($backtrace[$level]['line']) ? ' Line: '.$backtrace[$level]['line'] : '').
+				' Function: '.(isset($backtrace[$level] ['class']) ? '(class '.$backtrace[$level] ['class'].') ' : '').
 				(isset($backtrace[$level] ['type']) ? $backtrace[$level] ['type'].' ' : '').
 				$backtrace[$level] ['function'].'(';
-			if(isset($backtrace[$level] ['args']))
-				for($argument = 0, $argumentMax = count($backtrace[$level]['args']); $argument < $argumentMax; $argument++)
+			if (isset($backtrace[$level] ['args']))
+				for ($argument = 0, $argumentMax = count($backtrace[$level]['args']); $argument < $argumentMax; $argument++)
 					$message .= ($argument > 0 ? ', ' : '').
 						(is_object($backtrace[$level]['args'][$argument]) ? 'class '.get_class($backtrace[$level]['args'][$argument]) : json_encode($backtrace[$level]['args'][$argument]));
-			$message.=')';
+			$message .= ')';
 			$this->log($message, $line, $file, 'error');
 		}
 	}
@@ -292,7 +292,7 @@ 
 		$this->log("Database error: $msg", $line, $file, 'error');
 		if ($this->Errno != '0' || !in_array($this->Error, '', '()')) {
 			$sqlstate = mysqli_sqlstate($this->linkId);
-			$this->log("MySQLi SQLState: {$sqlstate}. Error: " . $this->Errno . ' (' . $this->Error . ')', $line, $file, 'error');
+			$this->log("MySQLi SQLState: {$sqlstate}. Error: ".$this->Errno.' ('.$this->Error.')', $line, $file, 'error');
 		}
 		$this->logBackTrace($msg, $line, $file);
 	}

src/Mdb2/Db.php

Spacing +8 added lines, -8 removed lines

@@ -36,10 +36,10 @@ 
 	public function quote($text = '', $type = 'text') {
 		switch ($type) {
 			case 'text':
-				return "'" . mysqli_real_escape_string($this->linkId, $text) . "'";
+				return "'".mysqli_real_escape_string($this->linkId, $text)."'";
 				break;
 			case 'integer':
-				return (int)$text;
+				return (int) $text;
 				break;
 			default:
 				return $text;
@@ -138,7 +138,7 @@ 
 			if ($this->connectionAttempt > 1)
 				error_log("MySQLi Connection Attempt #{$this->connectionAttempt}/{$this->maxConnectErrors}");
 			if ($this->connectionAttempt >= $this->maxConnectErrors) {
-				$this->halt("connect($host, $user, \$password) failed. " . $mysqli->connect_error);
+				$this->halt("connect($host, $user, \$password) failed. ".$mysqli->connect_error);
 				return 0;
 			}
 			$this->linkId = mysqli_init();
@@ -146,7 +146,7 @@ 
 			$this->linkId->real_connect($host, $user, $password, $database);
 			$this->linkId->set_charset($this->characterSet);
 			if ($this->linkId->connect_errno) {
-				$this->halt("connect($host, $user, \$password) failed. " . $mysqli->connect_error);
+				$this->halt("connect($host, $user, \$password) failed. ".$mysqli->connect_error);
 				return 0;
 			}
 		}
@@ -292,8 +292,8 @@ 
 		}
 		$this->haltOnError = $haltPrev;
 		if ($this->queryId === FALSE) {
-			$email = "MySQLi Error<br>\n".'Query: '.$queryString . "<br>\n".'Error #'.$this->Errno.': '.$this->Error . "<br>\n".'Line: '.$line . "<br>\n".'File: '.$file . "<br>\n" . (isset($GLOBALS['tf']) ?
-					'User: '.$GLOBALS['tf']->session->account_id . "<br>\n" : '');
+			$email = "MySQLi Error<br>\n".'Query: '.$queryString."<br>\n".'Error #'.$this->Errno.': '.$this->Error."<br>\n".'Line: '.$line."<br>\n".'File: '.$file."<br>\n".(isset($GLOBALS['tf']) ?
+					'User: '.$GLOBALS['tf']->session->account_id."<br>\n" : '');
 
 			$email .= '<br><br>Request Variables:<br>'.print_r($_REQUEST, TRUE);
 			$email .= '<br><br>Server Variables:<br>'.print_r($_SERVER, TRUE);
@@ -348,7 +348,7 @@ 
 		if ($status) {
 			$this->Row = $pos;
 		} else {
-			$this->halt("seek($pos) failed: result has " . $this->num_rows().' rows');
+			$this->halt("seek($pos) failed: result has ".$this->num_rows().' rows');
 			/* half assed attempt to save the day,
 			* but do not consider this documented or even
 			* desirable behaviour.
@@ -398,7 +398,7 @@ 
 		*/
 
 		if (!isset($table) || $table == '' || !isset($field) || $field == '')
-			return - 1;
+			return -1;
 
 		return @mysqli_insert_id($this->linkId);
 	}

src/Adodb/Db.php

Spacing +7 added lines, -7 removed lines

@@ -177,16 +177,16 @@ 
 			$this->queryId = $this->linkId->Execute($queryString);
 		}
 		catch (exception $e) {
-			$email = "MySQL Error<br>\n".'Query: '.$queryString . "<br>\n".'Error #'.print_r($e, TRUE) . "<br>\n".'Line: '.$line . "<br>\n".'File: '.$file . "<br>\n" . (isset($GLOBALS['tf']) ?
-					'User: '.$GLOBALS['tf']->session->account_id . "<br>\n" : '');
+			$email = "MySQL Error<br>\n".'Query: '.$queryString."<br>\n".'Error #'.print_r($e, TRUE)."<br>\n".'Line: '.$line."<br>\n".'File: '.$file."<br>\n".(isset($GLOBALS['tf']) ?
+					'User: '.$GLOBALS['tf']->session->account_id."<br>\n" : '');
 
 			$email .= '<br><br>Request Variables:<br>';
 			foreach ($_REQUEST as $key => $value)
-				$email .= $key.': '.$value . "<br>\n";
+				$email .= $key.': '.$value."<br>\n";
 
 			$email .= '<br><br>Server Variables:<br>';
 			foreach ($_SERVER as $key => $value)
-				$email .= $key.': '.$value . "<br>\n";
+				$email .= $key.': '.$value."<br>\n";
 			$subject = $_SERVER['HOSTNAME'].' MySQLi Error';
 			$headers = '';
 			$headers .= 'MIME-Version: 1.0'.PHP_EOL;
@@ -196,7 +196,7 @@ 
 			admin_mail($subject, $email, $headers, FALSE, 'admin/sql_error.tpl');
 			$this->halt('Invalid SQL: '.$queryString, $line, $file);
 		}
-		$this->log("ADOdb Query $queryString (S:$success) - " . count($this->Rows).' Rows', __LINE__, __FILE__);
+		$this->log("ADOdb Query $queryString (S:$success) - ".count($this->Rows).' Rows', __LINE__, __FILE__);
 		$this->Row = 0;
 
 		// Will return nada if it fails. That's fine.
@@ -232,7 +232,7 @@ 
 		if (isset($this->Rows[$pos])) {
 			$this->Row = $pos;
 		} else {
-			$this->halt("seek($pos) failed: result has " . count($this->Rows).' rows');
+			$this->halt("seek($pos) failed: result has ".count($this->Rows).' rows');
 			/* half assed attempt to save the day,
 			* but do not consider this documented or even
 			* desirable behaviour.
@@ -374,7 +374,7 @@ 
 	public function haltmsg($msg, $line = '', $file = '') {
 		$this->log("Database error: $msg", $line, $file, 'error');
 		if ($this->linkId->ErrorNo() != '0' && $this->linkId->ErrorMsg() != '')
-			$this->log('ADOdb SQL Error: ' . $this->linkId->ErrorMsg(), $line, $file, 'error');
+			$this->log('ADOdb SQL Error: '.$this->linkId->ErrorMsg(), $line, $file, 'error');
 		$this->logBackTrace($msg, $line, $file);
 	}
 

src/Pgsql/Db.php

Spacing +6 added lines, -6 removed lines

@@ -32,7 +32,7 @@ 
 	 */
 	public function ifadd($add, $me) {
 		if ('' != $add)
-			return ' '.$me . $add;
+			return ' '.$me.$add;
 		return '';
 	}
 
@@ -82,7 +82,7 @@ 
 	 */
 	public function connect() {
 		if (0 == $this->linkId) {
-			$connectString = 'dbname='.$this->database . $this->ifadd($this->host, 'host=') . $this->ifadd($this->port, 'port=') . $this->ifadd($this->user, 'user=') . $this->ifadd("'" . $this->password . "'", 'password=');
+			$connectString = 'dbname='.$this->database.$this->ifadd($this->host, 'host=').$this->ifadd($this->port, 'port=').$this->ifadd($this->user, 'user=').$this->ifadd("'".$this->password."'", 'password=');
 			$this->linkId = pg_pconnect($connectString);
 
 			if (!$this->linkId) {
@@ -262,21 +262,21 @@ 
 		* conditional code to the apps.
 		*/
 		if (!isset($table) || $table == '' || !isset($field) || $field == '')
-			return - 1;
+			return -1;
 
 		$oid = pg_getlastoid($this->queryId);
 		if ($oid == -1)
-			return - 1;
+			return -1;
 
 		$result = @pg_exec($this->linkId, "select $field from $table where oid=$oid");
 		if (!$result)
-			return - 1;
+			return -1;
 
 		$Record = @pg_fetch_array($result, 0);
 		@pg_freeresult($result);
 		if (!is_array($Record)) /* OID not found? */
 		{
-			return - 1;
+			return -1;
 		}
 
 		return $Record[0];