ezsystems / ezpublish-kernel

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/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

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/Bundle/EzPublishRestBundle/Features/Context/RestClient/BuzzDriver.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
     /**
      * Get response status code.
      *
-     * @return string
+     * @return integer
      *
      * @throws \RuntimeException If request hasn't been send already
      */

eZ/Publish/Core/Persistence/Cache/Tests/ContentHandlerTest.php

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

@@ -12,14 +12,13 @@
 use eZ\Publish\SPI\Persistence\Content\Relation as SPIRelation;
 use eZ\Publish\Core\Persistence\Cache\ContentHandler;
 use eZ\Publish\SPI\Persistence\Content;
-use eZ\Publish\SPI\Persistence\Content\Handler;
+use eZ\Publish\SPI\Persistence\Content\Handler as SPIContentHandler;
 use eZ\Publish\SPI\Persistence\Content\ContentInfo;
 use eZ\Publish\SPI\Persistence\Content\VersionInfo;
 use eZ\Publish\SPI\Persistence\Content\CreateStruct;
 use eZ\Publish\SPI\Persistence\Content\UpdateStruct;
 use eZ\Publish\SPI\Persistence\Content\MetadataUpdateStruct;
 use eZ\Publish\SPI\Persistence\Content\Relation\CreateStruct as RelationCreateStruct;
-use eZ\Publish\SPI\Persistence\Content\Handler as SPIContentHandler;
 use eZ\Publish\SPI\Persistence\Content\Location\Handler as SPILocationHandler;
 use Stash\Interfaces\ItemInterface;
 

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

Doc Comments +12 added lines, -12 removed lines

@@ -94,7 +94,7 @@ 
     /**
      * Loads list of aliases by given $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $custom
      * @param mixed $languageId
      *
@@ -264,7 +264,7 @@ 
      *
      * @param string $action
      * @param mixed $languageId
-     * @param mixed $newId
+     * @param integer $newId
      * @param mixed $parentId
      * @param string $textMD5
      */
@@ -438,9 +438,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)
     {
@@ -523,8 +523,8 @@ 
      *
      * Update includes history entries.
      *
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function reparent($oldParentId, $newParentId)
     {
@@ -670,7 +670,7 @@ 
     /**
      * Returns next value for "id" column.
      *
-     * @return mixed
+     * @return string
      */
     public function getNextId()
     {
@@ -813,7 +813,7 @@ 
      * Loads autogenerated entry id by given $action and optionally $parentId.
      *
      * @param string $action
-     * @param mixed|null $parentId
+     * @param integer $parentId
      *
      * @return array
      */
@@ -985,7 +985,7 @@ 
     /**
      * Deletes single custom alias row matched by composite primary key.
      *
-     * @param mixed $parentId
+     * @param integer $parentId
      * @param string $textMD5
      *
      * @return bool
@@ -1023,10 +1023,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/Core/Repository/ContentService.php

Doc Comments +3 added lines, -3 removed lines

@@ -1345,7 +1345,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      *
-     * @return array
+     * @return string[]
      */
     private function getUpdatedLanguageCodes(APIContentUpdateStruct $contentUpdateStruct)
     {
@@ -1372,7 +1372,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct $contentUpdateStruct
      * @param \eZ\Publish\API\Repository\Values\Content\Content $content
      *
-     * @return array
+     * @return string[]
      */
     protected function getLanguageCodesForUpdate(APIContentUpdateStruct $contentUpdateStruct, APIContent $content)
     {
@@ -1491,7 +1491,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\VersionInfo $versionInfo
      * @param int|null $publicationDate If null existing date is kept if there is one, otherwise current time is used.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Content
+     * @return LocationCreateStruct
      */
     protected function internalPublishVersion(APIVersionInfo $versionInfo, $publicationDate = null)
     {

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

Doc Comments +2 added lines, -2 removed lines

@@ -677,7 +677,7 @@ 
      *
      * @deprecated Since platform 1.0. Forwards the request to the new /views location, but returns a 301.
      *
-     * @return \eZ\Publish\Core\REST\Server\Values\RestExecutedView
+     * @return \Symfony\Component\HttpFoundation\Response
      */
     public function createView()
     {
@@ -706,7 +706,7 @@ 
     /**
      * @param \Symfony\Component\HttpFoundation\Request $request
      *
-     * @return mixed
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     protected function parseContentRequest(Request $request)
     {