diff --git a/htdocs/comm/propal/class/propal.class.php b/htdocs/comm/propal/class/propal.class.php
index 093418b4ba7..061e625e3ec 100644
--- a/htdocs/comm/propal/class/propal.class.php
+++ b/htdocs/comm/propal/class/propal.class.php
@@ -736,7 +736,8 @@ class Propal extends CommonObject
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($ranktouse > 0 && $ranktouse <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $ranktouse; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $ranktouse; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}
diff --git a/htdocs/commande/class/commande.class.php b/htdocs/commande/class/commande.class.php
index fe7f93ff86c..29f3f316f53 100644
--- a/htdocs/commande/class/commande.class.php
+++ b/htdocs/commande/class/commande.class.php
@@ -1650,7 +1650,8 @@ class Commande extends CommonOrder
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($ranktouse > 0 && $ranktouse <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $ranktouse; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $ranktouse; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}
diff --git a/htdocs/compta/facture/class/facture.class.php b/htdocs/compta/facture/class/facture.class.php
index a5b690f5a7c..19aa36e09bc 100644
--- a/htdocs/compta/facture/class/facture.class.php
+++ b/htdocs/compta/facture/class/facture.class.php
@@ -2090,7 +2090,8 @@ class Facture extends CommonInvoice
 
 			if (!empty($conf->global->MAIN_VIEW_LINE_NUMBER)) {
 				$facligne->rang = 1;
-				for ($ii = 1; $ii <= count($this->lines); $ii++) {
+				$linecount = count($this->lines);
+				for ($ii = 1; $ii <= $linecount; $ii++) {
 					$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii+1);
 				}
 			}
@@ -3364,7 +3365,8 @@ class Facture extends CommonInvoice
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($ranktouse > 0 && $ranktouse <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $ranktouse; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $ranktouse; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}
diff --git a/htdocs/fourn/class/fournisseur.commande.class.php b/htdocs/fourn/class/fournisseur.commande.class.php
index 72f0073cd21..624fc31f14f 100644
--- a/htdocs/fourn/class/fournisseur.commande.class.php
+++ b/htdocs/fourn/class/fournisseur.commande.class.php
@@ -1925,7 +1925,8 @@ class CommandeFournisseur extends CommonOrder
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($rang > 0 && $rang <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $rang; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $rang; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}
diff --git a/htdocs/fourn/class/fournisseur.facture.class.php b/htdocs/fourn/class/fournisseur.facture.class.php
index 1dee7815b14..04c49d1065e 100644
--- a/htdocs/fourn/class/fournisseur.facture.class.php
+++ b/htdocs/fourn/class/fournisseur.facture.class.php
@@ -1145,7 +1145,8 @@ class FactureFournisseur extends CommonInvoice
 
 			if (!empty($conf->global->MAIN_VIEW_LINE_NUMBER)) {
 				$facligne->rang = 1;
-				for ($ii = 1; $ii <= count($this->lines); $ii++) {
+				$linecount = count($this->lines);
+				for ($ii = 1; $ii <= $linecount; $ii++) {
 					$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii+1);
 				}
 			}
@@ -2016,7 +2017,8 @@ class FactureFournisseur extends CommonInvoice
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($rang > 0 && $rang <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $rang; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $rang; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}
diff --git a/htdocs/supplier_proposal/class/supplier_proposal.class.php b/htdocs/supplier_proposal/class/supplier_proposal.class.php
index c89b9657d4c..a53c772a5a3 100644
--- a/htdocs/supplier_proposal/class/supplier_proposal.class.php
+++ b/htdocs/supplier_proposal/class/supplier_proposal.class.php
@@ -623,7 +623,8 @@ class SupplierProposal extends CommonObject
 				if (!empty($fk_parent_line)) {
 					$this->line_order(true, 'DESC');
 				} elseif ($ranktouse > 0 && $ranktouse <= count($this->lines)) { // Update all rank of all other lines
-					for ($ii = $ranktouse; $ii <= count($this->lines); $ii++) {
+					$linecount = count($this->lines);
+					for ($ii = $ranktouse; $ii <= $linecount; $ii++) {
 						$this->updateRangOfLine($this->lines[$ii - 1]->id, $ii + 1);
 					}
 				}