Test Failed
Push — dev5 ( d8a4d3...9629db )
by Ron
08:46
created
app/Providers/FormGroupServiceProvider.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -25,14 +25,14 @@
 block discarded – undo
25 25
     public function boot()
26 26
     {
27 27
         //  Service provider to customize form inputs
28
-        Form::component('bsText',     'components.form.text',     ['name', 'label', 'value' => null, 'attributes' => []]);
28
+        Form::component('bsText', 'components.form.text', ['name', 'label', 'value' => null, 'attributes' => []]);
29 29
         Form::component('bsPassword', 'components.form.password', ['name', 'label', 'value' => null, 'attributes' => []]);
30
-        Form::component('bsNumber',   'components.form.number',   ['name', 'label', 'value' => null, 'attributes' => []]);
31
-        Form::component('bsEmail',    'components.form.email',    ['name', 'label', 'value' => null, 'attributes' => []]);
32
-        Form::component('bsDate',     'components.form.date',     ['name', 'label', 'value' => null, 'attributes' => []]);
30
+        Form::component('bsNumber', 'components.form.number', ['name', 'label', 'value' => null, 'attributes' => []]);
31
+        Form::component('bsEmail', 'components.form.email', ['name', 'label', 'value' => null, 'attributes' => []]);
32
+        Form::component('bsDate', 'components.form.date', ['name', 'label', 'value' => null, 'attributes' => []]);
33 33
         Form::component('bsTextarea', 'components.form.textarea', ['name', 'label', 'value' => null, 'attributes' => []]);
34 34
         Form::component('bsCheckbox', 'components.form.checkbox', ['name', 'label', 'value' => 'on', 'checked' => false, 'attributes' => []]);
35
-        Form::component('bsSelect',   'components.form.select',   ['name', 'label', 'list', 'selected' => null, 'attributes' => []]);
35
+        Form::component('bsSelect', 'components.form.select', ['name', 'label', 'list', 'selected' => null, 'attributes' => []]);
36 36
         //  Custom Submit button
37 37
         Form::component('bsSubmit', 'components.form.submit', ['name']);
38 38
         //  List of all states in a dropdown box
Please login to merge, or discard this patch.
app/Http/Controllers/Customers/CustomerSystemsController.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
     //  Get the possible system types that can be assigned to the customer
25 25
     public function index()
26 26
     {
27
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
27
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
28 28
         $sysList = new CategoriesCollection(SystemCategories::with('SystemTypes')->with('SystemTypes.SystemDataFields.SystemDataFieldTypes')->get());
29 29
 
30 30
         return $sysList;
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
     //  Store a new system for the customer
34 34
     public function store(Request $request)
35 35
     {
36
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
36
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
37 37
 
38 38
         $request->validate([
39 39
             'cust_id' => 'required',
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     //  Get the list of systems attached to the customer
77 77
     public function show($id)
78 78
     {
79
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
79
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
80 80
 
81 81
         $sysList = CustomerSystems::where('cust_id', $id)
82 82
                     ->with('SystemTypes')
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
     // Update the customers system data
106 106
     public function update(Request $request, $id)
107 107
     {
108
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
108
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
109 109
 
110 110
         $request->validate([
111 111
             'cust_id'    => 'required',
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 
125 125
         foreach($fields as $data)
126 126
         {
127
-            $fieldName = 'field_' . $data->field_id;
127
+            $fieldName = 'field_'.$data->field_id;
128 128
             if(isset($request->$fieldName))
129 129
             {
130 130
                 Log::debug($request->$fieldName);
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
     //  Delete a system attached to a customer
143 143
     public function destroy($id)
144 144
     {
145
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
145
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
146 146
 
147 147
         // return response('deleted '.$id);
148 148
         $system = CustomerSystems::find($id);
Please login to merge, or discard this patch.
app/Policies/GatePolicy.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -37,13 +37,13 @@  discard block
 block discarded – undo
37 37
     //  Determine if a user can see the Administration Nav Link
38 38
     public function seeAdminLink(User $user)
39 39
     {
40
-        if ($this->isInstaller($user))
40
+        if($this->isInstaller($user))
41 41
         {
42 42
             return true;
43 43
         }
44 44
 
45 45
         $data = UserRolePermissions::with('UserRolePermissionTypes')
46
-            ->whereHas('UserRolePermissionTypes', function ($query) {
46
+            ->whereHas('UserRolePermissionTypes', function($query) {
47 47
                 $query->where('description', 'Manage Users')
48 48
                     ->orWhere('description', 'Manage User Roles')
49 49
                     ->orWhere('description', 'Manage Customers')
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
             ->get();
55 55
 
56 56
         $allow = $data->isEmpty() ? 'Denied' : 'Allowed';
57
-        Log::debug('User ' . $user->full_name . ' is trying to see admin link.  Result - ' . $allow);
57
+        Log::debug('User '.$user->full_name.' is trying to see admin link.  Result - '.$allow);
58 58
 
59 59
         return  $data->isEmpty() ? false : true;
60 60
     }
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
         }
70 70
 
71 71
         $data = UserRolePermissions::with('UserRolePermissionTypes')
72
-            ->whereHas('UserRolePermissionTypes', function ($query) use ($task) {
72
+            ->whereHas('UserRolePermissionTypes', function($query) use ($task) {
73 73
                 $query->where('description', $task);
74 74
             })
75 75
             ->where('role_id', $user->role_id)
@@ -77,10 +77,10 @@  discard block
 block discarded – undo
77 77
             ->get();
78 78
 
79 79
         $allow = $data->isEmpty() ? 'false' : 'true';
80
-        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - ' . $allow);
81
-        if ($allow === 'Denied')
80
+        Log::debug('User '.$user->full_name.' is trying to access '.$task.'.  Result - '.$allow);
81
+        if($allow === 'Denied')
82 82
         {
83
-            Log::alert('User ' . $user->full_name . ' was denied from accessing '.$task.' link.');
83
+            Log::alert('User '.$user->full_name.' was denied from accessing '.$task.' link.');
84 84
         }
85 85
 
86 86
         return  $data->isEmpty() ? false : true;
Please login to merge, or discard this patch.
app/Http/Controllers/Customers/CustomerContactsController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -130,7 +130,7 @@
 block discarded – undo
130 130
 
131 131
         //  Update the primary contact information
132 132
         $details = Customers::find($request->cust_id);
133
-        if ($details->parent_id && $request->shared == 'true')
133
+        if($details->parent_id && $request->shared == 'true')
134 134
         {
135 135
             $request->cust_id = $details->parent_id;
136 136
         }
Please login to merge, or discard this patch.
app/Http/Controllers/Customers/CustomerNotesController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
 
32 32
         //  Determine if the note should go to the customer, or its parent
33 33
         $details = Customers::find($request->cust_id);
34
-        if ($details->parent_id && $request->shared == 'true')
34
+        if($details->parent_id && $request->shared == 'true')
35 35
         {
36 36
             $request->cust_id = $details->parent_id;
37 37
         }
@@ -56,21 +56,21 @@  discard block
 block discarded – undo
56 56
 
57 57
         //  Determine if there is a parent site with shared notes
58 58
         $parent = Customers::find($id)->parent_id;
59
-        if ($parent)
59
+        if($parent)
60 60
         {
61 61
             $parentList = CustomerNotes::where('cust_id', $parent)->where('shared', 1)->orderBy('urgent', 'desc')->get();
62 62
 
63 63
             $notes = $notes->merge($parentList);
64 64
         }
65 65
 
66
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
66
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
67 67
         return $notes;
68 68
     }
69 69
 
70 70
     //  Update a customer note
71 71
     public function update(Request $request, $id)
72 72
     {
73
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
73
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name.'. Submitted Data - ', $request->toArray());
74 74
 
75 75
         $request->validate([
76 76
             'cust_id' => 'required',
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         ]);
80 80
 
81 81
         $details = Customers::find($request->cust_id);
82
-        if ($details->parent_id && $request->shared == 'true')
82
+        if($details->parent_id && $request->shared == 'true')
83 83
         {
84 84
             $request->cust_id = $details->parent_id;
85 85
         }
Please login to merge, or discard this patch.
app/Http/Controllers/Installer/SettingsController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         ]);
81 81
 
82 82
         //  Update the site timezone
83
-        if (config('app.timezone') !== $request->timezone)
83
+        if(config('app.timezone') !== $request->timezone)
84 84
         {
85 85
             Settings::firstOrCreate(
86 86
                 ['key'   => 'app.timezone'],
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
             )->update(['value' => $request->timezone]);
89 89
         }
90 90
         //  Update the maximum file upload size
91
-        if (config('filesystems.paths.max_size') !== $request->filesize)
91
+        if(config('filesystems.paths.max_size') !== $request->filesize)
92 92
         {
93 93
             Settings::firstOrCreate(
94 94
                 ['key'   => 'filesystems.paths.max_size'],
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/AdminController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
         Log::notice('User '.Auth::user()->full_name.' updated User Password Policy');
90 90
 
91 91
         //  If the setting is changing from never to xx days, update all users
92
-        if ($request->passExpire == 0)
92
+        if($request->passExpire == 0)
93 93
         {
94 94
             User::whereNotNull('password_expires')->update([
95 95
                 'password_expires' => null
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
     public function roleSettings()
111 111
     {
112 112
         $this->authorize('hasAccess', 'Manage User Roles');
113
-        Log::debug('Route ' . Route::currentRouteName() . ' visited by ' . Auth::user()->full_name);
113
+        Log::debug('Route '.Route::currentRouteName().' visited by '.Auth::user()->full_name);
114 114
         $roles = UserRoleType::with(['UserRolePermissions' => function($query) {
115 115
             $query->join('user_role_permission_types', 'user_role_permission_types.perm_type_id', '=', 'user_role_permissions.perm_type_id');
116 116
         }])->get();
Please login to merge, or discard this patch.
app/Http/Controllers/Admin/UserController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -182,13 +182,13 @@  discard block
 block discarded – undo
182 182
 
183 183
         //  Good to go - get role information
184 184
         $roleArr = [];
185
-        foreach ($roles as $role)
185
+        foreach($roles as $role)
186 186
         {
187
-            if ($role->role_id == 1 && Auth::user()->role_id != 1)
187
+            if($role->role_id == 1 && Auth::user()->role_id != 1)
188 188
             {
189 189
                 continue;
190 190
             }
191
-            else if ($role->role_id == 2 && Auth::user()->role_id > 1)
191
+            else if($role->role_id == 2 && Auth::user()->role_id > 1)
192 192
             {
193 193
                 continue;
194 194
             }
@@ -285,13 +285,13 @@  discard block
 block discarded – undo
285 285
         $user = User::find($request->user_id);
286 286
 
287 287
         //  Verify this is a valid user ID
288
-        if (!$user)
288
+        if(!$user)
289 289
         {
290 290
             $success = false;
291 291
             $reason  = 'Cannot find user with this ID';
292 292
         }
293 293
         //  Make sure that the user is not trying to deactivate someone with more permissions
294
-        else if ($user->role_id < Auth::user()->role_id)
294
+        else if($user->role_id < Auth::user()->role_id)
295 295
         {
296 296
             $success = false;
297 297
             $reason  = 'You cannot change password for a user with higher permissions that you.  If this user has locked themselves out, have then use the reset link on the login page.';
Please login to merge, or discard this patch.
app/Files.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
         Log::debug('Preparing to clean a file. Current Name - '.$fileName.'. Current Path - '.$path);
20 20
         //  Remove all spaces
21 21
         $fileName = str_replace(' ', '_', $fileName);
22
-        Log::debug('Cleaned Filename. Current Name - ' . $fileName . '. Current Path - ' . $path);
22
+        Log::debug('Cleaned Filename. Current Name - '.$fileName.'. Current Path - '.$path);
23 23
 
24 24
         //  Determine if the filename already exists
25 25
         if(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName))
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
             //  Increase the number until one that is not in use is found
45 45
             do
46 46
             {
47
-                Log::debug('Filename ' . $fileName . ' already exists.  Appending name.');
47
+                Log::debug('Filename '.$fileName.' already exists.  Appending name.');
48 48
                 $fileName = $base.'('.++$number.')'.$extension;
49 49
             } while(Storage::exists($path.DIRECTORY_SEPARATOR.$fileName));
50 50
         }
Please login to merge, or discard this patch.