ezsystems / ezpublish-kernel

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

Doc Comments +3 added lines, -3 removed lines

@@ -284,7 +284,7 @@ 
     /**
      * Returns a Content Gateway mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -298,7 +298,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -312,7 +312,7 @@ 
     /**
      * Returns a Content mapper mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return ContentMapper
      */
     protected function getContentMapperMock()
     {

eZ/Publish/Core/Persistence/Legacy/User/Handler.php

Doc Comments +1 added lines, -1 removed lines

@@ -538,7 +538,7 @@
      * are an array of limitation values. The limitation parameter is optional.
      *
      * @param mixed $contentId The groupId or userId to assign the role to.
-     * @param mixed $roleId
+     * @param integer|null $roleId
      * @param array $limitation
      */
     public function assignRole($contentId, $roleId, array $limitation = null)

eZ/Publish/Core/Persistence/Database/DatabaseHandler.php

Doc Comments +9 added lines, -3 removed lines

@@ -27,16 +27,19 @@ 
 
     /**
      * Begin a transaction.
+     * @return void
      */
     public function beginTransaction();
 
     /**
      * Commit a transaction.
+     * @return void
      */
     public function commit();
 
     /**
      * Rollback a transaction.
+     * @return void
      */
     public function rollBack();
 
@@ -60,6 +63,7 @@ 
      * Execute a query against the database.
      *
      * @param string $query
+     * @return void
      */
     public function exec($query);
 
@@ -68,7 +72,8 @@ 
      *
      * Statements are ducktyped, but need to work like PDOStatement.
      *
-     * @return object
+     * @param string $query
+     * @return \Doctrine\DBAL\Driver\Statement
      */
     public function prepare($query);
 
@@ -138,8 +143,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);
@@ -185,7 +191,7 @@ 
     /**
      * Returns underlying connection (e.g. Doctrine connection object).
      *
-     * @return mixed
+     * @return \Doctrine\DBAL\Connection
      */
     public function getConnection();
 }

eZ/Publish/Core/Persistence/Legacy/Content/UrlAlias/Handler.php

Doc Comments +10 added lines, -10 removed lines

@@ -309,7 +309,7 @@ 
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
      * @throws \eZ\Publish\API\Repository\Exceptions\ForbiddenException
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param string $path
      * @param bool $forwarding
      * @param string $languageCode
@@ -466,7 +466,7 @@ 
      * @param string $text
      * @param string $textMD5
      *
-     * @return mixed
+     * @return integer
      */
     protected function insertNopEntry($parentId, $text, $textMD5)
     {
@@ -486,7 +486,7 @@ 
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\BadStateException
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $custom if true the user generated aliases are listed otherwise the autogenerated
      *
      * @return \eZ\Publish\SPI\Persistence\Content\UrlAlias[]
@@ -622,9 +622,9 @@ 
      *
      * This method triggers the change of the autogenerated aliases.
      *
-     * @param mixed $locationId
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function locationMoved($locationId, $oldParentId, $newParentId)
     {
@@ -652,9 +652,9 @@ 
      *
      * This method triggers the creation of the autogenerated aliases for the copied locations
      *
-     * @param mixed $locationId
-     * @param mixed $newLocationId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $newLocationId
+     * @param integer $newParentId
      */
     public function locationCopied($locationId, $newLocationId, $newParentId)
     {
@@ -983,7 +983,7 @@ 
     /**
      * Notifies the underlying engine that a location was deleted or moved to trash.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      */
     public function locationDeleted($locationId)
     {

eZ/Publish/Core/Repository/UserService.php

Doc Comments +4 added lines, -1 removed lines

@@ -173,7 +173,7 @@ 
     /**
      * Loads a user group for the given id.
      *
-     * @param mixed $id
+     * @param integer $id
      * @param string[] $prioritizedLanguages Used as prioritized language code on translated properties of returned object.
      *
      * @return \eZ\Publish\API\Repository\Values\User\UserGroup
@@ -1371,6 +1371,9 @@ 
             !empty($userUpdateStruct->maxLogin);
     }
 
+    /**
+     * @param integer|null $timestamp
+     */
     private function getDateTime(?int $timestamp): ?DateTimeInterface
     {
         if ($timestamp !== null) {

eZ/Publish/Core/Persistence/Legacy/Content/Gateway/DoctrineDatabase.php

Doc Comments +1 added lines, -3 removed lines

@@ -1966,6 +1966,4 @@
                 $alwaysAvailable ? 1 : self::REMOVE_ALWAYS_AVAILABLE_LANG_MASK_OPERAND
             );
 
-        return $query;
-    }
-}
+        return $query
\ No newline at end of file

eZ/Publish/Core/Repository/Strategy/ContentThumbnail/Field/ContentFieldStrategy.php

Doc Comments +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
     private $strategies = [];
 
     /**
-     * @param \eZ\Publish\SPI\Repository\Strategy\ContentThumbnail\Field\FieldTypeBasedThumbnailStrategy[]|Traversable $strategies
+     * @param Traversable $strategies
      */
     public function __construct(Traversable $strategies)
     {
@@ -66,7 +66,7 @@ 
     }
 
     /**
-     * @param \eZ\Publish\SPI\Repository\Strategy\ContentThumbnail\Field\FieldTypeBasedThumbnailStrategy[]|Traversable $thumbnailStrategies
+     * @param FieldTypeBasedThumbnailStrategy[] $thumbnailStrategies
      */
     public function setStrategies(array $thumbnailStrategies): void
     {

eZ/Publish/Core/Helper/PreviewLocationProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      * @see loadMainLocationByContent
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
      * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Location|null
      */

eZ/Publish/Core/Repository/Values/User/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      * @param string $fieldDefIdentifier
      * @param string $languageCode
      *
-     * @return mixed a primitive type or a field type Value object depending on the field type.
+     * @return Value|null a primitive type or a field type Value object depending on the field type.
      */
     public function getFieldValue(string $fieldDefIdentifier, ?string $languageCode = null): ?Value
     {