ezsystems / ezpublish-kernel

eZ/Publish/Core/Repository/RoleService.php

Doc Comments +4 added lines, -4 removed lines

@@ -191,7 +191,7 @@ 
      *
      * @param mixed $id
      *
-     * @return \eZ\Publish\API\Repository\Values\User\RoleDraft
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadRoleDraft($id)
     {
@@ -742,7 +742,7 @@ 
      *
      * @param mixed $id
      *
-     * @return \eZ\Publish\API\Repository\Values\User\Role
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadRole($id)
     {
@@ -1324,7 +1324,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\User\RoleCreateStruct $roleCreateStruct
      *
-     * @return \eZ\Publish\Core\FieldType\ValidationError[][][]
+     * @return \eZ\Publish\Core\FieldType\ValidationError[]
      */
     protected function validateRoleCreateStruct(APIRoleCreateStruct $roleCreateStruct)
     {
@@ -1354,7 +1354,7 @@ 
      * @param string $function
      * @param \eZ\Publish\API\Repository\Values\User\Limitation[] $limitations
      *
-     * @return \eZ\Publish\Core\FieldType\ValidationError[][]
+     * @return \eZ\Publish\Core\FieldType\ValidationError[]
      */
     protected function validatePolicy($module, $function, array $limitations)
     {

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/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/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/MVC/Symfony/Security/Authorization/Voter/ValueObjectVoter.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
      * @see \eZ\Publish\API\Repository\PermissionResolver::canUser()
      *
      * @param TokenInterface $token      A TokenInterface instance
-     * @param object         $object     The object to secure
+     * @param \stdClass         $object     The object to secure
      * @param array          $attributes An array of attributes associated with the method being invoked
      *
      * @return int either ACCESS_GRANTED, ACCESS_ABSTAIN, or ACCESS_DENIED

eZ/Publish/Core/Repository/ContentService.php

Doc Comments +4 added lines, -4 removed lines

@@ -220,7 +220,7 @@ 
      * @param mixed $contentId
      * @param int $versionNo the version number. If not given the current version is returned.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\VersionInfo
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadVersionInfoById($contentId, $versionNo = null)
     {
@@ -1397,7 +1397,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      *
-     * @return array
+     * @return string[]
      */
     private function getUpdatedLanguageCodes(APIContentUpdateStruct $contentUpdateStruct)
     {
@@ -1424,7 +1424,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      * @param \eZ\Publish\API\Repository\Values\Content\Content $content
      *
-     * @return array
+     * @return string[]
      */
     protected function getLanguageCodesForUpdate(APIContentUpdateStruct $contentUpdateStruct, APIContent $content)
     {
@@ -1553,7 +1553,7 @@ 
 
     /**
      * @param \eZ\Publish\API\Repository\Values\Content\VersionInfo $versionInfo
-     * @param array $translations
+     * @param string[] $translations
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\BadStateException
      * @throws \eZ\Publish\API\Repository\Exceptions\ContentFieldValidationException

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

Doc Comments +12 added lines, -5 removed lines

@@ -1038,7 +1038,7 @@ 
      * @param string $mainLanguageCode
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return array
      */
@@ -1112,10 +1112,9 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $spiFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      * @param \eZ\Publish\API\Repository\Values\Content\LocationCreateStruct[] $locationCreateStructs
-     * @param \eZ\Publish\SPI\Persistence\Content\ObjectState\Group[] $objectStateGroups
      * @param bool $execute
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentNonRedundantFieldSet(
         $mainLanguageCode,
@@ -2000,7 +1999,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentRequiredField(
         $mainLanguageCode,
@@ -4809,6 +4808,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -5010,6 +5013,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,
@@ -5740,7 +5747,7 @@ 
      * @param array $translations
      * @param bool $useAlwaysAvailable
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\API\Repository\Values\Content\Content
+     * @return APIContent
      */
     private function mockBuildContentDomainObject(SPIContent $spiContent, array $translations = null, bool $useAlwaysAvailable = null)
     {