kaliop-uk / ezmigrationbundle

Command/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -250,9 +250,9 @@ 
         $time = microtime(true) - $start;
         if ($input->getOption('separate-process')) {
             // in case of using subprocesses, we can not measure max memory used
-            $this->writeln("<info>Time taken: ".sprintf('%.3f', $time)." secs</info>");
+            $this->writeln("<info>Time taken: " . sprintf('%.3f', $time) . " secs</info>");
         } else {
-            $this->writeln("<info>Time taken: ".sprintf('%.3f', $time)." secs, memory: ".sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)). ' MB</info>');
+            $this->writeln("<info>Time taken: " . sprintf('%.3f', $time) . " secs, memory: " . sprintf('%.2f', (memory_get_peak_usage(true) / 1000000)) . ' MB</info>');
         }
 
         return $failed;
@@ -484,7 +484,7 @@ 
             $builderArgs[] = '--no-debug';
         }
         if ($input->getOption('siteaccess')) {
-            $builderArgs[] = '--siteaccess='.$input->getOption('siteaccess');
+            $builderArgs[] = '--siteaccess=' . $input->getOption('siteaccess');
         }
         switch ($this->verbosity) {
             case OutputInterface::VERBOSITY_VERBOSE:

Core/Matcher/ContentVersionMatcher.php

Spacing +3 added lines, -3 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;
@@ -204,14 +204,14 @@ 
         $i = 0;
         foreach ($contentVersions as $versionKey => $versionInfo) {
             foreach ($values as $acceptedVersionNo) {
-                if ($acceptedVersionNo > 0 ) {
+                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;
 

Core/Matcher/QueryBasedMatcher.php

Spacing +2 added lines, -2 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:
@@ -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;

Core/Matcher/UserMatcher.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
         $users = [];
 
         foreach ($userIds as $userId) {
-            try{
+            try {
                 // return unique contents
                 $user = $this->repository->getUserService()->loadUser($userId);
 

Core/StorageHandler/Database/Migration.php

Spacing +5 added lines, -5 removed lines

@@ -206,7 +206,7 @@ 
     public function endMigration(APIMigration $migration, $force = false)
     {
         if ($migration->status == APIMigration::STATUS_STARTED) {
-            throw new MigrationBundleException($this->getEntityName($migration)." '{$migration->name}' can not be ended as its status is 'started'...");
+            throw new MigrationBundleException($this->getEntityName($migration) . " '{$migration->name}' can not be ended as its status is 'started'...");
         }
 
         $this->createTableIfNeeded();
@@ -233,13 +233,13 @@ 
         if (!is_array($existingMigrationData)) {
             // commit to release the lock
             $conn->commit();
-            throw new MigrationBundleException($this->getEntityName($migration)." '{$migration->name}' can not be ended as it is not found");
+            throw new MigrationBundleException($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 MigrationBundleException($this->getEntityName($migration)." '{$migration->name}' can not be ended as it is not executing");
+            throw new MigrationBundleException($this->getEntityName($migration) . " '{$migration->name}' can not be ended as it is not executing");
         }
 
         $conn->update(
@@ -395,7 +395,7 @@ 
         if (!is_array($existingMigrationData)) {
             // commit immediately, to release the lock and avoid deadlocks
             $conn->commit();
-            throw new MigrationBundleException($this->getEntityName($migration)." '{$migration->name}' can not be resumed as it is not found");
+            throw new MigrationBundleException($this->getEntityName($migration) . " '{$migration->name}' can not be resumed as it is not found");
         }
 
         // migration exists
@@ -404,7 +404,7 @@ 
         if ($existingMigrationData['status'] != APIMigration::STATUS_SUSPENDED) {
             // commit to release the lock
             $conn->commit();
-            throw new MigrationBundleException($this->getEntityName($migration)." '{$migration->name}' can not be resumed as it is not suspended");
+            throw new MigrationBundleException($this->getEntityName($migration) . " '{$migration->name}' can not be resumed as it is not suspended");
         }
 
         $migration = new APIMigration(

Core/Executor/MigrationDefinitionExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -170,7 +170,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);

Core/Executor/ContentManager.php

Spacing +1 added lines, -1 removed lines

@@ -778,7 +778,7 @@
             },
             array_filter(
                 $this->repository->getContentLanguageService()->loadLanguages(),
-                function ($language) {
+                function($language) {
                     return $language->enabled;
                 }
             )

Core/Executor/ObjectStateManager.php

Spacing +2 added lines, -2 removed lines

@@ -268,10 +268,10 @@
                 $names = array();
                 $descriptions = array();
                 foreach ($objectState->languageCodes as $languageCode) {
-                    $names[$languageCode] =  $objectState->getName($languageCode);
+                    $names[$languageCode] = $objectState->getName($languageCode);
                 }
                 foreach ($objectState->languageCodes as $languageCode) {
-                    $descriptions[$languageCode] =  $objectState->getDescription($languageCode);
+                    $descriptions[$languageCode] = $objectState->getDescription($languageCode);
                 }
                 $groupData = array_merge(
                     $groupData,

Core/Executor/MigrationExecutor.php

Spacing +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
         }
         $step = new MigrationStep($dsl['type'], $dsl, $context);
 
-        switch($dsl['type']) {
+        switch ($dsl['type']) {
             case 'content':
                 return $this->contentManager->execute($step);
             case 'location':
@@ -178,7 +178,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));
             }
         }
     }