kaliop-uk / ezmigrationbundle

Core/ReferenceResolver/ChainResolver.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
         $refs = array();
 
         foreach ($this->resolvers as $resolver) {
-            if (! $resolver instanceof EnumerableReferenceResolverInterface) {
+            if (!$resolver instanceof EnumerableReferenceResolverInterface) {
                 // Disabled - we now allow chaining enumerable and non-enumerable sets
                 //throw new \Exception("Could not enumerate references because of chained resolver of type: " . get_class($resolver));
                 continue;

Core/Executor/BasePHPExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 
             $args = $dsl['arguments'];
 
-            foreach($args as &$val) {
+            foreach ($args as &$val) {
                 $val = $this->resolveReferencesRecursively($val);
             }
         } else {

Core/Executor/HTTPExecutor.php

Spacing +1 added lines, -1 removed lines

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

Core/Executor/LoopExecutor.php

Spacing +6 added lines, -6 removed lines

@@ -99,14 +99,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }
@@ -135,14 +135,14 @@ 
                     $subStep = new MigrationStep($type, $stepDef, array_merge($context, array()));
                     try {
                         $result = $stepExecutors[$j]->execute($subStep);
-                    } catch(MigrationStepSkippedException $e) {
+                    } catch (MigrationStepSkippedException $e) {
                         // all ok, continue the loop
-                    } catch(LoopContinueException $e) {
+                    } catch (LoopContinueException $e) {
                         // all ok, move to the next iteration
                         break;
                     }
                 }
-            } catch(LoopBreakException $e) {
+            } catch (LoopBreakException $e) {
                 // all ok, exit the loop
                 break;
             }

Core/Executor/ProcessExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         $builderArgs = array($this->referenceResolver->resolveReference($dsl['command']));
 
         if (isset($dsl['arguments'])) {
-            foreach($dsl['arguments'] as $arg) {
+            foreach ($dsl['arguments'] as $arg) {
                 /// @todo should this be recursive?
                 $builderArgs[] = $this->referenceResolver->resolveReference($arg);
             }

Core/Executor/ContentManager.php

Spacing +4 added lines, -4 removed lines

@@ -493,7 +493,7 @@ 
                             }
                             // we do allow array values for refs, but not multi-level
                             if (is_array($value)) {
-                                foreach($hashValue as $subValue) {
+                                foreach ($hashValue as $subValue) {
                                     if (is_array($subValue) || is_object($subValue)) {
                                         throw new \InvalidArgumentException('Content Manager does not support setting references for attribute ' . $reference['attribute'] . ': the given value is an array with a non scalar element');
                                     }
@@ -575,7 +575,7 @@ 
                     $locations = $locationService->loadLocations($content->contentInfo);
                     if (count($locations) > 1) {
                         $otherParentLocations = array();
-                        foreach($locations as $otherLocation) {
+                        foreach ($locations as $otherLocation) {
                             if ($otherLocation->id != $location->id) {
                                 $otherParentLocations[] = $otherLocation->parentLocationId;
                             }
@@ -626,7 +626,7 @@ 
                 }
 
                 $attributes = array();
-                foreach($languages as $lang) {
+                foreach ($languages as $lang) {
                     foreach ($content->getFieldsByLanguage($lang) as $fieldIdentifier => $field) {
                         $fieldDefinition = $contentType->getFieldDefinition($fieldIdentifier);
                         $fieldValue = $this->fieldHandlerManager->fieldValueToHash(
@@ -774,7 +774,7 @@ 
             },
             array_filter(
                 $this->repository->getContentLanguageService()->loadLanguages(),
-                function ($language) {
+                function($language) {
                     return $language->enabled;
                 }
             )

Core/Loader/Filesystem.php

Spacing +1 added lines, -1 removed lines

@@ -120,6 +120,6 @@
     {
         $rootDir = realpath($this->kernel->getRootDir() . '/..') . '/';
         // note: we handle the case of 'path = root dir', but path is expected to include a filename...
-        return $path === $rootDir ? './' : preg_replace('#^' . preg_quote($rootDir, '#'). '#', '', $path);
+        return $path === $rootDir ? './' : preg_replace('#^' . preg_quote($rootDir, '#') . '#', '', $path);
     }
 }

Core/FieldHandler/EzSelection.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 
         // allow user to pass in selection values by name
         $fieldSettings = null;
-        foreach($fieldValue as $key => $val) {
+        foreach ($fieldValue as $key => $val) {
 
             // NB: this might result in double reference-resolving when the original value is a string, given preResolveReferences...
             $val = $this->referenceResolver->resolveReference($val);
@@ -48,7 +48,7 @@ 
                     if ($fieldSettings === null) {
                         $fieldSettings = $this->loadFieldSettings($context['contentTypeIdentifier'], $context['fieldIdentifier']);
                     }
-                    foreach($fieldSettings['options'] as $pos => $name) {
+                    foreach ($fieldSettings['options'] as $pos => $name) {
                         if ($name === $val) {
                             $fieldValue[$key] = $pos;
                             break;

Core/Matcher/QueryBasedMatcher.php

Spacing +8 added lines, -8 removed lines

@@ -155,7 +155,7 @@ 
                     return new Query\Criterion\Field($attribute, self::$operatorsMap[$operator], $match);
                 } else {
                     /// @todo make the list of unary operators more flexible
-                    switch($spec) {
+                    switch ($spec) {
                         case 'empty':
                             return new Query\Criterion\IsFieldEmpty($attribute);
                         default:
@@ -180,7 +180,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::CREATED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_GROUP:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->groupMatcher->matchOneByKey($value)->id;
                     }
@@ -199,7 +199,7 @@ 
                 return new Query\Criterion\DateMetadata(Query\Criterion\DateMetadata::MODIFIED, self::$operatorsMap[$operator], $match);
 
             case self::MATCH_OBJECT_STATE:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->stateMatcher->matchOneByKey($value)->id;
                     }
@@ -207,7 +207,7 @@ 
                 return new Query\Criterion\ObjectStateId($values);
 
             case self::MATCH_OWNER:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->userMatcher->matchOneByKey($value)->id;
                     }
@@ -227,7 +227,7 @@ 
                 return new Query\Criterion\ParentLocationId($locationIds);
 
             case self::MATCH_SECTION:
-                foreach($values as &$value) {
+                foreach ($values as &$value) {
                     if (!ctype_digit($value)) {
                         $value = $this->sectionMatcher->matchOneByKey($value)->id;
                     }
@@ -248,7 +248,7 @@ 
 
             case self::MATCH_AND:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -256,7 +256,7 @@ 
 
             case self::MATCH_OR:
                 $subCriteria = array();
-                foreach($values as $subCriterion) {
+                foreach ($values as $subCriterion) {
                     $value = reset($subCriterion);
                     $subCriteria[] = $this->getQueryCriterion(key($subCriterion), $value);
                 }
@@ -301,7 +301,7 @@ 
 
             $direction = $this->hash2SortOrder($sortItem['sort_order']);
 
-            switch($sortItem['sort_field']) {
+            switch ($sortItem['sort_field']) {
                 case self::SORT_CONTENT_ID:
                     $out[] = new SortClause\ContentId($direction);
                     break;