kaliop-uk / ezmigrationbundle

Core/DefinitionParser/PHPDefinitionParser.php

Doc Comments +3 added lines

@@ -90,6 +90,9 @@
         );
     }
 
+    /**
+     * @param string $fileName
+     */
     protected function getClassNameFromFile($fileName)
     {
         $parts = explode('_', pathinfo($fileName, PATHINFO_FILENAME), 2);

Core/Executor/ContentTypeManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
      * @throws \InvalidArgumentException When trying to set
      *
      * @param \eZ\Publish\API\Repository\Values\ContentType\ContentType $contentType
-     * @return boolean
+     * @return false|null
      */
     protected function setReferences($contentType)
     {

Braces +1 added lines, -2 removed lines

@@ -393,8 +393,7 @@
                     $ret[$key] = $this->referenceResolver->getReferenceValue($val);
                 }
             }
-        }
-        else {
+        } else {
             $ret = $value;
 
             if ($this->referenceResolver->isReference($value)) {

Core/Loader/Filesystem.php

Braces +1 added lines, -2 removed lines

@@ -85,8 +85,7 @@
                             );
                     }
                 }
-            }
-            else {
+            } else {
                 throw new \Exception("Path '$path' is neither a file nor directory");
             }
         }

Core/ComplexField/ComplexFieldManager.php

Braces +1 added lines, -2 removed lines

@@ -32,8 +32,7 @@
         {
             $fieldService = $this->fieldTypeMap[$fieldTypeIdentifier];
             return $fieldService->createValue($fieldValueArray, $context);
-        }
-        else
+        } else
         {
             throw new \InvalidArgumentException("Field of type '$fieldTypeIdentifier' can not be handled as it does not have a complex field class defined");
         }

Core/MigrationService.php

Doc Comments -1 removed lines

@@ -94,7 +94,6 @@
     }
 
     /**
-     * @param Migration $migration
      */
     public function addMigration(MigrationDefinition $migrationDefinition)
     {

Core/StorageHandler/Database.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     /**
      * Creates and stores a new migration (leaving it in TODO status)
      * @param MigrationDefinition $migrationDefinition
-     * @return mixed
+     * @return Migration
      * @throws \Exception If the migration exists already (we rely on the PK for that)
      */
     public function addMigration(MigrationDefinition $migrationDefinition)

Tests/phpunit/1_GenerateTest.php

Doc Comments +6 added lines

@@ -48,6 +48,9 @@ 
         $this->checkGeneratedFile($this->saveGeneratedFile($output));
     }
 
+    /**
+     * @param null|string $output
+     */
     protected function saveGeneratedFile($output)
     {
 var_dump($output);
@@ -59,6 +62,9 @@ 
         return null;
     }
 
+    /**
+     * @param string|null $filePath
+     */
     protected function checkGeneratedFile($filePath)
     {
         $input = new ArrayInput(array('command' => 'kaliop:migration:status'));