librenms / librenmsv2

app/Settings.php

Spacing +8 added lines, -8 removed lines

@@ -69,11 +69,11 @@ 
                 if (!empty($key)) {
                     if (is_string($k) && !str_contains($k, '.') && DbConfig::exactKey($key)->exists() && DbConfig::key($key)->count() == 1) {
                         // check that we aren't trying to set an array onto an existing value only setting
-                        throw new \Exception("Attempting to set array value to existing non-array value at the key '" . $key . "'");
+                        throw new \Exception("Attempting to set array value to existing non-array value at the key '".$key."'");
                     }
                     else {
                         // we are not at the leaf yet, add this chunk to the key and recurse
-                        $this->set($key . '.' . $k, $v);
+                        $this->set($key.'.'.$k, $v);
                     }
                 }
                 else {
@@ -85,7 +85,7 @@ 
         else {
             // make sure we can save this
             if ($this->isReadOnly($key)) {
-                throw new \Exception("The setting '" . $key . "' is read only");
+                throw new \Exception("The setting '".$key."' is read only");
             }
 
             // flush the cache and save the value in db and cache
@@ -106,10 +106,10 @@ 
     public function get($key, $default = null)
     {
         // return value from cache or fetch it and return it
-        return Cache::tags(self::$cache_tag)->remember($key, $this->cache_time, function () use ($key, $default) {
+        return Cache::tags(self::$cache_tag)->remember($key, $this->cache_time, function() use ($key, $default) {
             // fetch the value from config.php first
-            if (Config::has('config.' . $key)) {
-                $config_data = Config::get('config.' . $key, $default);
+            if (Config::has('config.'.$key)) {
+                $config_data = Config::get('config.'.$key, $default);
                 if (!is_array($config_data)) {
                     // return the value from config.php if it is a value
                     return $config_data;
@@ -175,7 +175,7 @@ 
      */
     public function has($key)
     {
-        return (Cache::tags(self::$cache_tag)->has($key) || Config::has('config.' . $key) || DbConfig::key($key)->exists());
+        return (Cache::tags(self::$cache_tag)->has($key) || Config::has('config.'.$key) || DbConfig::key($key)->exists());
     }
 
     /**
@@ -186,7 +186,7 @@ 
      */
     public function isReadOnly($key)
     {
-        return Config::has('config.' . $key);
+        return Config::has('config.'.$key);
     }
 
     /**

Doc Comments +1 added lines, -1 removed lines

@@ -194,7 +194,7 @@
      * Forget a key and all children
      * This cannot forget variables set in config.php
      *
-     * @param $key string Explicit key to forget
+     * @param string $key string Explicit key to forget
      */
     public function forget($key)
     {

app/Http/Middleware/Authenticate.php

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
         if (Auth::guard($guard)->guest()) {
             if ($request->ajax() || $request->wantsJson()) {
                 return response('Unauthorized.', 401);
-            } else {
+            }
+            else {
                 return redirect()->guest('login');
             }
         }

app/Console/Commands/DeleteUser.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
     public function handle()
     {
         $user = $this->argument('user');
-        $user_list = User::select('username')->where('username', 'like', '%' . $user . '%')->orWhere('realname', 'like', '%' . $user . '%')->get();
+        $user_list = User::select('username')->where('username', 'like', '%'.$user.'%')->orWhere('realname', 'like', '%'.$user.'%')->get();
         $names = [];
 
         if (count($user_list) < 1) {
@@ -78,7 +78,7 @@ 
         else {
             $name = $names[0];
         }
-        if ($this->confirm('Do you wish to remove ' . $name . '?')) {
+        if ($this->confirm('Do you wish to remove '.$name.'?')) {
             User::where('username', $name)->delete();
             $this->info('User deleted.');
         }

app/Http/Middleware/SetViewVariable.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,9 +2,9 @@
 namespace App\Http\Middleware;
 
 use Closure;
-use Illuminate\Contracts\Auth\Guard;
 use Dingo\Api\Http;
 use Dingo\Api\Routing\Helpers;
+use Illuminate\Contracts\Auth\Guard;
 use Illuminate\Http\Request;
 
 class SetViewVariable