Fix curly braces

Fix can set desiredstock to (int) 0 from API Rest

htdocs/core/class/utils.class.php

@@ -292,17 +292,19 @@ class Utils
 		        dol_syslog("Run command ".$fullcommandcrypted);
 		        $handlein = popen($fullcommandclear, 'r');
 		        $i=0;
-		        while (!feof($handlein))
-		        {
-		            $i++;   // output line number
-		            $read = fgets($handlein);
-		            // Exclude warning line we don't want
-		            if ($i == 1 && preg_match('/Warning.*Using a password/i', $read)) continue;
-		            fwrite($handle,$read);
-		            if (preg_match('/'.preg_quote('-- Dump completed').'/i',$read)) $ok=1;
-		            elseif (preg_match('/'.preg_quote('SET SQL_NOTES=@OLD_SQL_NOTES').'/i',$read)) $ok=1;
+		        if ($handlein) {
+			        while (!feof($handlein))
+			        {
+			            $i++;   // output line number
+			            $read = fgets($handlein);
+			            // Exclude warning line we don't want
+			            if ($i == 1 && preg_match('/Warning.*Using a password/i', $read)) continue;
+			            fwrite($handle,$read);
+			            if (preg_match('/'.preg_quote('-- Dump completed').'/i',$read)) $ok=1;
+			            elseif (preg_match('/'.preg_quote('SET SQL_NOTES=@OLD_SQL_NOTES').'/i',$read)) $ok=1;
+			        }
+			        pclose($handlein);
 		        }
-		        pclose($handlein);
 
 		        if ($compression == 'none') fclose($handle);
 		        if ($compression == 'gz')   gzclose($handle);

htdocs/core/lib/functions.lib.php

@@ -4965,7 +4965,7 @@ function dol_string_is_good_iso($s)
 	$ok=1;
 	for($scursor=0;$scursor<$len;$scursor++)
 	{
-		$ordchar=ord($s{$scursor});
+		$ordchar=ord($s[$scursor]);
 		//print $scursor.'-'.$ordchar.'<br>';
 		if ($ordchar < 32 && $ordchar != 13 && $ordchar != 10) { $ok=0; break; }
 		if ($ordchar > 126 && $ordchar < 160) { $ok=0; break; }

htdocs/core/lib/json.lib.php

@@ -118,7 +118,7 @@ function _val($val)
 	     */
 	    for ($c = 0; $c < $strlen_var; ++$c) {
 
-	        $ord_var_c = ord($val{$c});
+	        $ord_var_c = ord($val[$c]);
 
 	        switch (true) {
 	            case $ord_var_c == 0x08:
@@ -141,7 +141,7 @@ function _val($val)
 	            case $ord_var_c == 0x2F:
 	            case $ord_var_c == 0x5C:
 	                // double quote, slash, slosh
-	                $ascii .= '\\'.$val{$c};
+	                $ascii .= '\\'.$val[$c];
 	                break;
 
 	            case (($ord_var_c >= 0x20) && ($ord_var_c <= 0x7F)):

htdocs/product/class/product.class.php

@@ -884,7 +884,7 @@ class Product extends CommonObject
 			$sql.= ", duration = '" . $this->db->escape($this->duration_value . $this->duration_unit) ."'";
 			$sql.= ", accountancy_code_buy = '" . $this->db->escape($this->accountancy_code_buy)."'";
 			$sql.= ", accountancy_code_sell= '" . $this->db->escape($this->accountancy_code_sell)."'";
-			$sql.= ", desiredstock = " . ((isset($this->desiredstock) && $this->desiredstock != '') ? $this->desiredstock : "null");
+			$sql.= ", desiredstock = " . ((isset($this->desiredstock) && is_numeric($this->desiredstock)) ? $this->desiredstock : "null");
 			$sql.= ", cost_price = " . ($this->cost_price != '' ? $this->db->escape($this->cost_price) : 'null');
 	        $sql.= ", fk_unit= " . (!$this->fk_unit ? 'NULL' : $this->fk_unit);
 	        $sql.= ", price_autogen = " . (!$this->price_autogen ? 0 : 1);