Prevent iso from being set by both legacy string and positional argument

Author: Pixelshaped

src/ZipCodeValidator/Constraints/ZipCode.php

@@ -33,6 +33,13 @@ public function __construct(
     )
     {
         if (is_string($options)) {
+            if (null !== $iso) {
+                throw new InvalidOptionsException(
+                    'Cannot pass both positional string $options and named "iso". Use one style.',
+                    ['options', 'iso']
+                );
+            }
+
             $options = ['iso' => $options];
         } elseif (null === $options) {
             $options = [];

tests/Constraints/ZipCodeTest.php

@@ -76,4 +76,10 @@ public function testUnknownLegacyOptionThrowsException(): void
         $this->expectException(InvalidOptionsException::class);
         new ZipCode(['foo' => 'bar', 'iso' => 'FR']);
     }
+
+    public function testLegacyStringOptionCannotBeCombinedWithNamedIso(): void
+    {
+        $this->expectException(InvalidOptionsException::class);
+        new ZipCode('DE', iso: 'FR');
+    }
 }