kaliop-uk / ezmigrationbundle

Core/Matcher/ObjectStateMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
                 // return unique contents
                 $objectState = $this->repository->getObjectStateService()->loadObjectState($objectStateId);
                 $objectStates[$objectState->id] = $objectState;
-            } catch(NotFoundException $e) {
+            } catch (NotFoundException $e) {
                 if (!$tolerateMisses) {
                     throw $e;
                 }

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

Spacing +6 added lines, -6 removed lines

@@ -178,7 +178,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();
@@ -205,13 +205,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(
@@ -366,7 +366,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
@@ -375,7 +375,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(
@@ -434,7 +434,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/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);
         }
     }