Completed
Pull Request — develop (#57)
by Tony
07:09
created
app/Http/Controllers/SettingsController.php 2 patches
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -1,19 +1,19 @@
 block discarded – undo
1 1
 <?php
2 2
 /**
3
- * Copyright (C) 2016 Tony Murray <[email protected]>
4
- * This program is free software: you can redistribute it and/or modify
5
- * it under the terms of the GNU General Public License as published by
6
- * the Free Software Foundation, either version 3 of the License, or
7
- * (at your option) any later version.
8
- *
9
- * This program is distributed in the hope that it will be useful,
10
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
11
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
12
- * GNU General Public License for more details.
13
- *
14
- * You should have received a copy of the GNU General Public License
15
- * along with this program.  If not, see <http://www.gnu.org/licenses/>.
16
- */
3
+     * Copyright (C) 2016 Tony Murray <[email protected]>
4
+     * This program is free software: you can redistribute it and/or modify
5
+     * it under the terms of the GNU General Public License as published by
6
+     * the Free Software Foundation, either version 3 of the License, or
7
+     * (at your option) any later version.
8
+     *
9
+     * This program is distributed in the hope that it will be useful,
10
+     * but WITHOUT ANY WARRANTY; without even the implied warranty of
11
+     * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
12
+     * GNU General Public License for more details.
13
+     *
14
+     * You should have received a copy of the GNU General Public License
15
+     * along with this program.  If not, see <http://www.gnu.org/licenses/>.
16
+     */
17 17
 
18 18
 namespace App\Http\Controllers;
19 19
 
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -17,8 +17,6 @@
 block discarded – undo
17 17
 
18 18
 namespace App\Http\Controllers;
19 19
 
20
-use App\Http\Requests;
21
-use App\Models\DbConfig;
22 20
 use Illuminate\Http\Request;
23 21
 use Settings;
24 22
 
Please login to merge, or discard this patch.
app/Models/DbConfig.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
     public function scopeKey($query, $key)
51 51
     {
52
-        return $query->where('config_name', 'LIKE', $key . '%');
52
+        return $query->where('config_name', 'LIKE', $key.'%');
53 53
     }
54 54
 
55 55
     public function scopeExactKey($query, $key)
Please login to merge, or discard this patch.
app/Settings.php 4 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 use App\Models\DbConfig;
29 29
 use Cache;
30 30
 use Config;
31
-use DB;
32 31
 use Illuminate\Contracts\Config\Repository as ConfigContract;
33 32
 
34 33
 // adds the possibility to replace the default Config facade
Please login to merge, or discard this patch.
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -15,13 +15,13 @@
 block discarded – undo
15 15
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
16 16
  */
17 17
 /**
18
- * Settings.php
19
- *
20
- * @package    LibreNMS
21
- * @author     Tony Murray <[email protected]>
22
- * @copyright  2016 Tony Murray
23
- * @license    @license http://opensource.org/licenses/GPL-3.0 GNU Public License v3 or later
24
- */
18
+     * Settings.php
19
+     *
20
+     * @package    LibreNMS
21
+     * @author     Tony Murray <[email protected]>
22
+     * @copyright  2016 Tony Murray
23
+     * @license    @license http://opensource.org/licenses/GPL-3.0 GNU Public License v3 or later
24
+     */
25 25
 namespace App\Settings;
26 26
 
27 27
 
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
     public function get($key, $default = null)
63 63
     {
64 64
         // return value from cache or fetch it and return it
65
-        return Cache::remember($key, $this->cache_time, function () use ($key, $default) {
65
+        return Cache::remember($key, $this->cache_time, function() use ($key, $default) {
66 66
             $db_data = DbConfig::key($key)->get(['config_name', 'config_value']);
67 67
 
68 68
             if (count($db_data) == 1 && $db_data->first()->config_name == $key) {
@@ -71,14 +71,14 @@  discard block
 block discarded – undo
71 71
             }
72 72
             elseif (count($db_data) >= 1) {
73 73
                 $result = self::collectionToArray($db_data, $key);
74
-                $config = Config::get('config.' . $key, $default);
74
+                $config = Config::get('config.'.$key, $default);
75 75
                 if (!is_null($config)) {
76 76
                     $result = array_replace_recursive($config, $result);
77 77
                 }
78 78
                 return $result;
79 79
             }
80 80
             else {
81
-                return Config::get('config.' . $key, $default);
81
+                return Config::get('config.'.$key, $default);
82 82
             }
83 83
 
84 84
         });
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
     private static function recursive_keys(array $array, $prefix = "", array $path = array())
160 160
     {
161 161
         if ($prefix != "") {
162
-            $prefix = trim($prefix, '.') . '.';
162
+            $prefix = trim($prefix, '.').'.';
163 163
         }
164 164
         $result = array();
165 165
         foreach ($array as $key => $val) {
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
                 $result = array_merge($result, self::recursive_keys($val, $prefix, $currentPath));
169 169
             }
170 170
             else {
171
-                $result[$prefix . join('.', $currentPath)] = $val;
171
+                $result[$prefix.join('.', $currentPath)] = $val;
172 172
             }
173 173
         }
174 174
         return $result;
Please login to merge, or discard this patch.
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -50,8 +50,7 @@  discard block
 block discarded – undo
50 50
                 DbConfig::updateOrCreate(['config_name' => $k], ['config_value' => $v]);
51 51
                 Cache::put($k, $v, $this->cache_time);
52 52
             }
53
-        }
54
-        else {
53
+        } else {
55 54
             DbConfig::updateOrCreate(['config_name' => $key], ['config_value' => $value]);
56 55
             Cache::put($key, $value, $this->cache_time);
57 56
         }
@@ -68,16 +67,14 @@  discard block
 block discarded – undo
68 67
             if (count($db_data) == 1 && $db_data->first()->config_name == $key) {
69 68
                 // return a bare value if we are getting one item
70 69
                 return $db_data->first()->config_value;
71
-            }
72
-            elseif (count($db_data) >= 1) {
70
+            } elseif (count($db_data) >= 1) {
73 71
                 $result = self::collectionToArray($db_data, $key);
74 72
                 $config = Config::get('config.' . $key, $default);
75 73
                 if (!is_null($config)) {
76 74
                     $result = array_replace_recursive($config, $result);
77 75
                 }
78 76
                 return $result;
79
-            }
80
-            else {
77
+            } else {
81 78
                 return Config::get('config.' . $key, $default);
82 79
             }
83 80
 
@@ -166,8 +163,7 @@  discard block
 block discarded – undo
166 163
             $currentPath = array_merge($path, array($key));
167 164
             if (is_array($val)) {
168 165
                 $result = array_merge($result, self::recursive_keys($val, $prefix, $currentPath));
169
-            }
170
-            else {
166
+            } else {
171 167
                 $result[$prefix . join('.', $currentPath)] = $val;
172 168
             }
173 169
         }
Please login to merge, or discard this patch.