kaliop-uk / ezmigrationbundle

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)

Spacing +3 added lines, -3 removed lines

@@ -116,7 +116,7 @@ 
         );
         try {
             $conn->insert($this->migrationsTableName, $this->migrationToArray($migration));
-        } catch(UniqueConstraintViolationException $e) {
+        } catch (UniqueConstraintViolationException $e) {
             throw new \Exception("Migration '{$migrationDefinition->name}' already exists");
         }
 
@@ -311,7 +311,7 @@ 
         $t->addColumn('execution_error', 'string', array('length' => 4000, 'notnull' => false));
         $t->setPrimaryKey(array('migration'));
 
-        foreach($schema->toSql($dbPlatform) as $sql) {
+        foreach ($schema->toSql($dbPlatform) as $sql) {
             $this->dbHandler->exec($sql);
         }
     }
@@ -326,7 +326,7 @@ 
     {
         /** @var \Doctrine\DBAL\Schema\AbstractSchemaManager $sm */
         $sm = $this->dbHandler->getConnection()->getSchemaManager();
-        foreach($sm->listTables() as $table) {
+        foreach ($sm->listTables() as $table) {
             if ($table->getName() == $tableName) {
                 return true;
             }

Command/MigrationCommand.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        if (! $input->getOption('add') && ! $input->getOption('delete')) {
+        if (!$input->getOption('add') && !$input->getOption('delete')) {
             throw new \InvalidArgumentException('You must specify whether you want to --add or --delete the specified migration.');
         }
 
@@ -79,7 +79,7 @@ 
                 throw new \InvalidArgumentException(sprintf('The path "%s" does not correspond to any migration definition.', $migrationNameOrPath));
             }
 
-            foreach($migrationDefinitionCollection as $migrationDefinition) {
+            foreach ($migrationDefinitionCollection as $migrationDefinition) {
                 $migrationName = basename($migrationDefinition->path);
 
                 $migration = $migrationService->getMigration($migrationNameOrPath);