larriereguichet / AdminBundle

src/Event/Subscriber/ExtraConfigurationSubscriber.php

Doc Comments +8 added lines, -1 removed lines

@@ -138,6 +138,10 @@ 
         ];
     }
 
+    /**
+     * @param string $entityClass
+     * @param string $adminName
+     */
     private function addDefaultFields(array &$configuration, $entityClass, $adminName)
     {
         $fieldsMapping = [
@@ -343,7 +347,7 @@ 
     /**
      * Return the default action field if found.
      *
-     * @param array $fields
+     * @param string[] $fields
      *
      * @return string|null
      */
@@ -420,6 +424,9 @@ 
         return $metadata;
     }
 
+    /**
+     * @param string $type
+     */
     private function getOperatorFromFieldType($type)
     {
         $mapping = [

src/Factory/FieldFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -172,7 +172,7 @@
     /**
      * @param array               $configuration
      * @param ActionConfiguration $actionConfiguration
-     * @return array
+     * @return string
      * @throws Exception
      */
     private function resolveTopLevelConfiguration(array $configuration, ActionConfiguration $actionConfiguration)

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/AdminTestBase.php

Doc Comments +12 added lines, -12 removed lines

@@ -161,7 +161,7 @@ 
     }
 
     /**
-     * @return KernelInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockKernel()
     {
@@ -172,8 +172,8 @@ 
     }
 
     /**
-     * @param $name
-     * @return ActionInterface | PHPUnit_Framework_MockObject_MockObject
+     * @param string $name
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockAction($name)
     {
@@ -189,7 +189,7 @@ 
     }
 
     /**
-     * @return ActionConfiguration | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockActionConfiguration()
     {
@@ -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()
     {
@@ -256,7 +256,7 @@ 
     }
 
     /**
-     * @return EntityManager | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockEntityManager()
     {
@@ -278,7 +278,7 @@ 
     }
 
     /**
-     * @return Registry | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      *
      * @deprecated
      */
@@ -296,7 +296,7 @@ 
     }
 
     /**
-     * @return ActionFactory | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockActionFactory()
     {
@@ -312,7 +312,7 @@ 
     }
 
     /**
-     * @return TokenStorageInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockTokenStorage()
     {
@@ -338,7 +338,7 @@ 
     }
 
     /**
-     * @return MessageHandlerInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockMessageHandler()
     {
@@ -351,7 +351,7 @@ 
     }
 
     /**
-     * @return TranslatorInterface | PHPUnit_Framework_MockObject_MockObject
+     * @return \PHPUnit\Framework\MockObject\MockObject | PHPUnit_Framework_MockObject_MockObject
      */
     protected function mockTranslator()
     {
@@ -392,7 +392,7 @@ 
     /**
      * @param $class
      *
-     * @return PHPUnit_Framework_MockObject_MockObject|mixed
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getMockWithoutConstructor($class)
     {

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)
     {