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/Gateway/DoctrineDatabase.php

Doc Comments +11 added lines, -11 removed lines

@@ -270,7 +270,7 @@ 
      *
      * @param string $action
      * @param mixed $languageId
-     * @param mixed $newId
+     * @param integer $newId
      * @param mixed $parentId
      * @param string $textMD5
      */
@@ -444,9 +444,9 @@ 
      *
      * Removes given $languageId from entry's language mask
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
-     * @param mixed $languageId
+     * @param integer $languageId
      */
     protected function removeTranslation($parentId, $textMD5, $languageId)
     {
@@ -529,8 +529,8 @@ 
      *
      * Update includes history entries.
      *
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function reparent($oldParentId, $newParentId)
     {
@@ -676,7 +676,7 @@ 
     /**
      * Returns next value for "id" column.
      *
-     * @return mixed
+     * @return string
      */
     public function getNextId()
     {
@@ -819,7 +819,7 @@ 
      * Loads autogenerated entry id by given $action and optionally $parentId.
      *
      * @param string $action
-     * @param mixed|null $parentId
+     * @param integer $parentId
      *
      * @return array
      */
@@ -1003,7 +1003,7 @@ 
     /**
      * Deletes single custom alias row matched by composite primary key.
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
      *
      * @return bool
@@ -1041,10 +1041,10 @@ 
      *
      * If $id is set only autogenerated entries will be removed.
      *
-     * @param mixed $action
-     * @param mixed|null $id
+     * @param string $action
+     * @param integer $id
      *
-     * @return bool
+     * @return boolean|null
      */
     public function remove($action, $id = null)
     {

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

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Doctrine\DBAL\Connection;
 use eZ\Publish\API\Repository\Exceptions\ContentFieldValidationException;
 use eZ\Publish\API\Repository\Tests\PHPUnitConstraint\ValidationErrorOccurs as PHPUnitConstraintValidationErrorOccurs;
-use eZ\Publish\API\Repository\Values\Content\Location;
 use EzSystems\EzPlatformSolrSearchEngine\Tests\SetupFactory\LegacySetupFactory as LegacySolrSetupFactory;
 use PHPUnit\Framework\TestCase;
 use eZ\Publish\API\Repository\Repository;

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

Doc Comments +6 added lines, -6 removed lines

@@ -763,7 +763,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Location[]
+     * @return LocationList
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildren
@@ -827,7 +827,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Location[]
+     * @return LocationList
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset, $limit)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildren
@@ -856,7 +856,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @param \eZ\Publish\API\Repository\Values\Content\Location[] $locations
+     * @param LocationList $locations
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset, $limit)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildrenWithOffsetAndLimit
@@ -1140,7 +1140,7 @@ 
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException
      * @throws \eZ\Publish\API\Repository\Exceptions\UnauthorizedException
      *
-     * @return int[]
+     * @return Content[]
      */
     public function testSwapLocationForMainAndSecondaryLocation()
     {
@@ -2342,7 +2342,7 @@ 
      * Assert generated aliases to expected alias return.
      *
      * @param \eZ\Publish\API\Repository\URLAliasService $urlAliasService
-     * @param array $expectedAliases
+     * @param string[] $expectedAliases
      */
     protected function assertGeneratedAliases($urlAliasService, array $expectedAliases)
     {
@@ -2354,7 +2354,7 @@ 
 
     /**
      * @param \eZ\Publish\API\Repository\URLAliasService $urlAliasService
-     * @param array $expectedSubItemAliases
+     * @param string[] $expectedSubItemAliases
      */
     private function assertAliasesBeforeCopy($urlAliasService, array $expectedSubItemAliases)
     {

eZ/Publish/Core/Persistence/Legacy/Content/Location/Gateway/DoctrineDatabase.php

Doc Comments +23 added lines, -20 removed lines

@@ -96,7 +96,7 @@ 
      *
      * @todo optimize
      *
-     * @param mixed $remoteId
+     * @param string $remoteId
      *
      * @return array
      */
@@ -250,7 +250,7 @@ 
     /**
      * Returns data for the first level children of the location identified by given $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
      * @return array
      */
@@ -377,6 +377,9 @@ 
         }
     }
 
+    /**
+     * @param string $pathString
+     */
     private function isHiddenByParent($pathString, array $rows)
     {
         $parentNodeIds = explode('/', trim($pathString, '/'));
@@ -577,8 +580,8 @@ 
      * 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
      */
@@ -754,7 +757,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)
@@ -882,8 +885,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)
     {
@@ -951,8 +954,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)
     {
@@ -1111,8 +1114,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
      */
@@ -1148,9 +1151,9 @@ 
      *
      * The associated content object is left untouched.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function trashLocation($locationId)
     {
@@ -1179,8 +1182,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
      */
@@ -1433,10 +1436,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)
@@ -1564,7 +1567,7 @@ 
     /**
      * Get Query Builder for fetching data of all Locations except the Root node.
      *
-     * @param array $columns list of columns to fetch
+     * @param string[] $columns list of columns to fetch
      *
      * @return \Doctrine\DBAL\Query\QueryBuilder
      */

eZ/Bundle/EzPublishCoreBundle/Imagine/Cache/Resolver/RelativeResolver.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     /**
      * Returns relative image path.
      *
-     * @param $url string
+     * @param string $url string
      * @return string
      */
     protected function rewriteUrl($url)