Throw an exception if maximum number of fields exceeds 10

birkof · birkof · commit 2ea2b6495f9d · 2020-06-25T02:05:02.000+03:00
diff --git a/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackSectionBlock.php b/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackSectionBlock.php
@@ -45,7 +45,9 @@ public function field(string $text, bool $markdown = true): self
         ];
 
         // Maximum number of items is 10
-        $this->options['fields'] = \array_slice($this->options['fields'], 0, 10);
+        if (10 <= \count($this->options['fields'])) {
+            throw new \LogicException('Maximum number of fields should not exceed 10.');
+        }
 
         return $this;
     }

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,9 @@ public function field(string $text, bool $markdown = true): self`
`45`	`45`	`];`
`46`	`46`
`47`	`47`	`// Maximum number of items is 10`
`48`		`- $this->options['fields'] = \array_slice($this->options['fields'], 0, 10);`
	`48`	`+ if (10 <= \count($this->options['fields'])) {`
	`49`	`+ throw new \LogicException('Maximum number of fields should not exceed 10.');`
	`50`	`+ }`
`49`	`51`
`50`	`52`	`return $this;`
`51`	`53`	`}`