kaliop-uk / ezmigrationbundle

Core/Matcher/ContentTypeMatcher.php

Spacing +3 added lines, -3 removed lines

@@ -108,7 +108,7 @@ 
                 // return unique contents
                 $contentType = $this->repository->getContentTypeService()->loadContentType($contentTypeId);
                 $contentTypes[$contentType->id] = $contentType;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -133,7 +133,7 @@ 
                 // return unique contents
                 $contentType = $this->repository->getContentTypeService()->loadContentTypeByIdentifier($contentTypeIdentifier);
                 $contentTypes[$contentType->id] = $contentType;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -157,7 +157,7 @@ 
                 // return unique contents
                 $contentType = $this->repository->getContentTypeService()->loadContentTypeByRemoteId($contentTypeRemoteId);
                 $contentTypes[$contentType->id] = $contentType;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

Core/Executor/MigrationExecutor.php

Spacing +3 added lines, -3 removed lines

@@ -117,13 +117,13 @@ 
         $dsl['mode'] = 'load';
         // be kind to users and allow them not to specify this explicitly
         if (isset($dsl['references'])) {
-            foreach($dsl['references'] as &$refDef) {
+            foreach ($dsl['references'] as &$refDef) {
                 $refDef['overwrite'] = true;
             }
         }
         $step = new MigrationStep($dsl['type'], $dsl, $context);
 
-        switch($dsl['type']) {
+        switch ($dsl['type']) {
             case 'content':
                 return $this->contentManager->execute($step);
             case 'location':
@@ -154,7 +154,7 @@ 
                     return $this->matchConditions($values);
 
                 default:
-                    throw new InvalidStepDefinitionException("Unknown until condition: '$key' when suspending a migration ".var_export($conditions, true));
+                    throw new InvalidStepDefinitionException("Unknown until condition: '$key' when suspending a migration " . var_export($conditions, true));
             }
         }
     }

Core/StorageHandler/Database/Context.php

Spacing +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
         foreach ($schema->toSql($dbPlatform) as $sql) {
             try {
                 $this->dbHandler->exec($sql);
-            } catch(QueryException $e) {
+            } catch (QueryException $e) {
                 // work around limitations in both Mysql and Doctrine
                 // @see https://github.com/kaliop-uk/ezmigrationbundle/issues/176
                 if (strpos($e->getMessage(), '1071 Specified key was too long; max key length is 767 bytes') !== false &&

Core/StorageHandler/Database/TableStorage.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      */
     protected function injectTableCreationOptions(Table $table)
     {
-        foreach($this->tableCreationOptions as $key => $value) {
+        foreach ($this->tableCreationOptions as $key => $value) {
             $table->addOption($key, $value);
         }
     }

Core/Executor/AbstractExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         if (is_string($key) && is_string($value)) {
             return array('identifier' => $key, 'attribute' => $value);
         }
-        if (!is_array($value) || !isset($value['identifier']) || ! isset($value['attribute'])) {
+        if (!is_array($value) || !isset($value['identifier']) || !isset($value['attribute'])) {
             throw new InvalidStepDefinitionException("Invalid reference definition for reference number $key");
         }
         return $value;

Core/Executor/NonScalarReferenceSetterTrait.php

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
         // q2: why not just check for Countable interface instead of AbstractCollection? Or at least allow ArrayIterators and ObjectIterators
         if (is_array($results) || $results instanceof AbstractCollection) {
             $expectedResultsCount = $this->expectedResultsCount($step);
-            switch($expectedResultsCount) {
+            switch ($expectedResultsCount) {
                 case self::$EXPECT_UNSPECIFIED:
                 case self::$EXPECT_ANY:
                     break;
@@ -62,7 +62,7 @@ 
      */
     protected function expectedResultsType($step)
     {
-        switch($this->expectedResultsCount($step)) {
+        switch ($this->expectedResultsCount($step)) {
             case 1:
                 return self::$RESULT_TYPE_SINGLE;
             case 0:
@@ -104,7 +104,7 @@ 
 
         // BC
         if (isset($step->dsl['references_type'])) {
-            switch($step->dsl['references_type']) {
+            switch ($step->dsl['references_type']) {
                 case 'array':
                     return self::$EXPECT_ANY;
                 case 'scalar':
@@ -130,7 +130,7 @@ 
      */
     protected function hasNonScalarReferences($referencesDefinition)
     {
-        foreach($referencesDefinition as $key => $referenceDefinition) {
+        foreach ($referencesDefinition as $key => $referenceDefinition) {
             $referenceDefinition = $this->parseReferenceDefinition($key, $referenceDefinition);
             if (!$this->isScalarReference($referenceDefinition))
             {

Core/Executor/RoleManager.php

Spacing +3 added lines, -3 removed lines

@@ -444,7 +444,7 @@ 
     protected function sortPolicyLimitationsDefinitions(array &$limitations)
     {
         usort($limitations, function($l1, $l2) {
-            if (($iComp = strcmp($l1['identifier'], $l2['identifier'])) != 0 ) {
+            if (($iComp = strcmp($l1['identifier'], $l2['identifier'])) != 0) {
                 return $iComp;
             }
             if (is_int($l1['values']) || is_float($l1['values'])) {
@@ -473,10 +473,10 @@ 
             return $this->compareArraysForSorting($p1['limitations'], $p2['limitations']);
             $p1LimIds = array();
             $p2LimIds = array();
-            foreach($p1['limitations'] as $lim) {
+            foreach ($p1['limitations'] as $lim) {
                 $p1LimIds = $lim['identifier'];
             }
-            foreach($p2['limitations'] as $lim) {
+            foreach ($p2['limitations'] as $lim) {
                 $p2LimIds = $lim['identifier'];
             }
             /// @todo if limitations identifier are the same, sort by lim. values...

Core/Executor/MigrationDefinitionExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -168,7 +168,7 @@ 
 
         foreach ($dsl['references'] as $key => $reference) {
             // BC
-            if (is_array($reference) && isset($reference['json_path']) && !isset($reference['attribute'] )) {
+            if (is_array($reference) && isset($reference['json_path']) && !isset($reference['attribute'])) {
                 $reference['attribute'] = $reference['json_path'];
             }
             $reference = $this->parseReferenceDefinition($key, $reference);
@@ -197,7 +197,7 @@ 
     {
         $migrationService = $this->migrationService;
         $executors = $migrationService->listExecutors();
-        foreach($executors as $key => $name) {
+        foreach ($executors as $key => $name) {
             $executor = $migrationService->getExecutor($name);
             if (!$executor instanceof MigrationGeneratorInterface) {
                 unset($executors[$key]);

Core/Executor/RepositoryExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -216,7 +216,7 @@ 
 
         $multivalued = ($this->expectedResultsType($step) == self::$RESULT_TYPE_MULTIPLE);
 
-        if ($item instanceof AbstractCollection  || is_array($item)) {
+        if ($item instanceof AbstractCollection || is_array($item)) {
             $items = $item;
         } else {
             $items = array($item);
@@ -261,7 +261,7 @@ 
         // allow setting *some* refs even when we have 0 or N matches
         foreach ($referencesDefinition as $key => $reference) {
             $reference = $this->parseReferenceDefinition($key, $reference);
-            switch($reference['attribute']) {
+            switch ($reference['attribute']) {
 
                 case 'count':
                     $value = count($entity);