ezsystems / ezpublish-kernel

eZ/Publish/Core/Repository/RoleService.php

Doc Comments +3 added lines, -3 removed lines

@@ -230,7 +230,7 @@ 
      *
      * @param mixed $id
      *
-     * @return \eZ\Publish\API\Repository\Values\User\RoleDraft
+     * @return \eZ\Publish\SPI\Persistence\User\Policy
      */
     public function loadRoleDraft($id)
     {
@@ -1396,7 +1396,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\User\RoleCreateStruct $roleCreateStruct
      *
-     * @return \eZ\Publish\Core\FieldType\ValidationError[][][]
+     * @return \eZ\Publish\Core\FieldType\ValidationError[]
      */
     protected function validateRoleCreateStruct(APIRoleCreateStruct $roleCreateStruct)
     {
@@ -1426,7 +1426,7 @@ 
      * @param string $function
      * @param \eZ\Publish\API\Repository\Values\User\Limitation[] $limitations
      *
-     * @return \eZ\Publish\Core\FieldType\ValidationError[][]
+     * @return \eZ\Publish\Core\FieldType\ValidationError[]
      */
     protected function validatePolicy($module, $function, array $limitations)
     {

eZ/Publish/Core/Repository/Tests/Service/Integration/Base.php

Doc Comments +2 added lines, -2 removed lines

@@ -123,7 +123,7 @@ 
      *
      * @param mixed[] $expectedValues
      * @param \eZ\Publish\API\Repository\Values\ValueObject $actualObject
-     * @param array $skipProperties
+     * @param string[] $skipProperties
      */
     protected function assertPropertiesCorrect(array $expectedValues, ValueObject $actualObject, array $skipProperties = array())
     {
@@ -158,7 +158,7 @@ 
      *
      * @param \eZ\Publish\API\Repository\Values\ValueObject $expectedValues
      * @param \eZ\Publish\API\Repository\Values\ValueObject $actualObject
-     * @param array $skipProperties
+     * @param string[] $skipProperties
      */
     protected function assertStructPropertiesCorrect(ValueObject $expectedValues, ValueObject $actualObject, array $skipProperties = array())
     {

eZ/Publish/Core/Repository/Tests/Service/Integration/ContentBase.php

Doc Comments +6 added lines, -1 removed lines

@@ -951,7 +951,7 @@ 
      * @param \eZ\Publish\API\Repository\Values\Content\Field[] $fields
      * @param string $mainLanguageCode
      *
-     * @return array an array of language code strings
+     * @return string[] an array of language code strings
      */
     protected function getLanguageCodesFromFields(array $fields, $mainLanguageCode)
     {
@@ -2298,6 +2298,11 @@ 
         $this->assertRelations($relations, $contentDraft->contentInfo, $mediaContentInfo);
     }
 
+    /**
+     * @param Relation[] $relations
+     * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $sourceContentInfo
+     * @param \eZ\Publish\API\Repository\Values\Content\ContentInfo $destinationContentInfo
+     */
     protected function assertRelations($relations, $sourceContentInfo, $destinationContentInfo)
     {
         self::assertInternalType('array', $relations);

eZ/Publish/Core/Repository/Tests/Service/Integration/ContentTypeBase.php

Doc Comments +11 added lines, -2 removed lines

@@ -367,7 +367,7 @@ 
      *
      * @covers \eZ\Publish\Core\Repository\ContentTypeService::loadContentTypeGroups
      *
-     * @return array
+     * @return \eZ\Publish\API\Repository\Values\ContentType\ContentTypeGroup[]
      */
     public function testLoadContentTypeGroups()
     {
@@ -2038,7 +2038,7 @@ 
      *
      * @todo when all fieldTypes are functional revisit this and simplify by testing against fixtures
      *
-     * @return array
+     * @return ContentType[]
      */
     public function testLoadContentTypes()
     {
@@ -2946,6 +2946,10 @@ 
         $this->assertAddFieldDefinitionStructValues($loadedType, $fieldDefCreate);
     }
 
+    /**
+     * @param \eZ\Publish\API\Repository\Values\ContentType\ContentTypeDraft $loadedType
+     * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinitionCreateStruct $fieldDefCreate
+     */
     public function assertAddFieldDefinitionStructValues($loadedType, $fieldDefCreate)
     {
         foreach ($loadedType->fieldDefinitions as $fieldDefinition) {
@@ -3276,6 +3280,11 @@ 
         );
     }
 
+    /**
+     * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition $originalField
+     * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinition $updatedField
+     * @param \eZ\Publish\API\Repository\Values\ContentType\FieldDefinitionUpdateStruct $updateStruct
+     */
     protected function assertUpdateFieldDefinitionStructValues($originalField, $updatedField, $updateStruct)
     {
         $this->assertPropertiesCorrect(

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