kaliop-uk / ezmigrationbundle

Core/Executor/LocationManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -227,7 +227,7 @@ 
 
     /**
      * @param int|string|array $locationKey
-     * @return mixed
+     * @return Location
      */
     public function matchLocationByKey($locationKey)
     {
@@ -325,7 +325,7 @@ 
      * The Location Manager currently supports setting references to location id.
      *
      * @throws \InvalidArgumentException When trying to set a reference to an unsupported attribute.
-     * @param \eZ\Publish\API\Repository\Values\Content\Location|LocationCollection $location
+     * @param LocationCollection|null $location
      * @return boolean
      */
     protected function setReferences($location)

Core/Matcher/TagMatcher.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
             return array(self::MATCH_TAG_ID => $key);
         }
 
-        throw new \Exception("Tag matcher can not uniquely identify the type of key used to match: ".key);
+        throw new \Exception("Tag matcher can not uniquely identify the type of key used to match: " . key);
     }
 
     /**
@@ -135,7 +135,7 @@ 
 
         foreach ($tagKeywords as $tagKeyword) {
             // return unique contents
-            foreach($this->tagService->loadTagsByKeyword($tagKeyword, $availableLanguages[0]) as $tag) {
+            foreach ($this->tagService->loadTagsByKeyword($tagKeyword, $availableLanguages[0]) as $tag) {
                 $tags[$tag->id] = $tag;
             }
         }

Core/ComplexField/ComplexFieldManager.php

Spacing +1 added lines, -2 removed lines

@@ -52,8 +52,7 @@
         if (isset($this->fieldTypeMap[$contentTypeIdentifier][$fieldTypeIdentifier]) || isset($this->fieldTypeMap['*'][$fieldTypeIdentifier]))
         {
             $fieldService = isset($this->fieldTypeMap[$contentTypeIdentifier][$fieldTypeIdentifier]) ?
-                $this->fieldTypeMap[$contentTypeIdentifier][$fieldTypeIdentifier] :
-                $this->fieldTypeMap['*'][$fieldTypeIdentifier];
+                $this->fieldTypeMap[$contentTypeIdentifier][$fieldTypeIdentifier] : $this->fieldTypeMap['*'][$fieldTypeIdentifier];
             return $fieldService->createValue($fieldValue, $context);
         }
         else

Core/StorageHandler/Database.php

Spacing +5 added lines, -5 removed lines

@@ -116,7 +116,7 @@ 
         );
         try {
             $conn->insert($this->migrationsTableName, $this->migrationToArray($migration));
-        } catch(UniqueConstraintViolationException $e) {
+        } catch (UniqueConstraintViolationException $e) {
             throw new \Exception("Migration '{$migrationDefinition->name}' already exists");
         }
 
@@ -273,7 +273,7 @@ 
                 $migrationDefinition->name,
                 md5($migrationDefinition->rawDefinition),
                 $migrationDefinition->path,
-                ($status == Migration::STATUS_SKIPPED ? null: time()),
+                ($status == Migration::STATUS_SKIPPED ? null : time()),
                 $status
             );
             $conn->update(
@@ -297,7 +297,7 @@ 
                 $migrationDefinition->name,
                 md5($migrationDefinition->rawDefinition),
                 $migrationDefinition->path,
-                ($status == Migration::STATUS_SKIPPED ? null: time()),
+                ($status == Migration::STATUS_SKIPPED ? null : time()),
                 $status
             );
             $conn->insert($this->migrationsTableName, $this->migrationToArray($migration));
@@ -362,7 +362,7 @@ 
         // and 767 bytes can be either 255 chars or 191 chars depending on charset utf8 or utf8mb4...
         //$t->addIndex(array('path'));
 
-        foreach($schema->toSql($dbPlatform) as $sql) {
+        foreach ($schema->toSql($dbPlatform) as $sql) {
             $this->dbHandler->exec($sql);
         }
     }
@@ -377,7 +377,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;
             }

Command/MigrateCommand.php

Spacing +9 added lines, -9 removed lines

@@ -113,15 +113,15 @@ 
             // 'optional' options
             // note: options 'clear-cache', 'ignore-failures' and 'no-transactions' we never propagate
             if ($input->getOption('default-language')) {
-                $builderArgs[]='--default-language='.$input->getOption('default-language');
+                $builderArgs[] = '--default-language=' . $input->getOption('default-language');
             }
             if ($input->getOption('no-transactions')) {
-                $builderArgs[]='--no-transactions';
+                $builderArgs[] = '--no-transactions';
             }
         }
 
         /** @var MigrationDefinition $migrationDefinition */
-        foreach($toExecute as $name => $migrationDefinition) {
+        foreach ($toExecute as $name => $migrationDefinition) {
 
             // let's skip migrations that we know are invalid - user was warned and he decided to proceed anyway
             if ($migrationDefinition->status == MigrationDefinition::STATUS_INVALID) {
@@ -143,7 +143,7 @@ 
                 $process->setTimeout(86400);
                 // and give immediate feedback to the user
                 $process->run(
-                    function ($type, $buffer) {
+                    function($type, $buffer) {
                         echo $buffer;
                     }
                 );
@@ -175,7 +175,7 @@ 
                         throw new \Exception($errorMsg);
                     }
 
-                } catch(\Exception $e) {
+                } catch (\Exception $e) {
                     if ($input->getOption('ignore-failures')) {
                         $output->writeln("\n<error>Migration failed! Reason: " . $e->getMessage() . "</error>\n");
                         continue;
@@ -192,7 +192,7 @@ 
                         !$input->getOption('no-transactions'),
                         $input->getOption('default-language')
                     );
-                } catch(\Exception $e) {
+                } catch (\Exception $e) {
                     if ($input->getOption('ignore-failures')) {
                         $output->writeln("\n<error>Migration failed! Reason: " . $e->getMessage() . "</error>\n");
                         continue;
@@ -227,7 +227,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] = $migrationService->parseMigrationDefinition($migrationDefinition);
             }
@@ -261,11 +261,11 @@ 
      *
      * @todo use a more compact output when there are *many* migrations
      */
-    protected function printMigrationsList($toExecute , InputInterface $input, OutputInterface $output)
+    protected function printMigrationsList($toExecute, InputInterface $input, OutputInterface $output)
     {
         $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>';