kaliop-uk / ezmigrationbundle

Core/ReferenceResolver/ContentResolver.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function getReferenceValue($stringIdentifier)
     {
         $ref = $this->getReferenceIdentifierByPrefix($stringIdentifier);
-        switch($ref['prefix']) {
+        switch ($ref['prefix']) {
             case 'content_remote_id:':
                 return $this->repository->getContentService()->loadContentByRemoteId($ref['identifier'])->id;
             //case 'content_id::':

Core/ReferenceResolver/AbstractResolver.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     public function __construct()
     {
         $quotedPrefixes = [];
-        foreach($this->referencePrefixes as $prefix) {
+        foreach ($this->referencePrefixes as $prefix) {
             $quotedPrefixes[] = preg_quote($prefix, '/');
         }
         $this->prefixMatchRegexp = '/^(' . implode('|', $quotedPrefixes) . ')/';
@@ -55,7 +55,7 @@ 
      */
     protected function getReferenceIdentifierByPrefix($stringIdentifier)
     {
-        foreach($this->referencePrefixes as $prefix) {
+        foreach ($this->referencePrefixes as $prefix) {
             $regexp = '/^' . preg_quote($prefix, '/') . '/';
             if (preg_match($regexp, $stringIdentifier)) {
                 return array('prefix' => $prefix, 'identifier' => preg_replace($regexp, '', $stringIdentifier));

Core/StorageHandler/Database.php

Spacing +2 added lines, -2 removed lines

@@ -282,7 +282,7 @@ 
         $t->addColumn('execution_error', 'string', array('length' => 4000, 'notnull' => false));
         $t->setPrimaryKey(array('migration'));
 
-        foreach($schema->toSql($dbPlatform) as $sql) {
+        foreach ($schema->toSql($dbPlatform) as $sql) {
             $this->dbHandler->exec($sql);
         }
     }
@@ -297,7 +297,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;
             }

Core/Helper/RoleHandler.php

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
                 }
                 break;
             case 'SiteAccess':
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     $name = "No Site Access Found";
                     if (array_key_exists($value, $this->siteAccessList)) {
                         $name = $this->siteAccessList[$value];
@@ -71,7 +71,7 @@ 
                 $availableLanguages = $this->translationHelper->getAvailableLanguages();
                 $primaryLanguage = $availableLanguages[0];
 
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     $contentType = $contentTypeService->loadContentType($value);
                     $retValues[] = $contentType->identifier;
                 }
@@ -123,7 +123,7 @@ 
                 }
                 break;
             case 'SiteAccess':
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     foreach ($this->siteAccessList as $key => $siteAccess) {
                         if ($value == $siteAccess) {
                             $retValues[] = (string)$key;
@@ -136,7 +136,7 @@ 
                 /** @var \eZ\Publish\API\Repository\ContentTypeService $contentTypeService */
                 $contentTypeService = $this->repository->getContentTypeService();
 
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     $contentTypeId = $value;
                     if (!is_int($value)) {
                         $contentType = $contentTypeService->loadContentTypeByIdentifier($value);

Core/ReferenceResolver/LocationResolver.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function getReferenceValue($stringIdentifier)
     {
         $ref = $this->getReferenceIdentifierByPrefix($stringIdentifier);
-        switch($ref['prefix']) {
+        switch ($ref['prefix']) {
             case 'content_remote_id:':
                 return $this->repository->getContentService()->loadContentByRemoteId($ref['identifier'])->id;
             //case 'content_id::':

Core/Executor/RoleManager.php

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
 
         if (array_key_exists('policies', $this->dsl)) {
             $ymlPolicies = $this->dsl['policies'];
-            foreach($ymlPolicies as $key => $ymlPolicy) {
+            foreach ($ymlPolicies as $key => $ymlPolicy) {
                 $this->addPolicy($role, $roleService, $ymlPolicy);
             }
         }
@@ -82,11 +82,11 @@ 
                 // Removing all policies so we can add them back.
                 // TODO: Check and update policies instead of remove and add.
                 $policies = $role->getPolicies();
-                foreach($policies as $policy) {
+                foreach ($policies as $policy) {
                     $roleService->deletePolicy($policy);
                 }
 
-                foreach($ymlPolicies as $key => $ymlPolicy) {
+                foreach ($ymlPolicies as $key => $ymlPolicy) {
                     $this->addPolicy($role, $roleService, $ymlPolicy);
                 }
             }
@@ -175,7 +175,7 @@ 
 
         $limitationValue = is_array($limitation['values']) ? $limitation['values'] : array($limitation['values']);
 
-        foreach($limitationValue as $id => $value) {
+        foreach ($limitationValue as $id => $value) {
             if ($this->referenceResolver->isReference($value)) {
                 $value = $this->referenceResolver->getReferenceValue($value);
                 $limitationValue[$id] = $value;

Command/GenerateCommand.php

Spacing +3 added lines, -3 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);

Core/Executor/ContentManager.php

Spacing +3 added lines, -3 removed lines

@@ -177,7 +177,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)) {
@@ -221,7 +221,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
@@ -248,7 +248,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

Command/MigrateCommand.php

Spacing +7 added lines, -7 removed lines

@@ -62,7 +62,7 @@ 
     {
         $migrationsService = $this->getMigrationService();
 
-        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path')) ;
+        $migrationDefinitions = $migrationsService->getMigrationsDefinitions($input->getOption('path'));
         $migrations = $migrationsService->getMigrations();
 
         if (!count($migrationDefinitions)) {
@@ -72,7 +72,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);
             }
@@ -89,7 +89,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>';
@@ -124,7 +124,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) {
@@ -136,12 +136,12 @@ 
 
             try {
                 $migrationsService->executeMigration($migrationDefinition);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 if ($input->getOption('ignore-failures')) {
-                    $output->writeln("\n<error>Migration failed! Reason: ".$this->getFullExceptionMessage($e)."</error>\n");
+                    $output->writeln("\n<error>Migration failed! Reason: " . $this->getFullExceptionMessage($e) . "</error>\n");
                     continue;
                 }
-                $output->writeln("\n<error>Migration aborted! Reason: ".$this->getFullExceptionMessage($e)."</error>");
+                $output->writeln("\n<error>Migration aborted! Reason: " . $this->getFullExceptionMessage($e) . "</error>");
                 return 1;
             }