ezsystems / ezpublish-kernel

eZ/Publish/Core/Persistence/Doctrine/SelectDoctrineQuery.php

Doc Comments -14 removed lines

@@ -47,7 +47,6 @@ 
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters..
      *
-     * @param string|array(string) $... Either a string with a column name or an array of column names.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery returns a pointer to $this.
      */
@@ -107,7 +106,6 @@ 
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters..
      *
-     * @param string|array(string) $... Either a string with a column name or an array of column names.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery returns a pointer to $this.
      */
@@ -131,7 +129,6 @@ 
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters.
      *
-     * @param string|array(string) $... Either a string with a table name or an array of table names.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery a pointer to $this
      */
@@ -183,8 +180,6 @@ 
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with inconsistent parameters or if
      *         invoked without preceding call to from().
      *
-     * @param string $table2,... The table to join with, followed by either the
-     *                           two join columns, or a join condition.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery
      */
@@ -236,8 +231,6 @@ 
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with inconsistent parameters or if
      *         invoked without preceding call to from().
      *
-     * @param string $table2,... The table to join with, followed by either the
-     *                           two join columns, or a join condition.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery
      */
@@ -289,8 +282,6 @@ 
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with inconsistent parameters or if
      *         invoked without preceding call to from().
      *
-     * @param string $table2,... The table to join with, followed by either the
-     *                           two join columns, or a join condition.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery
      */
@@ -313,8 +304,6 @@ 
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters.
      *
-     * @param string|array(string) $... Either a string with a logical expression name
-     * or an array with logical expressions.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery
      */
@@ -378,7 +367,6 @@ 
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters.
      *
-     * @param string $column a column name in the result set
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery a pointer to $this
      */
@@ -399,8 +387,6 @@ 
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException
      *         if called with no parameters.
      *
-     * @param string|array(string) $... Either a string with a logical expression name
-     *                             or an array with logical expressions.
      *
      * @return \eZ\Publish\Core\Persistence\Database\SelectQuery a pointer to $this
      */

eZ/Publish/Core/Persistence/Doctrine/UpdateDoctrineQuery.php

Doc Comments -2 removed lines

@@ -54,8 +54,6 @@
      *
      * @throws \eZ\Publish\Core\Persistence\Database\QueryException if called with no parameters.
      *
-     * @param string|array(string) $... Either a string with a logical expression name
-     * or an array with logical expressions.
      *
      * @return \eZ\Publish\Core\Persistence\Database\UpdateQuery
      */

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

Doc Comments +9 added lines, -9 removed lines

@@ -114,9 +114,9 @@ 
      * Inserts a new content object.
      *
      * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct
-     * @param mixed $currentVersionNo
+     * @param integer $currentVersionNo
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertContentObject(CreateStruct $struct, $currentVersionNo = 1)
     {
@@ -216,7 +216,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\VersionInfo $versionInfo
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $fields
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertVersion(VersionInfo $versionInfo, array $fields)
     {
@@ -610,7 +610,7 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field $field
      * @param \eZ\Publish\Core\Persistence\Legacy\Content\StorageFieldValue $value
      *
-     * @return int ID
+     * @return string ID
      */
     public function insertNewField(Content $content, Field $field, StorageFieldValue $value)
     {
@@ -820,8 +820,8 @@ 
      *
      * Returns an array with the relevant data.
      *
-     * @param mixed $contentId
-     * @param mixed $version
+     * @param integer $contentId
+     * @param integer $version
      * @param string[] $translations
      *
      * @return array
@@ -992,7 +992,7 @@ 
      *
      * Result is returned with oldest version first (using version id as it has index and is 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.
      *
@@ -1060,7 +1060,7 @@ 
     /**
      * Returns all version numbers for the given $contentId.
      *
-     * @param mixed $contentId
+     * @param integer $contentId
      *
      * @return int[]
      */
@@ -1698,7 +1698,7 @@ 
      *
      * @param \eZ\Publish\SPI\Persistence\Content\Relation\CreateStruct $createStruct
      *
-     * @return int ID the inserted ID
+     * @return string ID the inserted ID
      */
     public function insertRelation(RelationCreateStruct $createStruct)
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/UrlAlias/UrlAliasHandlerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -5003,7 +5003,7 @@
     protected $languageMaskGenerator;
 
     /**
-     * @param array $methods
+     * @param string[] $methods
      *
      * @return \eZ\Publish\Core\Persistence\Legacy\Content\UrlAlias\Handler|\PHPUnit_Framework_MockObject_MockObject
      */

eZ/Publish/Core/Repository/ContentService.php

Doc Comments +2 added lines, -2 removed lines

@@ -1338,7 +1338,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)
     {
@@ -1467,7 +1467,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/Repository/Tests/Service/Mock/ContentTest.php

Doc Comments +11 added lines, -4 removed lines

@@ -1052,7 +1052,7 @@ 
      * @param string $mainLanguageCode
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
-     * @param array $languageCodes
+     * @param string[] $languageCodes
      *
      * @return array
      */
@@ -1126,10 +1126,9 @@ 
      * @param \eZ\Publish\SPI\Persistence\Content\Field[] $spiFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      * @param \eZ\Publish\API\Repository\Values\Content\LocationCreateStruct[] $locationCreateStructs
-     * @param \eZ\Publish\SPI\Persistence\Content\ObjectState\Group[] $objectStateGroups
      * @param bool $execute
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentNonRedundantFieldSet(
         $mainLanguageCode,
@@ -2006,7 +2005,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $structFields
      * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition[] $fieldDefinitions
      *
-     * @return mixed
+     * @return ContentCreateStruct
      */
     protected function assertForTestCreateContentRequiredField(
         $mainLanguageCode,
@@ -4793,6 +4792,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -4991,6 +4994,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,

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

@@ -14,7 +14,7 @@ 
 use eZ\Publish\API\Repository\ContentTypeService as APIContentTypeService;
 use eZ\Publish\API\Repository\LocationService as APILocationService;
 use eZ\Publish\API\Repository\Exceptions\NotFoundException as APINotFoundException;
-use eZ\Publish\API\Repository\Values\Content\ContentInfo as APIContentInfo;
+use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use eZ\Publish\API\Repository\Values\ContentType\ContentType as APIContentType;
 use eZ\Publish\API\Repository\Values\Content\Location as APILocation;
 use eZ\Publish\API\Repository\Values\ContentType\FieldDefinition as APIFieldDefinition;
@@ -33,7 +33,6 @@ 
 use eZ\Publish\Core\Repository\Helper\NameSchemaService;
 use eZ\Publish\API\Repository\Values\Content\Field;
 use eZ\Publish\Core\FieldType\Value;
-use eZ\Publish\API\Repository\Values\Content\ContentInfo;
 use eZ\Publish\API\Repository\Values\Content\VersionInfo as APIVersionInfo;
 use eZ\Publish\Core\Repository\Values\ContentType\ContentType;
 use eZ\Publish\Core\Repository\Values\ContentType\FieldDefinition;

eZ/Publish/Core/Search/Common/FieldValueMapper/BooleanMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      *
      * @param \eZ\Publish\SPI\Search\Field $field
      *
-     * @return mixed
+     * @return boolean
      */
     public function map(Field $field)
     {

eZ/Publish/Core/Search/Common/FieldValueMapper/IntegerMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      *
      * @param \eZ\Publish\SPI\Search\Field $field
      *
-     * @return mixed
+     * @return integer
      */
     public function map(Field $field)
     {

eZ/Publish/Core/Search/Common/FieldValueMapper/PriceMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      *
      * @param \eZ\Publish\SPI\Search\Field $field
      *
-     * @return mixed
+     * @return double
      */
     public function map(Field $field)
     {