ezsystems / ezpublish-kernel

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

Doc Comments +2 added lines, -2 removed lines

@@ -136,7 +136,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\SPI\Persistence\Content\Language\Handler|\PHPUnit_Framework_MockObject_MockObject
+     * @return SPILanguage\Handler
      */
     protected function getLanguageHandlerMock()
     {
@@ -144,7 +144,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\SPI\Persistence\Content\Type\Handler|\PHPUnit_Framework_MockObject_MockObject
+     * @return \eZ\Publish\SPI\Persistence\Content\Type\Handler
      */
     protected function getTypeHandlerMock()
     {

eZ/Publish/Core/Repository/URLAliasService.php

Doc Comments +2 added lines, -2 removed lines

@@ -295,7 +295,7 @@ 
      * @param bool $showAllTranslations
      * @param string[] $prioritizedLanguageList
      *
-     * @return string|bool
+     * @return false|string
      */
     protected function extractPath(
         SPIURLAlias $spiUrlAlias,
@@ -640,7 +640,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Location $location
      * @param string $languageCode
      * @param null|bool $showAllTranslations
-     * @param null|string[] $prioritizedLanguageList
+     * @param string[] $prioritizedLanguageList
      *
      * @return \eZ\Publish\API\Repository\Values\Content\URLAlias
      */

eZ/Publish/Core/REST/Common/Tests/Output/ValueObjectVisitorDispatcherTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\REST\Common\Output\ValueObjectVisitor
+     * @return Common\Output\ValueObjectVisitor
      */
     private function getValueObjectVisitorMock()
     {
@@ -124,7 +124,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|Common\Output\Visitor
+     * @return Common\Output\Visitor
      */
     private function getOutputVisitorMock()
     {
@@ -138,7 +138,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|Common\Output\Generator
+     * @return Common\Output\Generator
      */
     private function getOutputGeneratorMock()
     {

eZ/Bundle/EzPublishLegacySearchEngineBundle/Command/CreateIndexCommand.php

Doc Comments +2 added lines, -2 removed lines

@@ -148,7 +148,7 @@ 
      * @param \eZ\Publish\Core\Persistence\Database\DatabaseHandler $databaseHandler
      * @param int $contentObjectStatus ContentInfo constant
      *
-     * @return int
+     * @return string
      */
     private function getContentObjectsTotalCount(DatabaseHandler $databaseHandler, $contentObjectStatus)
     {
@@ -168,7 +168,7 @@ 
      *
      * @param \Symfony\Component\Console\Output\OutputInterface $output
      * @param \Symfony\Component\Console\Helper\ProgressHelper $progress
-     * @param $message
+     * @param string $message
      */
     private function logWarning(OutputInterface $output, ProgressHelper $progress, $message)
     {

eZ/Publish/Core/Search/Legacy/Content/WordIndexer/Repository/SearchIndex.php

Doc Comments +6 added lines, -6 removed lines

@@ -127,15 +127,15 @@
      * Link word with specific content object (legacy db table: ezsearch_object_word_link).
      *
      * @param $wordId
-     * @param $contentId
-     * @param $frequency
-     * @param $placement
+     * @param integer $contentId
+     * @param integer $frequency
+     * @param integer $placement
      * @param $nextWordId
      * @param $prevWordId
-     * @param $contentTypeId
+     * @param integer $contentTypeId
      * @param $fieldTypeId
-     * @param $published
-     * @param $sectionId
+     * @param integer $published
+     * @param integer $sectionId
      * @param $identifier
      * @param $integerValue
      */

eZ/Publish/Core/Persistence/Database/DatabaseHandler.php

Doc Comments +9 added lines, -4 removed lines

@@ -26,16 +26,19 @@ 
 
     /**
      * Begin a transaction.
+     * @return void
      */
     public function beginTransaction();
 
     /**
      * Commit a transaction.
+     * @return void
      */
     public function commit();
 
     /**
      * Rollback a transaction.
+     * @return void
      */
     public function rollBack();
 
@@ -59,6 +62,7 @@ 
      * Execute a query against the database.
      *
      * @param string $query
+     * @return void
      */
     public function exec($query);
 
@@ -67,7 +71,7 @@ 
      *
      * Statements are ducktyped, but need to work like PDOStatement.
      *
-     * @return object
+     * @return \Doctrine\DBAL\Driver\Statement
      */
     public function prepare($query);
 
@@ -137,8 +141,9 @@ 
      *
      * This is an optimization and works around the ezcDB implementation.
      *
-     * @param string $identifier
      *
+     * @param string $name
+     * @param string $alias
      * @return string
      */
     public function alias($name, $alias);
@@ -167,7 +172,7 @@ 
      * @param string $table
      * @param string $column
      *
-     * @return mixed
+     * @return string
      */
     public function getAutoIncrementValue($table, $column);
 
@@ -184,7 +189,7 @@ 
     /**
      * Returns underlying connection (e.g. Doctrine connection object).
      *
-     * @return mixed
+     * @return \Doctrine\DBAL\Connection
      */
     public function getConnection();
 }

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

Doc Comments +26 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
      */
@@ -373,6 +373,9 @@ 
         }
     }
 
+    /**
+     * @param string $pathString
+     */
     private function isHiddenByParent($pathString, array $rows)
     {
         $parentNodeIds = explode('/', trim($pathString, '/'));
@@ -573,10 +576,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 false|null
      */
     public function swap($locationId1, $locationId2)
     {
@@ -777,7 +780,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)
@@ -905,8 +908,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)
     {
@@ -974,8 +977,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)
     {
@@ -1134,8 +1137,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
      */
@@ -1171,9 +1174,9 @@ 
      *
      * The associated content object is left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function trashLocation($locationId)
     {
@@ -1202,8 +1205,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
      */
@@ -1377,10 +1380,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)
     {
@@ -1446,10 +1449,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/UrlAlias/Gateway/DoctrineDatabase.php

Doc Comments +9 added lines, -9 removed lines

@@ -250,7 +250,7 @@ 
      *
      * @param string $action
      * @param mixed $languageId
-     * @param mixed $newId
+     * @param integer $newId
      * @param mixed $parentId
      * @param string $textMD5
      */
@@ -447,8 +447,8 @@ 
      *
      * Update includes history entries.
      *
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function reparent($oldParentId, $newParentId)
     {
@@ -594,7 +594,7 @@ 
     /**
      * Returns next value for "id" column.
      *
-     * @return mixed
+     * @return string
      */
     public function getNextId()
     {
@@ -909,7 +909,7 @@ 
     /**
      * Deletes single custom alias row matched by composite primary key.
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
      *
      * @return bool
@@ -947,10 +947,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)
     {
@@ -1040,7 +1040,7 @@ 
      * @param mixed $locationId1
      * @param mixed $locationId2
      *
-     * @return bool
+     * @return false|null
      */
     public function swap($locationId1, $locationId2)
     {

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

Doc Comments +12 added lines, -12 removed lines

@@ -101,8 +101,8 @@ 
      * $alwaysAvailable controls whether the url alias is accessible in all
      * languages.
      *
-     * @param mixed $locationId
-     * @param mixed $parentLocationId
+     * @param integer $locationId
+     * @param integer $parentLocationId
      * @param string $name the new name computed by the name schema or url alias schema
      * @param string $languageCode
      * @param bool $alwaysAvailable
@@ -231,7 +231,7 @@ 
      * If $languageCode is null the $alias is created in the system's default
      * language. $alwaysAvailable makes the alias available in all languages.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param string $path
      * @param bool $forwarding
      * @param string $languageCode
@@ -393,7 +393,7 @@ 
     /**
      * List of user generated or autogenerated url entries, pointing to $locationId.
      *
-     * @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[]
@@ -522,9 +522,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)
     {
@@ -552,9 +552,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)
     {
@@ -580,7 +580,7 @@ 
      * @param mixed $locationId1
      * @param mixed $locationId2
      *
-     * @return bool
+     * @return boolean|null
      */
     public function swap($locationId1, $locationId2)
     {
@@ -673,7 +673,7 @@ 
     /**
      * Notifies the underlying engine that a location was deleted or moved to trash.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      */
     public function locationDeleted($locationId)
     {