ezsystems / ezpublish-kernel

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
      */
@@ -380,6 +380,9 @@ 
         }
     }
 
+    /**
+     * @param string $pathString
+     */
     private function isHiddenByParent($pathString, array $rows)
     {
         $parentNodeIds = explode('/', trim($pathString, '/'));
@@ -579,10 +582,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)
     {
@@ -739,7 +742,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)
@@ -867,8 +870,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)
     {
@@ -936,8 +939,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)
     {
@@ -1096,8 +1099,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
      */
@@ -1133,9 +1136,9 @@ 
      *
      * The associated content object is left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function trashLocation($locationId)
     {
@@ -1164,8 +1167,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
      */
@@ -1339,10 +1342,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)
     {
@@ -1408,10 +1411,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/Repository/Helper/DomainMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -360,7 +360,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/MVC/Symfony/Controller/Controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     private static $contentId = 42;
 
     /**
-     * @return mixed
+     * @return integer
      */
     public static function getContentId()
     {

eZ/Publish/Core/Search/Elasticsearch/Content/FieldValueMapper/DateMapper.php

Doc Comments +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
      *
      * @param \eZ\Publish\SPI\Search\Field $field
      *
-     * @return mixed
+     * @return boolean
      */
     public function canMap(Field $field)
     {
@@ -39,7 +39,7 @@ 
      *
      * @param \eZ\Publish\SPI\Search\Field $field
      *
-     * @return mixed
+     * @return string
      */
     public function map(Field $field)
     {

eZ/Publish/Core/MVC/Symfony/Cache/Tests/Http/SignalSlot/AbstractSlotTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
     /**
      * Returns an array of allowed IPs for Http PURGE requests.
      *
-     * @return array
+     * @return string[]
      */
     protected function getInternalAllowedIPs()
     {

eZ/Publish/Core/MVC/Symfony/Templating/Tests/Twig/Extension/ContentExtensionTest.php

Doc Comments +3 added lines

@@ -111,6 +111,9 @@
         return $content;
     }
 
+    /**
+     * @return \eZ\Publish\Core\MVC\ConfigResolverInterface
+     */
     private function getConfigResolverMock()
     {
         $mock = $this->getMock(

eZ/Publish/Core/Persistence/Legacy/Tests/Content/TreeHandlerTest.php

Doc Comments +6 added lines, -6 removed lines

@@ -365,7 +365,7 @@ 
     /**
      * Returns Location Gateway mock.
      *
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Location\Gateway
+     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Gateway
      */
     protected function getLocationGatewayMock()
     {
@@ -386,7 +386,7 @@ 
     /**
      * Returns a Location Mapper mock.
      *
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper
+     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Mapper
      */
     protected function getLocationMapperMock()
     {
@@ -411,7 +411,7 @@ 
     /**
      * Returns Content Gateway mock.
      *
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Gateway
+     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Gateway
      */
     protected function getContentGatewayMock()
     {
@@ -432,7 +432,7 @@ 
     /**
      * Returns a Content Mapper mock.
      *
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\Mapper
+     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Mapper
      */
     protected function getContentMapperMock()
     {
@@ -457,7 +457,7 @@ 
     /**
      * Returns a FieldHandler mock.
      *
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler
+     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldHandler
      */
     protected function getFieldHandlerMock()
     {
@@ -475,7 +475,7 @@ 
     }
 
     /**
-     * @param array $methods
+     * @param string[] $methods
      *
      * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Publish\Core\Persistence\Legacy\Content\TreeHandler
      */

eZ/Bundle/EzPublishIOBundle/DependencyInjection/EzPublishIOExtension.php

Doc Comments +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
     }
 
     /**
-     * @return \eZ\Bundle\EzPublishIOBundle\DependencyInjection\ConfigurationFactory[]|\ArrayObject
+     * @return ArrayObject
      */
     public function getMetadataHandlerFactories()
     {
@@ -69,7 +69,7 @@ 
     }
 
     /**
-     * @return \eZ\Bundle\EzPublishIOBundle\DependencyInjection\ConfigurationFactory[]|\ArrayObject
+     * @return ArrayObject
      */
     public function getBinarydataHandlerFactories()
     {

eZ/Bundle/EzPublishRestBundle/Features/Context/SubContext/EzRest.php

Doc Comments +2 added lines

@@ -46,6 +46,8 @@
 
     /**
      * @When I set header :header with/for :object object
+     * @param string $header
+     * @param string $object
      */
     public function setHeaderWithObject($header, $object)
     {