ezsystems / ezpublish-kernel

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

Doc Comments +7 added lines, -7 removed lines

@@ -117,9 +117,9 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertContentObject(CreateStruct $struct, $currentVersionNo = 1)
     {
@@ -221,7 +221,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\VersionInfo $versionInfo
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $fields
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertVersion(VersionInfo $versionInfo, array $fields)
     {
@@ -615,7 +615,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field $field
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldValue $value
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertNewField(Content $content, Field $field, StorageFieldValue $value)
     {
@@ -1094,7 +1094,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.
      *
@@ -1162,7 +1162,7 @@ 
     /**
      * Returns all version numbers for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return int[]
      */
@@ -1860,7 +1860,7 @@ 
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Relation\CreateStruct $createStruct
      *
-     * @return int ID the inserted ID
+     * @return string ID the inserted ID
      */
     public function insertRelation(RelationCreateStruct $createStruct)
     {

eZ/Publish/Core/Repository/Helper/DomainMapper.php

Doc Comments +2 added lines, -2 removed lines

@@ -233,7 +233,7 @@ 
      * @throws InvalidArgumentType On invalid $contentType
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $spiFields
-     * @param ContentType|SPIType $contentType
+     * @param ContentType $contentType
      * @param array $prioritizedLanguages A language priority, filters returned fields and is used as prioritized language code on
      *                         returned value object. If not given all languages are returned.
      * @param string|null $alwaysAvailableLanguage Language code fallback if a given field is not found in $prioritizedLanguages
@@ -640,7 +640,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Location $parentLocation
      * @param mixed $mainLocation
      * @param mixed $contentId
-     * @param mixed $contentVersionNo
+     * @param null|integer $contentVersionNo
      *
      * @return \eZ\Publish\SPI\Persistence\Content\Location\CreateStruct
      */

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

Doc Comments +1 added lines, -1 removed lines

@@ -187,7 +187,7 @@
     /**
      * Returns if the given $value is considered empty by the field type.
      *
-     * @param mixed $value
+     * @param SPIValue $value
      *
      * @return bool
      */

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

Doc Comments +5 added lines, -1 removed lines

@@ -1270,7 +1270,7 @@ 
 
     /**
      * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $contentInfo
-     * @param $fieldDefinitionIdentifier
+     * @param string $fieldDefinitionIdentifier
      * @param array $fieldValues
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content
@@ -1387,6 +1387,10 @@ 
         $contentTypeService->publishContentTypeDraft($contentTypeDraft);
     }
 
+    /**
+     * @param string $expectedPath
+     * @param boolean $expectedIsHistory
+     */
     private function assertUrlAliasPropertiesCorrect(
         Location $expectedDestinationLocation,
         $expectedPath,

eZ/Publish/Core/Search/Legacy/Tests/Content/AbstractTestCase.php

Doc Comments +1 added lines

@@ -56,6 +56,7 @@
 
     /**
      * Assert that the elements are.
+     * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $searchResult
      */
     protected function assertSearchResults($expectedIds, $searchResult)
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerContentSortTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     /**
      * Returns a content mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentMapperMock()
     {
@@ -149,7 +149,7 @@ 
     /**
      * Returns a content field handler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentFieldHandlerMock()
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerLocationSortTest.php

Doc Comments +8 added lines, -1 removed lines

@@ -28,9 +28,16 @@ 
  */
 class HandlerLocationSortTest extends AbstractTestCase
 {
+    /**
+     * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $searchResult
+     */
     protected function getIds($searchResult)
     {
         $ids = array_map(
+
+            /**
+             * @param \eZ\Publish\API\Repository\Values\Content\Search\SearchResult $hit
+             */
             function ($hit) {
                 return $hit->valueObject->id;
             },
@@ -106,7 +113,7 @@ 
     /**
      * Returns a location mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getLocationMapperMock()
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerLocationTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
     /**
      * Returns a location mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getLocationMapperMock()
     {

eZ/Publish/Core/Search/Legacy/Tests/Content/HandlerContentTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     /**
      * Returns a content mapper mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentMapperMock()
     {
@@ -149,7 +149,7 @@ 
     /**
      * Returns a content field handler mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getContentFieldHandlerMock()
     {