ezsystems / ezpublish-kernel

eZ/Publish/Core/Helper/Tests/FieldsGroups/ArrayTranslatorFieldsGroupsListTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -65,6 +65,9 @@ 
         );
     }
 
+    /**
+     * @param string $default
+     */
     private function buildList($groups, $default)
     {
         return new SettingsFieldsGroupsList(
@@ -75,7 +78,7 @@ 
     }
 
     /**
-     * @return \Symfony\Component\Translation\TranslatorInterface|\PHPUnit_Framework_MockObject_MockObject
+     * @return \Symfony\Component\Translation\TranslatorInterface
      */
     private function getTranslatorMock()
     {

eZ/Publish/Core/Helper/Tests/FieldsGroups/RepositoryConfigFieldsGroupsListFactoryTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\eZ\Bundle\EzPublishCoreBundle\ApiLoader\RepositoryConfigurationProvider
+     * @return \eZ\Bundle\EzPublishCoreBundle\ApiLoader\RepositoryConfigurationProvider
      */
     protected function getRepositoryConfigMock()
     {
@@ -48,7 +48,7 @@ 
     }
 
     /**
-     * @return \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\Translation\TranslatorInterface
+     * @return \Symfony\Component\Translation\TranslatorInterface
      */
     protected function getTranslatorMock()
     {

eZ/Bundle/EzPublishRestBundle/Tests/Functional/RoleTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -119,6 +119,7 @@ 
      * @depends testCreateRole
      * @covers POST /user/roles/{roleId}
      *
+     * @param string $roleHref
      * @return string The created role draft href
      */
     public function testCreateRoleDraft($roleHref)
@@ -412,7 +413,6 @@ 
     /**
      * @covers       POST /user/users/{userId}/roles
      *
-     * @param string $roleHref
      * @param array $limitation
      *
      * @return string assigned role href

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace eZ\Bundle\EzPublishRestBundle\Tests\Functional;
 
 use eZ\Bundle\EzPublishRestBundle\Tests\Functional\TestCase as RESTFunctionalTestCase;
-use eZ\Publish\API\Repository\Values\User\Limitation;
 
 class RoleTest extends RESTFunctionalTestCase
 {

eZ/Publish/Core/Base/Container/ApiLoader/FieldTypeNameableCollectionFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      * Registers an eZ Publish field type.
      * Field types are being registered as a closure so that they will be lazy loaded.
      *
-     * @param string $nameableFieldTypeServiceId The field type nameable service Id
+     * @param string $fieldTypeServiceId The field type nameable service Id
      * @param string $fieldTypeAlias The field type alias (e.g. "ezstring")
      */
     public function registerNonNameableFieldType($fieldTypeServiceId, $fieldTypeAlias)

eZ/Bundle/EzPublishRestBundle/DependencyInjection/Configuration.php

Doc Comments +3 added lines

@@ -22,6 +22,9 @@
         return $treeBuilder;
     }
 
+    /**
+     * @param \Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition $rootNode
+     */
     public function addRestRootResourcesSection($rootNode)
     {
         $systemNode = $this->generateScopeBaseNode($rootNode);

eZ/Publish/API/Repository/Tests/LocationServiceTest.php

Doc Comments +5 added lines, -5 removed lines

@@ -762,7 +762,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Location[]
+     * @return LocationList
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildren
@@ -826,7 +826,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @return \eZ\Publish\API\Repository\Values\Content\Location[]
+     * @return LocationList
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset, $limit)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildren
@@ -855,7 +855,7 @@ 
     /**
      * Test for the loadLocationChildren() method.
      *
-     * @param \eZ\Publish\API\Repository\Values\Content\Location[] $locations
+     * @param LocationList $locations
      *
      * @see \eZ\Publish\API\Repository\LocationService::loadLocationChildren($location, $offset, $limit)
      * @depends eZ\Publish\API\Repository\Tests\LocationServiceTest::testLoadLocationChildrenWithOffsetAndLimit
@@ -2127,7 +2127,7 @@ 
      * Assert generated aliases to expected alias return.
      *
      * @param \eZ\Publish\API\Repository\URLAliasService $urlAliasService
-     * @param array $expectedAliases
+     * @param string[] $expectedAliases
      */
     protected function assertGeneratedAliases($urlAliasService, array $expectedAliases)
     {
@@ -2139,7 +2139,7 @@ 
 
     /**
      * @param \eZ\Publish\API\Repository\URLAliasService $urlAliasService
-     * @param array $expectedSubItemAliases
+     * @param string[] $expectedSubItemAliases
      */
     private function assertAliasesBeforeCopy($urlAliasService, array $expectedSubItemAliases)
     {

eZ/Publish/Core/Persistence/Legacy/Content/UrlAlias/Handler.php

Doc Comments +11 added lines, -11 removed lines

@@ -101,8 +101,8 @@ 
      * $alwaysAvailable controls whether the url alias is accessible in all
      * languages.
      *
-     * @param mixed $locationId
-     * @param mixed $parentLocationId
+     * @param integer $locationId
+     * @param integer $parentLocationId
      * @param string $name the new name computed by the name schema or url alias schema
      * @param string $languageCode
      * @param bool $alwaysAvailable
@@ -231,7 +231,7 @@ 
      * If $languageCode is null the $alias is created in the system's default
      * language. $alwaysAvailable makes the alias available in all languages.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param string $path
      * @param bool $forwarding
      * @param string $languageCode
@@ -393,7 +393,7 @@ 
     /**
      * List of user generated or autogenerated url entries, pointing to $locationId.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      * @param bool $custom if true the user generated aliases are listed otherwise the autogenerated
      *
      * @return \eZ\Publish\SPI\Persistence\Content\UrlAlias[]
@@ -522,9 +522,9 @@ 
      *
      * This method triggers the change of the autogenerated aliases.
      *
-     * @param mixed $locationId
-     * @param mixed $oldParentId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $oldParentId
+     * @param integer $newParentId
      */
     public function locationMoved($locationId, $oldParentId, $newParentId)
     {
@@ -552,9 +552,9 @@ 
      *
      * This method triggers the creation of the autogenerated aliases for the copied locations
      *
-     * @param mixed $locationId
-     * @param mixed $newLocationId
-     * @param mixed $newParentId
+     * @param integer $locationId
+     * @param integer $newLocationId
+     * @param integer $newParentId
      */
     public function locationCopied($locationId, $newLocationId, $newParentId)
     {
@@ -657,7 +657,7 @@ 
     /**
      * Notifies the underlying engine that a location was deleted or moved to trash.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      */
     public function locationDeleted($locationId)
     {

eZ/Publish/Core/MVC/Symfony/Cache/Http/LocationAwareStore.php

Doc Comments +4 added lines, -4 removed lines

@@ -97,7 +97,7 @@ 
      * @param string $tag    The tag key
      * @param string $digest The digest hash to store representing the cache item.
      *
-     * @return bool|void
+     * @return false|null
      */
     private function saveTag($tag, $digest)
     {
@@ -131,7 +131,7 @@ 
      *
      * @param \Symfony\Component\HttpFoundation\Request $request
      *
-     * @return bool True if purge was successful. False otherwise
+     * @return boolean|null True if purge was successful. False otherwise
      */
     public function purgeByRequest(Request $request)
     {
@@ -173,7 +173,7 @@ 
      *
      * @deprecated Use cache:clear, with multi tagging theoretically there shouldn't be need to delete all anymore from core.
      *
-     * @return bool
+     * @return boolean|null
      */
     public function purgeAllContent()
     {
@@ -186,7 +186,7 @@ 
      *
      * @param string $tag
      *
-     * @return bool
+     * @return false|null
      */
     private function purgeByCacheTag($tag)
     {

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

Doc Comments +1 added lines, -1 removed lines

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