kaliop-uk / ezmigrationbundle

Core/Matcher/ContentTypeGroupMatcher.php

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
                 // return unique contents
                 $contentTypeGroup = $this->repository->getContentTypeService()->loadContentTypeGroup($contentTypeGroupId);
                 $contentTypeGroups[$contentTypeGroup->id] = $contentTypeGroup;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }
@@ -125,7 +125,7 @@ 
                 // return unique contents
                 $contentTypeGroup = $this->repository->getContentTypeService()->loadContentTypeGroupByIdentifier($contentTypeGroupIdentifier);
                 $contentTypeGroups[$contentTypeGroup->id] = $contentTypeGroup;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

Core/Matcher/ContentMatcherDirectLoad.php

Spacing +4 added lines, -4 removed lines

@@ -37,11 +37,11 @@ 
             $contents = array();
             switch ($key) {
                 case self::MATCH_CONTENT_ID:
-                    foreach($match as $contentId) {
+                    foreach ($match as $contentId) {
                         try {
                             $content = $this->repository->getContentService()->loadContent($contentId);
                             $contents[$content->id] = $content;
-                        } catch(NotFoundException $e) {
+                        } catch (NotFoundException $e) {
                             if (!$tolerateMisses) {
                                 throw $e;
                             }
@@ -49,11 +49,11 @@ 
                     }
                     break;
                 case self::MATCH_CONTENT_REMOTE_ID:
-                    foreach($match as $contentRemoteId) {
+                    foreach ($match as $contentRemoteId) {
                         try {
                             $content = $this->repository->getContentService()->loadContentByRemoteId($contentRemoteId);
                             $contents[$content->id] = $content;
-                        } catch(NotFoundException $e) {
+                        } catch (NotFoundException $e) {
                             if (!$tolerateMisses) {
                                 throw $e;
                             }

Core/Matcher/ContentVersionMatcher.php

Spacing +7 added lines, -7 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace Kaliop\eZMigrationBundle\Core\Matcher;
 
-use eZ\Publish\API\Repository\Exceptions\UnauthorizedException ;
+use eZ\Publish\API\Repository\Exceptions\UnauthorizedException;
 use eZ\Publish\API\Repository\Repository;
 use eZ\Publish\API\Repository\Values\Content\Content;
 use \eZ\Publish\API\Repository\Values\Content\VersionInfo;
@@ -57,7 +57,7 @@ 
         $versions = array();
 
         $contentCollection = $this->contentMatcher->match($contentConditions, $sort, $offset, $limit, $tolerateMisses);
-        foreach($contentCollection as $content) {
+        foreach ($contentCollection as $content) {
             $versions = array_merge($versions, $this->matchContentVersions($versionConditions, $content));
         }
 
@@ -179,7 +179,7 @@ 
     {
         $versions = array();
         foreach ($this->findAllContentVersions($content) as $versionKey => $versionInfo) {
-            foreach($values as $acceptedStatus) {
+            foreach ($values as $acceptedStatus) {
                 if ($versionInfo->status == self::STATUS_MAP[$acceptedStatus]) {
                     $versions[$versionKey] = $versionInfo;
                     break;
@@ -203,15 +203,15 @@ 
         $contentVersionsCount = count($contentVersions);
         $i = 0;
         foreach ($contentVersions as $versionKey => $versionInfo) {
-            foreach($values as $acceptedVersionNo) {
-                if ($acceptedVersionNo > 0 ) {
+            foreach ($values as $acceptedVersionNo) {
+                if ($acceptedVersionNo > 0) {
                     if ($acceptedVersionNo == $versionInfo->versionNo) {
                         $versions[$versionKey] = $versionInfo;
                         break;
                     }
                 } else {
                     // negative $acceptedVersionNo means 'leave the last X versions', eg: -1 = leave the last version
-                    if ($i < $contentVersionsCount + $acceptedVersionNo)  {
+                    if ($i < $contentVersionsCount + $acceptedVersionNo) {
                         $versions[$versionKey] = $versionInfo;
                         break;
 
@@ -233,7 +233,7 @@ 
         $contentVersions = $this->repository->getContentService()->loadVersions($content->contentInfo);
         // different eZ kernels apparently sort versions in different order...
         $sortedVersions = array();
-        foreach($contentVersions as $versionInfo) {
+        foreach ($contentVersions as $versionInfo) {
             $sortedVersions[$content->contentInfo->id . '/' . $versionInfo->versionNo] = $versionInfo;
         }
         ksort($sortedVersions);

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