Merge pull request #218 from campino/master

db errors and update error mail

include/dba.php

@@ -8,189 +8,273 @@ require_once('include/datetime.php');
  *
  * For debugging, insert 'dbg(1);' anywhere in the program flow.
  * dbg(0); will turn it off. Logging is performed at LOGGER_DATA level.
- * When logging, all binary info is converted to text and html entities are escaped so that 
+ * When logging, all binary info is converted to
- * the debugging stream is safe to view within both terminals and web pages.
+ * text and html entities are escaped so that
+ * the debugging stream is safe to view
+ * within both terminals and web pages.
  *
  */
 
 if(! class_exists('dba')) {
-class dba {
 
-	private $debug = 0;
+	class dba {
-	private $db;
-	public  $mysqli = true;
-	public  $connected = false;
-	public  $error = false;
 
-	function __construct($server,$user,$pass,$db,$install = false) {
+		private $debug = 0;
+		private $db;
+		private $exceptions; 
 		
-		$server = trim($server);
+		public  $mysqli = true;
-		$user = trim($user);
+		public  $connected = false;
-		$pass = trim($pass);
+		public  $error = false;
-		$db = trim($db);
 
-		if (!(strlen($server) && strlen($user))){
+		function __construct($server,$user,$pass,$db,$install = false) {
-			$this->connected = false;
-			$this->db = null;
-			return;			
-		}
 
-		if($install) {
+			$server = trim($server);
-			if(strlen($server) && ($server !== 'localhost') && ($server !== '127.0.0.1')) {
+			$user = trim($user);
-				if(! dns_get_record($server, DNS_A + DNS_CNAME + DNS_PTR)) {
+			$pass = trim($pass);
-					$this->error = sprintf( t('Cannot locate DNS info for database server \'%s\''), $server);
+			$db = trim($db);
-					$this->connected = false;
+
-					$this->db = null;
+			//we need both, server and username, so fail if one is missing
-					return;
+			if (!(strlen($server) && strlen($user))){
+				$this->connected = false;
+				$this->db = null;
+				throw new InvalidArgumentException(t("Server name of user name are missing. "));
+			}
+
+			//when we are installing
+			if($install) {
+				if(strlen($server) && ($server !== 'localhost') && ($server !== '127.0.0.1')) {
+					if(! dns_get_record($server, DNS_A + DNS_CNAME + DNS_PTR)) {
+						$this->connected = false;
+						$this->db = null;
+						throw new InvalidArgumentException( t('Cannot locate DNS info for database server \'%s\''), $server);
+					}
+				}
+			}
+
+			if(class_exists('mysqli')) {
+				$this->db = new mysqli($server,$user,$pass,$db);
+				if(NULL === $this->db->connect_error) {
+					$this->connected = true;
+				} else {
+					throw new RuntimeException($this->db->connect_error);
+				}
+			} else {
+				$this->mysqli = false;
+				$this->db = mysql_connect($server,$user,$pass);
+				if($this->db && mysql_select_db($db,$this->db)) {
+					$this->connected = true;
+				} else {
+					throw new RuntimeException(mysql_error());
 				}
 			}
 		}
 
-		if(class_exists('mysqli')) {
+		public function excep($excep) {
-			$this->db = @new mysqli($server,$user,$pass,$db);
+			$this->exceptions=$excep; 
-			if(! mysqli_connect_errno()) {
+		}
-				$this->connected = true;
+		
+		public function getdb() {
+			return $this->db;
+		}
+
+		public function q($sql) {
+
+			if((! $this->db) || (! $this->connected)) {
+				$this->throwOrLog(new RuntimeException(t("There is no db connection. ")));
+				return;
 			}
-		}
-		else {
-			$this->mysqli = false;
-			$this->db = mysql_connect($server,$user,$pass);
-			if($this->db && mysql_select_db($db,$this->db)) {
-				$this->connected = true;
-			}
-		}
-		if(! $this->connected) {
-			$this->db = null;
-			if(! $install)
-				system_unavailable();
-		}
-	}
-
-	public function getdb() {
-		return $this->db;
-	}
-
-	public function q($sql) {
-		
-		if((! $this->db) || (! $this->connected))
-			return false;
-		
-		if($this->mysqli)
-			$result = @$this->db->query($sql);
-		else
-			$result = @mysql_query($sql,$this->db);
-
-		if($this->debug) {
-
-			$mesg = '';
 
 			if($this->mysqli) {
-				if($this->db->errno)
+				$result = $this->db->query($sql);
-					logger('dba: ' . $this->db->error);
+			} else {
-			}
+				$result = mysql_query($sql,$this->db);
-			elseif(mysql_errno($this->db))
-				logger('dba: ' . mysql_error($this->db));
-
-			if($result === false)
-				$mesg = 'false';
-			elseif($result === true)
-				$mesg = 'true';
-			else {
-				if($this->mysqli)
-					$mesg = $result->num_rows . ' results' . EOL;
-    			else
-					$mesg = mysql_num_rows($result) . ' results' . EOL;
 			}
 
-			$str =  'SQL = ' . printable($sql) . EOL . 'SQL returned ' . $mesg . EOL;
+			//on debug mode or fail, the query is written to the log.
+			//this won't work if logger can not read it's logging level
+			//from the db.
+			if($this->debug || FALSE === $result) {
 
-			logger('dba: ' . $str );
+				$mesg = '';
+
+				if($result === false) {
+					$mesg = 'false '.$this->error();
+				} elseif($result === true) {
+					$mesg = 'true';
+				} else {
+					if($this->mysqli) {
+						$mesg = $result->num_rows . t(' results') . EOL;
+					} else {
+						$mesg = mysql_num_rows($result) . t(' results') . EOL;
+					}
+				}
+
+				$str =  'SQL = ' . printable($sql) . EOL . t('SQL returned ') . $mesg . EOL;
+
+
+			 // If dbfail.out exists, we will write any failed calls directly to it,
+			 // regardless of any logging that may or may nor be in effect.
+			 // These usually indicate SQL syntax errors that need to be resolved.
+				if(file_exists('dbfail.out')) {
+					file_put_contents('dbfail.out', datetime_convert() . "\n" . $str . "\n", FILE_APPEND);
+				}
+				logger('dba: ' . $str );
+				if(FALSE===$result) {
+					$this->throwOrLog(new RuntimeException('dba: ' . $str));
+					return; 
+				}
+			}
+				
+
+			if($result === true) {
+				return $result;
+			}
+
+			$r = array();
+			if($this->mysqli) {
+				if($result->num_rows) {
+					while($x = $result->fetch_array(MYSQLI_ASSOC)) {
+						$r[] = $x;
+					}
+					$result->free_result();
+				}
+			} else {
+				if(mysql_num_rows($result)) {
+					while($x = mysql_fetch_array($result, MYSQL_ASSOC)) {
+						$r[] = $x;
+					}
+					mysql_free_result($result);
+				}
+			}
+
+
+			if($this->debug) {
+				logger('dba: ' . printable(print_r($r, true)));
+			}
+			return($r);
+		}
+
+		private function error() {
+			if($this->mysqli) {
+				return $this->db->error;
+			} else {
+				return mysql_error($this->db);
+			}
+		}
+		
+		private function throwOrLog(Exception $ex) {
+			if($this->exceptions) {
+				throw $ex; 
+			} else {
+				logger('dba: '.$ex->getMessage()); 
+			}
 		}
 		
 		/**
-		 * If dbfail.out exists, we will write any failed calls directly to it,
+		 * starts a transaction. Transactions need to be finished with 
-		 * regardless of any logging that may or may nor be in effect.
+		 * commit() or rollback(). Please mind that the db table engine may
-		 * These usually indicate SQL syntax errors that need to be resolved.
+		 * not support this. 
 		 */
-
+		public function beginTransaction() {
-		if($result === false) {
+			if($this->mysqli) {
-			logger('dba: ' . printable($sql) . ' returned false.');
+				return $this->db->autocommit(false);
-			if(file_exists('dbfail.out'))
+			} else {
-				file_put_contents('dbfail.out', datetime_convert() . "\n" . printable($sql) . ' returned false' . "\n", FILE_APPEND);
+				//no transaction support in mysql module...
-		}
+				mysql_query('SET AUTOCOMMIT = 0;', $db); 
-
-		if(($result === true) || ($result === false))
-			return $result;
-
-		$r = array();
-		if($this->mysqli) {
-			if($result->num_rows) {
-				while($x = $result->fetch_array(MYSQLI_ASSOC))
-					$r[] = $x;
-				$result->free_result();
-			}
-		}
-		else {
-			if(mysql_num_rows($result)) {
-				while($x = mysql_fetch_array($result, MYSQL_ASSOC))
-					$r[] = $x;
-				mysql_free_result($result);
 			}
 		}
 		
-    
+		/**
-		if($this->debug)
+		 * rollback a transaction. So, rollback anything that was done since the last call 
-			logger('dba: ' . printable(print_r($r, true)));
+		 * to beginTransaction(). 
-		return($r);
+		 */
-	}
+		public function rollback() {
-
+			if($this->mysqli) {
-	public function dbg($dbg) {
+				return $this->db->rollback();
-		$this->debug = $dbg;
+			} else {
-	}
+				//no transaction support in mysql module...
-
+				mysql_query('ROLLBACK;', $db);
-	public function escape($str) {
+			}
-		if($this->db && $this->connected) {
+			$this->stopTransaction(); 
-			if($this->mysqli)
-				return @$this->db->real_escape_string($str);
-			else
-				return @mysql_real_escape_string($str,$this->db);
 		}
-	}
 
-	function __destruct() {
+		/**
-		if ($this->db) 
+		 * commit a transaction. So, write any query to the database. 
-			if($this->mysqli)
+		 */
-				$this->db->close();
+		public function commit() {
-			else
+			if($this->mysqli) {
+				return $this->db->commit();
+			} else {
+				//no transaction support in mysql module...
+				mysql_query('COMMIT;', $db);
+			}
+			$this->stopTransaction();
+		}
+		
+		private function stopTransaction() {
+			if($this->mysqli) {
+				return $this->db->autocommit(true);
+			} else {
+				//no transaction support in mysql module...
+				mysql_query('SET AUTOCOMMIT = 1;', $db);
+			}
+		}
+		
+		public function dbg($dbg) {
+			$this->debug = $dbg;
+		}
+
+		public function escape($str) {
+			if($this->db && $this->connected) {
+				if($this->mysqli) {
+					return $this->db->real_escape_string($str);
+				} else {
+					return mysql_real_escape_string($str,$this->db);
+				}
+			}
+		}
+
+		function __destruct() {
+			if ($this->db) {
+				if($this->mysqli) {
+					$this->db->close();
+				}
+			} else {
 				mysql_close($this->db);
+			}
+		}
 	}
-}}
+}
 
 if(! function_exists('printable')) {
-function printable($s) {
+	function printable($s) {
-	$s = preg_replace("~([\x01-\x08\x0E-\x0F\x10-\x1F\x7F-\xFF])~",".", $s);
+		$s = preg_replace("~([\x01-\x08\x0E-\x0F\x10-\x1F\x7F-\xFF])~",".", $s);
-	$s = str_replace("\x00",'.',$s);
+		$s = str_replace("\x00",'.',$s);
-	if(x($_SERVER,'SERVER_NAME'))
+		if(x($_SERVER,'SERVER_NAME'))
-		$s = escape_tags($s);
+			$s = escape_tags($s);
-	return $s;
+		return $s;
-}}
+	}
+}
 
 // Procedural functions
 if(! function_exists('dbg')) {
-function dbg($state) {
+	function dbg($state) {
-	global $db;
+		global $db;
-	if($db)
+		if($db)
-	$db->dbg($state);
+			$db->dbg($state);
-}}
+	}
+}
 
 if(! function_exists('dbesc')) {
-function dbesc($str) {
+	function dbesc($str) {
-	global $db;
+		global $db;
-	if($db && $db->connected)
+		if($db && $db->connected)
-		return($db->escape($str));
+			return($db->escape($str));
-	else
+		else
-		return(str_replace("'","\\'",$str));
+			return(str_replace("'","\\'",$str));
-}}
+	}
+}
 
 
 
@@ -200,29 +284,30 @@ function dbesc($str) {
 //                   'user', 1);
 
 if(! function_exists('q')) {
-function q($sql) {
+	function q($sql) {
 
-	global $db;
+		global $db;
-	$args = func_get_args();
+		$args = func_get_args();
-	unset($args[0]);
+		unset($args[0]);
+
+		if($db && $db->connected) {
+			$stmt = vsprintf($sql,$args);
+			if($stmt === false)
+				logger('dba: vsprintf error: ' . print_r(debug_backtrace(),true));
+			return $db->q($stmt);
+		}
+
+		/**
+		 *
+		 * This will happen occasionally trying to store the
+		 * session data after abnormal program termination
+		 *
+		 */
+		logger('dba: no database: ' . print_r($args,true));
+		return false;
 
-	if($db && $db->connected) {
-		$stmt = vsprintf($sql,$args);
-		if($stmt === false)
-			logger('dba: vsprintf error: ' . print_r(debug_backtrace(),true));
-		return $db->q($stmt);
 	}
-
+}
-	/**
-	 *
-	 * This will happen occasionally trying to store the 
-	 * session data after abnormal program termination 
-	 *
-	 */
-	logger('dba: no database: ' . print_r($args,true));
-	return false; 
-
-}}
 
 /**
  *
@@ -231,15 +316,16 @@ function q($sql) {
  */
 
 if(! function_exists('dbq')) {
-function dbq($sql) {
+	function dbq($sql) {
 
-	global $db;
+		global $db;
-	if($db && $db->connected)
+		if($db && $db->connected)
-		$ret = $db->q($sql);
+			$ret = $db->q($sql);
-	else
+		else
-		$ret = false;
+			$ret = false;
-	return $ret;
+		return $ret;
-}}
+	}
+}
 
 
 // Caller is responsible for ensuring that any integer arguments to
@@ -249,17 +335,19 @@ function dbq($sql) {
 
 
 if(! function_exists('dbesc_array_cb')) {
-function dbesc_array_cb(&$item, $key) {
+	function dbesc_array_cb(&$item, $key) {
-	if(is_string($item))
+		if(is_string($item))
-		$item = dbesc($item);
+			$item = dbesc($item);
-}}
+	}
+}
 
 
 if(! function_exists('dbesc_array')) {
-function dbesc_array(&$arr) {
+	function dbesc_array(&$arr) {
-	if(is_array($arr) && count($arr)) {
+		if(is_array($arr) && count($arr)) {
-		array_walk($arr,'dbesc_array_cb');
+			array_walk($arr,'dbesc_array_cb');
+		}
 	}
-}}		
+}
 
 

view/de/strings.php

@@ -777,7 +777,7 @@ $a->strings["{0} tagged %s's post with #%s"] = "{0} hat %ss Beitrag mit dem Schl
 $a->strings["{0} mentioned you in a post"] = "{0} hat dich in einem Beitrag erwähnt";
 $a->strings["OpenID protocol error. No ID returned."] = "OpenID Protokollfehler. Keine ID zurückgegeben.";
 $a->strings["Account not found and OpenID registration is not permitted on this site."] = "Account wurde nicht gefunden und OpenID Registrierung auf diesem Server nicht gestattet.";
-$a->strings["Login failed."] = "Annmeldung fehlgeschlagen.";
+$a->strings["Login failed."] = "Anmeldung fehlgeschlagen.";
 $a->strings["Connect URL missing."] = "Connect-URL fehlt";
 $a->strings["This site is not configured to allow communications with other networks."] = "Diese Seite ist so konfiguriert, dass keine Kommunikation mit anderen Netzwerken erfolgen kann.";
 $a->strings["No compatible communication protocols or feeds were discovered."] = "Es wurden keine kompatiblen Kommunikationsprotokolle oder Feeds gefunden.";
@@ -1511,3 +1511,4 @@ $a->strings["[today]"] = "[heute]";
 $a->strings["Event Reminders"] = "Veranstaltungserinnerungen";
 $a->strings["Events this week:"] = "Veranstaltungen diese Woche";
 $a->strings["[No description]"] = "[keine Beschreibung]";
+$a->strings['Update Error at %s'] = 'Fehler beim Updaten von %s'; 

view/de/update_fail_eml.tpl

@@ -0,0 +1,12 @@
+Hey, 
+Ich bin's,  $sitename.   
+Die Friendica-Entwickler haben gerade Update $update freigegeben, 
+aber als ich es installieren wollte, ist irgendetwas schief gegangen.  
+Das sollte schnell repariert werden und alleine schaffe ich es nicht. 
+Wende dich bitte an einen Friendica-Entwickler, wenn du mir nicht selbst helfen kannst. 
+Meine Datenbank könnte ziemlich durcheinander sein. 
+
+Die Fehlermeldung ist '$error'.   
+
+Tut mir leid, 
+dein Friendica Server unter $siteurl

view/update_fail_eml.tpl

@@ -0,0 +1,11 @@
+Hey, 
+I'm $sitename.   
+The friendica developers released update $update recently, 
+but when I tried to install it, something went terribly wrong.  
+This needs to be fixed soon and I can't do it alone. Please contact a 
+friendica developer if you can not help me on your own. My database might be invalid. 
+
+The error message is '$error'.   
+
+I'm sorry, 
+your friendica server at $siteurl