ezsystems / ezpublish-kernel

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

Doc Comments +4 added lines

@@ -436,6 +436,10 @@
             ->getMock();
     }
 
+    /**
+     * @param integer $id
+     * @param string $url
+     */
     private function getApiUrl($id = null, $url = null)
     {
         return new URL(['id' => $id, 'url' => $url]);

eZ/Publish/Core/Repository/URLService.php

Doc Comments +3 added lines

@@ -245,6 +245,9 @@
         }
     }
 
+    /**
+     * @param integer $timestamp
+     */
     private function createDateTime($timestamp)
     {
         if ($timestamp > 0) {

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

Doc Comments +1 added lines, -2 removed lines

@@ -212,7 +212,6 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\ValueObject $expectedValues
      * @param \eZ\Publish\API\Repository\Values\ValueObject $actualObject
-     * @param array $propertyNames
      */
     protected function assertStructPropertiesCorrect(ValueObject $expectedValues, ValueObject $actualObject, array $additionalProperties = array())
     {
@@ -474,7 +473,7 @@ 
     /**
      * Create role of a given name with the given policies described by an array.
      *
-     * @param $roleName
+     * @param string $roleName
      * @param array $policiesData [['module' => 'content', 'function' => 'read', 'limitations' => []]
      *
      * @return \eZ\Publish\API\Repository\Values\User\Role

eZ/Publish/Core/Repository/SiteAccessAware/Tests/AbstractServiceTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
      *
      * @param array $arguments
      * @param int $languageArgumentIndex
-     * @param array $languages
+     * @param string[] $languages
      *
      * @return array
      */
@@ -166,7 +166,7 @@ 
      *
      * @param array $arguments
      * @param int $languageArgumentIndex
-     * @param array $languages
+     * @param string[] $languages
      *
      * @return array
      */

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/SiteAccess/Tests/RouterURIElement2Test.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -9,10 +9,9 @@
 namespace eZ\Publish\Core\MVC\Symfony\SiteAccess\Tests;
 
 use eZ\Publish\Core\MVC\Symfony\SiteAccess;
-use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement;
+use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement as URIElementMatcher;
 use PHPUnit\Framework\TestCase;
 use eZ\Publish\Core\MVC\Symfony\SiteAccess\Router;
-use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement as URIElementMatcher;
 use eZ\Publish\Core\MVC\Symfony\Routing\SimplifiedRequest;
 use eZ\Publish\Core\MVC\Symfony\SiteAccess\MatcherBuilder;
 use Psr\Log\LoggerInterface;

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,