ezsystems / ezpublish-kernel

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/Bundle/EzPublishCoreBundle/Features/Context/ContentTypeContext.php

Doc Comments +4 added lines, -4 removed lines

@@ -127,7 +127,7 @@ 
      * @param  string  $identifier       content type identifier
      * @param  bool $throwIfNotFound  if true, throws an exception if it is not found.
      *
-     * @return \eZ\Publish\API\Repository\Values\ContentType\ContentTypeGroup|null
+     * @return \eZ\Publish\API\Repository\Values\ContentType\ContentType
      */
     protected function loadContentTypeByIdentifier($identifier, $throwIfNotFound = true)
     {
@@ -151,9 +151,9 @@ 
      *
      * @param  string $groupIdentifier content type group identifier
      * @param  string $identifier      content type identifier
-     * @param  array $fields           content type fields definitions
+     * @param  TableNode $fields           content type fields definitions
      *
-     * @return eZ\Publish\API\Repository\Values\ContentType\ContentType
+     * @return \eZ\Publish\API\Repository\Values\ContentType\ContentType
      */
     public function createContentType($groupIdentifier, $identifier, $fields)
     {
@@ -201,7 +201,7 @@ 
     /**
      * Remove the given 'ContentType' object.
      *
-     * @param  eZ\Publish\API\Repository\Values\ContentType\ContentType $contentType
+     * @param  \eZ\Publish\API\Repository\Values\ContentType\ContentType $contentType
      */
     protected function removeContentType($contentType)
     {

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,

eZ/Publish/Core/IO/Tests/IOServiceTest.php

Doc Comments +2 added lines, -1 removed lines

@@ -422,8 +422,9 @@
      * Asserts that the given $ioCreateStruct is of the right type and that id matches the expected value.
      *
      * @param $ioCreateStruct
+     * @param string $spiId
      *
-     * @return bool
+     * @return \Closure
      */
     private function getSPIBinaryFileCreateStructCallback($spiId)
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Language/CachingLanguageHandlerTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -291,7 +291,7 @@ 
     /**
      * Returns a mock for the in-memory cache.
      *
-     * @return \eZ\Publish\Core\Persistence\Cache\InMemory\InMemoryCache|\PHPUnit\Framework\MockObject\MockObject
+     * @return InMemoryCache
      */
     protected function getLanguageCacheMock()
     {
@@ -305,7 +305,7 @@ 
     /**
      * Returns an array with 2 languages.
      *
-     * @return \eZ\Publish\SPI\Persistence\Content\Language[]
+     * @return Language[]
      */
     protected function getLanguagesFixture()
     {

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

Doc Comments +5 added lines, -5 removed lines

@@ -499,7 +499,7 @@ 
      * Returns a Content fixture.
      *
      * @param int $versionNo
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return \eZ\Publish\SPI\Persistence\Content
      */
@@ -527,7 +527,7 @@ 
     /**
      * Returns a Content Gateway mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -541,7 +541,7 @@ 
     /**
      * Returns a FieldValue converter mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\Converter
+     * @return Converter
      */
     protected function getFieldValueConverterMock()
     {
@@ -555,7 +555,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -569,7 +569,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/Tests/Content/Type/ContentUpdater/Action/RemoveFieldTest.php

Doc Comments +5 added lines, -3 removed lines

@@ -250,6 +250,8 @@ 
     /**
      * Returns a Content fixture.
      *
+     * @param integer $versionNo
+     * @param string[] $languageCodes
      * @return \eZ\Publish\SPI\Persistence\Content
      */
     protected function getContentFixture($versionNo, $languageCodes)
@@ -287,7 +289,7 @@ 
     /**
      * Returns a Content Gateway mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -301,7 +303,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -315,7 +317,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/Tests/TransactionHandlerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
     /**
      * Returns a mock object for the Content Gateway.
      *
-     * @return \eZ\Publish\Core\Persistence\Database\DatabaseHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return DatabaseHandler
      */
     protected function getDatabaseHandlerMock()
     {

eZ/Publish/Core/Repository/Tests/Permission/CachedPermissionServiceTest.php

Doc Comments +6 added lines

@@ -173,6 +173,9 @@ 
 
     protected $permissionResolverMock;
 
+    /**
+     * @return PermissionResolver
+     */
     protected function getPermissionResolverMock($methods = [])
     {
         // Tests first calls here with methods set before initiating PermissionCriterionResolver with same instance.
@@ -189,6 +192,9 @@ 
 
     protected $permissionCriterionResolverMock;
 
+    /**
+     * @return PermissionCriterionResolver
+     */
     protected function getPermissionCriterionResolverMock($methods = [])
     {
         // Tests first calls here with methods set before initiating PermissionCriterionResolver with same instance.