acacha / llum

src/Console/PackageListCommand.php

Spacing +1 added lines, -1 removed lines

@@ -48,6 +48,6 @@
      */
     private function parsePackageInfo($package)
     {
-        return 'Composer name: '.$package['name'];
+        return 'Composer name: '.$package[ 'name' ];
     }
 }

src/Console/GithubRepoCommand.php

Doc Comments -1 removed lines

@@ -111,7 +111,6 @@
     /**
      * Get credentials.
      *
-     * @param OutputInterface $input
      * @return array
      */
     public function getCredentials(OutputInterface $output)

Spacing +5 added lines, -5 removed lines

@@ -79,15 +79,15 @@ 
         $this->api->setCredentials($this->getCredentials($output));
         $name = $this->repoName($input);
         try {
-            $this->api->createRepo($name,$this->repoDescription($input));
+            $this->api->createRepo($name, $this->repoDescription($input));
         } catch (ServerException $se) {
             //TODO
             $output->writeln('<error>Server exception thrown</error>');
             die();
         } catch (ClientException $ce) {
-            $this->showError( $ce , $output );
+            $this->showError($ce, $output);
         }
-        $output->writeln('<info>Repository ' . $name . ' created</info>');
+        $output->writeln('<info>Repository '.$name.' created</info>');
     }
 
 
@@ -117,8 +117,8 @@ 
     public function getCredentials(OutputInterface $output)
     {
         $credentials = $this->parser->getCredentials();
-        if ( is_null($credentials)) {
-            $this->showErrorRunLlumInitFirst($output,"Credentials");
+        if (is_null($credentials)) {
+            $this->showErrorRunLlumInitFirst($output, "Credentials");
         };
         return $credentials;
     }

src/Filesystem/Filesystem.php

Doc Comments +2 added lines, -1 removed lines

@@ -16,7 +16,7 @@ 
      *
      * Overwrite file with provided content.
      *
-     * @param $file
+     * @param string $file
      * @param $content
      */
     public function overwrite($file, $content)
@@ -42,6 +42,7 @@ 
 
     /**
      * Get file contents
+     * @param string $file
      */
     public function get($file) {
         return file_get_contents($file);

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      */
     public function make($file, $content)
     {
-        if ( $this->exists($file))
+        if ($this->exists($file))
         {
             throw new FileAlreadyExists;
         }

src/Parser/LlumRCParser.php

Spacing +4 added lines, -4 removed lines

@@ -50,8 +50,8 @@ 
     public function getCredentials()
     {
         $rc_file = $this->parse();
-        if ( array_key_exists('username',$rc_file) && array_key_exists('username',$rc_file)) {
-            return [$rc_file['username'],$rc_file['token']];
+        if (array_key_exists('username', $rc_file) && array_key_exists('username', $rc_file)) {
+            return [ $rc_file[ 'username' ], $rc_file[ 'token' ] ];
         }
     }
 
@@ -63,8 +63,8 @@ 
     public function getGitHubUsername()
     {
         $rc_file = $this->parse();
-        if ( array_key_exists('username',$rc_file)) {
-            return $rc_file['username'];
+        if (array_key_exists('username', $rc_file)) {
+            return $rc_file[ 'username' ];
         }
     }
 

src/Github/GithubAPI.php

Spacing +8 added lines, -8 removed lines

@@ -70,7 +70,7 @@ 
      * @return string
      */
     protected function repo_json_stub() {
-        return __DIR__ . '/stubs/repo_json.stub';
+        return __DIR__.'/stubs/repo_json.stub';
     }
 
     /**
@@ -79,7 +79,7 @@ 
      * @return string
      */
     protected function authorization_url() {
-        return $this->api_url . $this->authorizations_uri;
+        return $this->api_url.$this->authorizations_uri;
     }
 
     /**
@@ -88,7 +88,7 @@ 
      * @return string
      */
     protected function create_repo_url() {
-        return $this->api_url . $this->repos_uri;
+        return $this->api_url.$this->repos_uri;
     }
 
     /**
@@ -102,7 +102,7 @@ 
     {
         $response = $this->client->request('POST', $this->authorization_url(),
             [
-                "auth" => [ $username, $password],
+                "auth" => [ $username, $password ],
                 "json" => $this->authorizationsRequestJson()
             ]
         );
@@ -114,7 +114,7 @@ 
     /**
      * @return array
      */
-    protected function authorizationsRequestJson(){
+    protected function authorizationsRequestJson() {
         return [
             'scopes' => [ 'public_repo' ],
             'note' =>  uniqid('llum_')
@@ -133,7 +133,7 @@ 
         return $this->client->request('POST', $this->create_repo_url(),
             [
                 "auth" => $this->credentials(),
-                "json" => json_decode($this->compileStub($repo_name, $repo_description),true),
+                "json" => json_decode($this->compileStub($repo_name, $repo_description), true),
             ]
         );
     }
@@ -175,7 +175,7 @@ 
             "REPO_DESCRIPTION" => $repo_description
         ];
         return $this->compile(
-            $this->filesystem->get($this->repo_json_stub()) ,
+            $this->filesystem->get($this->repo_json_stub()),
             $data);
     }
 
@@ -188,7 +188,7 @@ 
      */
     protected function compile($template, $data)
     {
-        foreach($data as $key => $value)
+        foreach ($data as $key => $value)
         {
             $template = preg_replace("/\\$$key\\$/i", $value, $template);
         }

Braces +3 added lines, -2 removed lines

@@ -156,8 +156,9 @@
      */
     protected function credentials()
     {
-        if (isset($this->credentials))
-            return $this->credentials;
+        if (isset($this->credentials)) {
+                    return $this->credentials;
+        }
         throw new CredentialsException;
     }
 

Doc Comments +2 added lines, -2 removed lines

@@ -126,7 +126,7 @@ 
      *
      * @param $repo_name
      * @param $repo_description
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     public function createRepo($repo_name, $repo_description)
     {
@@ -182,7 +182,7 @@ 
     /**
      * Compile the template using the given data.
      *
-     * @param $template
+     * @param string $template
      * @param $data
      * @return mixed
      */

src/Compiler/RCFileCompiler.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function compile($template, $data)
     {
-        foreach($data as $key => $value)
+        foreach ($data as $key => $value)
         {
             $template = preg_replace("/\\$$key\\$/i", $value, $template);
         }

src/LlumRCFile.php

Spacing +1 added lines, -1 removed lines

@@ -18,6 +18,6 @@
      */
     public function path()
     {
-        return getenv("HOME") . $this->relative_file_path;
+        return getenv("HOME").$this->relative_file_path;
     }
 }
\ No newline at end of file

src/Console/GithubCommand.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
      */
     protected function showErrorRunLlumInitFirst(OutputInterface $output, $varname)
     {
-        $output->writeln('<error>' . $varname
+        $output->writeln('<error>'.$varname
             . ' not found in config file. Run git init before executing this command!</error>');
         die();
     }
@@ -31,6 +31,6 @@ 
      * @param OutputInterface $output
      */
     protected function showMessageRunningCommand(OutputInterface $output, $commandName) {
-        $output->writeln('<info>Running command ' . $commandName . '...</info>');
+        $output->writeln('<info>Running command '.$commandName.'...</info>');
     }
 }
\ No newline at end of file

src/Console/LlumCommand.php

Spacing +6 added lines, -6 removed lines

@@ -149,7 +149,7 @@ 
         $process = new Process($composer.' require '.$package.''.$this->getDevOption(),
                        null, null, null, null);
         $this->output->writeln('<info>Running composer require '.$package.$this->getDevOption().'</info>');
-        $process->run(function ($type, $line) {
+        $process->run(function($type, $line) {
             $this->output->write($line);
         });
     }
@@ -264,7 +264,7 @@ 
     private function getPackageNameByComposerName($composerPackageName)
     {
         foreach ($this->config->all() as $key => $configItem) {
-            if ($configItem['name'] == $composerPackageName) {
+            if ($configItem[ 'name' ] == $composerPackageName) {
                 return $key;
             }
         }
@@ -301,9 +301,9 @@ 
         $this->setName($name)
              ->setDescription($description);
         if ($command->argument() != null) {
-            $this->addArgument($command->argument()['name'],
-                $command->argument()['type'],
-                $command->argument()['description']
+            $this->addArgument($command->argument()[ 'name' ],
+                $command->argument()[ 'type' ],
+                $command->argument()[ 'description' ]
             );
         }
     }
@@ -328,7 +328,7 @@ 
             return;
         }
 
-        $this->$method($input,$output);
+        $this->$method($input, $output);
     }
 
     /**

Doc Comments +1 added lines, -1 removed lines

@@ -217,7 +217,7 @@
     /**
      * Obtain package.
      *
-     * @param $name
+     * @param string $name
      *
      * @return array|int
      */