From b7280f652414fb41a3d35880c30f0db489ff4389 Mon Sep 17 00:00:00 2001
From: altairis-noe <noe.cendrier@altairis.fr>
Date: Thu, 8 Apr 2021 15:15:58 +0200
Subject: [PATCH] product masks are relevant only if we are in advanced mode

---
 htdocs/product/admin/product_lot.php | 120 ++++++++++++++-------------
 1 file changed, 62 insertions(+), 58 deletions(-)

diff --git a/htdocs/product/admin/product_lot.php b/htdocs/product/admin/product_lot.php
index 2ae8ef9810e..874f4b9e8a9 100644
--- a/htdocs/product/admin/product_lot.php
+++ b/htdocs/product/admin/product_lot.php
@@ -182,38 +182,40 @@ foreach ($dirmodels as $reldir) {
 				}
 			}
 			closedir($handle);
-			print '<tr class="oddeven"><td>Option</td><td>'."\n";
-			print $langs->trans('CustomMasks');
-			print '</td>';
+			if ($conf->global->PRODUCTBATCH_LOT_ADDON == 'mod_lot_advanced') {
+				print '<tr class="oddeven"><td>Option</td><td>'."\n";
+				print $langs->trans('CustomMasks');
+				print '</td>';
 
-			// Show example of numbering model
-			print '<td class="nowrap">';
-			$tmp = 'NoExample';
-			if (preg_match('/^Error/', $tmp)) print '<div class="error">'.$langs->trans($tmp).'</div>';
-			elseif ($tmp == 'NotConfigured') print $langs->trans($tmp);
-			else print $langs->trans($tmp);
-			print '</td>'."\n";
+				// Show example of numbering model
+				print '<td class="nowrap">';
+				$tmp = 'NoExample';
+				if (preg_match('/^Error/', $tmp)) print '<div class="error">'.$langs->trans($tmp).'</div>';
+				elseif ($tmp == 'NotConfigured') print $langs->trans($tmp);
+				else print $langs->trans($tmp);
+				print '</td>'."\n";
 
-			print '<td class="center">';
-			if ($conf->global->PRODUCTBATCH_LOT_USE_PRODUCT_MASKS) {
-				print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskslot&amp;value=0">';
-				print img_picto($langs->trans("Activated"), 'switch_on');
-				print '</a>';
-			} else {
-				print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskslot&amp;value=1">';
-				print img_picto($langs->trans("Disabled"), 'switch_off');
-				print '</a>';
+				print '<td class="center">';
+				if ($conf->global->PRODUCTBATCH_LOT_USE_PRODUCT_MASKS) {
+					print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskslot&amp;value=0">';
+					print img_picto($langs->trans("Activated"), 'switch_on');
+					print '</a>';
+				} else {
+					print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskslot&amp;value=1">';
+					print img_picto($langs->trans("Disabled"), 'switch_off');
+					print '</a>';
+				}
+				print '</td>';
+
+				// Info
+				$htmltooltip = $langs->trans("LotProductTooltip");
+
+				print '<td class="center">';
+				print $form->textwithpicto('', $htmltooltip, 1, 0);
+				print '</td>';
+
+				print "</tr>\n";
 			}
-			print '</td>';
-
-			// Info
-			$htmltooltip = $langs->trans("LotProductTooltip");
-
-			print '<td class="center">';
-			print $form->textwithpicto('', $htmltooltip, 1, 0);
-			print '</td>';
-
-			print "</tr>\n";
 		}
 	}
 }
@@ -306,38 +308,40 @@ foreach ($dirmodels as $reldir) {
 				}
 			}
 			closedir($handle);
-			print '<tr class="oddeven"><td>Option</td><td>'."\n";
-			print $langs->trans('CustomMasks');
-			print '</td>';
+			if ($conf->global->PRODUCTBATCH_SN_ADDON == 'mod_sn_advanced') {
+				print '<tr class="oddeven"><td>Option</td><td>'."\n";
+				print $langs->trans('CustomMasks');
+				print '</td>';
 
-			// Show example of numbering model
-			print '<td class="nowrap">';
-			$tmp = 'NoExample';
-			if (preg_match('/^Error/', $tmp)) print '<div class="error">'.$langs->trans($tmp).'</div>';
-			elseif ($tmp == 'NotConfigured') print $langs->trans($tmp);
-			else print $langs->trans($tmp);
-			print '</td>'."\n";
+				// Show example of numbering model
+				print '<td class="nowrap">';
+				$tmp = 'NoExample';
+				if (preg_match('/^Error/', $tmp)) print '<div class="error">'.$langs->trans($tmp).'</div>';
+				elseif ($tmp == 'NotConfigured') print $langs->trans($tmp);
+				else print $langs->trans($tmp);
+				print '</td>'."\n";
 
-			print '<td class="center">';
-			if ($conf->global->PRODUCTBATCH_SN_USE_PRODUCT_MASKS) {
-				print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskssn&amp;value=0">';
-				print img_picto($langs->trans("Activated"), 'switch_on');
-				print '</a>';
-			} else {
-				print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskssn&amp;value=1">';
-				print img_picto($langs->trans("Disabled"), 'switch_off');
-				print '</a>';
+				print '<td class="center">';
+				if ($conf->global->PRODUCTBATCH_SN_USE_PRODUCT_MASKS) {
+					print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskssn&amp;value=0">';
+					print img_picto($langs->trans("Activated"), 'switch_on');
+					print '</a>';
+				} else {
+					print '<a class="reposition" href="'.$_SERVER["PHP_SELF"].'?action=setmaskssn&amp;value=1">';
+					print img_picto($langs->trans("Disabled"), 'switch_off');
+					print '</a>';
+				}
+				print '</td>';
+
+				// Info
+				$htmltooltip = $langs->trans("SNProductTooltip");
+
+				print '<td class="center">';
+				print $form->textwithpicto('', $htmltooltip, 1, 0);
+				print '</td>';
+
+				print "</tr>\n";
 			}
-			print '</td>';
-
-			// Info
-			$htmltooltip = $langs->trans("SNProductTooltip");
-
-			print '<td class="center">';
-			print $form->textwithpicto('', $htmltooltip, 1, 0);
-			print '</td>';
-
-			print "</tr>\n";
 		}
 	}
 }