ezsystems / ezpublish-kernel

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/MVC/Symfony/Routing/RouteReference.php

Doc Comments +5 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
     }
 
     /**
-     * @param mixed $route
+     * @param string $route
      */
     public function setRoute($route)
     {
@@ -69,7 +69,7 @@ 
      * Returns a route parameter.
      *
      * @param string $parameterName
-     * @param mixed $defaultValue
+     * @param string $defaultValue
      * @param bool $deep
      *
      * @return mixed
@@ -79,6 +79,9 @@ 
         return $this->params->get($parameterName, $defaultValue, $deep);
     }
 
+    /**
+     * @param string $parameterName
+     */
     public function has($parameterName)
     {
         return $this->params->has($parameterName);

eZ/Publish/Core/REST/Server/Controller/User.php

Doc Comments +5 added lines, -5 removed lines

@@ -141,7 +141,7 @@ 
      *
      * @param $groupPath
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\RestUserGroup
+     * @return Values\CachedValue
      */
     public function loadUserGroup($groupPath)
     {
@@ -178,7 +178,7 @@ 
      *
      * @param $userId
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\RestUser
+     * @return Values\CachedValue
      */
     public function loadUser($userId)
     {
@@ -711,7 +711,7 @@ 
      *
      * @param $groupPath
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\UserGroupList|\eZ\Publish\Core\REST\Server\Values\UserGroupRefList
+     * @return Values\CachedValue
      */
     public function loadSubUserGroups($groupPath, Request $request)
     {
@@ -767,7 +767,7 @@ 
      *
      * @param $userId
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\UserGroupRefList
+     * @return Values\CachedValue
      */
     public function loadUserGroupsOfUser($userId, Request $request)
     {
@@ -807,7 +807,7 @@ 
      *
      * @param $groupPath
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\UserList|\eZ\Publish\Core\REST\Server\Values\UserRefList
+     * @return Values\CachedValue
      */
     public function loadUsersFromGroup($groupPath, Request $request)
     {

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

@@ -11,10 +11,9 @@
 namespace eZ\Publish\API\Repository\Tests;
 
 use EzSystems\EzPlatformSolrSearchEngine\Tests\SetupFactory\LegacySetupFactory as LegacySolrSetupFactory;
-use eZ\Publish\API\Repository\Tests\SetupFactory\LegacyElasticsearch as LegacyElasticsearchSetupFactory;
+use eZ\Publish\API\Repository\Tests\SetupFactory\LegacyElasticsearch;
 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\ContentInfo;
 use eZ\Publish\API\Repository\Values\Content\LocationQuery;
 use eZ\Publish\API\Repository\Values\Content\Query;

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/Core/Persistence/Legacy/Content/TreeHandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
      *
      * Result is returned with oldest version first (sorted by created, or version id as long as it's auto increment).
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      * @param mixed|null $status Optional argument to filter versions by status, like {@see VersionInfo::STATUS_ARCHIVED}.
      * @param int $limit Limit for items returned, -1 means none.
      *
@@ -173,7 +173,7 @@ 
      *
      * @param mixed $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function removeSubtree($locationId)
     {

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)