larriereguichet / AdminBundle

src/LAG/AdminBundle/Message/MessageHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      * using the logger using info channel
      *
      * @param $flashMessage
-     * @param null $logMessage
+     * @param string $logMessage
      */
     public function handleSuccess($flashMessage, $logMessage = null)
     {

src/LAG/AdminBundle/Tests/Utils/FakeEntityManager.php

Doc Comments +4 added lines, -4 removed lines

@@ -274,7 +274,7 @@ 
     /**
      * Check if the Entity manager is open or closed.
      *
-     * @return bool
+     * @return boolean|null
      */
     public function isOpen()
     {
@@ -345,7 +345,7 @@ 
     /**
      * Checks whether the state of the filter collection is clean.
      *
-     * @return boolean True, if the filter collection is clean.
+     * @return boolean|null True, if the filter collection is clean.
      */
     public function isFiltersStateClean()
     {
@@ -355,7 +355,7 @@ 
     /**
      * Checks whether the Entity Manager has filters.
      *
-     * @return boolean True, if the EM has a filter collection.
+     * @return boolean|null True, if the EM has a filter collection.
      */
     public function hasFilters()
     {
@@ -517,7 +517,7 @@ 
      *
      * @param object $object
      *
-     * @return bool
+     * @return boolean|null
      */
     public function contains($object)
     {

src/LAG/AdminBundle/Tests/AdminTestBase.php

Spacing +3 added lines, -3 removed lines

@@ -65,12 +65,12 @@
         // initialise database
         if (!self::$isDatabaseCreated) {
             // TODO remove database at the end of the tests
-            exec(__DIR__ . '/app/console doctrine:database:create --if-not-exists', $output);
-            exec(__DIR__ . '/app/console doctrine:schema:update --force', $output);
+            exec(__DIR__.'/app/console doctrine:database:create --if-not-exists', $output);
+            exec(__DIR__.'/app/console doctrine:schema:update --force', $output);
 
             foreach ($output as $line) {
                 // TODO only in verbose mode
-                fwrite(STDOUT, $line . "\n");
+                fwrite(STDOUT, $line."\n");
             }
             fwrite(STDOUT, "\n");
             self::$isDatabaseCreated = true;

Doc Comments +5 added lines, -5 removed lines

@@ -172,7 +172,7 @@ 
     }
 
     /**
-     * @param $name
+     * @param string $name
      * @return ActionInterface | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockAction($name)
@@ -246,7 +246,7 @@ 
     /**
      * Return a mock of an entity repository
      *
-     * @return RepositoryInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockEntityRepository()
     {
@@ -312,7 +312,7 @@ 
     }
 
     /**
-     * @return TokenStorageInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockTokenStorage()
     {
@@ -351,7 +351,7 @@ 
     }
 
     /**
-     * @return TranslatorInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockTranslator()
     {
@@ -362,7 +362,7 @@ 
 
     /**
      * @param array $entities
-     * @return DataProviderInterface|PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function mockDataProvider($entities = [])
     {

src/LAG/AdminBundle/Tests/AdminBundle/DependencyInjection/LAGAdminExtensionTest.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
         $logger = new Definition(Logger::class, [
             'default'
         ]);
-        $session= new Definition(Session::class);
+        $session = new Definition(Session::class);
 
         $entityManager = new Definition();
         $entityManager->setClass(FakeEntityManager::class);

src/LAG/AdminBundle/Tests/AdminBundle/DependencyInjection/FieldCompilerPassTest.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
         $this->assertEquals('setConfiguration', $fieldCalls[0][0]);
 
         $this->assertInstanceOf(Reference::class, $fieldCalls[0][1][0]);
-        $this->assertEquals('lag.admin.application', (string)$fieldCalls[0][1][0]);
+        $this->assertEquals('lag.admin.application', (string) $fieldCalls[0][1][0]);
 
     }
 
@@ -73,7 +73,7 @@ 
 
         $this->assertExceptionRaised(
             InvalidConfigurationException::class,
-            function () use ($compilerPass, $containerBuilder) {
+            function() use ($compilerPass, $containerBuilder) {
                 // process the compiler pass
                 $compilerPass->process($containerBuilder);
             }

src/LAG/AdminBundle/Tests/AdminBundle/Action/Registry/RegistryTest.php

Spacing +2 added lines, -2 removed lines

@@ -23,11 +23,11 @@
         $registry->add('my.action', $action);
 
         // an exception SHOULD be thrown if an Action with the same has already been registered
-        $this->assertExceptionRaised(Exception::class, function () use ($registry, $action) {
+        $this->assertExceptionRaised(Exception::class, function() use ($registry, $action) {
             $registry->add('my.action', $action);
         });
     
-        $this->assertExceptionRaised(Exception::class, function () use ($registry, $action) {
+        $this->assertExceptionRaised(Exception::class, function() use ($registry, $action) {
             $registry->get('badName');
         });
     

src/LAG/AdminBundle/Tests/AdminBundle/Action/Factory/ActionFactoryTest.php

Spacing +2 added lines, -2 removed lines

@@ -239,13 +239,13 @@
             $actionRegistry
         );
     
-        $this->assertExceptionRaised(Exception::class, function () use ($actionFactory) {
+        $this->assertExceptionRaised(Exception::class, function() use ($actionFactory) {
             $actionFactory->getActions('test', [
                 'lol',
             ]);
         });
     
-        $this->assertExceptionRaised(LogicException::class, function () use ($actionFactory) {
+        $this->assertExceptionRaised(LogicException::class, function() use ($actionFactory) {
             $actionFactory->getActions('test', [
                 'actions' => [
                     'my-action' => [],

src/LAG/AdminBundle/Tests/AdminBundle/Admin/Registry/RegistryTest.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $registry->add($admin);
 
         // an exception SHOULD be thrown if an Admin with the same has already been registered
-        $this->assertExceptionRaised(Exception::class, function () use ($registry, $admin) {
+        $this->assertExceptionRaised(Exception::class, function() use ($registry, $admin) {
             $registry->add($admin);
         });
     

src/LAG/AdminBundle/Tests/AdminBundle/Admin/AdminTest.php

Spacing +1 added lines, -1 removed lines

@@ -419,7 +419,7 @@
         $authorizationChecker
             ->expects($this->exactly(2))
             ->method('isGranted')
-            ->willReturnCallback(function () use (&$i) {
+            ->willReturnCallback(function() use (&$i) {
                 $i++;
     
                 return $i <= 1;