[CS Fix] Consistent coding-style of concatenation operator usage

Author: canni

Options.php

@@ -225,7 +225,7 @@ public function get($option)
         $this->reading = true;
 
         if (!array_key_exists($option, $this->options)) {
-            throw new \OutOfBoundsException('The option "' . $option . '" does not exist.');
+            throw new \OutOfBoundsException('The option "'.$option.'" does not exist.');
         }
 
         if (isset($this->lazy[$option])) {
@@ -459,7 +459,7 @@ private function resolve($option)
                 }
             }
 
-            throw new OptionDefinitionException('The options "' . implode('", "', $conflicts) . '" have a cyclic dependency.');
+            throw new OptionDefinitionException('The options "'.implode('", "', $conflicts).'" have a cyclic dependency.');
         }
 
         $this->lock[$option] = true;
@@ -497,7 +497,7 @@ private function normalize($option)
                 }
             }
 
-            throw new OptionDefinitionException('The options "' . implode('", "', $conflicts) . '" have a cyclic dependency.');
+            throw new OptionDefinitionException('The options "'.implode('", "', $conflicts).'" have a cyclic dependency.');
         }
 
         /** @var \Closure $normalizer */

OptionsResolver.php

@@ -253,7 +253,7 @@ private function validateOptionsExistence(array $options)
             ksort($diff);
 
             throw new InvalidOptionsException(sprintf(
-                (count($diff) > 1 ? 'The options "%s" do not exist.' : 'The option "%s" does not exist.') . ' Known options are: "%s"',
+                (count($diff) > 1 ? 'The options "%s" do not exist.' : 'The option "%s" does not exist.').' Known options are: "%s"',
                 implode('", "', array_keys($diff)),
                 implode('", "', array_keys($this->knownOptions))
             ));
@@ -320,7 +320,7 @@ private function validateOptionTypes(array $options)
             $allowedTypes = (array) $allowedTypes;
 
             foreach ($allowedTypes as $type) {
-                $isFunction = 'is_' . $type;
+                $isFunction = 'is_'.$type;
 
                 if (function_exists($isFunction) && $isFunction($value)) {
                     continue 2;

Tests/OptionsResolverTest.php

@@ -63,7 +63,7 @@ public function testResolveLazyDependencyOnOptional()
         $this->resolver->setDefaults(array(
             'one' => '1',
             'two' => function (Options $options) {
-                return $options['one'] . '2';
+                return $options['one'].'2';
             },
         ));
 
@@ -115,7 +115,7 @@ public function testResolveLazyDependencyOnOptionalWithoutDefault()
                 /* @var \PHPUnit_Framework_TestCase $test */
                 $test->assertTrue(isset($options['one']));
 
-                return $options['one'] . '2';
+                return $options['one'].'2';
             },
         ));
 
@@ -136,7 +136,7 @@ public function testResolveLazyDependencyOnRequired()
         ));
         $this->resolver->setDefaults(array(
             'two' => function (Options $options) {
-                return $options['one'] . '2';
+                return $options['one'].'2';
             },
         ));
 
@@ -591,7 +591,7 @@ public function testNormalizersTransformFinalOptions()
         ));
         $this->resolver->setNormalizers(array(
             'foo' => function (Options $options, $value) {
-                return $options['bam'] . '[' . $value . ']';
+                return $options['bam'].'['.$value.']';
             },
         ));
 

Tests/OptionsTest.php

@@ -225,7 +225,7 @@ public function testNormalizerReceivesUnnormalizedValue()
         $this->options->set('foo', 'bar');
 
         $this->options->setNormalizer('foo', function (Options $options, $value) {
-            return 'normalized[' . $value . ']';
+            return 'normalized['.$value.']';
         });
 
         $this->assertEquals('normalized[bar]', $this->options->get('foo'));