ezsystems / ezpublish-kernel

eZ/Publish/Core/Repository/ContentService.php

Doc Comments +4 added lines, -4 removed lines

@@ -1416,7 +1416,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      *
-     * @return array
+     * @return string[]
      */
     private function getUpdatedLanguageCodes(APIContentUpdateStruct $contentUpdateStruct)
     {
@@ -1443,7 +1443,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)
     {
@@ -1572,7 +1572,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
@@ -1642,7 +1642,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\VersionInfo $versionInfo
      * @param int|null $publicationDate If null existing date is kept if there is one, otherwise current time is used.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Content
+     * @return LocationCreateStruct
      */
     protected function internalPublishVersion(APIVersionInfo $versionInfo, $publicationDate = null)
     {

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use eZ\Publish\API\Repository\Repository as RepositoryInterface;
 use eZ\Publish\Core\Repository\Values\Content\Location;
 use eZ\Publish\API\Repository\Values\Content\Language;
-use eZ\Publish\SPI\Persistence\Content\Type;
 use eZ\Publish\SPI\Persistence\Handler;
 use eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct as APIContentUpdateStruct;
 use eZ\Publish\API\Repository\Values\ContentType\ContentType;

eZ/Publish/Core/Repository/URLWildcardService.php

Doc Comments +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException if the url wild card was not found
      *
-     * @param mixed $id
+     * @param string $id
      *
      * @return \eZ\Publish\API\Repository\Values\Content\UrlWildcard
      */

eZ/Bundle/EzPublishCoreBundle/Features/Context/UserContext.php

Doc Comments +11 added lines, -8 removed lines

@@ -62,9 +62,9 @@ 
      * Search User with given username, optionally at given location.
      *
      * @param string $username name of User to search for
-     * @param string $parentGroupLocationId where to search, in User Group tree
+     * @param string $parentGroupId where to search, in User Group tree
      *
-     * @return User found
+     * @return null|\eZ\Publish\API\Repository\Values\User\User found
      */
     public function searchUserByLogin($username, $parentGroupId = null)
     {
@@ -95,7 +95,7 @@ 
      * @param string $name name of User Group to search for
      * @param string $parentLocationId (optional) parent location id to search in
      *
-     * @return search results
+     * @return \eZ\Publish\API\Repository\Values\Content\Search\SearchHit[] results
      */
     public function searchUserGroups($name, $parentLocationId = null)
     {
@@ -118,12 +118,12 @@ 
     /**
      * Create user inside given User Group; DELETES existing User if login already exists!
      *
-     * @param $username username of the user to create
-     * @param $email email address of user to create
-     * @param $password account password for user to create
-     * @param $parentGroup pathstring wherein to create user
+     * @param string $username username of the user to create
+     * @param string $email email address of user to create
+     * @param string $password account password for user to create
+     * @param \eZ\Publish\API\Repository\Values\User\UserGroup $parentGroup pathstring wherein to create user
      *
-     * @return eZ\Publish\API\Repository\Values\User\User
+     * @return \eZ\Publish\API\Repository\Values\User\User
      */
     protected function createUser($username, $email, $password, $parentGroup = null, $fields = [])
     {
@@ -317,6 +317,9 @@ 
         return false;
     }
 
+    /**
+     * @param integer $type
+     */
     public function createPasswordHash($login, $password, $type)
     {
         switch ($type) {

eZ/Bundle/EzPublishIOBundle/BinaryStreamResponse.php

Doc Comments +2 added lines, -4 removed lines

@@ -44,7 +44,6 @@ 
      * @param array               $headers            An array of response headers
      * @param bool                $public             Files are public by default
      * @param null|string         $contentDisposition The type of Content-Disposition to set automatically with the filename
-     * @param bool                $autoEtag           Whether the ETag header should be automatically set
      * @param bool                $autoLastModified   Whether the Last-Modified header should be automatically set
      */
     public function __construct(BinaryFile $binaryFile, IOServiceInterface $ioService, $status = 200, $headers = [], $public = true, $contentDisposition = null, $autoLastModified = true)
@@ -63,12 +62,11 @@ 
     /**
      * Sets the file to stream.
      *
-     * @param \SplFileInfo|string $file The file to stream
+     * @param BinaryFile $file The file to stream
      * @param string $contentDisposition
-     * @param bool $autoEtag
      * @param bool $autoLastModified
      *
-     * @return BinaryFileResponse
+     * @return BinaryStreamResponse
      */
     public function setFile($file, $contentDisposition = null, $autoLastModified = true)
     {

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

Doc Comments +1 added lines, -2 removed lines

@@ -215,7 +215,6 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\ValueObject $expectedValues
      * @param \eZ\Publish\API\Repository\Values\ValueObject $actualObject
-     * @param array $propertyNames
      */
     protected function assertStructPropertiesCorrect(ValueObject $expectedValues, ValueObject $actualObject, array $additionalProperties = [])
     {
@@ -477,7 +476,7 @@ 
     /**
      * Create role of a given name with the given policies described by an array.
      *
-     * @param $roleName
+     * @param string $roleName
      * @param array $policiesData [['module' => 'content', 'function' => 'read', 'limitations' => []]
      *
      * @return \eZ\Publish\API\Repository\Values\User\Role

eZ/Publish/Core/Limitation/Tests/ParentContentTypeLimitationTypeTest.php

Doc Comments +3 added lines

@@ -476,6 +476,9 @@
         ];
     }
 
+    /**
+     * @param boolean $persistenceCalled
+     */
     protected function assertContentHandlerExpectations($callNo, $persistenceCalled, $contentId, $contentInfo)
     {
         $this->getPersistenceMock()

eZ/Publish/Core/Persistence/TransformationProcessor/PreprocessedBased.php

Doc Comments -1 removed lines

@@ -19,7 +19,6 @@
      * Constructor.
      *
      * @param \eZ\Publish\Core\Persistence\TransformationProcessor\PcreCompiler $compiler
-     * @param string $installDir Base dir for rule loading
      * @param array $ruleFiles
      */
     public function __construct(PcreCompiler $compiler, array $ruleFiles = [])

eZ/Publish/Core/Limitation/Tests/Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     /**
      * @param array $mockMethods For specifying the methods to mock, all by default
      *
-     * @return \eZ\Publish\SPI\Persistence\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return SPIHandler
      */
     public function getPersistenceMock(array $mockMethods = [])
     {

eZ/Bundle/EzPublishCoreBundle/Command/ReindexCommand.php

Doc Comments +8 added lines, -1 removed lines

@@ -218,6 +218,9 @@ 
         }
     }
 
+    /**
+     * @param boolean $commit
+     */
     protected function indexIncrementally(InputInterface $input, OutputInterface $output, $iterationCount, $commit)
     {
         if ($contentIds = $input->getOption('content-ids')) {
@@ -285,6 +288,10 @@ 
         $progress->finish();
     }
 
+    /**
+     * @param integer $processCount
+     * @param integer $iterationCount
+     */
     private function runParallelProcess(ProgressBar $progress, Statement $stmt, $processCount, $iterationCount, $commit)
     {
         /** @var \Symfony\Component\Process\Process[]|null[] */
@@ -341,7 +348,7 @@ 
     }
 
     /**
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $count
      *
      * @return \Doctrine\DBAL\Driver\Statement