Imprve

Author: yoanm

src/App/Helper/MinMaxHelper.php

@@ -81,23 +81,12 @@ private function appendCollectionDoc(CollectionDoc $doc, Constraint $constraint)
         } elseif ($constraint instanceof Assert\NotBlank && null === $doc->getMinItem()) {
             // Not blank so minimum 1 item
             $doc->setMinItem(1);
-        } elseif ($constraint instanceof Assert\GreaterThan || $constraint instanceof Assert\GreaterThanOrEqual) {
-            $doc->setMinItem(
-                $constraint instanceof Assert\GreaterThanOrEqual
-                    ? $constraint->value
-                    : $constraint->value + 1
-            );
-        } elseif ($constraint instanceof Assert\LessThan || $constraint instanceof Assert\LessThanOrEqual) {
-            $doc->setMaxItem(
-                $constraint instanceof Assert\LessThanOrEqual
-                    ? $constraint->value
-                    : $constraint->value - 1
-            );
         } /* Documentation does not mention array, counter to NotBlank constraint
          elseif ($constraint instanceof Assert\Blank && null === $doc->getMaxItem()) {
             // Blank so maximum 0 item
             $doc->setMaxItem(0);
         }*/
+        $this->appendLessGreaterThanMinMaxItem($doc, $constraint);
     }
 
     /**
@@ -123,4 +112,25 @@ private function appendNumberMinMax(NumberDoc $doc, Constraint $constraint) : vo
             $doc->setMin($constraint->value);
         }
     }
+
+    /**
+     * @param CollectionDoc $doc
+     * @param Constraint    $constraint
+     */
+    private function appendLessGreaterThanMinMaxItem(CollectionDoc $doc, Constraint $constraint): void
+    {
+        if ($constraint instanceof Assert\GreaterThan || $constraint instanceof Assert\GreaterThanOrEqual) {
+            $doc->setMinItem(
+                $constraint instanceof Assert\GreaterThanOrEqual
+                    ? $constraint->value
+                    : $constraint->value + 1
+            );
+        } elseif ($constraint instanceof Assert\LessThan || $constraint instanceof Assert\LessThanOrEqual) {
+            $doc->setMaxItem(
+                $constraint instanceof Assert\LessThanOrEqual
+                    ? $constraint->value
+                    : $constraint->value - 1
+            );
+        }
+    }
 }