kaliop-uk / ezmigrationbundle

Core/ReferenceResolver/ChainResolver.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
         $refs = array();
 
         foreach ($this->resolvers as $resolver) {
-            if (! $resolver instanceof EnumerableReferenceResolverInterface) {
+            if (!$resolver instanceof EnumerableReferenceResolverInterface) {
                 throw new \Exception("Could not enumerate references because of chained resolver of type: " . get_class($resolver));
             }
 

Core/Matcher/ReferenceMatcher.php

Spacing +2 added lines, -2 removed lines

@@ -81,7 +81,7 @@ 
 
             switch ($key) {
                 case self::MATCH_AND:
-                    foreach($values as $subCriterion) {
+                    foreach ($values as $subCriterion) {
                         $value = $this->match($subCriterion);
                         if (!reset($value)) {
                             return $value;
@@ -90,7 +90,7 @@ 
                     return array(true);
 
                 case self::MATCH_OR:
-                    foreach($values as $subCriterion) {
+                    foreach ($values as $subCriterion) {
                         $value = $this->match($subCriterion);
                         if (reset($value)) {
                             return $value;

Core/ContextHandler.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     public function storeCurrentContext($migrationName)
     {
         $context = array();
-        foreach($this->providers as $label => $provider) {
+        foreach ($this->providers as $label => $provider) {
             $context[$label] = $provider->getCurrentContext($migrationName);
         }
 
@@ -52,7 +52,7 @@ 
         if (!is_array($context)) {
             throw new \Exception("No execution context found associated with migration '$migrationName'");
         }
-        foreach($this->providers as $label => $provider) {
+        foreach ($this->providers as $label => $provider) {
             if (isset($context[$label])) {
                 $provider->restoreContext($migrationName, $context[$label]);
             }

Command/MigrationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 
             $output->writeln('<info>Migration: ' . $migration->name . '</info>');
             $output->writeln('Status: ' . $status);
-            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--'). '</info>');
+            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--') . '</info>');
             $output->writeln('Execution notes: <info>' . $migration->executionError . '</info>');
 
             if ($migration->status == Migration::STATUS_SUSPENDED) {

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/Executor/HTTPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         $body = isset($dsl['body']) ? $this->resolveReferencesInText($dsl['body']) : null;
 
         if (isset($dsl['client'])) {
-            $client = $this->container->get('httplug.client.'.$dsl['client']);
+            $client = $this->container->get('httplug.client.' . $dsl['client']);
         } else {
             $client = $this->container->get('httplug.client');
         }

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)));

Spacing +5 added lines, -5 removed lines

@@ -171,7 +171,7 @@ 
     public function endMigration(APIMigration $migration, $force = false)
     {
         if ($migration->status == APIMigration::STATUS_STARTED) {
-            throw new \Exception($this->getEntityName($migration)." '{$migration->name}' can not be ended as its status is 'started'...");
+            throw new \Exception($this->getEntityName($migration) . " '{$migration->name}' can not be ended as its status is 'started'...");
         }
 
         $this->createTableIfNeeded();
@@ -198,13 +198,13 @@ 
         if (!is_array($existingMigrationData)) {
             // commit to release the lock
             $conn->commit();
-            throw new \Exception($this->getEntityName($migration)." '{$migration->name}' can not be ended as it is not found");
+            throw new \Exception($this->getEntityName($migration) . " '{$migration->name}' can not be ended as it is not found");
         }
 
         if (($existingMigrationData['status'] != APIMigration::STATUS_STARTED) && !$force) {
             // commit to release the lock
             $conn->commit();
-            throw new \Exception($this->getEntityName($migration)." '{$migration->name}' can not be ended as it is not executing");
+            throw new \Exception($this->getEntityName($migration) . " '{$migration->name}' can not be ended as it is not executing");
         }
 
         $conn->update(
@@ -355,7 +355,7 @@ 
         if (!is_array($existingMigrationData)) {
             // commit immediately, to release the lock and avoid deadlocks
             $conn->commit();
-            throw new \Exception($this->getEntityName($migration)." '{$migration->name}' can not be resumed as it is not found");
+            throw new \Exception($this->getEntityName($migration) . " '{$migration->name}' can not be resumed as it is not found");
         }
 
         // migration exists
@@ -364,7 +364,7 @@ 
         if ($existingMigrationData['status'] != APIMigration::STATUS_SUSPENDED) {
             // commit to release the lock
             $conn->commit();
-            throw new \Exception($this->getEntityName($migration)." '{$migration->name}' can not be resumed as it is not suspended");
+            throw new \Exception($this->getEntityName($migration) . " '{$migration->name}' can not be resumed as it is not suspended");
         }
 
         $migration = new APIMigration(

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/ProcessExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         $builderArgs = array($this->referenceResolver->resolveReference($dsl['command']));
 
         if (isset($dsl['arguments'])) {
-            foreach($dsl['arguments'] as $arg) {
+            foreach ($dsl['arguments'] as $arg) {
                 $builderArgs[] = $this->referenceResolver->resolveReference($arg);
             }
         }

Command/ResumeCommand.php

Spacing +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
         $executed = 0;
         $failed = 0;
 
-        foreach($suspendedMigrations as $suspendedMigration) {
+        foreach ($suspendedMigrations as $suspendedMigration) {
             $output->writeln("<info>Resuming {$suspendedMigration->name}</info>");
 
             try {
@@ -113,7 +113,7 @@ 
 
         $time = microtime(true) - $start;
         $output->writeln("Resumed $executed migrations, failed $failed");
-        $output->writeln("Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB');
+        $output->writeln("Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB');
 
         if ($failed) {
             return 2;