ezsystems / ezpublish-kernel

eZ/Publish/API/Repository/Tests/ContentServiceTest.php

Doc Comments +4 added lines, -4 removed lines

@@ -3048,7 +3048,7 @@ 
     /**
      * Test for the addRelation() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Content
+     * @return Relation[]
      *
      * @see \eZ\Publish\API\Repository\ContentService::addRelation()
      * @depends eZ\Publish\API\Repository\Tests\ContentServiceTest::testPublishVersionFromContentDraft
@@ -5326,7 +5326,7 @@ 
      * given properties are available in $actualAliases and not more.
      *
      * @param array $expectedAliasProperties
-     * @param array $actualAliases
+     * @param URLAlias[] $actualAliases
      */
     private function assertAliasesCorrect(array $expectedAliasProperties, array $actualAliases)
     {
@@ -5495,7 +5495,7 @@ 
     /**
      * Returns the default fixture of fields used in most tests.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Field[]
+     * @return Field[]
      */
     private function createFieldsFixture()
     {
@@ -6014,7 +6014,7 @@ 
      * @throws \eZ\Publish\API\Repository\Exceptions\BadStateException
      * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      *
-     * @return object
+     * @return anonymous//eZ/Publish/API/Repository/Tests/ContentServiceTest.php@return object
      */
     private function createContentWithReverseRelations(array $drafts)
     {

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

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return int ID
      */

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

Doc Comments +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return int ID
      */
@@ -1237,7 +1237,7 @@ 
      *
      * Result is returned with oldest version first (using version id as it has index and is auto increment).
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      * @param mixed|null $status Optional argument to filter versions by status, like {@see VersionInfo::STATUS_ARCHIVED}.
      * @param int $limit Limit for items returned, -1 means none.
      *
@@ -1305,7 +1305,7 @@ 
     /**
      * Returns all version numbers for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return int[]
      */

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

Doc Comments +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
      * Creates a Content from the given $struct and $currentVersionNo.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -92,7 +92,7 @@ 
      * Creates a new version for the given $struct and $versionNo.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $versionNo
+     * @param integer $versionNo
      *
      * @return \eZ\Publish\SPI\Persistence\Content\VersionInfo
      */
@@ -125,7 +125,7 @@ 
      * Creates a new version for the given $content.
      *
      * @param \eZ\Publish\SPI\Persistence\Content $content
-     * @param mixed $versionNo
+     * @param integer $versionNo
      * @param mixed $userId
      * @param string|null $languageCode
      *

eZ/Publish/Core/REST/Client/ContentService.php

Doc Comments -1 removed lines

@@ -441,7 +441,6 @@
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $contentInfo
      * @param \eZ\Publish\API\Repository\Values\Content\VersionInfo $versionInfo
-     * @param \eZ\Publish\API\Repository\Values\User\User $user if set given user is used to create the draft - otherwise the current user is used
      * @param \eZ\Publish\API\Repository\Values\Content\Language|null if not set the draft is created with the initialLanguage code of the source version or if not present with the main language.
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content - the newly created content draft

eZ/Publish/Core/SignalSlot/ContentService.php

Doc Comments -1 removed lines

@@ -441,7 +441,6 @@
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $contentInfo
      * @param \eZ\Publish\API\Repository\Values\Content\VersionInfo $versionInfo
-     * @param \eZ\Publish\API\Repository\Values\User\User $user if set given user is used to create the draft - otherwise the current user is used
      * @param \eZ\Publish\API\Repository\Values\Content\Language|null if not set the draft is created with the initialLanguage code of the source version or if not present with the main language.
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content - the newly created content draft

eZ/Publish/API/Repository/Tests/BaseURLServiceTest.php

Doc Comments +6 added lines

@@ -20,6 +20,9 @@ 
 {
     private const URL_CONTENT_TYPE_IDENTIFIER = 'link_ct';
 
+    /**
+     * @param integer $expectedTotalCount
+     */
     protected function doTestFindUrls(URLQuery $query, array $expectedUrls, $expectedTotalCount = null, $ignoreOrder = true)
     {
         $repository = $this->getRepository();
@@ -37,6 +40,9 @@ 
         $this->assertSearchResultItems($searchResult, $expectedUrls, $ignoreOrder);
     }
 
+    /**
+     * @param boolean $ignoreOrder
+     */
     protected function assertSearchResultItems(SearchResult $searchResult, array $expectedUrls, $ignoreOrder)
     {
         $this->assertCount(count($expectedUrls), $searchResult->items);

eZ/Publish/Core/Search/Legacy/Content/Common/Gateway/CriterionHandler/Field.php

Doc Comments +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      * @param \eZ\Publish\API\Repository\Values\Content\Query\Criterion $criterion
      * @param array $languageSettings
      *
-     * @return \eZ\Publish\Core\Persistence\Database\Expression
+     * @return string
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\InvalidArgumentException
      * @throws \eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\Converter\Exception\NotFound

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,