Adding some validation

matomo-org · Nov 3, 2023 · 0101e7f · 0101e7f
1 parent ddf6ddf
commit 0101e7f
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/Model/Container.php b/Model/Container.php
@@ -19,6 +19,7 @@
 use Piwik\Plugins\TagManager\Input\IdSite;
 use Piwik\Plugins\TagManager\Input\Name;
 use Piwik\Plugins\TagManager\Model\Container\ContainerIdGenerator;
+use Piwik\Validators\NumberRange;
 
 
 class Container extends BaseModel
@@ -143,7 +144,7 @@ public function getContainerInstallInstructions($idSite, $idContainer, $environm
         }
     }
 
-    private function validateContainer($idSite, $name, $description)
+    private function validateContainer($idSite, $name, $description, $ignoreGtmDataLayer)
     {
         $site = new IdSite($idSite);
         $site->check();
@@ -153,11 +154,14 @@ private function validateContainer($idSite, $name, $description)
 
         $description = new Description($description);
         $description->check();
+
+        $numberRange = new NumberRange(0, 1);
+        $numberRange->validate($ignoreGtmDataLayer);
     }
 
     public function addContainer($idSite, $context, $name, $description, $ignoreGtmDataLayer)
     {
-        $this->validateContainer($idSite, $name, $description);
+        $this->validateContainer($idSite, $name, $description, $ignoreGtmDataLayer);
         $this->contextProvider->checkIsValidContext($context);
 
         $createdDate = $this->getCurrentDateTime();
@@ -175,7 +179,7 @@ public function addContainer($idSite, $context, $name, $description, $ignoreGtmD
 
     public function updateContainer($idSite, $idContainer, $name, $description, $ignoreGtmDataLayer)
     {
-        $this->validateContainer($idSite, $name, $description);
+        $this->validateContainer($idSite, $name, $description, $ignoreGtmDataLayer);
 
         $columns = array(
             'name' => $name,