ezsystems / ezpublish-kernel

eZ/Publish/Core/Persistence/Doctrine/ConnectionHandler.php

Doc Comments +5 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
     protected $connection;
 
     /**
-     * @param array|string $dsn
+     * @param string $dsn
      *
      * @return \Doctrine\DBAL\Connection
      */
@@ -68,7 +68,7 @@ 
     /**
      * Create a Connection Handler with corresponding Doctrine connection from DSN.
      *
-     * @param array|string $dsn
+     * @param string $dsn
      *
      * @return ConnectionHandler
      */
@@ -459,8 +459,9 @@ 
      *
      * This is an optimization and works around the ezcDB implementation.
      *
-     * @param string $identifier
      *
+     * @param string $name
+     * @param string $alias
      * @return string
      */
     public function alias($name, $alias)
@@ -495,7 +496,7 @@ 
      * @param string $table
      * @param string $column
      *
-     * @return mixed
+     * @return string
      */
     public function getAutoIncrementValue($table, $column)
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/ContentHandlerTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -1478,7 +1478,7 @@ 
      *
      * @param string[] $methods
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Handler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getPartlyMockedHandler(array $methods)
     {
@@ -1502,7 +1502,7 @@ 
     /**
      * Returns a TreeHandler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\TreeHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return TreeHandler
      */
     protected function getTreeHandlerMock()
     {
@@ -1516,7 +1516,7 @@ 
     /**
      * Returns a ContentTypeHandler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Type\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return Type\Handler
      */
     protected function getContentTypeHandlerMock()
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/TreeHandlerTest.php

Doc Comments +6 added lines, -6 removed lines

@@ -351,7 +351,7 @@ 
     /**
      * Returns Location Gateway mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Gateway|\PHPUnit\Framework\MockObject\MockObject
+     * @return LocationGateway
      */
     protected function getLocationGatewayMock()
     {
@@ -368,7 +368,7 @@ 
     /**
      * Returns a Location Mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return LocationMapper
      */
     protected function getLocationMapperMock()
     {
@@ -385,7 +385,7 @@ 
     /**
      * Returns Content Gateway mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Gateway|\PHPUnit\Framework\MockObject\MockObject
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -402,7 +402,7 @@ 
     /**
      * Returns a Content Mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return Mapper
      */
     protected function getContentMapperMock()
     {
@@ -419,7 +419,7 @@ 
     /**
      * Returns a FieldHandler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return FieldHandler
      */
     protected function getFieldHandlerMock()
     {
@@ -431,7 +431,7 @@ 
     }
 
     /**
-     * @param array $methods
+     * @param string[] $methods
      *
      * @return \eZ\Publish\Core\Persistence\Legacy\Content\TreeHandler|\PHPUnit\Framework\MockObject\MockObject
      */

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/ContentUpdater/Action/AddFieldTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -510,7 +510,7 @@ 
      * Returns a Content fixture.
      *
      * @param int $versionNo
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return \eZ\Publish\SPI\Persistence\Content
      */
@@ -566,7 +566,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -580,7 +580,7 @@ 
     /**
      * Returns a Content mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return ContentMapper
      */
     protected function getContentMapperMock()
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/ContentUpdater/Action/RemoveFieldTest.php

Doc Comments +4 added lines, -2 removed lines

@@ -266,6 +266,8 @@ 
     /**
      * Returns a Content fixture.
      *
+     * @param integer $versionNo
+     * @param string[] $languageCodes
      * @return \eZ\Publish\SPI\Persistence\Content
      */
     protected function getContentFixture($versionNo, $languageCodes)
@@ -317,7 +319,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -331,7 +333,7 @@ 
     /**
      * Returns a Content mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return ContentMapper
      */
     protected function getContentMapperMock()
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/Base.php

Doc Comments +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\Core\Repository\Helper\ContentTypeDomainMapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return ContentTypeDomainMapper
      */
     protected function getContentTypeDomainMapperMock()
     {
@@ -258,7 +258,7 @@ 
      *
      * @param int $id
      *
-     * @return \eZ\Publish\API\Repository\Values\User\User
+     * @return null|\eZ\Publish\API\Repository\Values\User\UserReference
      */
     protected function getStubbedUser($id)
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/PermissionTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -656,7 +656,7 @@ 
      * Returns Role stub.
      *
      * @param array $policiesData
-     * @param mixed $roleId
+     * @param integer $roleId
      *
      * @return \eZ\Publish\SPI\Persistence\User\Role
      */
@@ -1090,7 +1090,7 @@ 
     protected $roleDomainMapperMock;
 
     /**
-     * @return \eZ\Publish\Core\Repository\Helper\RoleDomainMapper|\PHPUnit\Framework\MockObject\MockObject
+     * @return integer
      */
     protected function getRoleDomainMapperMock($methods = [])
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/UrlAliasTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -3443,7 +3443,7 @@
     }
 
     /**
-     * @return \eZ\Publish\Core\Repository\Helper\NameSchemaService|\PHPUnit_Framework_MockObject_MockObject
+     * @return NameSchemaService
      */
     protected function getNameSchemaServiceMock()
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/UrlTest.php

Doc Comments +10 added lines

@@ -441,6 +441,9 @@ 
             ->willReturn($hasAccess);
     }
 
+    /**
+     * @param URL $object
+     */
     protected function configureUrlViewPermission($object, $hasAccess = false)
     {
         $this->permissionResolver
@@ -454,6 +457,9 @@ 
             ->willReturn($hasAccess);
     }
 
+    /**
+     * @param URL $object
+     */
     protected function configureUrlUpdatePermission($object, $hasAccess = false)
     {
         $this->permissionResolver
@@ -488,6 +494,10 @@ 
             ->getMock();
     }
 
+    /**
+     * @param integer $id
+     * @param string $url
+     */
     private function getApiUrl($id = null, $url = null)
     {
         return new URL(['id' => $id, 'url' => $url]);