kaliop-uk / ezmigrationbundle

MigrationVersions/20100101000200_MigrateV1ToV2.php

Spacing +4 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
         $this->dbHandler = $this->container->get('ezpublish.connection');
 
         $this->activeBundles = array();
-        foreach($this->container->get('kernel')->getBundles() as $bundle)
+        foreach ($this->container->get('kernel')->getBundles() as $bundle)
         {
             $this->activeBundles[$bundle->getName()] = $bundle->getPath();
         }
@@ -57,7 +57,7 @@ 
 
         // we need to decide of a random time to stamp existing migrations. We use 'now - 1 minute'...
         $executionDate = time() - 60;
-        foreach($toMigrate as $legacyMigration) {
+        foreach ($toMigrate as $legacyMigration) {
             $name = $legacyMigration['version'];
 
             $path = $this->getLegacyMigrationDefinition($legacyMigration['version'], $legacyMigration['bundle']);
@@ -104,7 +104,7 @@ 
     {
         /** @var \Doctrine\DBAL\Schema\AbstractSchemaManager $sm */
         $sm = $this->dbHandler->getConnection()->getSchemaManager();
-        foreach($sm->listTables() as $table) {
+        foreach ($sm->listTables() as $table) {
             if ($table->getName() == $tableName) {
                 return true;
             }
@@ -151,7 +151,7 @@ 
             return false;
         }
 
-        foreach($versionDefinitions as $key => $versionDefinition) {
+        foreach ($versionDefinitions as $key => $versionDefinition) {
             if (!in_array(pathinfo($versionDefinition, PATHINFO_EXTENSION), array('php', 'yml', 'sql'))) {
                 unset($versionDefinitions[$key]);
             }

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/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),

Core/MigrationService.php

Spacing +7 added lines, -7 removed lines

@@ -43,7 +43,7 @@ 
 
     public function addExecutor(ExecutorInterface $executor)
     {
-        foreach($executor->supportedTypes() as $type) {
+        foreach ($executor->supportedTypes() as $type) {
             $this->executors[$type] = $executor;
         }
     }
@@ -58,8 +58,8 @@ 
     {
         // we try to be flexible in file types we support, and the same time avoid loading all files in a directory
         $handledDefinitions = array();
-        foreach($this->loader->listAvailableDefinitions($paths) as $migrationName => $definitionPath) {
-            foreach($this->DefinitionParsers as $definitionParser) {
+        foreach ($this->loader->listAvailableDefinitions($paths) as $migrationName => $definitionPath) {
+            foreach ($this->DefinitionParsers as $definitionParser) {
                 if ($definitionParser->supports($migrationName)) {
                     $handledDefinitions[] = $definitionPath;
                 }
@@ -106,13 +106,13 @@ 
      */
     public function parseMigrationDefinition(MigrationDefinition $migrationDefinition)
     {
-        foreach($this->DefinitionParsers as $definitionParser) {
+        foreach ($this->DefinitionParsers as $definitionParser) {
             if ($definitionParser->supports($migrationDefinition->name)) {
                 // parse the source file
                 $migrationDefinition = $definitionParser->parseMigrationDefinition($migrationDefinition);
 
                 // and make sure we know how to handle all steps
-                foreach($migrationDefinition->steps as $step) {
+                foreach ($migrationDefinition->steps as $step) {
                     if (!isset($this->executors[$step->type])) {
                         return new MigrationDefinition(
                             $migrationDefinition->name,
@@ -154,7 +154,7 @@ 
         $this->repository->beginTransaction();
         try {
 
-            foreach($migrationDefinition->steps as $step) {
+            foreach ($migrationDefinition->steps as $step) {
                 // we validated the fact that we have a good executor at parsing time
                 $executor = $this->executors[$step->type];
                 $executor->execute($step);
@@ -173,7 +173,7 @@ 
 
             $this->repository->commit();
 
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             $this->repository->rollBack();
 
             /// set migration as failed

Core/ComplexField/EzBinaryFile.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      */
     public function createValue(array $fieldValueArray, array $context = array())
     {
-        $filePath = dirname($context['path']) .  '/files/' . $fieldValueArray['path'];
+        $filePath = dirname($context['path']) . '/files/' . $fieldValueArray['path'];
 
         return new BinaryFileValue(
             array(

Core/ComplexField/EzAuthor.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     public function createValue(array $fieldValueArray, array $context = array())
     {
         $authors = array();
-        foreach($fieldValueArray['authors'] as $author) {
+        foreach ($fieldValueArray['authors'] as $author) {
             $authors[] = new Author($author);
         }