ezsystems / ezpublish-kernel

eZ/Publish/Core/Repository/Tests/Service/Mock/ContentTest.php

Doc Comments +11 added lines, -4 removed lines

@@ -1054,7 +1054,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
      */
@@ -1128,10 +1128,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,
@@ -2008,7 +2007,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 assertForTestCreateContentThrowsContentValidationExceptionRequiredField(
         $mainLanguageCode,
@@ -4795,6 +4794,10 @@ 
         );
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentValidationExceptionRequiredField(
         $initialLanguageCode,
         $structFields,
@@ -4993,6 +4996,10 @@ 
         }
     }
 
+    /**
+     * @param Field[] $existingFields
+     * @param FieldDefinition[] $fieldDefinitions
+     */
     public function assertForTestUpdateContentThrowsContentFieldValidationException(
         $initialLanguageCode,
         $structFields,

eZ/Publish/Core/Repository/Tests/Service/Mock/NameSchemaTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\API\Repository\Values\Content\Field[]
+     * @return Field[]
      */
     protected function getFields()
     {
@@ -204,7 +204,7 @@ 
     }
 
     /**
-     * @return \eZ\Publish\Core\Repository\Values\ContentType\FieldDefinition[]
+     * @return FieldDefinition[]
      */
     protected function getFieldDefinitions()
     {

eZ/Publish/Core/Repository/Tests/Service/Mock/RelationProcessorTest.php

Doc Comments +5 added lines

@@ -504,6 +504,11 @@
         );
     }
 
+    /**
+     * @param integer $id
+     * @param null|integer $fieldDefinitionId
+     * @param integer $contentId
+     */
     protected function getStubbedRelation($id, $type, $fieldDefinitionId, $contentId)
     {
         return new \eZ\Publish\Core\Repository\Values\Content\Relation(

eZ/Publish/Core/Repository/Tests/Service/Mock/RepositoryTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -628,7 +628,7 @@
      * Returns Role stub.
      *
      * @param array $policiesData
-     * @param mixed $roleId
+     * @param integer $roleId
      *
      * @return \eZ\Publish\SPI\Persistence\User\Role
      */

eZ/Publish/Core/Repository/TrashService.php

Doc Comments +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
      *
      * @param \eZ\Publish\API\Repository\Values\Content\Location $location
      *
-     * @return null|\eZ\Publish\API\Repository\Values\Content\TrashItem null if location was deleted, otherwise TrashItem
+     * @return TrashItem|null null if location was deleted, otherwise TrashItem
      */
     public function trash(Location $location)
     {

eZ/Publish/Core/Repository/UserService.php

Doc Comments +1 added lines, -1 removed lines

@@ -502,7 +502,7 @@
      *
      * @param mixed $userId
      *
-     * @return \eZ\Publish\API\Repository\Values\User\User
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException if a user with the given id was not found
      */

eZ/Publish/Core/Repository/Values/ContentType/FieldType.php

Doc Comments +2 added lines, -2 removed lines

@@ -154,7 +154,7 @@ 
      * Returns the fallback default value of field type when no such default
      * value is provided in the field definition in content types.
      *
-     * @return mixed
+     * @return Value
      */
     public function getEmptyValue()
     {
@@ -184,7 +184,7 @@ 
      *
      * @param mixed $hash
      *
-     * @return mixed
+     * @return Value
      */
     public function fromHash($hash)
     {

eZ/Publish/Core/REST/Client/ContentTypeService.php

Doc Comments +6 added lines, -6 removed lines

@@ -122,7 +122,7 @@ 
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If group can not be found
      *
-     * @param mixed $contentTypeGroupId
+     * @param string $contentTypeGroupId
      *
      * @return \eZ\Publish\API\Repository\Values\ContentType\ContentTypeGroup
      */
@@ -174,7 +174,7 @@ 
     /**
      * Get all Content Type Groups.
      *
-     * @return \eZ\Publish\API\Repository\Values\ContentType\ContentTypeGroup[]
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadContentTypeGroups()
     {
@@ -535,9 +535,9 @@ 
      * ATTENTION: This is not an API method and only meant for internal use in
      * the REST Client implementation.
      *
-     * @param mixed $fieldDefinitionListReference
+     * @param string $fieldDefinitionListReference
      *
-     * @return \eZ\Publish\Core\REST\Client\Values\FieldDefinitionList
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadFieldDefinitionList($fieldDefinitionListReference)
     {
@@ -558,9 +558,9 @@ 
      * ATTENTION: This is not an API method and only meant for internal use in
      * the REST Client implementation.
      *
-     * @param mixed $contentTypeGroupListReference
+     * @param string $contentTypeGroupListReference
      *
-     * @return \eZ\Publish\Core\REST\Client\Values\ContentTypeGroupRefList
+     * @return \eZ\Publish\API\Repository\Values\ValueObject
      */
     public function loadContentTypeGroupList($contentTypeGroupListReference)
     {

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

@@ -29,7 +29,8 @@
 use eZ\Publish\Core\REST\Common\RequestParser;
 use eZ\Publish\Core\REST\Common\Input\Dispatcher;
 use eZ\Publish\Core\REST\Common\Output\Visitor;
-use eZ\Publish\Core\REST\Common\Message; use eZ\Publish\Core\REST\Client\Exceptions\InvalidArgumentValue;
+use eZ\Publish\Core\REST\Common\Message;
+use eZ\Publish\Core\REST\Client\Exceptions\InvalidArgumentValue;
 use eZ\Publish\Core\REST\Common\Exceptions\InvalidArgumentException;
 use eZ\Publish\Core\REST\Common\Exceptions\ForbiddenException;
 use eZ\Publish\Core\REST\Client\Exceptions\BadStateException;

eZ/Publish/Core/REST/Client/Input/Parser/Relation.php

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      * @param array $data
      * @param \eZ\Publish\Core\REST\Common\Input\ParsingDispatcher $parsingDispatcher
      *
-     * @return \eZ\Publish\API\Repository\Values\Relation\Version
+     * @return Values\Content\Relation
      *
      * @todo Error handling
      * @todo Should the related ContentInfo structs really be loaded here or do