ezsystems / ezpublish-kernel

eZ/Bundle/EzPublishRestBundle/Tests/Functional/TestCase.php

Doc Comments +7 added lines

@@ -111,6 +111,7 @@ 
     }
 
     /**
+     * @param string $method
      * @return HttpRequest
      */
     public function createHttpRequest($method, $uri, $contentType = '', $acceptType = '')
@@ -172,6 +173,9 @@ 
         return $errorMessageString;
     }
 
+    /**
+     * @param string $header
+     */
     protected function assertHttpResponseHasHeader(HttpResponse $response, $header, $expectedValue = null)
     {
         $headerValue = $response->getHeader($header);
@@ -181,6 +185,9 @@ 
         }
     }
 
+    /**
+     * @param string $typeString
+     */
     protected function generateMediaTypeString($typeString)
     {
         return "application/vnd.ez.api.$typeString";

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

Doc Comments +9 added lines, -9 removed lines

@@ -251,7 +251,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
@@ -413,7 +413,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[]
@@ -542,9 +542,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)
     {
@@ -572,9 +572,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)
     {
@@ -799,7 +799,7 @@ 
     /**
      * Notifies the underlying engine that a location was deleted or moved to trash.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      */
     public function locationDeleted($locationId)
     {

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

Doc Comments +2 added lines, -2 removed lines

@@ -742,8 +742,8 @@
      * $parentLocationIdList.
      *
      * @param string $contentName
-     * @param $contentDescription
-     * @param array $parentLocationIdList
+     * @param string $contentDescription
+     * @param integer[] $parentLocationIdList
      *
      * @return \eZ\Publish\API\Repository\Values\Content\Content
      */

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

@@ -8,11 +8,10 @@
  */
 namespace eZ\Publish\API\Repository\Tests;
 
-use eZ\Publish\API\Repository\Tests\SetupFactory\LegacyElasticsearch as LegacyElasticsearchSetupFactory;
+use eZ\Publish\API\Repository\Tests\SetupFactory\LegacyElasticsearch;
 use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use eZ\Publish\API\Repository\Exceptions\NotFoundException;
 use eZ\Publish\API\Repository\SearchService;
-use eZ\Publish\API\Repository\Tests\SetupFactory\LegacyElasticsearch;
 use eZ\Publish\API\Repository\Values\Content\LocationQuery;
 use eZ\Publish\API\Repository\Values\Content\Query;
 use eZ\Publish\API\Repository\Values\Content\Query\Criterion;

eZ/Publish/Core/Helper/Tests/ContentInfoLocationLoader/SudoMainLocationLoaderTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     }
 
     /**
-     * @return \eZ\Publish\Core\Repository\Repository|\PHPUnit_Framework_MockObject_MockObject
+     * @return \eZ\Publish\API\Repository\Repository
      */
     private function getRepositoryMock()
     {

eZ/Publish/API/Repository/Tests/Values/User/Limitation/RolePolicyLimitationTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
     /**
      * Perform search by the Subtree Criterion for the given subtree path and return results count.
      *
-     * @param $subtreePathString
+     * @param string $subtreePathString
      * @return int|null
      */
     protected function getSubtreeLocationsCount($subtreePathString)

eZ/Publish/Core/Search/Common/Indexer.php

Doc Comments +2 added lines, -2 removed lines

@@ -70,7 +70,7 @@ 
     /**
      * Get PDOStatement to fetch metadata about content objects to be indexed.
      *
-     * @param array $fields Select fields
+     * @param string[] $fields Select fields
      * @return \PDOStatement
      */
     protected function getContentDbFieldsStmt(array $fields)
@@ -108,7 +108,7 @@ 
      * Log warning while progress bar is shown.
      *
      * @param \Symfony\Component\Console\Helper\ProgressBar $progress
-     * @param $message
+     * @param string $message
      */
     protected function logWarning(ProgressBar $progress, $message)
     {

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

Doc Comments +4 added lines, -4 removed lines

@@ -2858,7 +2858,7 @@ 
     /**
      * Test for the addRelation() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Content
+     * @return Relation[]
      *
      * @see \eZ\Publish\API\Repository\ContentService::addRelation()
      * @depends eZ\Publish\API\Repository\Tests\ContentServiceTest::testPublishVersionFromContentDraft
@@ -4768,7 +4768,7 @@ 
     /**
      * Simplify creating custom role with limited set of policies.
      *
-     * @param $roleName
+     * @param string $roleName
      * @param array $policies e.g. [ ['content', 'create'], ['content', 'edit'], ]
      */
     private function createRoleWithPolicies($roleName, array $policies)
@@ -4791,7 +4791,7 @@ 
      * given properties are available in $actualAliases and not more.
      *
      * @param array $expectedAliasProperties
-     * @param array $actualAliases
+     * @param URLAlias[] $actualAliases
      */
     private function assertAliasesCorrect(array $expectedAliasProperties, array $actualAliases)
     {
@@ -4953,7 +4953,7 @@ 
     /**
      * Returns the default fixture of fields used in most tests.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Field[]
+     * @return Field[]
      */
     private function createFieldsFixture()
     {

eZ/Publish/Core/Persistence/Cache/LocationHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -270,9 +270,9 @@
      * Return relevant content and location tags so cache can be purged reliably.
      *
      * @param Location $location
-     * @param array $tags Optional, can be used to specify additional tags.
+     * @param string[] $tags Optional, can be used to specify additional tags.
      *
-     * @return array
+     * @return string[]
      */
     private function getCacheTags(Location $location, $tags = [])
     {

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

Doc Comments +2 added lines, -2 removed lines

@@ -888,7 +888,7 @@ 
      * Asserts that the given $actualDefinition is correctly created from the
      * create struct in $expectedCreate.
      *
-     * @param \eZ\Publish\API\Repository\Values\FieldDefinitionCreateStruct $expectedDefinitionCreate
+     * @param \eZ\Publish\API\Repository\Values\FieldDefinitionCreateStruct $expectedCreate
      * @param \eZ\Publish\API\Repository\Values\FieldDefinition $actualDefinition
      */
     protected function assertFieldDefinitionsEqual($expectedCreate, $actualDefinition)
@@ -3007,7 +3007,7 @@ 
     /**
      * @param \eZ\Publish\API\Repository\Values\ContentType\ContentType $originalType
      * @param \eZ\Publish\API\Repository\Values\ContentType\ContentType $copiedType
-     * @param array $excludedProperties
+     * @param string[] $excludedProperties
      */
     private function assertCopyContentTypeValues($originalType, $copiedType, $excludedProperties = [])
     {