From d7500b16097ac6346d381cab3ff18bdee58e5813 Mon Sep 17 00:00:00 2001 From: Florian HENRY Date: Tue, 2 Mar 2021 09:45:49 +0100 Subject: [PATCH 1/5] fix php warning --- htdocs/compta/facture/class/facture.class.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/htdocs/compta/facture/class/facture.class.php b/htdocs/compta/facture/class/facture.class.php index ef0962a0169..dd081a87870 100644 --- a/htdocs/compta/facture/class/facture.class.php +++ b/htdocs/compta/facture/class/facture.class.php @@ -1217,8 +1217,9 @@ class Facture extends CommonInvoice $object->lines[$i]->date_end = $newLast; } } - - $object->lines[$i]->ref_ext = ''; // Do not clone ref_ext + if ($object->lines[$i]) { + $object->lines[$i]->ref_ext = ''; + }// Do not clone ref_ext } // Create clone From 593ae78761958c20148a9d3ead44cc34eeb7a7d3 Mon Sep 17 00:00:00 2001 From: Florian HENRY Date: Tue, 2 Mar 2021 10:55:23 +0100 Subject: [PATCH 2/5] avoid php wrning --- htdocs/core/lib/modulebuilder.lib.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/core/lib/modulebuilder.lib.php b/htdocs/core/lib/modulebuilder.lib.php index ff14ec56e18..375ec6f7ed5 100644 --- a/htdocs/core/lib/modulebuilder.lib.php +++ b/htdocs/core/lib/modulebuilder.lib.php @@ -51,7 +51,7 @@ function rebuildObjectClass($destdir, $module, $objectname, $newmask, $readdir = } // Check parameters - if (count($addfieldentry) > 0) + if (!empty($addfieldentry) && count($addfieldentry) > 0) { if (empty($addfieldentry['name'])) { From 1d642f49f7869ba78e56348dc5f3923d40016171 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Tue, 2 Mar 2021 12:37:24 +0100 Subject: [PATCH 3/5] Update facture.class.php --- htdocs/compta/facture/class/facture.class.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/htdocs/compta/facture/class/facture.class.php b/htdocs/compta/facture/class/facture.class.php index dd081a87870..40d220ac717 100644 --- a/htdocs/compta/facture/class/facture.class.php +++ b/htdocs/compta/facture/class/facture.class.php @@ -1192,6 +1192,7 @@ class Facture extends CommonInvoice if (($object->lines[$i]->info_bits & 0x02) == 0x02) // We do not clone line of discounts { unset($object->lines[$i]); + continue; } // Bloc to update dates of service (month by month only if previously filled and similare to start and end of month) @@ -1217,9 +1218,8 @@ class Facture extends CommonInvoice $object->lines[$i]->date_end = $newLast; } } - if ($object->lines[$i]) { - $object->lines[$i]->ref_ext = ''; - }// Do not clone ref_ext + + $object->lines[$i]->ref_ext = ''; // Do not clone ref_ext } // Create clone From 35cb48d64976fa679dcd0c6ffce0f947234f77e0 Mon Sep 17 00:00:00 2001 From: stickler-ci Date: Tue, 2 Mar 2021 11:37:46 +0000 Subject: [PATCH 4/5] Fixing style errors. --- htdocs/compta/facture/class/facture.class.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/compta/facture/class/facture.class.php b/htdocs/compta/facture/class/facture.class.php index 40d220ac717..6656141593c 100644 --- a/htdocs/compta/facture/class/facture.class.php +++ b/htdocs/compta/facture/class/facture.class.php @@ -1218,7 +1218,7 @@ class Facture extends CommonInvoice $object->lines[$i]->date_end = $newLast; } } - + $object->lines[$i]->ref_ext = ''; // Do not clone ref_ext } From ef99b0f98c33ede42d80cf03264fadd73eea1f44 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Tue, 2 Mar 2021 12:38:17 +0100 Subject: [PATCH 5/5] Update modulebuilder.lib.php --- htdocs/core/lib/modulebuilder.lib.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/htdocs/core/lib/modulebuilder.lib.php b/htdocs/core/lib/modulebuilder.lib.php index 375ec6f7ed5..f0da3bad3ea 100644 --- a/htdocs/core/lib/modulebuilder.lib.php +++ b/htdocs/core/lib/modulebuilder.lib.php @@ -51,7 +51,7 @@ function rebuildObjectClass($destdir, $module, $objectname, $newmask, $readdir = } // Check parameters - if (!empty($addfieldentry) && count($addfieldentry) > 0) + if (is_array($addfieldentry) && count($addfieldentry) > 0) { if (empty($addfieldentry['name'])) {