ezsystems / ezpublish-kernel

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/ContentUpdater/Action/AddFieldTest.php

Doc Comments +5 added lines, -5 removed lines

@@ -499,7 +499,7 @@ 
      * Returns a Content fixture.
      *
      * @param int $versionNo
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return \eZ\Publish\SPI\Persistence\Content
      */
@@ -527,7 +527,7 @@ 
     /**
      * Returns a Content Gateway mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -541,7 +541,7 @@ 
     /**
      * Returns a FieldValue converter mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\Converter
+     * @return Converter
      */
     protected function getFieldValueConverterMock()
     {
@@ -555,7 +555,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -569,7 +569,7 @@ 
     /**
      * Returns a Content mapper mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return ContentMapper
      */
     protected function getContentMapperMock()
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/ContentUpdater/Action/RemoveFieldTest.php

Doc Comments +5 added lines, -3 removed lines

@@ -250,6 +250,8 @@ 
     /**
      * Returns a Content fixture.
      *
+     * @param integer $versionNo
+     * @param string[] $languageCodes
      * @return \eZ\Publish\SPI\Persistence\Content
      */
     protected function getContentFixture($versionNo, $languageCodes)
@@ -287,7 +289,7 @@ 
     /**
      * Returns a Content Gateway mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -301,7 +303,7 @@ 
     /**
      * Returns a Content StorageHandler mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\StorageHandler
+     * @return StorageHandler
      */
     protected function getContentStorageHandlerMock()
     {
@@ -315,7 +317,7 @@ 
     /**
      * Returns a Content mapper mock.
      *
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return ContentMapper
      */
     protected function getContentMapperMock()
     {

eZ/Publish/Core/Persistence/Legacy/Tests/TransactionHandlerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -165,7 +165,7 @@
     /**
      * Returns a mock object for the Content Gateway.
      *
-     * @return \eZ\Publish\Core\Persistence\Database\DatabaseHandler|\PHPUnit\Framework\MockObject\MockObject
+     * @return DatabaseHandler
      */
     protected function getDatabaseHandlerMock()
     {

eZ/Publish/Core/Repository/Tests/Permission/CachedPermissionServiceTest.php

Doc Comments +6 added lines

@@ -173,6 +173,9 @@ 
 
     protected $permissionResolverMock;
 
+    /**
+     * @return PermissionResolver
+     */
     protected function getPermissionResolverMock($methods = [])
     {
         // Tests first calls here with methods set before initiating PermissionCriterionResolver with same instance.
@@ -189,6 +192,9 @@ 
 
     protected $permissionCriterionResolverMock;
 
+    /**
+     * @return PermissionCriterionResolver
+     */
     protected function getPermissionCriterionResolverMock($methods = [])
     {
         // Tests first calls here with methods set before initiating PermissionCriterionResolver with same instance.

eZ/Publish/Core/Repository/Tests/Permission/PermissionCriterionResolverTest.php

Doc Comments +3 added lines

@@ -384,6 +384,9 @@
 
     protected $limitationServiceMock;
 
+    /**
+     * @return integer
+     */
     protected function getLimitationServiceMock($methods = [])
     {
         // Tests first calls here with methods set before initiating PermissionCriterionResolver with same instance.

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

Doc Comments +3 added lines, -3 removed lines

@@ -142,7 +142,7 @@ 
     }
 
     /**
-     * @return \PHPUnit\Framework\MockObject\MockObject|\eZ\Publish\Core\Repository\Helper\ContentTypeDomainMapper
+     * @return ContentTypeDomainMapper
      */
     protected function getContentTypeDomainMapperMock()
     {
@@ -156,7 +156,7 @@ 
     /**
      * Returns a persistence Handler mock.
      *
-     * @return \eZ\Publish\SPI\Persistence\Handler|\PHPUnit\Framework\MockObject\MockObject
+     * @return Handler
      */
     protected function getPersistenceMock()
     {
@@ -253,7 +253,7 @@ 
      *
      * @param int $id
      *
-     * @return \eZ\Publish\API\Repository\Values\User\User
+     * @return null|\eZ\Publish\API\Repository\Values\User\UserReference
      */
     protected function getStubbedUser($id)
     {

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