ezsystems / ezpublish-kernel

eZ/Publish/Core/FieldType/Tests/RichText/InternalLinkValidatorTest.php

Doc Comments +12 added lines

@@ -268,6 +268,9 @@ 
         $this->assertContainsEzRemoteInvalidLinkError($contentRemoteId, $errors);
     }
 
+    /**
+     * @param integer $locationId
+     */
     private function assertContainsEzLocationInvalidLinkError($locationId, array $errors)
     {
         $format = 'Invalid link "ezlocation://%d": target location cannot be found';
@@ -275,6 +278,9 @@ 
         $this->assertContains(sprintf($format, $locationId), $errors);
     }
 
+    /**
+     * @param integer $contentId
+     */
     private function assertContainsEzContentInvalidLinkError($contentId, array $errors)
     {
         $format = 'Invalid link "ezcontent://%d": target content cannot be found';
@@ -282,6 +288,9 @@ 
         $this->assertContains(sprintf($format, $contentId), $errors);
     }
 
+    /**
+     * @param string $contentId
+     */
     private function assertContainsEzRemoteInvalidLinkError($contentId, array $errors)
     {
         $format = 'Invalid link "ezremote://%s": target content cannot be found';
@@ -303,6 +312,9 @@ 
             ->getMock();
     }
 
+    /**
+     * @param string $scheme
+     */
     private function createInputDocument($scheme, $id)
     {
         $url = $scheme . '://' . $id;

eZ/Publish/Core/Persistence/Doctrine/ConnectionHandler/PostgresConnectionHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      * @param string $table
      * @param string $column
      *
-     * @return mixed
+     * @return string
      */
     public function getAutoIncrementValue($table, $column)
     {

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

Doc Comments +6 added lines

@@ -15,6 +15,9 @@ 
  */
 abstract class BaseURLServiceTest extends BaseTest
 {
+    /**
+     * @param integer $exectedTotalCount
+     */
     protected function doTestFindUrls(URLQuery $query, array $expectedUrls, $exectedTotalCount = null, $ignoreOrder = true)
     {
         $repository = $this->getRepository();
@@ -32,6 +35,9 @@ 
         $this->assertSearchResultItems($searchResult, $expectedUrls, $ignoreOrder);
     }
 
+    /**
+     * @param boolean $ignoreOrder
+     */
     protected function assertSearchResultItems(SearchResult $searchResult, array $expectedUrls, $ignoreOrder)
     {
         $this->assertCount(count($expectedUrls), $searchResult->items);

eZ/Publish/Core/MVC/Symfony/Routing/Tests/UrlAliasRouterTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
     }
 
     /**
-     * @param $pathInfo
+     * @param string $pathInfo
      *
      * @return Request
      */

eZ/Publish/Core/MVC/Symfony/SiteAccess/Tests/Compound/CompoundOrTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     }
 
     /**
-     * @return \eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\Compound\LogicalAnd
+     * @return LogicalOr
      */
     public function testConstruct()
     {

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

Doc Comments +3 added lines

@@ -150,6 +150,9 @@
         return $content;
     }
 
+    /**
+     * @param string $tpl
+     */
     private function getTemplatePath($tpl)
     {
         return 'templates/' . $tpl;

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

Doc Comments +8 added lines, -8 removed lines

@@ -140,7 +140,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
      */
@@ -356,7 +356,7 @@ 
     /**
      * Returns the metadata object for a content identified by $remoteId.
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo
      */
@@ -372,7 +372,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
@@ -524,7 +524,7 @@ 
      *
      * @param int $contentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteContent($contentId)
     {
@@ -556,7 +556,7 @@ 
      * @param int $contentId
      * @param int $versionNo
      *
-     * @return bool
+     * @return boolean|null
      */
     public function deleteVersion($contentId, $versionNo)
     {
@@ -677,7 +677,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,
@@ -691,7 +691,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,
@@ -712,7 +712,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/Location/Handler.php

Doc Comments +11 added lines, -11 removed lines

@@ -365,10 +365,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)
     {
@@ -411,7 +411,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)
     {
@@ -424,7 +424,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)
     {
@@ -439,10 +439,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)
     {
@@ -489,9 +489,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)
     {
@@ -502,7 +502,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/Tests/Content/LocationHandlerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -687,7 +687,7 @@
      *
      * @param string[] $methods
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Location\Handler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getPartlyMockedHandler(array $methods)
     {