kaliop-uk / ezmigrationbundle

Core/MigrationService.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
     }
 
     /**
-     * @return string[]
+     * @return integer[]
      */
     public function listExecutors()
     {

Core/Executor/LanguageManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
     /**
      * Sets references to certain language attributes.
      *
-     * @param \eZ\Publish\API\Repository\Values\Content\Language|LanguageCollection $language
+     * @param \eZ\Publish\API\Repository\Values\Content\Language $language
      * @throws \InvalidArgumentException When trying to set a reference to an unsupported attribute
      * @return boolean
      */

Core/Executor/ContentManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -648,7 +648,7 @@
      * @param string $contentTypeIdentifier
      * @param array $context
      * @throws \InvalidArgumentException
-     * @return mixed
+     * @return \Kaliop\eZMigrationBundle\API\Value\MigrationStep
      */
     protected function getFieldValue($value, FieldDefinition $fieldDefinition, $contentTypeIdentifier, array $context = array())
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use eZ\Publish\API\Repository\Values\ContentType\ContentType;
 use eZ\Publish\API\Repository\Values\ContentType\FieldDefinition;
-use eZ\Publish\API\Repository\Values\Content\Location;
 use eZ\Publish\API\Repository\Values\Content\Content;
 use eZ\Publish\API\Repository\Values\Content\ContentCreateStruct;
 use eZ\Publish\API\Repository\Values\Content\ContentUpdateStruct;

Core/Executor/LocationManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -244,7 +244,7 @@
      * The Location Manager currently supports setting references to location id.
      *
      * @throws \InvalidArgumentException When trying to set a reference to an unsupported attribute.
-     * @param \eZ\Publish\API\Repository\Values\Content\Location|LocationCollection $location
+     * @param LocationCollection|null $location
      * @return boolean
      */
     protected function setReferences($location, $step)

Core/Executor/MigrationDefinitionExecutor.php

Doc Comments +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
 
     /**
      * @todo cache this for faster access
-     * @return array
+     * @return integer[]
      */
     protected function getGeneratingExecutors()
     {

Core/Executor/MigrationExecutor.php

Doc Comments +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function cancel($dsl, $context)
@@ -70,7 +70,7 @@ 
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function suspend($dsl, $context)

Core/Executor/MailExecutor.php

Doc Comments +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     /**
      * @param array $dsl
      * @param array $context
-     * @return true
+     * @return boolean
      * @throws \Exception
      */
     protected function send($dsl, $context)

Core/StorageHandler/Database/Migration.php

Doc Comments +3 added lines

@@ -447,6 +447,9 @@
         );
     }
 
+    /**
+     * @param APIMigration $migration
+     */
     protected function getEntityName($migration)
     {
         return end(explode('\\', get_class($migration)));

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 use Kaliop\eZMigrationBundle\API\Collection\MigrationCollection;
 use Kaliop\eZMigrationBundle\API\Value\Migration as APIMigration;
 use Kaliop\eZMigrationBundle\API\Value\MigrationDefinition;
-
 use Kaliop\eZMigrationBundle\API\ConfigResolverInterface;
 
 /**

Core/Executor/ContentTypeManager.php

Doc Comments +4 added lines, -1 removed lines

@@ -652,6 +652,9 @@ 
         return $fieldDefinitionUpdateStruct;
     }
 
+    /**
+     * @param string $contentTypeIdentifier
+     */
     protected function getFieldSettings($value, $fieldTypeIdentifier, $contentTypeIdentifier)
     {
         // 1st update any references in the value array
@@ -704,7 +707,7 @@ 
      *
      * @param string|array $newValue if passed a string, we will use $lang as its language. If passed an array, it will be used as is - expected format [ "eng-GB": "Name", "fre-FR": "Nom", ... ]
      * @param string $lang ex: eng-GB. Not used when $newValue is an array
-     * @param array $currentValue current set of values in all known languages. Will be merged with $newValue, $newValue taking precedence
+     * @param string[] $currentValue current set of values in all known languages. Will be merged with $newValue, $newValue taking precedence
      * @return array in the format [ "eng-GB": "Name", "fre-FR": "Nom", ... ]
      */
     protected function getMultilingualValue($newValue, $lang, $currentValue = array())