Laurent Destailleur 3 anos atrás
pai
commit
3b8ece6c0b

+ 4 - 3
htdocs/product/class/product.class.php

@@ -874,9 +874,10 @@ class Product extends CommonObject
 
 		$arrayofnonnegativevalue = array('weight'=>'Weight', 'width'=>'Width', 'height'=>'Height', 'length'=>'Length', 'surface'=>'Surface', 'volume'=>'Volume');
 		foreach ($arrayofnonnegativevalue as $key => $value) {
-			if (property_exists($this, $key) && $this->$key < 0) {
-				$langs->load("other");
-				$this->errors[] = $langs->trans("FieldCannotBeNegative", $langs->transnoentitiesnoconv($value));
+			if (property_exists($this, $key) && !empty($this->$key) && ($this->$key < 0)) {
+				$langs->loadLangs(array("main", "other"));
+				$this->error = $langs->trans("FieldCannotBeNegative", $langs->transnoentitiesnoconv($value));
+				$this->errors[] = $this->error;
 				$result = -4;
 			}
 		}

+ 1 - 1
test/phpunit/ProductTest.php

@@ -199,7 +199,7 @@ class ProductTest extends PHPUnit\Framework\TestCase
 		$localobject->note_private = 'New private note after update';
 		$result=$localobject->update($localobject->id, $user);
 		print __METHOD__." id=".$localobject->id." result=".$result."\n";
-		$this->assertLessThan($result, 0);
+		$this->assertLessThan($result, 0, 'Error '.$localobject->error);
 
 		return $localobject;
 	}