Remove unneeded extra variables

pierredup · pierredup · commit 379de0d65d04 · 2017-09-26T23:09:22.000+02:00
diff --git a/src/Symfony/Component/OptionsResolver/OptionsResolver.php b/src/Symfony/Component/OptionsResolver/OptionsResolver.php
@@ -803,17 +803,12 @@ public function offsetGet($option)
             }
 
             if (!$valid) {
-                $message = 'The option "%s" with value %s is expected to be of type "%s"';
-
-                $message .= ', but is of type "%s".';
-                $types = implode('|', array_unique($invalidTypes));
-
                 throw new InvalidOptionsException(sprintf(
-                    $message,
+                    'The option "%s" with value %s is expected to be of type "%s"',
                     $option,
                     $this->formatValue($value),
                     implode('" or "', $this->allowedTypes[$option]),
-                    $types
+                    implode('|', array_unique($invalidTypes))
                 ));
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -803,17 +803,12 @@ public function offsetGet($option)`
`803`	`803`	`}`
`804`	`804`
`805`	`805`	`if (!$valid) {`
`806`		`- $message = 'The option "%s" with value %s is expected to be of type "%s"';`
`807`		`-`
`808`		`- $message .= ', but is of type "%s".';`
`809`		`- $types = implode('\|', array_unique($invalidTypes));`
`810`		`-`
`811`	`806`	`throw new InvalidOptionsException(sprintf(`
`812`		`- $message,`
	`807`	`+ 'The option "%s" with value %s is expected to be of type "%s"',`
`813`	`808`	`$option,`
`814`	`809`	`$this->formatValue($value),`
`815`	`810`	`implode('" or "', $this->allowedTypes[$option]),`
`816`		`- $types`
	`811`	`+ implode('\|', array_unique($invalidTypes))`
`817`	`812`	`));`
`818`	`813`	`}`
`819`	`814`	`}`