fixes CS

Author: fabpot

DependencyInjection/CompilerPass/RegisterMappingsPass.php

@@ -75,20 +75,23 @@ abstract class RegisterMappingsPass implements CompilerPassInterface
 
     /**
      * Naming pattern for the configuration service id, for example
-     * 'doctrine.orm.%s_configuration'
+     * 'doctrine.orm.%s_configuration'.
+     *
      * @var string
      */
     private $configurationPattern;
 
     /**
      * Method name to call on the configuration service. This depends on the
-     * Doctrine implementation. For example addEntityNamespace
+     * Doctrine implementation. For example addEntityNamespace.
+     *
      * @var string
      */
     private $registerAliasMethodName;
 
     /**
      * Map of alias to namespace.
+     *
      * @var string[]
      */
     private $aliasMap;

Form/Type/DoctrineType.php

@@ -136,15 +136,13 @@ public function configureOptions(OptionsResolver $resolver)
 
             // Unless the choices are given explicitly, load them on demand
             if (null === $options['choices']) {
-
                 $hash = null;
                 $qbParts = null;
 
                 // If there is no QueryBuilder we can safely cache DoctrineChoiceLoader,
                 // also if concrete Type can return important QueryBuilder parts to generate
                 // hash key we go for it as well
                 if (!$options['query_builder'] || false !== ($qbParts = $type->getQueryBuilderPartsForCachingHash($options['query_builder']))) {
-
                     $hash = CachingFactoryDecorator::generateHash(array(
                         $options['em'],
                         $options['class'],

Tests/DependencyInjection/DoctrineExtensionTest.php

@@ -162,10 +162,10 @@ public function testFixManagersAutoMappings(array $originalEm1, array $originalE
 
         $newEmConfigs = $method->invoke($this->extension, $emConfigs, $bundles);
 
-        $this->assertEquals($newEmConfigs["em1"], array_merge(array(
+        $this->assertEquals($newEmConfigs['em1'], array_merge(array(
             'auto_mapping' => false,
         ), $expectedEm1));
-        $this->assertEquals($newEmConfigs["em2"], array_merge(array(
+        $this->assertEquals($newEmConfigs['em2'], array_merge(array(
             'auto_mapping' => false,
         ), $expectedEm2));
     }

Tests/Validator/Constraints/LegacyUniqueEntityValidatorLegacyApiTest.php

@@ -15,6 +15,7 @@
 
 /**
  * @since  2.5.4
+ *
  * @author Bernhard Schussek <[email protected]>
  * @group  legacy
  */