realshadow / satis-control-panel

app/Satis/BuildCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
 
     /**
      * @param CommandContextInterface $commandContext
-     * @return mixed
+     * @return integer|null
      */
     protected function exec(CommandContextInterface $commandContext) {
         $commandChunks = $this->compile($commandContext);

Spacing +9 added lines, -9 removed lines

@@ -67,16 +67,16 @@ 
             $this->buildDirectory
         ]);
 
-        if($this->item !== null) {
+        if ($this->item !== null) {
             $chunks->push($this->item);
         }
 
         $chunks->push($commandContext->getOutputRedirection($this->logFile));
         $chunks->push($commandContext->getShouldUnlockOnCompletion());
 
-        foreach(['http', 'https'] as $protocol) {
+        foreach (['http', 'https'] as $protocol) {
             $proxy = $this->proxySettings->get($protocol);
-            if($proxy !== null) {
+            if ($proxy !== null) {
                 $chunks->prepend(strtoupper($protocol) . '_PROXY=' . $proxy);
             }
         }
@@ -119,7 +119,7 @@ 
 
         chdir($this->currentDirectory);
 
-        if($commandContext instanceof SyncCommand) {
+        if ($commandContext instanceof SyncCommand) {
             $logger->info('Command output => ' . implode(PHP_EOL, $output));
         } else {
             $logger->notice('Command output can be found in "' . $this->logFile . '".');
@@ -168,11 +168,11 @@ 
      * @param string $item
      */
     public function setItem($item) {
-        if($item === null) {
+        if ($item === null) {
             return;
         }
 
-        if(filter_var($item, FILTER_VALIDATE_URL)) {
+        if (filter_var($item, FILTER_VALIDATE_URL)) {
             $this->item = '--repository-url ' . escapeshellarg($item);
         } else {
             $this->item = escapeshellarg($item);
@@ -208,8 +208,8 @@ 
     public function run($asyncMode = true) {
         # -- force sync
         #if(true) {
-        if($asyncMode === false || $this->isWindows() === true) {
-            if($this->isWindows() === true) {
+        if ($asyncMode === false || $this->isWindows() === true) {
+            if ($this->isWindows() === true) {
                 $this->getLogger()
                     ->warn('OS does not support async mode, forcing sync.');
             }
@@ -231,7 +231,7 @@ 
 
         $result = $this->exec(new SyncCommand());
 
-        if($result !== 0) {
+        if ($result !== 0) {
             throw new PackageBuildFailedException('Package build failed. Check build log for details.');
         }
 

app/Satis/ConfigManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -121,7 +121,7 @@ 
      * @param \App\Satis\Collections\PackageCollection $packageCollection
      * @param string $packageId
      * @param \Illuminate\Support\Collection $input
-     * @return \App\Satis\Collections\RepositoryCollection
+     * @return PackageCollection
      * @throws \App\Satis\Exceptions\PackageNotFoundException
      */
     protected function _addOrUpdatePackage(PackageCollection $packageCollection, $packageId,
@@ -179,7 +179,7 @@ 
     /**
      * @param \App\Satis\Collections\PackageCollection $packageCollection
      * @param string $packageId
-     * @return \App\Satis\Collections\RepositoryCollection
+     * @return PackageCollection
      * @throws \App\Satis\Exceptions\RepositoryNotFoundException
      */
     protected function _deletePackage(PackageCollection $packageCollection, $packageId) {

Indentation +2 added lines, -2 removed lines

@@ -9,14 +9,14 @@
  * @author Lukas Homza <lukashomz@gmail.com>
  */
 class PrivateRepository extends BuildContext {
-	/**
+    /**
      * @return int
      */
     public function getType() {
         return ConfigBuilder::PRIVATE_REPOSITORY;
     }
 
-	/**
+    /**
      * @return string
      */
     public function getConfigFile() {

Spacing +21 added lines, -21 removed lines

@@ -89,16 +89,16 @@ 
         $repository->setUrl($input->get('url'));
         $repository->setType($input->get('type', config('satis.default_repository_type')));
 
-        if($input->has('async_mode')) {
+        if ($input->has('async_mode')) {
             $this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
         }
 
-        if($input->has('disable_build')) {
+        if ($input->has('disable_build')) {
             $this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
         }
 
-        if($repositoryId !== null) {
-            if($repositoryCollection->has($repositoryId)) {
+        if ($repositoryId !== null) {
+            if ($repositoryCollection->has($repositoryId)) {
                 $repositoryCollection->put($repositoryId, $repository);
             } else {
                 throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
@@ -130,16 +130,16 @@ 
         $package = new Package();
         $package->setName($input->get('name'));
 
-        if($input->has('async_mode')) {
+        if ($input->has('async_mode')) {
             $this->configBuilder->setAsyncMode(filter_var($input->get('async_mode'), FILTER_VALIDATE_BOOLEAN));
         }
 
-        if($input->has('disable_build')) {
+        if ($input->has('disable_build')) {
             $this->setDisableBuild(filter_var($input->get('disable_build'), FILTER_VALIDATE_BOOLEAN));
         }
 
-        if($packageId !== null) {
-            if($packageCollection->has($packageId)) {
+        if ($packageId !== null) {
+            if ($packageCollection->has($packageId)) {
                 $packageCollection->put($packageId, $package);
             } else {
                 throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
@@ -165,7 +165,7 @@ 
      * @throws \App\Satis\Exceptions\RepositoryNotFoundException
      */
     protected function _deleteRepository(RepositoryCollection $repositoryCollection, $repositoryId) {
-        if($repositoryCollection->has($repositoryId)) {
+        if ($repositoryCollection->has($repositoryId)) {
             $repositoryCollection->forget($repositoryId);
         } else {
             throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
@@ -183,7 +183,7 @@ 
      * @throws \App\Satis\Exceptions\RepositoryNotFoundException
      */
     protected function _deletePackage(PackageCollection $packageCollection, $packageId) {
-        if($packageCollection->has($packageId)) {
+        if ($packageCollection->has($packageId)) {
             $packageCollection->forget($packageId);
         } else {
             throw new RepositoryNotFoundException('Package with ID "' . $packageId . '" does not exist.');
@@ -226,11 +226,11 @@ 
     public function __call($method, array $arguments) {
         $method = $this->hiddenEntityPrefix . $method;
 
-        if(method_exists($this, $method)) {
+        if (method_exists($this, $method)) {
             $logger = $this->getLogger();
 
             $logger->info(str_repeat('=', 30));
-            $logger->info('Request => ' . \Request::method() . ' (' . $method .') => '. json_encode($arguments));
+            $logger->info('Request => ' . \Request::method() . ' (' . $method . ') => ' . json_encode($arguments));
 
             $type = str_plural(last(explode('_', snake_case($method))), 2);
 
@@ -238,7 +238,7 @@ 
 
             try {
                 $items = call_user_func_array([$this, $method], $arguments);
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 $logger->info('Response => ' . $e->getMessage());
                 $logger->info(str_repeat('=', 30));
 
@@ -258,9 +258,9 @@ 
      * @return mixed
      */
     public function __get($property) {
-        $property = $this->hiddenEntityPrefix.$property;
+        $property = $this->hiddenEntityPrefix . $property;
 
-        if(isset($this->lazyLoadedProperties[$property])) {
+        if (isset($this->lazyLoadedProperties[$property])) {
             $this->{$property} = $this->{$this->lazyLoadedProperties[$property]}();
             
             unset($this->lazyLoadedProperties[$property]);
@@ -287,8 +287,8 @@ 
     public function getRepositories($repositoryId = null) {
         $repositoryCollection = $this->satis->getRepositories();
 
-        if($repositoryId !== null) {
-            if(!$repositoryCollection->has($repositoryId)) {
+        if ($repositoryId !== null) {
+            if (!$repositoryCollection->has($repositoryId)) {
                 throw new RepositoryNotFoundException('Repository with ID "' . $repositoryId . '" does not exist.');
             }
 
@@ -308,8 +308,8 @@ 
     public function getPackages($packageId = null) {
         $packageCollection = $this->satis->getPackages();
 
-        if($packageId !== null) {
-            if(!$packageCollection->has($packageId)) {
+        if ($packageId !== null) {
+            if (!$packageCollection->has($packageId)) {
                 throw new PackageNotFoundException('Package with ID "' . $packageId . '" does not exist.');
             }
 
@@ -325,7 +325,7 @@ 
      * @param \App\Satis\BuildContext $buildContext
      */
     public function forceBuild(BuildContext $buildContext) {
-        if($buildContext->getItemName() !== null) {
+        if ($buildContext->getItemName() !== null) {
             $buildContext->setItemId(self::nameToId($buildContext->getItemName()));
         }
 
@@ -356,7 +356,7 @@ 
 
         $this->configPersister->updateWith($satisConfig);
 
-        if($this->disableBuild === false) {
+        if ($this->disableBuild === false) {
             return $this->configBuilder->build();
         }
 

app/Console/Commands/Inspire.php

Spacing +1 added lines, -1 removed lines

@@ -28,6 +28,6 @@
      */
     public function handle()
     {
-        $this->comment(PHP_EOL.Inspiring::quote().PHP_EOL);
+        $this->comment(PHP_EOL . Inspiring::quote() . PHP_EOL);
     }
 }

app/Console/Commands/Permissions.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 
         $configFile = config('satis.config');
 
-        if($this->filesystem->exists($configFile)) {
+        if ($this->filesystem->exists($configFile)) {
             chmod($configFile, 777);
         } else {
             $this->error("\n" . 'Satis config file "' . $configFile . '" does not exist.

app/Exceptions/Handler.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     {
         if ($e instanceof ModelNotFoundException) {
             $e = new NotFoundHttpException($e->getMessage(), $e);
-        } elseif($e instanceof HttpException && \App::isLocal() === false) {
+        } elseif ($e instanceof HttpException && \App::isLocal() === false) {
             Response::json(['result' => false, 'message' => $e->getMessage()])
                 ->setStatusCode($e->getStatusCode())
                 ->send();

app/Http/Controllers/Api/RepositoryController.php

Indentation +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
             $repositories = $configManager->getRepositories($repositoryId);
 
             Response::make($repositories)
-              ->send();
+                ->send();
         } catch(RepositoryNotFoundException $e) {
             Response::json()
                 ->setStatusCode(404)
@@ -117,5 +117,5 @@ 
                 ->setStatusCode(500)
                 ->send();
         }
-	}
+    }
 }

Spacing +7 added lines, -7 removed lines

@@ -34,7 +34,7 @@ 
             ]
         );
 
-        if($validator->fails()) {
+        if ($validator->fails()) {
             Response::json($validator->errors()->first('url'))
                 ->setStatusCode(400)
                 ->send();
@@ -44,11 +44,11 @@ 
 
                 Response::json(['command_output' => $output])
                     ->send();
-            } catch(PackageBuildFailedException $e) {
+            } catch (PackageBuildFailedException $e) {
                 Response::json()
                     ->setStatusCode(500)
                     ->send();
-            } catch(RepositoryNotFoundException $e) {
+            } catch (RepositoryNotFoundException $e) {
                 Response::json()
                     ->setStatusCode(404)
                     ->send();
@@ -66,11 +66,11 @@ 
 
             Response::make($repositories)
               ->send();
-        } catch(RepositoryNotFoundException $e) {
+        } catch (RepositoryNotFoundException $e) {
             Response::json()
                 ->setStatusCode(404)
                 ->send();
-        } catch(PackageBuildFailedException $e) {
+        } catch (PackageBuildFailedException $e) {
             Response::json($e->getMessage())
                 ->setStatusCode(500)
                 ->send();
@@ -108,11 +108,11 @@ 
 
             Response::json()
                 ->send();
-        } catch(RepositoryNotFoundException $e) {
+        } catch (RepositoryNotFoundException $e) {
             Response::json()
                 ->setStatusCode(404)
                 ->send();
-        } catch(PackageBuildFailedException $e) {
+        } catch (PackageBuildFailedException $e) {
             Response::json($e->getMessage())
                 ->setStatusCode(500)
                 ->send();

app/Http/Controllers/Controller.php

Indentation +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
  * @author Lukas Homza <lukashomz@gmail.com>
  */
 class Controller extends BaseController {
-	/**
+    /**
      * @param \App\Satis\ConfigManager $configManager
      * @param \App\Satis\BuildContext $buildContext
      * @param \Illuminate\Http\Request $request
@@ -73,7 +73,7 @@ 
         ]);
     }
 
-	/**
+    /**
      * @param \App\Satis\ConfigManager $configManager
      * @param \Illuminate\Http\Request $request
      */
@@ -81,7 +81,7 @@ 
         $this->build($configManager, new PrivateRepository(), $request);
     }
 
-	/**
+    /**
      * @param \App\Satis\ConfigManager $configManager
      * @param \Illuminate\Http\Request $request
      */

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      * @param \Illuminate\Http\Request $request
      */
     protected function build(ConfigManager $configManager, BuildContext $buildContext, Request $request) {
-        if($request->ajax()) {
+        if ($request->ajax()) {
             $buildContext->setItemName($request->get('what'));
 
             $configManager->forceBuild($buildContext);
@@ -53,9 +53,9 @@ 
                 ->setConfig($configManager->getDefinition())
                 ->setRepositoryTypes(config('satis.repository_types'))
                 ->isLoaded(true);
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $message = $e->getMessage();
-            if($e instanceof FileNotFoundException) {
+            if ($e instanceof FileNotFoundException) {
                 $message = trans('satis.not_found');
             }
 

app/Providers/RouteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function map(Router $router)
     {
-        $router->group(['namespace' => $this->namespace], function ($router) {
+        $router->group(['namespace' => $this->namespace], function($router) {
             require app_path('Http/routes.php');
         });
     }

app/Satis/Collections/PackageCollection.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      * @return $this
      */
     public function put($key, $value) {
-        if(!$value instanceof Package) {
+        if (!$value instanceof Package) {
             throw new InvalidArgumentException('PackageCollection accepts only elements of "Package" type.');
         }