ezsystems / ezpublish-kernel

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

Doc Comments +9 added lines, -9 removed lines

@@ -150,7 +150,7 @@ 
      * Will contain always a complete list of fields.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct Content creation struct.
-     * @param mixed $versionNo Used by self::copy() to maintain version numbers
+     * @param integer $versionNo Used by self::copy() to maintain version numbers
      *
      * @return \eZ\Publish\SPI\Persistence\Content Content value object
      */
@@ -437,7 +437,7 @@ 
     /**
      * Returns the metadata object for a content identified by $remoteId.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -634,7 +634,7 @@ 
      *
      * @param int $contentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteContent($contentId)
     {
@@ -666,7 +666,7 @@ 
      * @param int $contentId
      * @param int $versionNo
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteVersion($contentId, $versionNo)
     {
@@ -704,8 +704,8 @@ 
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If content or version is not found
      *
-     * @param mixed $contentId
-     * @param mixed|null $versionNo Copy all versions if left null
+     * @param integer $contentId
+     * @param integer $versionNo Copy all versions if left null
      * @param int|null $newOwnerId
      *
      * @return \eZ\Publish\SPI\Persistence\Content
@@ -793,7 +793,7 @@ 
      *
      * @todo Should the existence verifications happen here or is this supposed to be handled at a higher level?
      *
-     * @param mixed $relationId
+     * @param integer $relationId
      * @param int $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::LINK,
@@ -807,7 +807,7 @@ 
     /**
      * Loads relations from $sourceContentId. Optionally, loads only those with $type and $sourceContentVersionNo.
      *
-     * @param mixed $sourceContentId Source Content ID
+     * @param integer $sourceContentId Source Content ID
      * @param mixed|null $sourceContentVersionNo Source Content Version, null if not specified
      * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
@@ -836,7 +836,7 @@ 
      *
      * Only loads relations against published versions.
      *
-     * @param mixed $destinationContentId Destination Content ID
+     * @param integer $destinationContentId Destination Content ID
      * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::LINK,

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

Doc Comments +15 added lines, -16 removed lines

@@ -330,7 +330,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Type $type
      * @param mixed|null $typeId
      *
-     * @return mixed Type ID
+     * @return integer Type ID
      */
     public function insertType(Type $type, $typeId = null)
     {
@@ -425,8 +425,8 @@ 
     /**
      * Insert assignment of $typeId to $groupId.
      *
-     * @param mixed $groupId
-     * @param mixed $typeId
+     * @param integer $groupId
+     * @param integer $typeId
      * @param int $status
      */
     public function insertGroupAssignment($groupId, $typeId, $status)
@@ -457,8 +457,8 @@ 
     /**
      * Deletes a group assignments for a Type.
      *
-     * @param mixed $groupId
-     * @param mixed $typeId
+     * @param integer $groupId
+     * @param integer $typeId
      * @param int $status
      */
     public function deleteGroupAssignment($groupId, $typeId, $status)
@@ -507,7 +507,7 @@ 
     /**
      * Loads data about Group with $identifier.
      *
-     * @param mixed $identifier
+     * @param string $identifier
      *
      * @return string[][]
      */
@@ -566,7 +566,7 @@ 
     /**
      * Loads data for all Types in $status in $groupId.
      *
-     * @param mixed $groupId
+     * @param integer $groupId
      * @param int $status
      *
      * @return string[][]
@@ -587,12 +587,12 @@ 
     /**
      * Inserts a $fieldDefinition for $typeId.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      * @param \eZ\Publish\SPI\Persistence\Content\Type\FieldDefinition $fieldDefinition
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldDefinition $storageFieldDef
      *
-     * @return mixed Field definition ID
+     * @return integer Field definition ID
      */
     public function insertFieldDefinition(
         $typeId,
@@ -808,9 +808,9 @@ 
     /**
      * Deletes a field definition.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
-     * @param mixed $fieldDefinitionId
+     * @param integer $fieldDefinitionId
      */
     public function deleteFieldDefinition($typeId, $status, $fieldDefinitionId)
     {
@@ -851,7 +851,7 @@ 
     /**
      * Updates a $fieldDefinition for $typeId.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      * @param \eZ\Publish\SPI\Persistence\Content\Type\FieldDefinition $fieldDefinition
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldDefinition $storageFieldDef
@@ -994,9 +994,8 @@ 
     /**
      * Update a type with $updateStruct.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
-     * @param \eZ\Publish\SPI\Persistence\Content\Type\UpdateStruct $updateStruct
      */
     public function updateType($typeId, $status, Type $type)
     {
@@ -1039,7 +1038,7 @@ 
     /**
      * Loads an array with data about $typeId in $status.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      *
      * @return array Data rows.
@@ -1081,7 +1080,7 @@ 
      * Loads an array with data about the type referred to by $remoteId in
      * $status.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      * @param int $status
      *
      * @return array(int=>array(string=>mixed)) Data rows.

eZ/Bundle/EzPublishCoreBundle/Command/ReindexCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -332,7 +332,7 @@
     }
 
     /**
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $count
      *
      * @return \Doctrine\DBAL\Driver\Statement

eZ/Publish/Core/MVC/Symfony/EventListener/Tests/ContentViewTwigVariablesSubscriberTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 final class ContentViewTwigVariablesSubscriberTest extends TestCase
 {
     /**
-     * @return \eZ\Publish\Core\MVC\Symfony\View\ContentView|\PHPUnit\Framework\MockObject\MockObject
+     * @return View
      */
     private function getContentViewMock(): ContentView
     {

eZ/Publish/Core/FieldType/BinaryBase/BinaryBaseStorage.php

Doc Comments +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
      * Removes the old file referenced by $fieldId in $versionNo, if not
      * referenced else where.
      *
-     * @param mixed $fieldId
+     * @param integer $fieldId
      * @param string $versionNo
      * @param array $context
      */

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

Doc Comments +10 added lines, -4 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);
@@ -168,7 +174,7 @@ 
      * @param string $table
      * @param string $column
      *
-     * @return mixed
+     * @return string
      */
     public function getAutoIncrementValue($table, $column);
 
@@ -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/Language/CachingHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
     /**
      * Get language by id.
      *
-     * @param mixed $id
+     * @param integer $id
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If language could not be found by $id
      *
@@ -179,7 +179,7 @@ 
     /**
      * Delete a language.
      *
-     * @param mixed $id
+     * @param integer $id
      */
     public function delete($id)
     {

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

Doc Comments +2 added lines, -2 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)
     {

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