ezsystems / ezpublish-kernel

eZ/Bundle/EzPublishCoreBundle/Tests/Converter/AbstractParamConverterTest.php

Doc Comments +3 added lines

@@ -13,6 +13,9 @@
 
 abstract class AbstractParamConverterTest extends TestCase
 {
+    /**
+     * @param string $name
+     */
     public function createConfiguration($class = null, $name = null)
     {
         $config = $this

eZ/Publish/Core/Repository/Tests/Service/Mock/UrlTest.php

Doc Comments +10 added lines

@@ -441,6 +441,9 @@ 
             ->willReturn($hasAccess);
     }
 
+    /**
+     * @param URL $object
+     */
     protected function configureUrlViewPermission($object, $hasAccess = false)
     {
         $this->permissionResolver
@@ -454,6 +457,9 @@ 
             ->will($this->returnValue($hasAccess));
     }
 
+    /**
+     * @param URL $object
+     */
     protected function configureUrlUpdatePermission($object, $hasAccess = false)
     {
         $this->permissionResolver
@@ -488,6 +494,10 @@ 
             ->getMock();
     }
 
+    /**
+     * @param integer $id
+     * @param string $url
+     */
     private function getApiUrl($id = null, $url = null)
     {
         return new URL(['id' => $id, 'url' => $url]);

eZ/Publish/Core/Repository/ObjectStateService.php

Doc Comments +5 added lines, -5 removed lines

@@ -615,7 +615,7 @@ 
      * @param string $identifier
      * @param string $defaultLanguageCode
      * @param string[] $names
-     * @param string[]|null $descriptions
+     * @param string[] $descriptions
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ObjectState\InputStruct
      */
@@ -682,8 +682,8 @@ 
      * @param \eZ\Publish\API\Repository\Values\ObjectState\ObjectState $objectState
      * @param string|null $identifier
      * @param string|null $defaultLanguageCode
-     * @param string[]|null $names
-     * @param string[]|null $descriptions
+     * @param string[] $names
+     * @param string[] $descriptions
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ObjectState\InputStruct
      */
@@ -755,8 +755,8 @@ 
      * @param \eZ\Publish\API\Repository\Values\ObjectState\ObjectStateGroup $objectStateGroup
      * @param string|null $identifier
      * @param string|null $defaultLanguageCode
-     * @param string[]|null $names
-     * @param string[]|null $descriptions
+     * @param string[] $names
+     * @param string[] $descriptions
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ObjectState\InputStruct
      */

eZ/Publish/Core/Helper/Tests/PreviewLocationProviderTest.php

Doc Comments +3 added lines

@@ -125,6 +125,9 @@
         $this->assertNull($this->provider->loadMainLocation($contentId));
     }
 
+    /**
+     * @param integer $mainLocationId
+     */
     private function getContentMock(int $contentId, ?int $mainLocationId = null, bool $published = false): Content
     {
         $contentInfo = new APIContentInfo([

eZ/Publish/Core/Repository/UserService.php

Doc Comments +4 added lines, -1 removed lines

@@ -497,7 +497,7 @@ 
      * @param mixed $userId
      * @param string[] $prioritizedLanguages Used as prioritized language code on translated properties of returned object.
      *
-     * @return \eZ\Publish\API\Repository\Values\User\User
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException if a user with the given id was not found
      */
@@ -1528,6 +1528,9 @@ 
             !empty($userUpdateStruct->maxLogin);
     }
 
+    /**
+     * @param integer|null $timestamp
+     */
     private function getDateTime(?int $timestamp): ?DateTimeInterface
     {
         if ($timestamp !== null) {

eZ/Publish/Core/MVC/Symfony/View/Tests/AbstractViewTest.php

Doc Comments +3 added lines

@@ -15,6 +15,9 @@
 
 abstract class AbstractViewTest extends TestCase
 {
+    /**
+     * @param string $template
+     */
     abstract protected function createViewUnderTest($template = null, array $parameters = [], $viewType = 'full'): View;
 
     /**

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

Doc Comments +2 added lines, -2 removed lines

@@ -241,7 +241,7 @@ 
      * @throws InvalidArgumentType On invalid $contentType
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $spiFields
-     * @param \eZ\Publish\API\Repository\Values\ContentType\ContentType|\eZ\Publish\SPI\Persistence\Content\Type $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
@@ -684,7 +684,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/Persistence/Cache/Adapter/TransactionalInMemoryCacheAdapter.php

Doc Comments +1 added lines

@@ -68,6 +68,7 @@
 
     /**
      * {@inheritdoc}
+     * @param string $key
      */
     public function getItem($key)
     {

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

Doc Comments +7 added lines, -7 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
      */
@@ -432,7 +432,7 @@ 
     /**
      * Returns the metadata object for a content identified by $remoteId.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -593,7 +593,7 @@ 
      *
      * @param int $contentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteContent($contentId)
     {
@@ -625,7 +625,7 @@ 
      * @param int $contentId
      * @param int $versionNo
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteVersion($contentId, $versionNo)
     {
@@ -750,7 +750,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,
@@ -764,7 +764,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,
@@ -785,7 +785,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,