ezsystems / ezpublish-kernel

eZ/Publish/Core/Persistence/Legacy/Content/FieldValue/Converter/RelationListConverter.php

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
      * "sort_key_int" or "sort_key_string". This column is then used for
      * filtering and sorting for this type.
      *
-     * @return bool
+     * @return string
      */
     public function getIndexColumn()
     {

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

Doc Comments +1 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return int ID
      */
@@ -331,8 +331,6 @@ 
      * Loads data of related to/from $contentId.
      *
      * @param int $contentId
-     * @param bool $reverse Reverse relation, default false
-     * @param int $contentVersionNo
      * @param int $relationType
      *
      * @return mixed[][] Content data, array structured like {@see \eZ\Publish\Core\Persistence\Legacy\Content\Gateway::load()}

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

Doc Comments +9 added lines, -9 removed lines

@@ -103,9 +103,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)
     {
@@ -205,7 +205,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)
     {
@@ -599,7 +599,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)
     {
@@ -809,8 +809,8 @@ 
      *
      * Returns an array with the relevant data.
      *
-     * @param mixed $contentId
-     * @param mixed $version
+     * @param integer $contentId
+     * @param integer $version
      * @param string[] $translations
      *
      * @return array
@@ -979,7 +979,7 @@ 
     /**
      * Returns all version data for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return string[][]
      */
@@ -1030,7 +1030,7 @@ 
     /**
      * Returns all version numbers for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return int[]
      */
@@ -1668,7 +1668,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/Core/Persistence/Legacy/Content/Gateway/ExceptionConversion.php

Doc Comments +1 added lines, -4 removed lines

@@ -65,7 +65,7 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
      * @return int ID
      */
@@ -618,8 +618,6 @@ 
      * Loads data of related to/from $contentId.
      *
      * @param int $contentId
-     * @param bool $reverse Reverse relation, default false
-     * @param int $contentVersionNo
      * @param int $relationType
      *
      * @return mixed[][] Content data, array structured like {@see \eZ\Publish\Core\Persistence\Legacy\Content\Gateway::load()}
@@ -658,7 +656,6 @@ 
     /**
      * Inserts a new relation database record.
      *
-     * @param \eZ\Publish\SPI\Persistence\Content\Relation\CreateStruct $createStruct
      *
      * @return int ID the inserted ID
      */

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

Doc Comments +11 added lines, -11 removed lines

@@ -142,7 +142,7 @@ 
      * Will contain always a complete list of fields.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct Content creation struct.
-     * @param mixed $versionNo Used by self::copy() to maintain version numbers
+     * @param integer $versionNo Used by self::copy() to maintain version numbers
      *
      * @return \eZ\Publish\SPI\Persistence\Content Content value object
      */
@@ -240,9 +240,9 @@ 
      * also be an entry in the `eznode_assignment` created for the draft. This
      * is ignored in this implementation.
      *
-     * @param mixed $contentId
-     * @param mixed $srcVersion
-     * @param mixed $userId
+     * @param integer $contentId
+     * @param integer $srcVersion
+     * @param integer $userId
      *
      * @return \eZ\Publish\SPI\Persistence\Content
      */
@@ -344,7 +344,7 @@ 
     /**
      * Returns the metadata object for a content identified by $remoteId.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -360,7 +360,7 @@ 
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If version is not found
      *
-     * @param int|string $contentId
+     * @param integer $contentId
      * @param int $versionNo Version number to load
      *
      * @return \eZ\Publish\SPI\Persistence\Content\VersionInfo
@@ -512,7 +512,7 @@ 
      *
      * @param int $contentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteContent($contentId)
     {
@@ -544,7 +544,7 @@ 
      * @param int $contentId
      * @param int $versionNo
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteVersion($contentId, $versionNo)
     {
@@ -657,7 +657,7 @@ 
      *
      * @todo Should the existence verifications happen here or is this supposed to be handled at a higher level?
      *
-     * @param mixed $relationId
+     * @param integer $relationId
      * @param int $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::LINK,
@@ -671,7 +671,7 @@ 
     /**
      * Loads relations from $sourceContentId. Optionally, loads only those with $type and $sourceContentVersionNo.
      *
-     * @param mixed $sourceContentId Source Content ID
+     * @param integer $sourceContentId Source Content ID
      * @param mixed|null $sourceContentVersionNo Source Content Version, null if not specified
      * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
@@ -692,7 +692,7 @@ 
      *
      * Only loads relations against published versions.
      *
-     * @param mixed $destinationContentId Destination Content ID
+     * @param integer $destinationContentId Destination Content ID
      * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::EMBED,
      *                 \eZ\Publish\API\Repository\Values\Content\Relation::LINK,

eZ/Publish/Core/Persistence/Legacy/Content/Language/CachingHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
     /**
      * Delete a language.
      *
-     * @param mixed $id
+     * @param integer $id
      */
     public function delete($id)
     {

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

Doc Comments +23 added lines, -23 removed lines

@@ -91,7 +91,7 @@ 
      *
      * @todo optimize
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return array
      */
@@ -246,7 +246,7 @@ 
     /**
      * Returns data for the first level children of the location identified by given $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
      * @return array
      */
@@ -542,10 +542,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)
     {
@@ -702,7 +702,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)
@@ -830,8 +830,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)
     {
@@ -899,8 +899,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)
     {
@@ -1059,8 +1059,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
      */
@@ -1096,9 +1096,9 @@ 
      *
      * The associated content object is left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function trashLocation($locationId)
     {
@@ -1127,8 +1127,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
      */
@@ -1302,10 +1302,10 @@ 
     /**
      * Set section on all content objects in the subtree.
      *
-     * @param mixed $pathString
-     * @param mixed $sectionId
+     * @param string $pathString
+     * @param integer $sectionId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function setSectionForSubtree($pathString, $sectionId)
     {
@@ -1371,10 +1371,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)

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

Doc Comments +13 added lines, -13 removed lines

@@ -210,8 +210,8 @@ 
      *
      * @todo update subtree modification time, optionally retain dates and set creator
      *
-     * @param mixed $sourceId
-     * @param mixed $destinationParentId
+     * @param integer $sourceId
+     * @param integer $destinationParentId
      *
      * @return Location the newly created Location.
      */
@@ -331,10 +331,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)
     {
@@ -373,7 +373,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)
     {
@@ -386,7 +386,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)
     {
@@ -401,10 +401,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)
     {
@@ -451,9 +451,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)
     {
@@ -464,7 +464,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/Persistence/Legacy/Content/Location/Trash/Handler.php

Doc Comments +4 added lines, -4 removed lines

@@ -97,11 +97,11 @@ 
      * Moves all locations in the subtree to the Trash. The associated content
      * objects are left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
      * @todo Handle field types actions
      *
-     * @return null|\eZ\Publish\SPI\Persistence\Content\Location\Trashed null if location was deleted, otherwise Trashed object
+     * @return Trashed null if location was deleted, otherwise Trashed object
      */
     public function trashSubtree($locationId)
     {
@@ -154,8 +154,8 @@ 
      *
      * Returns newly restored location Id.
      *
-     * @param mixed $trashedId
-     * @param mixed $newParentId
+     * @param integer $trashedId
+     * @param integer $newParentId
      *
      * @return int Newly restored location id
      *