diff --git a/htdocs/modulebuilder/index.php b/htdocs/modulebuilder/index.php
index 06907b49742..82567e05fac 100644
--- a/htdocs/modulebuilder/index.php
+++ b/htdocs/modulebuilder/index.php
@@ -1295,8 +1295,8 @@ if ($dirins && $action == 'addproperty' && empty($cancel) && !empty($module) &&
 			$error++;
 			setEventMessages($langs->trans("ErrorFieldRequired", $langs->transnoentities("Type")), null, 'errors');
 		}
-
 	}
+
 	if (!$error) {
 		$addfieldentry = array(
 			'name'=>GETPOST('propname', 'aZ09'),
diff --git a/htdocs/product/class/api_products.class.php b/htdocs/product/class/api_products.class.php
index b6230ce119e..c3b22adedb2 100644
--- a/htdocs/product/class/api_products.class.php
+++ b/htdocs/product/class/api_products.class.php
@@ -1945,7 +1945,7 @@ class Products extends DolibarrApi
 
 		unset($object->supplierprices);	// Mut use another API to get them
 
-		if(!DolibarrApiAccess::$user->rights->stock->lire){
+		if (empty(DolibarrApiAccess::$user->rights->stock->lire)) {
 			unset($object->stock_reel);
 			unset($object->stock_theorique);
 		}