Completed
Pull Request — master (#237)
by
unknown
02:21 queued 25s
created
app/Entities/ACSNode.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
     public function heartbeatWarning()
47 47
     {
48 48
         //If the last heartbeat was more than an hour ago there is an issue
49
-        if ( $this->monitor_heartbeat && ($this->last_heartbeat < Carbon::now()->subHour())) {
49
+        if ($this->monitor_heartbeat && ($this->last_heartbeat < Carbon::now()->subHour())) {
50 50
             return true;
51 51
         }
52 52
 
Please login to merge, or discard this patch.
app/Exceptions/Handler.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
         ModelNotFoundException::class,
21 21
         MethodNotAllowedHttpException::class,
22 22
         FormValidationException::class,
23
-        AuthenticationException::class,         //These are logged separately below
23
+        AuthenticationException::class, //These are logged separately below
24 24
 	];
25 25
 
26 26
 	/**
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         }
64 64
 
65 65
         if ($e instanceof NotImplementedException) {
66
-            \Notification::error("NotImplementedException: ".$e->getMessage());
66
+            \Notification::error("NotImplementedException: " . $e->getMessage());
67 67
             \Log::warning($e);
68 68
             return redirect()->back()->withInput();
69 69
         }
@@ -72,8 +72,8 @@  discard block
 block discarded – undo
72 72
             if ($request->wantsJson()) {
73 73
                 return \Response::json(['error' => $e->getMessage()], 403);
74 74
             }
75
-            $userString = \Auth::guest() ? "A guest": \Auth::user()->name;
76
-            \Log::warning($userString." tried to access something they weren't supposed to.");
75
+            $userString = \Auth::guest() ? "A guest" : \Auth::user()->name;
76
+            \Log::warning($userString . " tried to access something they weren't supposed to.");
77 77
 
78 78
             return \Response::view('errors.403', [], 403);
79 79
         }
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
             $e = new HttpException(404, $e->getMessage());
83 83
         }
84 84
 
85
-        if (config('app.debug') && $this->shouldReport($e) && !$request->wantsJson())
85
+        if (config('app.debug') && $this->shouldReport($e) && ! $request->wantsJson())
86 86
         {
87 87
             return $this->renderExceptionWithWhoops($e);
88 88
         }
Please login to merge, or discard this patch.
app/Http/Controllers/ACSSparkController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
     function __construct(ACSNodeRepository $acsNodeRepository, KeyFobAccess $keyFobAccess, PaymentRepository $paymentRepository)
25 25
     {
26 26
         $this->acsNodeRepository = $acsNodeRepository;
27
-        $this->keyFobAccess     = $keyFobAccess;
27
+        $this->keyFobAccess = $keyFobAccess;
28 28
         $this->paymentRepository = $paymentRepository;
29 29
     }
30 30
 
Please login to merge, or discard this patch.
app/Http/Controllers/CCTVController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
                     'ContentType'   => 'image/jpg',
40 40
                     'ServerSideEncryption' => 'AES256',
41 41
                 ));
42
-            } catch(\Exception $e) {
42
+            } catch (\Exception $e) {
43 43
                 \Log::exception($e);
44 44
             }
45 45
             //Log::debug('Image saved :https://s3-eu-west-1.amazonaws.com/buildbrighton-bbms/'.$newFilename);
Please login to merge, or discard this patch.
app/Http/Controllers/EquipmentController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -225,7 +225,7 @@
 block discarded – undo
225 225
 
226 226
                 $equipment->addPhoto($newFilename);
227 227
 
228
-            } catch(\Exception $e) {
228
+            } catch (\Exception $e) {
229 229
                 \Log::error($e);
230 230
                 throw new ImageFailedException($e->getMessage());
231 231
             }
Please login to merge, or discard this patch.
app/Http/Controllers/ExpensesController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -83,13 +83,13 @@
 block discarded – undo
83 83
                 $ext = \Input::file('file')->guessClientExtension();
84 84
                 $mimeType = \Input::file('file')->getMimeType();
85 85
 
86
-                $newFilename = \App::environment().'/expenses/' . str_random() . '.' . $ext;
86
+                $newFilename = \App::environment() . '/expenses/' . str_random() . '.' . $ext;
87 87
 
88 88
                 Storage::put($newFilename, file_get_contents($filePath), 'public');
89 89
 
90 90
                 $data['file'] = $newFilename;
91 91
 
92
-            } catch(\Exception $e) {
92
+            } catch (\Exception $e) {
93 93
                 \Log::error($e);
94 94
                 throw new ImageFailedException($e->getMessage());
95 95
             }
Please login to merge, or discard this patch.
app/Http/Controllers/KeyFobController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
         $fob = KeyFob::findOrFail($id);
70 70
         $fob->markLost();
71 71
         \Notification::success("Key Fob marked as lost/broken");
72
-        return \Redirect::route('account.show',$fob->user_id);
72
+        return \Redirect::route('account.show', $fob->user_id);
73 73
     }
74 74
 
75 75
 
Please login to merge, or discard this patch.
app/Http/Controllers/PaymentController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -265,11 +265,11 @@  discard block
 block discarded – undo
265 265
     {
266 266
         $user = User::findWithPermission($userId);
267 267
 
268
-        if ( ! \Auth::user()->hasRole('admin') &&  ! \Auth::user()->hasRole('finance')) {
268
+        if ( ! \Auth::user()->hasRole('admin') && ! \Auth::user()->hasRole('finance')) {
269 269
             throw new \BB\Exceptions\AuthenticationException;
270 270
         }
271 271
 
272
-        \Log::debug('Manual payment endpoint getting hit. account/{id}/payment. paymentController@store '.json_encode(\Input::all()));
272
+        \Log::debug('Manual payment endpoint getting hit. account/{id}/payment. paymentController@store ' . json_encode(\Input::all()));
273 273
 
274 274
         $reason = \Input::get('reason');
275 275
 
@@ -407,7 +407,7 @@  discard block
 block discarded – undo
407 407
     {
408 408
         $payment = $this->paymentRepository->getById($paymentId);
409 409
 
410
-        switch($request->get('change')) {
410
+        switch ($request->get('change')) {
411 411
             case 'assign-unknown-to-user':
412 412
                 $newUserId = $request->get('user_id');
413 413
                 try {
Please login to merge, or discard this patch.
app/Http/Controllers/ReminderController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
             throw new FormValidationException('Error', $validator->errors());
34 34
         }
35 35
 
36
-        $response = Password::sendResetLink($request->only('email'), function(Message $message)
36
+        $response = Password::sendResetLink($request->only('email'), function (Message $message)
37 37
         {
38 38
             $message->subject('Reset your password');
39 39
         });
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
         //We aren't using a confirm password box so this can be faked
90 90
         $credentials['password_confirmation'] = $credentials['password'];
91 91
 
92
-        $response = Password::reset($credentials, function($user, $password) {
92
+        $response = Password::reset($credentials, function ($user, $password) {
93 93
             $user->password = $password;
94 94
 
95 95
             $user->save();
Please login to merge, or discard this patch.