kaliop-uk / ezmigrationbundle

Command/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
                 $builderArgs[] = '--no-transactions';
             }
             if ($input->getOption('siteaccess')) {
-                $builderArgs[]='--siteaccess='.$input->getOption('siteaccess');
+                $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
             }
         }
 
@@ -233,9 +233,9 @@ 
         $this->writeln("Executed $executed migrations, failed $failed, skipped $skipped");
         if ($input->getOption('separate-process')) {
             // in case of using subprocesses, we can not measure max memory used
-            $this->writeln("Time taken: ".sprintf('%.2f', $time)." secs");
+            $this->writeln("Time taken: " . sprintf('%.2f', $time) . " secs");
         } else {
-            $this->writeln("Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB');
+            $this->writeln("Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB');
         }
     }
 

Core/MigrationService.php

Spacing +8 added lines, -8 removed lines

@@ -259,7 +259,7 @@ 
         }
 
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition). " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not execute " . $this->getEntityName($migrationDefinition) . " '{$migrationDefinition->name}': {$migrationDefinition->parsingError}");
         }
 
         /// @todo add support for setting in $migrationContext a userContentType ?
@@ -293,7 +293,7 @@ 
 
         try {
 
-            $i = $stepOffset+1;
+            $i = $stepOffset + 1;
             $finalStatus = Migration::STATUS_DONE;
             $finalMessage = null;
 
@@ -418,12 +418,12 @@ 
     public function resumeMigration(Migration $migration, $useTransaction = true)
     {
         if ($migration->status != Migration::STATUS_SUSPENDED) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': it is not in suspended status");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': it is not in suspended status");
         }
 
         $migrationDefinitions = $this->getMigrationsDefinitions(array($migration->path));
         if (!count($migrationDefinitions)) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': its definition is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': its definition is missing");
         }
 
         $defs = $migrationDefinitions->getArrayCopy();
@@ -431,18 +431,18 @@ 
 
         $migrationDefinition = $this->parseMigrationDefinition($migrationDefinition);
         if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': {$migrationDefinition->parsingError}");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': {$migrationDefinition->parsingError}");
         }
 
         // restore context
         $this->contextHandler->restoreCurrentContext($migration->name);
 
         if (!isset($this->migrationContext[$migration->name])) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is missing");
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is missing");
         }
         $restoredContext = $this->migrationContext[$migration->name];
-        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'] )) {
-            throw new \Exception("Can not resume ".$this->getEntityName($migration)." '{$migration->name}': the stored context is invalid");
+        if (!is_array($restoredContext) || !isset($restoredContext['context']) || !isset($restoredContext['step'])) {
+            throw new \Exception("Can not resume " . $this->getEntityName($migration) . " '{$migration->name}': the stored context is invalid");
         }
 
         // update migration status

Core/Executor/ServiceExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
         } else {
             $args = array();
         }
-        foreach($args as &$val) {
+        foreach ($args as &$val) {
             $val = $this->resolveReferencesRecursively($val);
         }
 

Core/ReferenceResolver/EmbeddedRegexpReferenceResolverTrait.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     public function hasEmbeddedReferences($string)
     {
         $regexp = $this->getEmbeddedRegexp();
-        return (bool) preg_match_all($regexp, $string, $matches);
+        return (bool)preg_match_all($regexp, $string, $matches);
     }
     
     /**
@@ -52,7 +52,7 @@ 
     {
         // we need to alter the regexp we usr for std ref resolving, as it will be used to match parts of text, not the whole string
         $regexp = substr($this->getRegexp(), 1, -1);
-        return '/' . preg_quote($this->beginToken). preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
+        return '/' . preg_quote($this->beginToken) . preg_replace(array('/^\^/'), array(''), $regexp) . '[^' . $this->endToken . ']+' . preg_quote($this->endToken) . '/';
     }
 
     /**

Core/FieldHandler/EzRichText.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new \Exception("Reference resolver injected into EzRichText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Core/FieldHandler/EzXmlText.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function setReferenceResolver(ReferenceResolverInterface $referenceResolver)
     {
-        if (! $referenceResolver instanceof EmbeddedReferenceResolverInterface) {
+        if (!$referenceResolver instanceof EmbeddedReferenceResolverInterface) {
             throw new \Exception("Reference resolver injected into EzXmlText field handler should implement EmbeddedReferenceResolverInterface");
         }
         parent::setReferenceResolver($referenceResolver);

Core/ReferenceResolver/LoopResolver.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      */
     public function getReferenceValue($identifier)
     {
-        switch(substr($identifier, 5)) {
+        switch (substr($identifier, 5)) {
             case 'iteration':
                 return end($this->stack);
             case 'depth':

Command/TestBenchCommand.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
 echo 'B';*/
 
         $ms->executeMigration(new MigrationDefinition(
-            'test_'.time(),
+            'test_' . time(),
             '/hello',
             '123',
             MigrationDefinition::STATUS_PARSED,
@@ -62,8 +62,8 @@ 
 
         $sh = $this->getContainer()->get('ez_migration_bundle.storage_handler');
 
-        $m1 = 'hello_'.time();
-        $m2 = 'there_'.time();
+        $m1 = 'hello_' . time();
+        $m2 = 'there_' . time();
         $sh->startMigration(new MigrationDefinition(
             $m1,
             '/hello',