openrat-cms

commit 032095769e90b64d1c7bef0d622aed9da4eb81fe
parent 6956093c4bbd1559ab4f954fb91a0c55452d62f8
Author: dankert <devnull@localhost>
Date:   Tue, 20 Feb 2007 22:33:23 +0100

Warnungen in "foreach" vermeiden.

Diffstat:
serviceClasses/Sql.class.php
 | 
52
+++++++++++++++++++++++++++-------------------------

1 file changed, 27 insertions(+), 25 deletions(-)
diff --git a/serviceClasses/Sql.class.php b/serviceClasses/Sql.class.php
@@ -176,39 +176,41 @@ class Sql
 	 */
 	function setParam( $name,$value,$dieIfUnknown=true)
 	{
-//		Html::debug($this->src);
-//		Html::debug($this->param,'vor setParam() ('.$name.'='.$value.')');
-
-//   Nett gemeint, führt aber aktuell zu Fehlern, weil an vielen Stellen zu viele Parameter gefüllt werden.
-//   Daher erstmal deaktiviert.
-//		if	( !isset($this->param[$name]) )
-//		{
-//			if	( $dieIfUnknown )
-//				die("parameter '$name' unknown. SQL=".$this->src);
-//			else
-//				return;
-//		}
-
-		foreach( $this->param[$name] as $idx=>$xyz )
+
+		//   Nett gemeint, führt aber aktuell zu Fehlern, weil an vielen Stellen zu viele Parameter gefüllt werden.
+		//   Daher erstmal deaktiviert.
+		//		if	( !isset($this->param[$name]) )
+		//		{
+		//			if	( $dieIfUnknown )
+		//				die("parameter '$name' unknown. SQL=".$this->src);
+		//			else
+		//				return;
+		//		}
+
+		if	( !isset($this->param[$name]) )
+			return; // Parameter nicht vorhanden.
+
+		if	( is_array($this->param[$name]) )
 		{
-			$pos = $this->param[$name][$idx];
-			
-			$this->query = substr( $this->query,0,$pos ).$value.substr( $this->query,$pos );
-		
-			foreach( $this->param as $pn=>$par)
+			foreach( $this->param[$name] as $idx=>$xyz )
 			{
-				foreach( $par as $i=>$p )
+				$pos = $this->param[$name][$idx];
+				
+				$this->query = substr( $this->query,0,$pos ).$value.substr( $this->query,$pos );
+			
+				foreach( $this->param as $pn=>$par)
 				{
-					if	( $p > $pos )
-						$this->param[$pn][$i]=$p+strlen($value);
+					foreach( $par as $i=>$p )
+					{
+						if	( $p > $pos )
+							$this->param[$pn][$i]=$p+strlen($value);
+					}
 				}
+				
 			}
-			
 		}
 		
 		unset( $this->param[$name] );
-		
-//		Html::debug($this->param,'nach setParam()');
 	}