ezsystems / ezpublish-kernel

eZ/Bundle/EzPublishCoreBundle/EventListener/ConfigScopeListener.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
     /** @var \eZ\Publish\Core\MVC\Symfony\View\ViewProvider|\eZ\Publish\Core\MVC\Symfony\SiteAccess\SiteAccessAware */
     private $viewProviders;
 
+    /**
+     * @param \PHPUnit\Framework\MockObject\MockObject[] $configResolvers
+     */
     public function __construct(
         iterable $configResolvers,
         ViewManagerInterface $viewManager

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

Doc Comments +12 added lines, -5 removed lines

@@ -1082,7 +1082,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
      *
@@ -1158,10 +1158,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,
@@ -2045,7 +2044,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,
@@ -4873,6 +4872,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -5081,6 +5084,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,
@@ -5820,7 +5827,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)
     {

eZ/Publish/API/Repository/Exceptions/InvalidVariationException.php

Doc Comments +4 added lines

@@ -12,6 +12,10 @@
 
 class InvalidVariationException extends InvalidArgumentException
 {
+    /**
+     * @param string $variationName
+     * @param string $variationType
+     */
     public function __construct($variationName, $variationType, $code = 0, Exception $previous = null)
     {
         parent::__construct("Invalid variation '$variationName' for $variationType", $code, $previous);

eZ/Bundle/EzPublishCoreBundle/DependencyInjection/Configuration/Parser/IO.php

Doc Comments +1 added lines

@@ -100,6 +100,7 @@
 
     /**
      * Applies dependencies of complex $parameter in $scope.
+     * @param string $parameter
      */
     private function addComplexParametersDependencies($parameter, $scope, ContainerBuilder $container)
     {

eZ/Bundle/EzPublishIOBundle/EventListener/StreamFileListener.php

Doc Comments +1 added lines

@@ -77,6 +77,7 @@
      * Tests if $uri is an IO file uri root.
      *
      * @param string $uri
+     * @param string $urlPrefix
      *
      * @return bool
      */

eZ/Publish/Core/Repository/ContentService.php

Doc Comments +3 added lines, -3 removed lines

@@ -1481,7 +1481,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      *
-     * @return array
+     * @return string[]
      */
     private function getUpdatedLanguageCodes(APIContentUpdateStruct $contentUpdateStruct)
     {
@@ -1508,7 +1508,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)
     {
@@ -1632,7 +1632,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/RoleService.php

Doc Comments +1 added lines, -1 removed lines

@@ -1052,7 +1052,7 @@
      *
      * @param \eZ\Publish\API\Repository\Values\User\RoleCreateStruct $roleCreateStruct
      *
-     * @return \eZ\Publish\Core\FieldType\ValidationError[][][]
+     * @return \eZ\Publish\Core\FieldType\ValidationError[]
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\InvalidArgumentException
      */

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

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/Persistence/Legacy/Content/UrlAlias/Gateway/DoctrineDatabase.php

Doc Comments +12 added lines, -12 removed lines

@@ -104,7 +104,7 @@ 
     /**
      * Loads list of aliases by given $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $custom
      * @param mixed $languageId
      *
@@ -274,7 +274,7 @@ 
      *
      * @param string $action
      * @param mixed $languageId
-     * @param mixed $newId
+     * @param integer $newId
      * @param mixed $parentId
      * @param string $textMD5
      */
@@ -448,9 +448,9 @@ 
      *
      * Removes given $languageId from entry's language mask
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
-     * @param mixed $languageId
+     * @param integer $languageId
      */
     protected function removeTranslation($parentId, $textMD5, $languageId)
     {
@@ -533,8 +533,8 @@ 
      *
      * Update includes history entries.
      *
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function reparent($oldParentId, $newParentId)
     {
@@ -680,7 +680,7 @@ 
     /**
      * Returns next value for "id" column.
      *
-     * @return mixed
+     * @return integer
      */
     public function getNextId()
     {
@@ -823,7 +823,7 @@ 
      * Loads autogenerated entry id by given $action and optionally $parentId.
      *
      * @param string $action
-     * @param mixed|null $parentId
+     * @param integer $parentId
      *
      * @return array
      */
@@ -1007,7 +1007,7 @@ 
     /**
      * Deletes single custom alias row matched by composite primary key.
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
      *
      * @return bool
@@ -1045,10 +1045,10 @@ 
      *
      * If $id is set only autogenerated entries will be removed.
      *
-     * @param mixed $action
-     * @param mixed|null $id
+     * @param string $action
+     * @param integer $id
      *
-     * @return bool
+     * @return boolean|null
      */
     public function remove($action, $id = null)
     {