baleen / cli

src/CommandBus/Repository/CreateHandler.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 use Baleen\Cli\Exception\CliException;
 use Baleen\Migrations\Migration\SimpleMigration;
 use League\Flysystem\Filesystem;
-use phpDocumentor\Reflection\DocBlock\Tag;
 use Zend\Code\Generator\ClassGenerator;
 use Zend\Code\Generator\DocBlock\Tag\GenericTag;
 use Zend\Code\Generator\DocBlockGenerator;

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
         }
 
         $timestamp = date('YmdHis');
-        $className = ['v'.$timestamp];
+        $className = ['v' . $timestamp];
         $title = $input->getArgument('title');
         if (!empty($title)) {
             $title = preg_replace('/[^A-Za-z\d_]+/', '', $title);
@@ -161,12 +161,12 @@ 
     {
         $className = $class->getName();
         $file = new FileGenerator([
-            'fileName' => $className.'.php',
+            'fileName' => $className . '.php',
             'classes' => [$class],
         ]);
         $contents = $file->generate();
 
-        $relativePath = $destinationDir.DIRECTORY_SEPARATOR.$file->getFilename();
+        $relativePath = $destinationDir . DIRECTORY_SEPARATOR . $file->getFilename();
         if ($filesystem->has($relativePath)) {
             throw new CliException(sprintf(
                 'Could not generate migration. File already exists: %s',

test/CommandBus/Config/StatusHandlerTest.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 use Baleen\Migrations\Migration\MigrationInterface;
 use Baleen\Migrations\Repository\RepositoryInterface;
 use Baleen\Migrations\Storage\StorageInterface;
-use Baleen\Migrations\Version as V;
 use Baleen\Migrations\Version;
 use Baleen\Migrations\Version\Collection\LinkedVersions;
 use Baleen\Migrations\Version\Collection\MigratedVersions;

Spacing +14 added lines, -15 removed lines

@@ -197,8 +197,7 @@ 
         $this->command->setRepository($this->repository);
         $this->command->setStorage($this->storage);
         $currentMsg = $migrated->last() === false ?
-            '/[Nn]othing has been migrated/' :
-            '/[Cc]urrent version.*?' . $migrated->last()->getId() . '.*?$/';
+            '/[Nn]othing has been migrated/' : '/[Cc]urrent version.*?' . $migrated->last()->getId() . '.*?$/';
         $this->output->shouldReceive('writeln')->once()->with($currentMsg);
         if ($pendingCount > 0) {
             $this->output->shouldReceive('writeln')->with(m::on(function($messages) use ($pendingCount) {
@@ -238,7 +237,7 @@ 
         // the number of pending migrations with the foreach loop below (see comment below).
         $repVersions = [
             [],
-            Version::fromArray(range(1,10)),
+            Version::fromArray(range(1, 10)),
         ];
         $repositories = [];
         foreach ($repVersions as $versions) {
@@ -247,7 +246,7 @@ 
         }
         $storageVersions = [
             [],
-            Version::fromArray(range(1,3)),
+            Version::fromArray(range(1, 3)),
         ];
         $storages = [];
         foreach ($storageVersions as $versions) {
@@ -258,7 +257,7 @@ 
         $combinations = $this->combinations([$repositories, $storages]);
 
         // calculate pending number of migrations and set that as the third parameter. See note in function header.
-        foreach($combinations as &$combination) { // NB: addressing by reference!
+        foreach ($combinations as &$combination) { // NB: addressing by reference!
             $pending = $combination[0]->count() - $combination[1]->count();
             $combination[] = $pending;
         }
@@ -319,16 +318,16 @@ 
         $file3 = '/var/log/http/website/error.log';
         $dir1 = '/var';
         return [
-            ['', '', ''],                           // empty
-            ['/', '/', ''],                         // from root to root
-            ['/var', '/var', ''],                   // from directory to directory
-            ['/var/log.php', '/var/log.php', ''],   // from file to file
-            ['/', '/var/log.php', 'var/log.php'],   // from root to file
-            ['', '/var/log.php', 'var/log.php'],    // without $from (same as from root)
-            ['/var', '', '..'],                     // towards root
-            ['/var', '', '..'],                     // without $to (same as towards root)
-            [$file1, $file2, '../../..' . $file2],  // backwards traversal
-            [$file1, $file3, 'website/error.log'],  // inward traversal
+            ['', '', ''], // empty
+            ['/', '/', ''], // from root to root
+            ['/var', '/var', ''], // from directory to directory
+            ['/var/log.php', '/var/log.php', ''], // from file to file
+            ['/', '/var/log.php', 'var/log.php'], // from root to file
+            ['', '/var/log.php', 'var/log.php'], // without $from (same as from root)
+            ['/var', '', '..'], // towards root
+            ['/var', '', '..'], // without $to (same as towards root)
+            [$file1, $file2, '../../..' . $file2], // backwards traversal
+            [$file1, $file3, 'website/error.log'], // inward traversal
             [$dir1, $file1, 'log/http/access.log'], // inward traversal from directory
         ];
     }

test/Provider/HelperSetProviderTest.php

Unused Use Statements -1 removed lines

@@ -19,7 +19,6 @@
 
 namespace BaleenTest\Cli\Provider;
 
-use Baleen\Cli\Helper\ConfigHelper;
 use Baleen\Cli\Provider\Services;
 use Mockery as m;
 use Symfony\Component\Console\Helper\HelperSet;

src/CommandBus/Config/InitHandler.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
             $msg = sprintf('Config file created at "<info>%s</info>".', $message->getConfig()->getFileName());
         } else {
             $msg = sprintf(
-                '<error>Error: Could not create and write file "<info>%s</info>". '.
+                '<error>Error: Could not create and write file "<info>%s</info>". ' .
                 'Please check file and directory permissions.</error>',
                 $message->getConfig()->getFileName()
             );

src/CommandBus/Config/StatusHandler.php

Spacing +1 added lines, -2 removed lines

@@ -60,8 +60,7 @@
         $headVersion = $migratedVersions->last();
 
         $currentMessage = $headVersion ?
-            "Current version: <comment>{$headVersion->getId()}</comment>" :
-            'Nothing has been migrated yet.';
+            "Current version: <comment>{$headVersion->getId()}</comment>" : 'Nothing has been migrated yet.';
         $output->writeln($currentMessage);
 
         $diff = array_diff($availableMigrations->toArray(), $migratedVersions->toArray());

src/CommandBus/Repository/ListHandler.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
     protected function outputVersions(LinkedVersions $versions, OutputInterface $output)
     {
         foreach ($versions as $version) {
-            $output->writeln('<comment>('.$version->getId().')</comment> '.get_class($version->getMigration()));
+            $output->writeln('<comment>(' . $version->getId() . ')</comment> ' . get_class($version->getMigration()));
         }
     }
 }

src/CommandBus/Timeline/ExecuteHandler.php

Spacing +2 added lines, -3 removed lines

@@ -38,8 +38,7 @@ 
         $version = (string) $input->getArgument(ExecuteMessage::ARG_VERSION);
 
         $direction = $input->getArgument(ExecuteMessage::ARG_DIRECTION) == Options::DIRECTION_DOWN ?
-            Options::DIRECTION_DOWN :
-            Options::DIRECTION_UP;
+            Options::DIRECTION_DOWN : Options::DIRECTION_UP;
         $dryRun = (bool) $input->getOption(ExecuteMessage::OPT_DRY_RUN);
         $forced = true; // because we're executing a single migration
 
@@ -47,7 +46,7 @@ 
 
         $canExecute = true;
         if ($input->isInteractive()) {
-            $output->writeln('<error>WARNING!</error> You are about to manually execute a database migration that '.
+            $output->writeln('<error>WARNING!</error> You are about to manually execute a database migration that ' .
                 'could result in schema changes and data lost.');
             $question = sprintf('Are you sure you wish to migrate "%s" (y/n)? ', $direction);
             $canExecute = $command->getCliCommand()

src/Config/Config.php

Spacing +3 added lines, -3 removed lines

@@ -151,7 +151,7 @@ 
      */
     public function getMigrationsDirectoryPath()
     {
-        return getcwd().DIRECTORY_SEPARATOR.$this->getMigrationsDirectory();
+        return getcwd() . DIRECTORY_SEPARATOR . $this->getMigrationsDirectory();
     }
 
     /**
@@ -175,7 +175,7 @@ 
      */
     public function getStorageFilePath()
     {
-        return getcwd().DIRECTORY_SEPARATOR.$this->getStorageFile();
+        return getcwd() . DIRECTORY_SEPARATOR . $this->getStorageFile();
     }
 
     /**
@@ -191,7 +191,7 @@ 
      */
     public function getConfigFilePath()
     {
-        return getcwd().DIRECTORY_SEPARATOR.$this->getFileName();
+        return getcwd() . DIRECTORY_SEPARATOR . $this->getFileName();
     }
 
     /**

src/Provider/CommandsProvider.php

Spacing +3 added lines, -3 removed lines

@@ -114,7 +114,7 @@ 
 
         // add all message classes to the container
         foreach ($commands as $alias => $config) {
-            $container->add($alias, function (Container $container, $config) {
+            $container->add($alias, function(Container $container, $config) {
                 /** @var MessageFactoryInterface $factory */
                 $factory = !empty($config['factory']) ? $config['factory'] : DefaultFactory::class;
                 $factory = $container->get($factory);
@@ -130,7 +130,7 @@ 
         }
 
         // setup the command bus to know which handler to use for each message class
-        $container->singleton(Services::COMMAND_BUS, function () use ($commands) {
+        $container->singleton(Services::COMMAND_BUS, function() use ($commands) {
             $map = [];
             foreach ($commands as $alias => $config) {
                 $message = $config['message'];
@@ -142,7 +142,7 @@ 
         });
 
         // create a service (that's just an array) that has a list of all the commands for the app
-        $container->add(Services::COMMANDS, function (ContainerInterface $container) use ($commands) {
+        $container->add(Services::COMMANDS, function(ContainerInterface $container) use ($commands) {
             $commandList = [];
             foreach ($commands as $alias => $config) {
                 $commandList[] = new BaseCommand($container, $alias, $config['message']);