ezsystems / ezpublish-kernel

eZ/Publish/Core/FieldType/Author/Type.php

Doc Comments +2 added lines, -2 removed lines

@@ -212,7 +212,7 @@ 
      *
      * @param string $name
      *
-     * @return \eZ\Publish\SPI\FieldType\ValidationError|null
+     * @return ValidationError|null
      */
     private function validateSettingName($name)
     {
@@ -236,7 +236,7 @@ 
      * @param string $name
      * @param string $value
      *
-     * @return \eZ\Publish\SPI\FieldType\ValidationError|null
+     * @return ValidationError|null
      */
     private function validateDefaultAuthorSetting($name, $value)
     {

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

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 namespace eZ\Publish\API\Repository\Tests;
 
 use eZ\Publish\API\Repository\Exceptions\UnauthorizedException;
-use eZ\Publish\API\Repository\Values\Content\Location;
 use eZ\Publish\API\Repository\Values\User\Limitation\ObjectStateLimitation;
 use eZ\Publish\Core\Repository\Repository;
 use eZ\Publish\Core\Repository\TrashService;

eZ/Publish/Core/Helper/Tests/FieldsGroups/ArrayTranslatorFieldsGroupsListTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -66,6 +66,9 @@ 
         );
     }
 
+    /**
+     * @param string $default
+     */
     private function buildList($groups, $default)
     {
         return new ArrayTranslatorFieldsGroupsList(
@@ -76,7 +79,7 @@ 
     }
 
     /**
-     * @return \Symfony\Contracts\Translation\TranslatorInterface|\PHPUnit\Framework\MockObject\MockObject
+     * @return TranslatorInterface
      */
     private function getTranslatorMock()
     {

eZ/Publish/Core/Helper/Tests/FieldsGroups/RepositoryConfigFieldsGroupsListFactoryTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     }
 
     /**
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Bundle\EzPublishCoreBundle\ApiLoader\RepositoryConfigurationProvider
+     * @return RepositoryConfigurationProvider
      */
     protected function getRepositoryConfigMock()
     {
@@ -47,7 +47,7 @@ 
     }
 
     /**
-     * @return \PHPUnit\Framework\MockObject\MockObject|\Symfony\Contracts\Translation\TranslatorInterface
+     * @return TranslatorInterface
      */
     protected function getTranslatorMock()
     {

eZ/Publish/Core/MVC/Symfony/Templating/Twig/Extension/ContentExtension.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     /**
      * Returns a list of functions to add to the existing list.
      *
-     * @return array
+     * @return TwigFunction[]
      */
     public function getFunctions()
     {

eZ/Publish/Core/Persistence/Cache/UrlAliasHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -383,7 +383,7 @@
      * @param \eZ\Publish\SPI\Persistence\Content\UrlAlias $urlAlias
      * @param array $tags Optional, can be used to specify other tags.
      *
-     * @return array
+     * @return string[]
      */
     private function getCacheTags(UrlAlias $urlAlias, array $tags = [])
     {

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

Doc Comments +2 added lines, -3 removed lines

@@ -814,7 +814,6 @@ 
      *
      * @param string $text Content Item field value text (to be indexed)
      * @param string $searchForText text based on which Content Item should be found
-     * @param array $ignoreForSetupFactories list of SetupFactories to be ignored
      * @dataProvider getSpecialFullTextCases
      */
     public function testIndexingSpecialFullTextCases($text, $searchForText)
@@ -1184,8 +1183,8 @@ 
      * $parentLocationIdList.
      *
      * @param string $contentName
-     * @param $contentDescription
-     * @param array $parentLocationIdList
+     * @param string $contentDescription
+     * @param integer[] $parentLocationIdList
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content
      */

eZ/Publish/Core/Search/Common/Indexer.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
     /**
      * Get PDOStatement to fetch metadata about content objects to be indexed.
      *
-     * @param array $fields Select fields
+     * @param string[] $fields Select fields
      * @return \PDOStatement
      */
     protected function getContentDbFieldsStmt(array $fields)

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

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 use Exception;
 use eZ\Publish\Core\Persistence\Legacy\Content\Location\Gateway as LocationGateway;
-use eZ\Publish\SPI\Persistence\Content\Field;
 use eZ\Publish\SPI\Persistence\Content\Handler as BaseContentHandler;
 use eZ\Publish\SPI\Persistence\Content\Type\Handler as ContentTypeHandler;
 use eZ\Publish\Core\Persistence\Legacy\Content\UrlAlias\SlugConverter;

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,