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

Spacing +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
         }
 
         $activeBundles = array();
-        foreach($this->getApplication()->getKernel()->getBundles() as $bundle)
+        foreach ($this->getApplication()->getKernel()->getBundles() as $bundle)
         {
             $activeBundles[] = $bundle->getName();
         }
@@ -155,7 +155,7 @@ 
                     $className = 'Migration';
                 }
                 // Make sure that php class names are unique, not only migration definition file names
-                $existingMigrations = count(glob($migrationDirectory.'/*_' . $className .'*.php'));
+                $existingMigrations = count(glob($migrationDirectory . '/*_' . $className . '*.php'));
                 if ($existingMigrations) {
                     $className = $className . sprintf('%03d', $existingMigrations + 1);
                 }
@@ -201,7 +201,7 @@ 
         if ($migrationType == 'role') {
             $code = $this->generateRoleTemplate($parameters['role']);
         } else {
-            $code = $this->getContainer()->get('twig')->render($this->thisBundle . ':MigrationTemplate:'.$template, $parameters);
+            $code = $this->getContainer()->get('twig')->render($this->thisBundle . ':MigrationTemplate:' . $template, $parameters);
         }
 
         file_put_contents($path, $code);
@@ -267,7 +267,7 @@ 
         /** @var \eZ\Publish\API\Repository\SectionService $sectionService */
         $sectionService = $repository->getSectionService();
 
-        foreach($limitation->limitationValues as $value)
+        foreach ($limitation->limitationValues as $value)
         {
 
         }

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.');
         }