kaliop-uk / ezmigrationbundle

Command/MigrationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 
             $output->writeln('<info>Migration: ' . $migration->name . '</info>');
             $output->writeln('Status: ' . $status);
-            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--'). '</info>');
+            $output->writeln('Executed on: <info>' . ($migration->executionDate != null ? date("Y-m-d H:i:s", $migration->executionDate) : '--') . '</info>');
             $output->writeln('Execution notes: <info>' . $migration->executionError . '</info>');
 
             if ($migration->status == Migration::STATUS_SUSPENDED) {

Core/Executor/HTTPExecutor.php

Spacing +1 added lines, -1 removed lines

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

Core/StorageHandler/Database/Migration.php

Spacing +5 added lines, -5 removed lines

@@ -171,7 +171,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();
@@ -198,13 +198,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(
@@ -355,7 +355,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
@@ -364,7 +364,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(

Core/Executor/ProcessExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         $builderArgs = array($this->referenceResolver->resolveReference($dsl['command']));
 
         if (isset($dsl['arguments'])) {
-            foreach($dsl['arguments'] as $arg) {
+            foreach ($dsl['arguments'] as $arg) {
                 $builderArgs[] = $this->referenceResolver->resolveReference($arg);
             }
         }

Core/Executor/MigrationExecutor.php

Spacing +3 added lines, -3 removed lines

@@ -105,13 +105,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':
@@ -139,7 +139,7 @@ 
                     return $this->matchConditions($values);
 
                 default:
-                    throw new \Exception("Unknown until condition: '$key' when suspending a migration ".var_export($conditions, true));
+                    throw new \Exception("Unknown until condition: '$key' when suspending a migration " . var_export($conditions, true));
             }
         }
     }

Command/ResumeCommand.php

Spacing +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
         $executed = 0;
         $failed = 0;
 
-        foreach($suspendedMigrations as $suspendedMigration) {
+        foreach ($suspendedMigrations as $suspendedMigration) {
             $output->writeln("<info>Resuming {$suspendedMigration->name}</info>");
 
             try {
@@ -113,7 +113,7 @@ 
 
         $time = microtime(true) - $start;
         $output->writeln("Resumed $executed migrations, failed $failed");
-        $output->writeln("Time taken: ".sprintf('%.2f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB');
+        $output->writeln("Time taken: " . sprintf('%.2f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB');
 
         if ($failed) {
             return 2;

Core/Matcher/ContentMatcher.php

Spacing +5 added lines, -5 removed lines

@@ -46,15 +46,15 @@ 
 
         foreach ($conditions as $key => $values) {
 
-            switch($key) {
+            switch ($key) {
 
                 case self::MATCH_RELATES_TO:
                     $contentService = $this->repository->getContentService();
                     $contents = array();
                     // allow to specify the objects to relate to using different ways
                     $relatedContents = $this->match($values);
-                    foreach($relatedContents as $relatedContent) {
-                        foreach($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
+                    foreach ($relatedContents as $relatedContent) {
+                        foreach ($contentService->loadReverseRelations($relatedContent->contentInfo) as $relatingContent) {
                             $contents[$relatingContent->contentInfo->id] = $relatingContent;
                         }
                     }
@@ -65,8 +65,8 @@ 
                     $contents = array();
                     // allow to specify the objects we relate to using different ways
                     $relatingContents = $this->match($values);
-                    foreach($relatingContents as $relatingContent) {
-                        foreach($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
+                    foreach ($relatingContents as $relatingContent) {
+                        foreach ($contentService->loadRelations($relatingContent->contentInfo) as $relatedContent) {
                             $contents[$relatedContent->contentInfo->id] = $relatedContent;
                         }
                     }

Braces +12 added lines, -4 removed lines

@@ -119,7 +119,9 @@ 
                     $query = new Query();
                     $query->limit = $limit != 0 ? $limit : $this->queryLimit;
                     $query->offset = $offset;
-                    if (isset($query->performCount)) $query->performCount = false;
+                    if (isset($query->performCount)) {
+                        $query->performCount = false;
+                    }
                     $query->filter = $this->getQueryCriterion($key, $values);
                     if (!empty($sort)) {
                         $query->sortClauses = $this->getSortClauses($sort);
@@ -245,7 +247,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ParentLocationId($parentLocationIds);
         $results = $this->repository->getSearchService()->findContent($query);
 
@@ -285,7 +289,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeIdentifier($contentTypeIdentifiers);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries
@@ -313,7 +319,9 @@ 
     {
         $query = new Query();
         $query->limit = $this->queryLimit;
-        if (isset($query->performCount)) $query->performCount = false;
+        if (isset($query->performCount)) {
+            $query->performCount = false;
+        }
         $query->filter = new Query\Criterion\ContentTypeId($contentTypeIds);
         // sort objects by depth, lower to higher, so that deleting them has less chances of failure
         // NB: we only do this in eZP versions that allow depth sorting on content queries

Core/FieldHandler/EzDateAndTime.php

Spacing +7 added lines, -7 removed lines

@@ -58,26 +58,26 @@
     {
         $format = array();
         if (0 != $dateInterval->y) {
-            $format[] = $dateInterval->y.' years';
+            $format[] = $dateInterval->y . ' years';
         }
         if (0 != $dateInterval->m) {
-            $format[] = $dateInterval->m.' months';
+            $format[] = $dateInterval->m . ' months';
         }
         if (0 != $dateInterval->d) {
-            $format[] = $dateInterval->d.' days';
+            $format[] = $dateInterval->d . ' days';
         }
         //if (0 < $dateInterval->h || 0 < $dateInterval->i || 0 < $dateInterval->s) {
         //    $format .= 'T';
         //}
         if (0 != $dateInterval->h) {
-            $format[] = $dateInterval->h.' hours';
+            $format[] = $dateInterval->h . ' hours';
         }
         if (0 != $dateInterval->i) {
-            $format[] = $dateInterval->i.' minutes';
+            $format[] = $dateInterval->i . ' minutes';
         }
         if (0 != $dateInterval->s) {
-            $format[] = $dateInterval->s.' seconds';
+            $format[] = $dateInterval->s . ' seconds';
         }
-        return implode( ', ', $format);
+        return implode(', ', $format);
     }
 }

Core/Matcher/ContentMatcherDirectLoad.php

Spacing +2 added lines, -2 removed lines

@@ -22,13 +22,13 @@
             $contents = array();
             switch ($key) {
                 case self::MATCH_CONTENT_ID:
-                    foreach($match as $contentId) {
+                    foreach ($match as $contentId) {
                         $content = $this->repository->getContentService()->loadContent($contentId);
                         $contents[$content->id] = $content;
                     }
                     break;
                 case self::MATCH_CONTENT_REMOTE_ID:
-                    foreach($match as $contentRemoteId) {
+                    foreach ($match as $contentRemoteId) {
                         $content = $this->repository->getContentService()->loadContentByRemoteId($contentRemoteId);
                         $contents[$content->id] = $content;
                     }