edno / kleis

app/Http/Controllers/UserController.php

Unused Use Statements -4 removed lines

@@ -2,10 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use Illuminate\Http\Request;
-
-use App\Http\Requests;
-use App\User;
 use Auth;
 
 class UserController extends Controller

app/Http/Helpers/Installer/EnvironmentManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,6 +29,7 @@ 
     /**
      * Get the content of the .env file.
      *
+     * @param Request $request
      * @return string
      */
     public function getEnvContent($request)
@@ -48,7 +49,6 @@ 
     /**
      * Save the edited content to the file.
      *
-     * @param Request $input
      * @return boolean
      */
     public function saveFile(Request $request)

Spacing +4 added lines, -4 removed lines

@@ -36,9 +36,9 @@ 
         $data = $request->session()->all();
 
         $env = '';
-        foreach($data as $key => $value) {
-            if(preg_match('/^[A-Z]+_[A-Z]+$/', $key)) {
-                $env .= "${key}=${value}" . PHP_EOL;
+        foreach ($data as $key => $value) {
+            if (preg_match('/^[A-Z]+_[A-Z]+$/', $key)) {
+                $env .= "${key}=${value}".PHP_EOL;
             }
         }
 
@@ -58,7 +58,7 @@ 
 
         try {
             file_put_contents($this->envPath, $request->get('envConfig'));
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $status = false;
         }
 

app/Http/Helpers/Installer/PermissionsChecker.php

Doc Comments +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
      *
      * @param $folder
      * @param $permission
-     * @param $isSet
+     * @param boolean $isSet
      */
     private function addFileAndSetErrors($folder, $permission, $isSet)
     {

Spacing +2 added lines, -2 removed lines

@@ -29,9 +29,9 @@
      */
     public function check(array $folders)
     {
-        foreach($folders as $folder => $permission)
+        foreach ($folders as $folder => $permission)
         {
-            if(!($this->getPermission($folder) >= $permission))
+            if (!($this->getPermission($folder) >= $permission))
             {
                 $this->addFileAndSetErrors($folder, $permission, false);
             }

Braces +1 added lines, -2 removed lines

@@ -34,8 +34,7 @@
             if(!($this->getPermission($folder) >= $permission))
             {
                 $this->addFileAndSetErrors($folder, $permission, false);
-            }
-            else {
+            } else {
                 $this->addFile($folder, $permission, true);
             }
         }

app/Http/Middleware/RedirectIfNotInstalled.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Http\Middleware;
 
 use Closure;
-use Illuminate\Support\Facades\Auth;
 use App\Http\Middleware\RedirectIfInstalled;
 
 class RedirectIfNotInstalled extends RedirectIfInstalled

Doc Comments -1 removed lines

@@ -13,7 +13,6 @@
      *
      * @param  \Illuminate\Http\Request  $request
      * @param  \Closure  $next
-     * @param  string|null  $guard
      * @return mixed
      */
     public function handle($request, Closure $next)

app/Console/Commands/ExportCategories.php

Indentation +22 added lines, -22 removed lines

@@ -40,31 +40,31 @@
      *
      * @return mixed
      */
-     public function handle()
-     {
-         $categories = Category::get();
+        public function handle()
+        {
+            $categories = Category::get();
 
-         if ($categories->isEmpty()) {
-             $this->info("No categories to export");
-             return;
-         }
+            if ($categories->isEmpty()) {
+                $this->info("No categories to export");
+                return;
+            }
 
-         foreach ($categories as $category){
-             $name = static::stringNormalise($category->name);
-             $filename = 'export/categories/' . $name . '.txt';
-             $accounts = Account::where('status', 1)
+            foreach ($categories as $category){
+                $name = static::stringNormalise($category->name);
+                $filename = 'export/categories/' . $name . '.txt';
+                $accounts = Account::where('status', 1)
                             ->where('category_id', $category->id)
                             ->orderBy('netlogin', 'desc')
                             ->get();
-             Storage::put($filename, '');
-             $count = count($accounts);
-             $bar = $this->output->createProgressBar($count);
-             foreach ($accounts as $account) {
-                 Storage::prepend($filename, "{$account->netlogin}:{$account->netpass}");
-                 $bar->advance();
-             }
-             $bar->finish();
-             $this->info("\n{$count} accounts '{$category->name}' exported into file 'storage/app/{$filename}'");
-         }
-     }
+                Storage::put($filename, '');
+                $count = count($accounts);
+                $bar = $this->output->createProgressBar($count);
+                foreach ($accounts as $account) {
+                    Storage::prepend($filename, "{$account->netlogin}:{$account->netpass}");
+                    $bar->advance();
+                }
+                $bar->finish();
+                $this->info("\n{$count} accounts '{$category->name}' exported into file 'storage/app/{$filename}'");
+            }
+        }
 }

Spacing +2 added lines, -2 removed lines

@@ -49,9 +49,9 @@
              return;
          }
 
-         foreach ($categories as $category){
+         foreach ($categories as $category) {
              $name = static::stringNormalise($category->name);
-             $filename = 'export/categories/' . $name . '.txt';
+             $filename = 'export/categories/'.$name.'.txt';
              $accounts = Account::where('status', 1)
                             ->where('category_id', $category->id)
                             ->orderBy('netlogin', 'desc')

app/Console/Commands/ExportGroups.php

Indentation +22 added lines, -22 removed lines

@@ -40,31 +40,31 @@
      *
      * @return mixed
      */
-     public function handle()
-     {
-         $groups = Group::get();
+        public function handle()
+        {
+            $groups = Group::get();
 
-         if ($groups->isEmpty()) {
-             $this->info("No groups to export");
-             return;
-         }
+            if ($groups->isEmpty()) {
+                $this->info("No groups to export");
+                return;
+            }
 
-         foreach ($groups as $group){
-             $name = static::stringNormalise($group->name);
-             $filename = 'export/groups/' . $name . '.txt';
-             $accounts = Account::where('status', 1)
+            foreach ($groups as $group){
+                $name = static::stringNormalise($group->name);
+                $filename = 'export/groups/' . $name . '.txt';
+                $accounts = Account::where('status', 1)
                             ->where('group_id', $group->id)
                             ->orderBy('netlogin', 'desc')
                             ->get();
-             Storage::put($filename, '');
-             $count = count($accounts);
-             $bar = $this->output->createProgressBar($count);
-             foreach ($accounts as $account) {
-                 Storage::prepend($filename, "{$account->netlogin}:{$account->netpass}");
-                 $bar->advance();
-             }
-             $bar->finish();
-             $this->info("\n{$count} accounts '{$group->name}' exported into file 'storage/app/{$filename}'");
-         }
-     }
+                Storage::put($filename, '');
+                $count = count($accounts);
+                $bar = $this->output->createProgressBar($count);
+                foreach ($accounts as $account) {
+                    Storage::prepend($filename, "{$account->netlogin}:{$account->netpass}");
+                    $bar->advance();
+                }
+                $bar->finish();
+                $this->info("\n{$count} accounts '{$group->name}' exported into file 'storage/app/{$filename}'");
+            }
+        }
 }

Spacing +2 added lines, -2 removed lines

@@ -49,9 +49,9 @@
              return;
          }
 
-         foreach ($groups as $group){
+         foreach ($groups as $group) {
              $name = static::stringNormalise($group->name);
-             $filename = 'export/groups/' . $name . '.txt';
+             $filename = 'export/groups/'.$name.'.txt';
              $accounts = Account::where('status', 1)
                             ->where('group_id', $group->id)
                             ->orderBy('netlogin', 'desc')

app/Http/Middleware/RedirectIfInstalled.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      */
     public function handle($request, Closure $next)
     {
-        if($this->alreadyInstalled() || !config('kleis.installer')) {
+        if ($this->alreadyInstalled() || !config('kleis.installer')) {
             return redirect('/');
         }
 

app/Http/Helpers/Installer/RequirementsChecker.php

Spacing +2 added lines, -2 removed lines

@@ -15,11 +15,11 @@
     {
         $results = [];
 
-        foreach($requirements as $requirement)
+        foreach ($requirements as $requirement)
         {
             $results['requirements'][$requirement] = true;
 
-            if(!extension_loaded($requirement))
+            if (!extension_loaded($requirement))
             {
                 $results['requirements'][$requirement] = false;
 

app/Http/Helpers/Installer/DatabaseManager.php

Indentation +2 added lines, -2 removed lines

@@ -72,8 +72,8 @@
     }
 
         /**
-     * check database type. If SQLite, then create the database file.
-     */
+         * check database type. If SQLite, then create the database file.
+         */
     private function sqlite()
     {
         if(DB::connection() instanceof SQLiteConnection) {

Spacing +7 added lines, -7 removed lines

@@ -29,10 +29,10 @@ 
      */
     private function migrate()
     {
-        try{
-            Artisan::call('migrate', ["--force"=> true ]);
+        try {
+            Artisan::call('migrate', ["--force"=> true]);
         }
-        catch(Exception $e){
+        catch (Exception $e) {
             return $this->response($e->getMessage());
         }
 
@@ -46,10 +46,10 @@ 
      */
     private function seed()
     {
-        try{
+        try {
             Artisan::call('db:seed');
         }
-        catch(Exception $e){
+        catch (Exception $e) {
             return $this->response($e->getMessage());
         }
 
@@ -76,9 +76,9 @@ 
      */
     private function sqlite()
     {
-        if(DB::connection() instanceof SQLiteConnection) {
+        if (DB::connection() instanceof SQLiteConnection) {
             $database = DB::connection()->getDatabaseName();
-            if(!file_exists($database)) {
+            if (!file_exists($database)) {
                 touch($database);
                 DB::reconnect(Config::get('database.default'));
             }

Braces +2 added lines, -4 removed lines

@@ -31,8 +31,7 @@ 
     {
         try{
             Artisan::call('migrate', ["--force"=> true ]);
-        }
-        catch(Exception $e){
+        } catch(Exception $e){
             return $this->response($e->getMessage());
         }
 
@@ -48,8 +47,7 @@ 
     {
         try{
             Artisan::call('db:seed');
-        }
-        catch(Exception $e){
+        } catch(Exception $e){
             return $this->response($e->getMessage());
         }