ezsystems / ezpublish-kernel

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

Doc Comments +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
     /**
      * Create user.
      *
-     * @param user $user
+     * @param User $user
      *
      * @return mixed
      */
@@ -338,7 +338,7 @@ 
     /**
      * Remove role from user or user group, by assignment ID.
      *
-     * @param mixed $roleAssignmentId
+     * @param integer $roleAssignmentId
      */
     public function removeRoleAssignmentById($roleAssignmentId)
     {

eZ/Publish/Core/Persistence/Legacy/User/Gateway/ExceptionConversion.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     /**
      * Create user.
      *
-     * @param user $user
+     * @param User $user
      *
      * @return mixed
      */

eZ/Publish/Core/Persistence/Legacy/User/Handler.php

Doc Comments +3 added lines, -3 removed lines

@@ -286,7 +286,7 @@ 
     /**
      * Loads a role draft by the original role ID.
      *
-     * @param mixed $roleId ID of the role the draft was created from.
+     * @param integer $roleId ID of the role the draft was created from.
      *
      * @throws \eZ\Publish\API\Repository\Exceptions\NotFoundException If role is not found
      *
@@ -461,7 +461,7 @@ 
     /**
      * Returns the user policies associated with the user (including inherited policies from user groups).
      *
-     * @param mixed $userId
+     * @param integer $userId
      *
      * @return \eZ\Publish\SPI\Persistence\User\Policy[]
      */
@@ -572,7 +572,7 @@ 
      *
      * Role Assignments with same roleId and limitationIdentifier will be merged together into one.
      *
-     * @param mixed $groupId In legacy storage engine this is the content object id roles are assigned to in ezuser_role.
+     * @param integer $groupId In legacy storage engine this is the content object id roles are assigned to in ezuser_role.
      *                      By the nature of legacy this can currently also be used to get by $userId.
      * @param bool $inherit If true also return inherited role assignments from user groups.
      *

eZ/Publish/Core/Persistence/Legacy/User/Role/Gateway/DoctrineDatabase.php

Doc Comments +3 added lines, -3 removed lines

@@ -433,7 +433,7 @@ 
     /**
      * Loads role assignment for specified assignment ID.
      *
-     * @param mixed $roleAssignmentId
+     * @param integer $roleAssignmentId
      *
      * @return array
      */
@@ -464,7 +464,7 @@ 
     /**
      * Loads role assignments for specified content ID.
      *
-     * @param mixed $groupId
+     * @param integer $groupId
      * @param bool $inherited
      *
      * @return array
@@ -509,7 +509,7 @@ 
     /**
      * Loads role assignments for given role ID.
      *
-     * @param mixed $roleId
+     * @param integer $roleId
      *
      * @return array
      */

eZ/Publish/Core/Persistence/TransformationProcessor/PcreCompiler.php

Doc Comments +2 added lines, -2 removed lines

@@ -196,7 +196,7 @@ 
      * @param string $operator
      * @param string $value
      *
-     * @return callback
+     * @return \Closure
      */
     protected function getTransposeClosure($operator, $value)
     {
@@ -216,7 +216,7 @@ 
      *
      * @param string $char
      *
-     * @return callback
+     * @return \Closure
      */
     protected function compileTargetCharacter($char)
     {

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/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/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()
     {