Completed
Pull Request — develop (#143)
by Tony
05:40
created
app/DataTables/DeviceDataTable.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -25,7 +25,6 @@
 block discarded – undo
25 25
 
26 26
 namespace App\DataTables;
27 27
 
28
-use App\Models\DeviceGroup;
29 28
 use App\Models\Device;
30 29
 
31 30
 class DeviceDataTable extends BaseDataTable
Please login to merge, or discard this patch.
app/Http/Controllers/DeviceController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace App\Http\Controllers;
4 4
 
5 5
 use App\DataTables\DeviceDataTable;
6
-use App\DataTables\DeviceGroupDataTable;
7 6
 use App\Models\DeviceGroup;
8 7
 use Dingo\Api\Http;
9 8
 use Dingo\Api\Routing\Helpers;
Please login to merge, or discard this patch.
app/Http/Controllers/DeviceGroupController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace App\Api\Controllers;
4 4
 
5 5
 use DB;
6
-use App\Http\Requests;
7 6
 use Illuminate\Http\Request;
8 7
 
9 8
 class ApiController extends Controller
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -195,7 +195,8 @@
 block discarded – undo
195 195
         $user = \Auth::user();
196 196
         if (!is_null($user) && $user->isAdmin()) {
197 197
             return false;
198
-        } else {
198
+        }
199
+        else {
199 200
             return 'auth';
200 201
         }
201 202
     }
Please login to merge, or discard this patch.
app/Models/DeviceGroup.php 1 patch
Braces   +14 added lines, -7 removed lines patch added patch discarded remove patch
@@ -144,7 +144,8 @@  discard block
 block discarded – undo
144 144
         $query = null;
145 145
         if (count($tables) == 1) {
146 146
             $query = DB::table($tables[0])->select('device_id');
147
-        } else {
147
+        }
148
+        else {
148 149
             $query = DB::table('devices')->select('devices.device_id')->distinct();
149 150
 
150 151
             foreach ($tables as $table) {
@@ -221,14 +222,16 @@  discard block
 block discarded – undo
221 222
             if (starts_with($cur, '%')) {
222 223
                 // table and column
223 224
                 $out .= substr($cur, 1).' ';
224
-            } elseif (substr($cur, -1) == '~') {
225
+            }
226
+            elseif (substr($cur, -1) == '~') {
225 227
                 // like operator
226 228
                 $content = $parts[++$i]; // grab the content so we can format it
227 229
 
228 230
                 if (str_contains($content, '@')) {
229 231
                     // contains wildcard
230 232
                     $content = str_replace('@', '%', $content);
231
-                } else {
233
+                }
234
+                else {
232 235
                     // assume substring
233 236
                     $content = '%'.$content.'%';
234 237
                 }
@@ -240,14 +243,18 @@  discard block
 block discarded – undo
240 243
 
241 244
                 $out .= "LIKE('".$content."') ";
242 245
 
243
-            } elseif ($cur == '&&') {
246
+            }
247
+            elseif ($cur == '&&') {
244 248
                 $out .= 'AND ';
245
-            } elseif ($cur == '||') {
249
+            }
250
+            elseif ($cur == '||') {
246 251
                 $out .= 'OR ';
247
-            } elseif (in_array($cur, $ops)) {
252
+            }
253
+            elseif (in_array($cur, $ops)) {
248 254
                 // passthrough operators
249 255
                 $out .= $cur.' ';
250
-            } else {
256
+            }
257
+            else {
251 258
                 // user supplied input
252 259
                 $out .= "'".trim($cur, '"\'')."' "; // TODO: remove trim, only needed with invalid input
253 260
             }
Please login to merge, or discard this patch.