ezsystems / ezpublish-kernel

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

Doc Comments +7 added lines, -7 removed lines

@@ -117,9 +117,9 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertContentObject(CreateStruct $struct, $currentVersionNo = 1)
     {
@@ -221,7 +221,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\VersionInfo $versionInfo
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $fields
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertVersion(VersionInfo $versionInfo, array $fields)
     {
@@ -615,7 +615,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field $field
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldValue $value
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertNewField(Content $content, Field $field, StorageFieldValue $value)
     {
@@ -1094,7 +1094,7 @@ 
      *
      * Result is returned with oldest version first (using version id as it has index and is auto increment).
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      * @param mixed|null $status Optional argument to filter versions by status, like {@see VersionInfo::STATUS_ARCHIVED}.
      * @param int $limit Limit for items returned, -1 means none.
      *
@@ -1162,7 +1162,7 @@ 
     /**
      * Returns all version numbers for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return int[]
      */
@@ -1860,7 +1860,7 @@ 
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Relation\CreateStruct $createStruct
      *
-     * @return int ID the inserted ID
+     * @return string ID the inserted ID
      */
     public function insertRelation(RelationCreateStruct $createStruct)
     {

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

Doc Comments +1 added lines

@@ -886,6 +886,7 @@
 
     /**
      * Get Test Double for TrashItem for exception testing and similar.
+     * @param integer $trashId
      */
     private function getTrashItemDouble($trashId, $contentId = 44, $parentLocationId = 2)
     {

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

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

Doc Comments +9 added lines, -9 removed lines

@@ -311,7 +311,7 @@ 
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
      * @throws \eZ\Publish\API\Repository\Exceptions\ForbiddenException
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param string $path
      * @param bool $forwarding
      * @param string $languageCode
@@ -479,7 +479,7 @@ 
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\BadStateException
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $custom if true the user generated aliases are listed otherwise the autogenerated
      *
      * @return \eZ\Publish\SPI\Persistence\Content\UrlAlias[]
@@ -615,9 +615,9 @@ 
      *
      * This method triggers the change of the autogenerated aliases.
      *
-     * @param mixed $locationId
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function locationMoved($locationId, $oldParentId, $newParentId)
     {
@@ -645,9 +645,9 @@ 
      *
      * This method triggers the creation of the autogenerated aliases for the copied locations
      *
-     * @param mixed $locationId
-     * @param mixed $newLocationId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $newLocationId
+     * @param integer $newParentId
      */
     public function locationCopied($locationId, $newLocationId, $newParentId)
     {
@@ -976,7 +976,7 @@ 
     /**
      * Notifies the underlying engine that a location was deleted or moved to trash.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      */
     public function locationDeleted($locationId)
     {

eZ/Publish/Core/Repository/LocationService.php

Doc Comments +1 added lines, -1 removed lines

@@ -723,7 +723,7 @@
      * Instantiates a new location create class.
      *
      * @param mixed $parentLocationId the parent under which the new location should be created
-     * @param eZ\Publish\API\Repository\Values\ContentType\ContentType|null $contentType
+     * @param null|ContentType $contentType
      *
      * @return \eZ\Publish\API\Repository\Values\Content\LocationCreateStruct
      */

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

Doc Comments +13 added lines, -13 removed lines

@@ -209,8 +209,8 @@ 
      * @todo Either move to async/batch or find ways toward optimizing away operations per object.
      * @todo Optionally retain dates and set creator
      *
-     * @param mixed $sourceId
-     * @param mixed $destinationParentId
+     * @param integer $sourceId
+     * @param integer $destinationParentId
      * @param int|null $newOwnerId
      *
      * @return Location the newly created Location.
@@ -368,10 +368,10 @@ 
      * destination, identified by $destinationParentId. Relations do not need
      * to be updated, since they refer to Content. URLs are not touched.
      *
-     * @param mixed $sourceId
-     * @param mixed $destinationParentId
+     * @param integer $sourceId
+     * @param integer $destinationParentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function move($sourceId, $destinationParentId)
     {
@@ -414,7 +414,7 @@ 
     /**
      * Sets a location to be hidden, and it self + all children to invisible.
      *
-     * @param mixed $id Location ID
+     * @param integer $id Location ID
      */
     public function hide($id)
     {
@@ -427,7 +427,7 @@ 
      * Sets a location to be unhidden, and self + children to visible unless a parent is hiding the tree.
      * If not make sure only children down to first hidden node is marked visible.
      *
-     * @param mixed $id
+     * @param integer $id
      */
     public function unHide($id)
     {
@@ -442,10 +442,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)
     {
@@ -492,9 +492,9 @@ 
      * main Location will get the first of its other Locations assigned as the
      * new main Location.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function removeSubtree($locationId)
     {
@@ -505,7 +505,7 @@ 
      * Set section on all content objects in the subtree.
      *
      * @param mixed $locationId
-     * @param mixed $sectionId
+     * @param integer $sectionId
      */
     public function setSectionForSubtree($locationId, $sectionId)
     {

eZ/Publish/Core/FieldType/BinaryBase/Type.php

Doc Comments +1 added lines, -1 removed lines

@@ -152,7 +152,7 @@
      *
      * @param \eZ\Publish\Core\FieldType\BinaryBase\Value $value
      *
-     * @return mixed
+     * @return boolean
      */
     protected function getSortInfo(BaseValue $value)
     {