ezsystems / ezpublish-kernel

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

Doc Comments +11 added lines, -11 removed lines

@@ -365,10 +365,10 @@ 
      * destination, identified by $destinationParentId. Relations do not need
      * to be updated, since they refer to Content. URLs are not touched.
      *
-     * @param mixed $sourceId
-     * @param mixed $destinationParentId
+     * @param integer $sourceId
+     * @param integer $destinationParentId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function move($sourceId, $destinationParentId)
     {
@@ -411,7 +411,7 @@ 
     /**
      * Sets a location to be hidden, and it self + all children to invisible.
      *
-     * @param mixed $id Location ID
+     * @param integer $id Location ID
      */
     public function hide($id)
     {
@@ -424,7 +424,7 @@ 
      * Sets a location to be unhidden, and self + children to visible unless a parent is hiding the tree.
      * If not make sure only children down to first hidden node is marked visible.
      *
-     * @param mixed $id
+     * @param integer $id
      */
     public function unHide($id)
     {
@@ -439,10 +439,10 @@ 
      * Make the location identified by $locationId1 refer to the Content
      * referred to by $locationId2 and vice versa.
      *
-     * @param mixed $locationId1
-     * @param mixed $locationId2
+     * @param integer $locationId1
+     * @param integer $locationId2
      *
-     * @return bool
+     * @return boolean|null
      */
     public function swap($locationId1, $locationId2)
     {
@@ -489,9 +489,9 @@ 
      * main Location will get the first of its other Locations assigned as the
      * new main Location.
      *
-     * @param mixed $locationId
+     * @param integer $locationId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function removeSubtree($locationId)
     {
@@ -502,7 +502,7 @@ 
      * Set section on all content objects in the subtree.
      *
      * @param mixed $locationId
-     * @param mixed $sectionId
+     * @param integer $sectionId
      */
     public function setSectionForSubtree($locationId, $sectionId)
     {

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/ContentTypeHandlerTest.php

Doc Comments +3 added lines, -3 removed lines

@@ -1126,9 +1126,9 @@ 
     /**
      * Returns a handler to test with $methods mocked.
      *
-     * @param array $methods
+     * @param string[] $methods
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\Type\Handler
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getPartlyMockedHandler(array $methods)
     {
@@ -1163,7 +1163,7 @@ 
     /**
      * Returns a mapper mock.
      *
-     * @param array $methods
+     * @param string[] $methods
      *
      * @return \eZ\Publish\Core\Persistence\Legacy\Content\Type\Mapper
      */

eZ/Publish/Core/Persistence/Legacy/Tests/Content/Type/MapperTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -383,7 +383,7 @@ 
     /**
      * Returns a Mapper with conversion methods mocked.
      *
-     * @return Mapper
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getNonConvertingMapper()
     {
@@ -413,7 +413,7 @@ 
     /**
      * Returns a converter registry mock.
      *
-     * @return \eZ\Publish\Core\Persistence\Legacy\Content\FieldValue\ConverterRegistry
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     protected function getConverterRegistryMock()
     {

eZ/Publish/Core/SignalSlot/Tests/ServiceTest.php

Doc Comments +4 added lines, -4 removed lines

@@ -154,8 +154,8 @@ 
     /**
      * Creates a User object from $userId, $userRemoteId and $userVersionNo.
      *
-     * @param mixed $userId
-     * @param mixed $userRemoteId
+     * @param integer $userId
+     * @param string $userRemoteId
      * @param int $userVersionNo
      *
      * @return \eZ\Publish\Core\Repository\Values\User\User
@@ -177,8 +177,8 @@ 
     /**
      * Returns a new UserGroup.
      *
-     * @param mixed $groupId
-     * @param mixed $groupRemoteId
+     * @param integer $groupId
+     * @param string $groupRemoteId
      * @param int $groupVersioNo
      *
      * @return \eZ\Publish\Core\Repository\Values\User\UserGroup

eZ/Publish/Core/MVC/Symfony/SiteAccess/Tests/RouterURIElementTest.php

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

@@ -9,10 +9,9 @@
 namespace eZ\Publish\Core\MVC\Symfony\SiteAccess\Tests;
 
 use eZ\Publish\Core\MVC\Symfony\SiteAccess;
-use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement;
+use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement as URIElementMatcher;
 use PHPUnit\Framework\TestCase;
 use eZ\Publish\Core\MVC\Symfony\SiteAccess\Router;
-use eZ\Publish\Core\MVC\Symfony\SiteAccess\Matcher\URIElement as URIElementMatcher;
 use eZ\Publish\Core\MVC\Symfony\Routing\SimplifiedRequest;
 use eZ\Publish\Core\MVC\Symfony\SiteAccess\MatcherBuilder;
 use Psr\Log\LoggerInterface;

eZ/Publish/SPI/Tests/FieldType/ImageIntegrationTest.php

Doc Comments +3 added lines

@@ -100,6 +100,9 @@
         );
     }
 
+    /**
+     * @return DeprecationWarnerInterface
+     */
     public function getDeprecationWarnerMock()
     {
         if (!isset($this->deprecationWarnerMock)) {

eZ/Bundle/EzPublishCoreBundle/EventListener/ConfigScopeListener.php

Doc Comments +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     /**
      * Sets the complete list of view providers.
      *
-     * @param array $viewProviders
+     * @param \PHPUnit\Framework\MockObject\MockObject $viewProviders
      */
     public function setViewProviders(array $viewProviders)
     {

eZ/Bundle/EzPublishCoreBundle/Tests/EventListener/BackgroundIndexingTerminateListenerTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     /**
      * @dataProvider indexingProvider
      *
-     * @param array|null $value
+     * @param array|null $values
      * @param \Psr\Log\LoggerInterface|\PHPUnit\Framework\MockObject\MockObject|null $logger
      */
     public function testIndexing(array $values = null, $logger = null)

eZ/Publish/Core/FieldType/Tests/FieldTypeTest.php

Doc Comments +1 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
     private $fieldTypeUnderTest;
 
     /**
-     * @return \eZ\Publish\Core\Persistence\TransformationProcessor|\PHPUnit\Framework\MockObject\MockObject
+     * @return TransformationProcessor
      */
     protected function getTransformationProcessorMock()
     {
@@ -682,7 +682,6 @@ 
     }
 
     /**
-     * @param mixed $inputValue
      * @param array $expectedResult
      *
      * @dataProvider provideInputForFromHash
@@ -830,7 +829,6 @@ 
     }
 
     /**
-     * @param mixed $inputConfiguration
      *
      * @dataProvider provideValidFieldSettings
      */
@@ -858,7 +856,6 @@ 
     }
 
     /**
-     * @param mixed $inputConfiguration
      *
      * @dataProvider provideValidFieldSettings
      */