ezsystems / ezpublish-kernel

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

Doc Comments +1 added lines, -2 removed lines

@@ -212,7 +212,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 = array())
     {
@@ -474,7 +473,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/Repository/ContentService.php

Doc Comments +3 added lines, -3 removed lines

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

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

Doc Comments +4 added lines, -1 removed lines

@@ -205,6 +205,9 @@ 
         return $this->persistenceMock;
     }
 
+    /**
+     * @return RelationProcessor
+     */
     protected function getRelationProcessorMock()
     {
         return $this->getMock(RelationProcessor::class,
@@ -254,7 +257,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/REST/Server/Controller/Content.php

Doc Comments +2 added lines, -2 removed lines

@@ -677,7 +677,7 @@ 
      *
      * @deprecated Since platform 1.0. Forwards the request to the new /views location, but returns a 301.
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\RestExecutedView
+     * @return \Symfony\Component\HttpFoundation\Response
      */
     public function createView()
     {
@@ -706,7 +706,7 @@ 
     /**
      * @param \Symfony\Component\HttpFoundation\Request $request
      *
-     * @return mixed
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     protected function parseContentRequest(Request $request)
     {

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

Doc Comments +5 added lines, -1 removed lines

@@ -1270,7 +1270,7 @@ 
 
     /**
      * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $contentInfo
-     * @param $fieldDefinitionIdentifier
+     * @param string $fieldDefinitionIdentifier
      * @param array $fieldValues
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content
@@ -1387,6 +1387,10 @@ 
         $contentTypeService->publishContentTypeDraft($contentTypeDraft);
     }
 
+    /**
+     * @param string $expectedPath
+     * @param boolean $expectedIsHistory
+     */
     private function assertUrlAliasPropertiesCorrect(
         Location $expectedDestinationLocation,
         $expectedPath,

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

Doc Comments +24 added lines, -21 removed lines

@@ -95,7 +95,7 @@ 
      *
      * @todo optimize
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return array
      */
@@ -249,7 +249,7 @@ 
     /**
      * Returns data for the first level children of the location identified by given $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
      * @return array
      */
@@ -376,6 +376,9 @@ 
         }
     }
 
+    /**
+     * @param string $pathString
+     */
     private function isHiddenByParent($pathString, array $rows)
     {
         $parentNodeIds = explode('/', trim($pathString, '/'));
@@ -576,10 +579,10 @@ 
      * Make the location identified by $locationId1 refer to the Content
      * referred to by $locationId2 and vice versa.
      *
-     * @param mixed $locationId1
-     * @param mixed $locationId2
+     * @param integer $locationId1
+     * @param integer $locationId2
      *
-     * @return bool
+     * @return boolean|null
      */
     public function swap($locationId1, $locationId2)
     {
@@ -736,7 +739,7 @@ 
      * Create an entry in the node assignment table.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Location\CreateStruct $createStruct
-     * @param mixed $parentNodeId
+     * @param string $parentNodeId
      * @param int $type
      */
     public function createNodeAssignment(CreateStruct $createStruct, $parentNodeId, $type = self::NODE_ASSIGNMENT_OP_CODE_CREATE_NOP)
@@ -864,8 +867,8 @@ 
      *
      * Convert existing node assignments into real locations.
      *
-     * @param mixed $contentId
-     * @param mixed $versionNo
+     * @param integer $contentId
+     * @param integer $versionNo
      */
     public function createLocationsFromNodeAssignments($contentId, $versionNo)
     {
@@ -933,8 +936,8 @@ 
     /**
      * Updates all Locations of content identified with $contentId with $versionNo.
      *
-     * @param mixed $contentId
-     * @param mixed $versionNo
+     * @param integer $contentId
+     * @param integer $versionNo
      */
     public function updateLocationsContentVersionNo($contentId, $versionNo)
     {
@@ -1093,8 +1096,8 @@ 
      * the node identified by given $locationId (current main node).
      * Assumes that content has more than one location.
      *
-     * @param mixed $contentId
-     * @param mixed $locationId
+     * @param integer $contentId
+     * @param integer $locationId
      *
      * @return array
      */
@@ -1130,9 +1133,9 @@ 
      *
      * The associated content object is left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function trashLocation($locationId)
     {
@@ -1161,8 +1164,8 @@ 
      * at the old position. If this is not possible ( because the old location
      * does not exist any more) and exception is thrown.
      *
-     * @param mixed $locationId
-     * @param mixed|null $newParentId
+     * @param integer $locationId
+     * @param integer $newParentId
      *
      * @return \eZ\Publish\SPI\Persistence\Content\Location
      */
@@ -1415,10 +1418,10 @@ 
      * Updates ezcontentobject_tree table for the given $contentId and eznode_assignment table for the given
      * $contentId, $parentLocationId and $versionNo
      *
-     * @param mixed $contentId
-     * @param mixed $locationId
-     * @param mixed $versionNo version number, needed to update eznode_assignment table
-     * @param mixed $parentLocationId parent location of location identified by $locationId, needed to update
+     * @param integer $contentId
+     * @param integer $locationId
+     * @param integer $versionNo version number, needed to update eznode_assignment table
+     * @param integer $parentLocationId parent location of location identified by $locationId, needed to update
      *        eznode_assignment table
      */
     public function changeMainLocation($contentId, $locationId, $versionNo, $parentLocationId)
@@ -1546,7 +1549,7 @@ 
     /**
      * Get Query Builder for fetching data of all Locations except the Root node.
      *
-     * @param array $columns list of columns to fetch
+     * @param string[] $columns list of columns to fetch
      *
      * @return \Doctrine\DBAL\Query\QueryBuilder
      */

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

Doc Comments +11 added lines, -11 removed lines

@@ -270,7 +270,7 @@ 
      *
      * @param string $action
      * @param mixed $languageId
-     * @param mixed $newId
+     * @param integer $newId
      * @param mixed $parentId
      * @param string $textMD5
      */
@@ -444,9 +444,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)
     {
@@ -529,8 +529,8 @@ 
      *
      * Update includes history entries.
      *
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function reparent($oldParentId, $newParentId)
     {
@@ -676,7 +676,7 @@ 
     /**
      * Returns next value for "id" column.
      *
-     * @return mixed
+     * @return string
      */
     public function getNextId()
     {
@@ -819,7 +819,7 @@ 
      * Loads autogenerated entry id by given $action and optionally $parentId.
      *
      * @param string $action
-     * @param mixed|null $parentId
+     * @param integer $parentId
      *
      * @return array
      */
@@ -1003,7 +1003,7 @@ 
     /**
      * Deletes single custom alias row matched by composite primary key.
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
      *
      * @return bool
@@ -1041,10 +1041,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)
     {

eZ/Publish/Core/Repository/Helper/DomainMapper.php

Doc Comments +2 added lines, -2 removed lines

@@ -323,7 +323,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Location $spiLocation
      * @param \eZ\Publish\SPI\Persistence\Content\ContentInfo $spiContentInfo pre-loaded Content Info
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Location
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      *
      * @throws \eZ\Publish\Core\Base\Exceptions\InvalidArgumentException In case if the given Content does not belong to the given Location
      */
@@ -394,7 +394,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/Repository/Tests/Service/Mock/UrlAliasTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -3585,7 +3585,7 @@
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Repository\Helper\NameSchemaService
+     * @return \eZ\Publish\Core\Repository\Helper\NameSchemaService
      */
     protected function getNameSchemaServiceMock()
     {