kaliop-uk / ezmigrationbundle

Command/MigrateCommand.php

Spacing +7 added lines, -7 removed lines

@@ -61,7 +61,7 @@ 
     {
         $migrationsService = $this->getMigrationService();
 
-        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path')) ;
+        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path'));
         $migrations = $migrationsService->getMigrations();
 
         if (!count($migrationDefinitions)) {
@@ -71,7 +71,7 @@ 
 
         // filter away all migrations except 'to do' ones
         $toExecute = array();
-        foreach($migrationDefinitions as $name => $migrationDefinition) {
+        foreach ($migrationDefinitions as $name => $migrationDefinition) {
             if (!isset($migrations[$name]) || (($migration = $migrations[$name]) && $migration->status == Migration::STATUS_TODO)) {
                 $toExecute[$name] = $migrationsService->parseMigrationDefinition($migrationDefinition);
             }
@@ -88,7 +88,7 @@ 
 
         $data = array();
         $i = 1;
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
             $notes = '';
             if ($migrationDefinition->status != MigrationDefinition::STATUS_PARSED) {
                 $notes = '<error>' . $migrationDefinition->parsingError . '</error>';
@@ -123,7 +123,7 @@ 
             $output->writeln("=============================================\n");
         }
 
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
 
             // let's skip migrations that we know are invalid - user was warned and he decide to proceed anyway
             if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
@@ -135,12 +135,12 @@ 
 
             try {
                 $migrationsService->executeMigration($migrationDefinition);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 if ($input->getOption('ignore-failures')) {
-                    $output->writeln("\n<error>Migration failed!".$e->getMessage()."</error>\n");
+                    $output->writeln("\n<error>Migration failed!" . $e->getMessage() . "</error>\n");
                     continue;
                 }
-                $output->writeln("\n<error>Migration aborted!".$e->getMessage()."</error>");
+                $output->writeln("\n<error>Migration aborted!" . $e->getMessage() . "</error>");
                 return 1;
             }
 

Command/StatusCommand.php

Spacing +5 added lines, -5 removed lines

@@ -41,7 +41,7 @@ 
     {
         $migrationsService = $this->getMigrationService();
 
-        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path')) ;
+        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path'));
         $migrations = $migrationsService->getMigrations();
 
         if (!count($migrationDefinitions) && !count($migrations)) {
@@ -51,10 +51,10 @@ 
 
         // create a unique ist of all migrations (coming from db) and definitions (coming from disk)
         $index = array();
-        foreach($migrationDefinitions as $migrationDefinition) {
+        foreach ($migrationDefinitions as $migrationDefinition) {
             $index[$migrationDefinition->name] = array('definition' => $migrationDefinition);
         }
-        foreach($migrations as $migration) {
+        foreach ($migrations as $migration) {
             if (isset($index[$migration->name])) {
                 $index[$migration->name]['migration'] = $migration;
             } else {
@@ -68,7 +68,7 @@ 
                         {
                             $index[$migration->name]['definition'] = reset($migrationDefinitionCollection);
                         }
-                    } catch(\Exception $e) {
+                    } catch (\Exception $e) {
                         /// @todo one day we should be able to limit the kind of exceptions we have to catch here...
                     }
                 }
@@ -80,7 +80,7 @@ 
 
         $data = array();
         $i = 1;
-        foreach($index as $name => $value) {
+        foreach ($index as $name => $value) {
             if (!isset($value['migration'])) {
                 $migrationDefinition = $migrationsService->parseMigrationDefinition($value['definition']);
                 $notes = '';

Command/MigrationCommand.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        if (/*! $input->getOption('add') &&*/ ! $input->getOption('delete')) {
+        if (/*! $input->getOption('add') &&*/ !$input->getOption('delete')) {
             throw new \InvalidArgumentException('You must specify whether you want to --delete the specified migration.');
         }
 

Core/Executor/UserManager.php

Spacing +4 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
                 $this->dsl['user_id'] = array($this->dsl['user_id']);
             }
             foreach ($this->dsl['user_id'] as $id) {
-                array_push($usersToDelete,$userService->loadUser($id));
+                array_push($usersToDelete, $userService->loadUser($id));
             }
         }
         if (isset($this->dsl['email'])) {
@@ -140,7 +140,7 @@ 
                 $this->dsl['email'] = array($this->dsl['email']);
             }
             foreach ($this->dsl['email'] as $email) {
-                $usersToDelete = array_merge($usersToDelete,$userService->loadUsersByEmail($email));
+                $usersToDelete = array_merge($usersToDelete, $userService->loadUsersByEmail($email));
             }
         }
         if (isset($this->dsl['username'])) {
@@ -148,11 +148,11 @@ 
                 $this->dsl['username'] = array($this->dsl['username']);
             }
             foreach ($this->dsl['username'] as $username) {
-                array_push($usersToDelete,$userService->loadUserByLogin($username));
+                array_push($usersToDelete, $userService->loadUserByLogin($username));
             }
         }
 
-        foreach($usersToDelete as $user){
+        foreach ($usersToDelete as $user) {
             $userService->deleteUser($user);
         }
     }

Core/Executor/TagManager.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 
     protected $tagService;
 
-    public function __construct(TagsService $tagService=null)
+    public function __construct(TagsService $tagService = null)
     {
         $this->tagService = $tagService;
     }

Core/Executor/ContentManager.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
         }
 
         $draft = $contentService->createContentDraft($contentInfo);
-        $contentService->updateContent($draft->versionInfo,$contentUpdateStruct);
+        $contentService->updateContent($draft->versionInfo, $contentUpdateStruct);
         $content = $contentService->publishVersion($draft->versionInfo);
 
         if (array_key_exists('new_remote_id', $this->dsl)) {
@@ -158,7 +158,7 @@ 
 
         $contentService = $this->repository->getContentService();
 
-        if (isset($this->dsl['object_id'])){
+        if (isset($this->dsl['object_id'])) {
             if (!is_array($this->dsl['object_id'])) {
                 $this->dsl['object_id'] = array($this->dsl['object_id']);
             }
@@ -186,7 +186,7 @@ 
      * @param ContentCreateStruct $createStruct
      * @param array $fields
      */
-    protected function setFields(ContentCreateStruct &$createStruct, array $fields)
+    protected function setFields(ContentCreateStruct & $createStruct, array $fields)
     {
         foreach ($fields as $field) {
             // each $field is one key value pair
@@ -214,7 +214,7 @@ 
      * @param ContentUpdateStruct $updateStruct
      * @param array $fields
      */
-    protected function setFieldsToUpdate(ContentUpdateStruct &$updateStruct, array $fields, ContentType $contentType)
+    protected function setFieldsToUpdate(ContentUpdateStruct & $updateStruct, array $fields, ContentType $contentType)
     {
         foreach ($fields as $field) {
             // each $field is one key value pair

Core/Executor/UserGroupManager.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
             $groupIds = array($groupIds);
         }
 
-        foreach($groupIds as $groupId) {
+        foreach ($groupIds as $groupId) {
             $userGroup = $userService->loadUserGroup($groupId);
             $userService->deleteUserGroup($userGroup);
         }

Core/Executor/LocationManager.php

Indentation +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             !isset($match['parent_location_id']) &&
             !isset($match['parent_location_remote_id'])
         ) {
-          throw new \Exception('Either the ID or remote ID of a content, the ID or remote ID of a location or the id or remote ID
+            throw new \Exception('Either the ID or remote ID of a content, the ID or remote ID of a location or the id or remote ID
           of the parent location of the contents you want to create a new location are required to create a new location.');
         }
 

Core/Loader/Filesystem.php

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         if (empty($paths)) {
             $paths = array();
             /** @var $bundle \Symfony\Component\HttpKernel\Bundle\BundleInterface */
-            foreach($this->kernel->getBundles() as $bundle)
+            foreach ($this->kernel->getBundles() as $bundle)
             {
                 $path = $bundle->getPath() . "/" . $this->versionDirectory;
                 if (is_dir($path)) {
@@ -67,7 +67,7 @@ 
         }
 
         $definitions = array();
-        foreach($paths as $path) {
+        foreach ($paths as $path) {
             if (is_file($path)) {
                 $definitions[basename($path)] = $returnFilename ? $path : new MigrationDefinition(
                     basename($path),

Braces +1 added lines, -2 removed lines

@@ -85,8 +85,7 @@
                             );
                     }
                 }
-            }
-            else {
+            } else {
                 throw new \Exception("Path '$path' is neither a file nor directory");
             }
         }