Merge pull request #561 from TomHAnderson/hotfix/phpcs

Hotfix/phpcs

Author: TomHAnderson

.travis.yml

@@ -69,7 +69,7 @@ install:
 script:
   - if [[ $TEST_COVERAGE == 'true' ]]; then composer test-coverage ; else composer test ; fi
   - if [[ $TEST_COVERAGE == 'true' ]]; then php build/coverage-checker.php build/clover.xml 70 ; fi
-  - if [[ $CHECK_CS == 'true' ]]; then composer cs-check ; fi
+  - if [[ $CS_CHECK == 'true' ]]; then composer cs-check ; fi
 
 after_script:
   - if [[ $TEST_COVERAGE == 'true' ]]; then wget https://scrutinizer-ci.com/ocular.phar; fi

src/DoctrineORMModule/CliConfigurator.php

@@ -123,7 +123,7 @@ private function getObjectManagerName()
     {
         $arguments = new ArgvInput();
 
-        if (!$arguments->hasParameterOption('--object-manager')) {
+        if (! $arguments->hasParameterOption('--object-manager')) {
             return $this->defaultObjectManagerName;
         }
 

src/DoctrineORMModule/Service/ConfigurationFactory.php

@@ -135,8 +135,8 @@ public function __invoke(ContainerInterface $container, $requestedName, array $o
             $config->setSecondLevelCacheEnabled();
             $config->setSecondLevelCacheConfiguration($cacheConfiguration);
         }
-        
-        if($filterSchemaAssetsExpression = $options->getFilterSchemaAssetsExpression()) {
+
+        if ($filterSchemaAssetsExpression = $options->getFilterSchemaAssetsExpression()) {
             $config->setFilterSchemaAssetsExpression($filterSchemaAssetsExpression);
         }
 

src/DoctrineORMModule/Service/DBALConnectionFactory.php

@@ -41,8 +41,7 @@ public function __invoke(ContainerInterface $container, $requestedName, array $o
         ];
         $params = array_merge($params, $options->getParams());
 
-        if (
-            array_key_exists('platform', $params)
+        if (array_key_exists('platform', $params)
             && is_string($params['platform'])
             && $container->has($params['platform'])
         ) {

tests/DoctrineORMModuleTest/Service/ConfigurationFactoryTest.php

@@ -219,7 +219,10 @@ public function testDefaultMetadatFactory()
         $this->serviceManager->setService('config', $config);
         $factory = new ConfigurationFactory('test_default');
         $ormConfig = $factory->createService($this->serviceManager);
-        $this->assertEquals(\Doctrine\ORM\Mapping\ClassMetadataFactory::class, $ormConfig->getClassMetadataFactoryName());
+        $this->assertEquals(
+            \Doctrine\ORM\Mapping\ClassMetadataFactory::class,
+            $ormConfig->getClassMetadataFactoryName()
+        );
     }
 
     public function testWillInstantiateConfigWithoutEntityListenerResolverSetting()
@@ -236,7 +239,10 @@ public function testWillInstantiateConfigWithoutEntityListenerResolverSetting()
 
         $ormConfig = $this->factory->createService($this->serviceManager);
 
-        $this->assertInstanceOf(\Doctrine\ORM\Mapping\EntityListenerResolver::class, $ormConfig->getEntityListenerResolver());
+        $this->assertInstanceOf(
+            \Doctrine\ORM\Mapping\EntityListenerResolver::class,
+            $ormConfig->getEntityListenerResolver()
+        );
     }
 
     public function testWillInstantiateConfigWithEntityListenerResolverObject()