ezsystems / ezpublish-kernel

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

Doc Comments +13 added lines, -14 removed lines

@@ -423,8 +423,8 @@ 
     /**
      * Insert assignment of $typeId to $groupId.
      *
-     * @param mixed $groupId
-     * @param mixed $typeId
+     * @param integer $groupId
+     * @param integer $typeId
      * @param int $status
      */
     public function insertGroupAssignment($groupId, $typeId, $status)
@@ -455,8 +455,8 @@ 
     /**
      * Deletes a group assignments for a Type.
      *
-     * @param mixed $groupId
-     * @param mixed $typeId
+     * @param integer $groupId
+     * @param integer $typeId
      * @param int $status
      */
     public function deleteGroupAssignment($groupId, $typeId, $status)
@@ -505,7 +505,7 @@ 
     /**
      * Loads data about Group with $identifier.
      *
-     * @param mixed $identifier
+     * @param string $identifier
      *
      * @return string[][]
      */
@@ -564,7 +564,7 @@ 
     /**
      * Loads data for all Types in $status in $groupId.
      *
-     * @param mixed $groupId
+     * @param integer $groupId
      * @param int $status
      *
      * @return string[][]
@@ -585,7 +585,7 @@ 
     /**
      * Inserts a $fieldDefinition for $typeId.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      * @param \eZ\Publish\SPI\Persistence\Content\Type\FieldDefinition $fieldDefinition
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldDefinition $storageFieldDef
@@ -803,9 +803,9 @@ 
     /**
      * Deletes a field definition.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
-     * @param mixed $fieldDefinitionId
+     * @param integer $fieldDefinitionId
      */
     public function deleteFieldDefinition($typeId, $status, $fieldDefinitionId)
     {
@@ -846,7 +846,7 @@ 
     /**
      * Updates a $fieldDefinition for $typeId.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      * @param \eZ\Publish\SPI\Persistence\Content\Type\FieldDefinition $fieldDefinition
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldDefinition $storageFieldDef
@@ -989,9 +989,8 @@ 
     /**
      * Update a type with $updateStruct.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
-     * @param \eZ\Publish\SPI\Persistence\Content\Type\UpdateStruct $updateStruct
      */
     public function updateType($typeId, $status, Type $type)
     {
@@ -1034,7 +1033,7 @@ 
     /**
      * Loads an array with data about $typeId in $status.
      *
-     * @param mixed $typeId
+     * @param integer $typeId
      * @param int $status
      *
      * @return array Data rows.
@@ -1076,7 +1075,7 @@ 
      * Loads an array with data about the type referred to by $remoteId in
      * $status.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      * @param int $status
      *
      * @return array(int=>array(string=>mixed)) Data rows.

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

Doc Comments +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
      * Creates a Content from the given $struct and $currentVersionNo.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -92,7 +92,7 @@ 
      * Creates a new version for the given $struct and $versionNo.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $versionNo
+     * @param integer $versionNo
      *
      * @return \eZ\Publish\SPI\Persistence\Content\VersionInfo
      */
@@ -125,7 +125,7 @@ 
      * Creates a new version for the given $content.
      *
      * @param \eZ\Publish\SPI\Persistence\Content $content
-     * @param mixed $versionNo
+     * @param integer $versionNo
      * @param mixed $userId
      * @param string|null $languageCode
      *

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

Doc Comments +6 added lines

@@ -20,6 +20,9 @@ 
 {
     private const URL_CONTENT_TYPE_IDENTIFIER = 'link_ct';
 
+    /**
+     * @param integer $expectedTotalCount
+     */
     protected function doTestFindUrls(URLQuery $query, array $expectedUrls, $expectedTotalCount = null, $ignoreOrder = true)
     {
         $repository = $this->getRepository();
@@ -37,6 +40,9 @@ 
         $this->assertSearchResultItems($searchResult, $expectedUrls, $ignoreOrder);
     }
 
+    /**
+     * @param boolean $ignoreOrder
+     */
     protected function assertSearchResultItems(SearchResult $searchResult, array $expectedUrls, $ignoreOrder)
     {
         $this->assertCount(count($expectedUrls), $searchResult->items);

eZ/Publish/Core/Search/Legacy/Content/Common/Gateway/CriterionHandler/Field.php

Doc Comments +1 added lines, -1 removed lines

@@ -127,7 +127,7 @@
      * @param \eZ\Publish\API\Repository\Values\Content\Query\Criterion $criterion
      * @param array $languageSettings
      *
-     * @return \eZ\Publish\Core\Persistence\Database\Expression
+     * @return string
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\InvalidArgumentException
      * @throws \eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\Converter\Exception\NotFound

eZ/Bundle/EzPublishIOBundle/Tests/EventListener/StreamFileListenerTest.php

Doc Comments +1 added lines

@@ -143,6 +143,7 @@
     }
 
     /**
+     * @param string $semanticPath
      * @return Request
      */
     protected function createRequest($semanticPath, $host = 'localhost')

eZ/Publish/Core/MVC/Symfony/SiteAccess/SiteAccessService.php

Doc Comments +3 added lines

@@ -65,6 +65,9 @@
         return $this->get($this->siteAccess->name);
     }
 
+    /**
+     * @param  $siteAccess
+     */
     public function getSiteAccessesRelation(?SiteAccess $siteAccess = null): array
     {
         $siteAccess = $siteAccess ?? $this->siteAccess;

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)
     {