ArthurGuy / BBMembershipSystem

app/Http/Controllers/EquipmentController.php

Spacing +1 added lines, -1 removed lines

@@ -225,7 +225,7 @@
 
                 $equipment->addPhoto($newFilename);
 
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 \Log::error($e);
                 throw new ImageFailedException($e->getMessage());
             }

app/Http/Controllers/ExpensesController.php

Indentation +49 added lines, -49 removed lines

@@ -33,8 +33,8 @@ 
      * @return Response
      * @throws \BB\Exceptions\AuthenticationException
      */
-	public function index()
-	{
+    public function index()
+    {
         $userId       = \Request::get('user_id', '');
         $sortBy       = \Request::get('sortBy', 'created_at');
         $direction    = \Request::get('direction', 'desc');
@@ -49,18 +49,18 @@ 
         $expenses = $this->expenseRepository->getPaginated(compact('sortBy', 'direction'));
 
         return \View::make('expenses.index')->with('expenses', $expenses);
-	}
+    }
 
 
-	/**
-	 * Show the form for creating a new resource.
-	 *
-	 * @return Response
-	 */
-	public function create()
-	{
-		//
-	}
+    /**
+     * Show the form for creating a new resource.
+     *
+     * @return Response
+     */
+    public function create()
+    {
+        //
+    }
 
 
     /**
@@ -70,9 +70,9 @@ 
      * @throws ImageFailedException
      * @throws \BB\Exceptions\FormValidationException
      */
-	public function store()
-	{
-		$data = \Request::only(['user_id', 'category', 'description', 'amount', 'expense_date', 'file']);
+    public function store()
+    {
+        $data = \Request::only(['user_id', 'category', 'description', 'amount', 'expense_date', 'file']);
 
         $this->expenseValidator->validate($data);
 
@@ -107,31 +107,31 @@ 
         event(new NewExpenseSubmitted($expense));
 
         return \Response::json($expense, 201);
-	}
+    }
 
 
-	/**
-	 * Display the specified resource.
-	 *
-	 * @param  int  $id
-	 * @return Response
-	 */
-	public function show($id)
-	{
-		return \BB\Entities\Expense::findOrFail($id);
-	}
+    /**
+     * Display the specified resource.
+     *
+     * @param  int  $id
+     * @return Response
+     */
+    public function show($id)
+    {
+        return \BB\Entities\Expense::findOrFail($id);
+    }
 
 
-	/**
-	 * Show the form for editing the specified resource.
-	 *
-	 * @param  int  $id
-	 * @return Response
-	 */
-	public function edit($id)
-	{
-		//
-	}
+    /**
+     * Show the form for editing the specified resource.
+     *
+     * @param  int  $id
+     * @return Response
+     */
+    public function edit($id)
+    {
+        //
+    }
 
 
     /**
@@ -141,8 +141,8 @@ 
      * @return Response
      * @throws \BB\Exceptions\AuthenticationException
      */
-	public function update($id)
-	{
+    public function update($id)
+    {
         /**
         if (\Request::ajax()) {
             $data = \Request::only(['category', 'description', 'amount', 'expense_date']);
@@ -152,7 +152,7 @@ 
 
             return $expense;
         }
-        */
+         */
 
         if ( ! \Auth::user()->hasRole('admin')) {
             throw new \BB\Exceptions\AuthenticationException();
@@ -167,20 +167,20 @@ 
         }
 
         return \Redirect::route('expenses.index');
-	}
+    }
 
 
-	/**
-	 * Remove the specified resource from storage.
-	 *
-	 * @param  int  $id
-	 * @return Response
-	 */
-	public function destroy($id)
-	{
+    /**
+     * Remove the specified resource from storage.
+     *
+     * @param  int  $id
+     * @return Response
+     */
+    public function destroy($id)
+    {
         //$expense = \BB\Entities\Expense::findOrFail($id);
         //$expense->delete();
-	}
+    }
 
 
 }

Spacing +2 added lines, -2 removed lines

@@ -83,13 +83,13 @@
                 $ext = \Input::file('file')->guessClientExtension();
                 $mimeType = \Input::file('file')->getMimeType();
 
-                $newFilename = \App::environment().'/expenses/' . str_random() . '.' . $ext;
+                $newFilename = \App::environment() . '/expenses/' . str_random() . '.' . $ext;
 
                 Storage::put($newFilename, file_get_contents($filePath), 'public');
 
                 $data['file'] = $newFilename;
 
-            } catch(\Exception $e) {
+            } catch (\Exception $e) {
                 \Log::error($e);
                 throw new ImageFailedException($e->getMessage());
             }

Braces +2 added lines, -1 removed lines

@@ -6,7 +6,8 @@
 use Illuminate\Support\Facades\Response;
 use Illuminate\Support\Facades\Storage;
 
-class ExpensesController extends Controller {
+class ExpensesController extends Controller
+{
 
     /**
      * @var \BB\Validators\ExpenseValidator

app/Http/Controllers/KeyFobController.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
         $fob = KeyFob::findOrFail($id);
         $fob->markLost();
         \Notification::success("Key Fob marked as lost/broken");
-        return \Redirect::route('account.show',$fob->user_id);
+        return \Redirect::route('account.show', $fob->user_id);
     }
 
 

app/Http/Controllers/PaymentController.php

Spacing +3 added lines, -3 removed lines

@@ -265,11 +265,11 @@ 
     {
         $user = User::findWithPermission($userId);
 
-        if ( ! \Auth::user()->hasRole('admin') &&  ! \Auth::user()->hasRole('finance')) {
+        if ( ! \Auth::user()->hasRole('admin') && ! \Auth::user()->hasRole('finance')) {
             throw new \BB\Exceptions\AuthenticationException;
         }
 
-        \Log::debug('Manual payment endpoint getting hit. account/{id}/payment. paymentController@store '.json_encode(\Input::all()));
+        \Log::debug('Manual payment endpoint getting hit. account/{id}/payment. paymentController@store ' . json_encode(\Input::all()));
 
         $reason = \Input::get('reason');
 
@@ -407,7 +407,7 @@ 
     {
         $payment = $this->paymentRepository->getById($paymentId);
 
-        switch($request->get('change')) {
+        switch ($request->get('change')) {
             case 'assign-unknown-to-user':
                 $newUserId = $request->get('user_id');
                 try {

app/Http/Controllers/ReminderController.php

Indentation +26 added lines, -26 removed lines

@@ -9,16 +9,16 @@ 
 class ReminderController extends Controller
 {
 
-	/**
-	 * Display the password reminder view.
-	 *
-	 * @return Response
-	 */
-	public function create()
-	{
+    /**
+     * Display the password reminder view.
+     *
+     * @return Response
+     */
+    public function create()
+    {
 
         return view('password.create');
-	}
+    }
 
     /**
      * Handle a POST request to remind a user of their password.
@@ -26,8 +26,8 @@ 
      * @param Request $request
      * @return \Illuminate\Http\RedirectResponse|null
      */
-	public function store(Request $request)
-	{
+    public function store(Request $request)
+    {
         $validator = app('Illuminate\Contracts\Validation\Factory')->make($request->all(), ['email' => 'required|email']);
         if ($validator->fails()) {
             throw new FormValidationException('Error', $validator->errors());
@@ -39,31 +39,31 @@ 
         });
 
         switch ($response) {
-			case Password::INVALID_USER:
+            case Password::INVALID_USER:
                 \Notification::error(trans($response));
                 return redirect()->back();
 
-			case Password::RESET_LINK_SENT:
+            case Password::RESET_LINK_SENT:
                 \Notification::success(trans($response));
                 return redirect()->back();
-		}
-	}
-
-	/**
-	 * Display the password reset view for the given token.
-	 *
-	 * @param  string  $token
-	 * @return Response
-	 */
-	public function getReset($token = null)
-	{
+        }
+    }
+
+    /**
+     * Display the password reset view for the given token.
+     *
+     * @param  string  $token
+     * @return Response
+     */
+    public function getReset($token = null)
+    {
         if (is_null($token))
         {
             throw new NotFoundHttpException;
         }
 
         return view('password.reset')->with('token', $token);
-	}
+    }
 
     /**
      * Handle a POST request to reset a user's password.
@@ -71,8 +71,8 @@ 
      * @param Request $request
      * @return \Illuminate\Http\RedirectResponse|null
      */
-	public function postReset(Request $request)
-	{
+    public function postReset(Request $request)
+    {
         $credentials = $request->only(
             'email', 'password', 'password_confirmation', 'token'
         );

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
             throw new FormValidationException('Error', $validator->errors());
         }
 
-        $response = Password::sendResetLink($request->only('email'), function(Message $message)
+        $response = Password::sendResetLink($request->only('email'), function (Message $message)
         {
             $message->subject('Reset your password');
         });
@@ -89,7 +89,7 @@ 
         //We aren't using a confirm password box so this can be faked
         $credentials['password_confirmation'] = $credentials['password'];
 
-        $response = Password::reset($credentials, function($user, $password) {
+        $response = Password::reset($credentials, function ($user, $password) {
             $user->password = $password;
 
             $user->save();

Braces +2 added lines, -4 removed lines

@@ -33,8 +33,7 @@ 
             throw new FormValidationException('Error', $validator->errors());
         }
 
-        $response = Password::sendResetLink($request->only('email'), function(Message $message)
-        {
+        $response = Password::sendResetLink($request->only('email'), function(Message $message) {
             $message->subject('Reset your password');
         });
 
@@ -57,8 +56,7 @@ 
 	 */
 	public function getReset($token = null)
 	{
-        if (is_null($token))
-        {
+        if (is_null($token)) {
             throw new NotFoundHttpException;
         }
 

app/Http/Controllers/SessionController.php

Indentation +25 added lines, -25 removed lines

@@ -16,27 +16,27 @@ 
     }
 
 
-	/**
-	 * Show the form for creating a new resource.
-	 *
-	 * @return Response
-	 */
-	public function create()
-	{
+    /**
+     * Show the form for creating a new resource.
+     *
+     * @return Response
+     */
+    public function create()
+    {
         if ( ! Auth::guest()) {
             return redirect()->to('account/' . \Auth::id());
         }
         return \View::make('session.create');
-	}
+    }
 
 
-	/**
-	 * Store a newly created resource in storage.
-	 *
-	 * @return \Illuminate\Http\RedirectResponse
-	 */
-	public function store()
-	{
+    /**
+     * Store a newly created resource in storage.
+     *
+     * @return \Illuminate\Http\RedirectResponse
+     */
+    public function store()
+    {
         $input = \Input::only('email', 'password');
 
         $this->loginForm->validate($input);
@@ -47,23 +47,23 @@ 
 
         \Notification::error("Invalid login details");
         return redirect()->back()->withInput();
-	}
+    }
 
 
-	/**
-	 * Remove the specified resource from storage.
-	 *
-	 * @param  int  $id
-	 * @return \Illuminate\Http\RedirectResponse
-	 */
-	public function destroy($id = null)
-	{
+    /**
+     * Remove the specified resource from storage.
+     *
+     * @param  int  $id
+     * @return \Illuminate\Http\RedirectResponse
+     */
+    public function destroy($id = null)
+    {
         Auth::logout();
 
         \Notification::success('Logged Out');
 
         return redirect()->home();
-	}
+    }
 
     /**
      * @param Request $request

app/Http/Middleware/Authenticate.php

Indentation +38 added lines, -38 removed lines

@@ -5,46 +5,46 @@
 
 class Authenticate {
 
-	/**
-	 * The Guard implementation.
-	 *
-	 * @var Guard
-	 */
-	protected $auth;
+    /**
+     * The Guard implementation.
+     *
+     * @var Guard
+     */
+    protected $auth;
 
-	/**
-	 * Create a new filter instance.
-	 *
-	 * @param  Guard  $auth
-	 * @return void
-	 */
-	public function __construct(Guard $auth)
-	{
-		$this->auth = $auth;
-	}
+    /**
+     * Create a new filter instance.
+     *
+     * @param  Guard  $auth
+     * @return void
+     */
+    public function __construct(Guard $auth)
+    {
+        $this->auth = $auth;
+    }
 
-	/**
-	 * Handle an incoming request.
-	 *
-	 * @param  \Illuminate\Http\Request  $request
-	 * @param  \Closure  $next
-	 * @return mixed
-	 */
-	public function handle($request, Closure $next)
-	{
-		if ($this->auth->guest())
-		{
-			if ($request->ajax())
-			{
-				return response('Unauthorized.', 401);
-			}
-			else
-			{
-				return redirect()->guest('auth/login');
-			}
-		}
+    /**
+     * Handle an incoming request.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Closure  $next
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
+        if ($this->auth->guest())
+        {
+            if ($request->ajax())
+            {
+                return response('Unauthorized.', 401);
+            }
+            else
+            {
+                return redirect()->guest('auth/login');
+            }
+        }
 
-		return $next($request);
-	}
+        return $next($request);
+    }
 
 }

Braces +5 added lines, -8 removed lines

@@ -3,7 +3,8 @@ 
 use Closure;
 use Illuminate\Contracts\Auth\Guard;
 
-class Authenticate {
+class Authenticate
+{
 
 	/**
 	 * The Guard implementation.
@@ -32,14 +33,10 @@ 
 	 */
 	public function handle($request, Closure $next)
 	{
-		if ($this->auth->guest())
-		{
-			if ($request->ajax())
-			{
+		if ($this->auth->guest()) {
+			if ($request->ajax()) {
 				return response('Unauthorized.', 401);
-			}
-			else
-			{
+			} else {
 				return redirect()->guest('auth/login');
 			}
 		}

app/Http/Middleware/BrandingHeaders.php

Indentation +11 added lines, -11 removed lines

@@ -4,21 +4,21 @@
 
 class BrandingHeaders {
 
-	/**
-	 * Handle an incoming request.
-	 *
-	 * @param  \Illuminate\Http\Request  $request
-	 * @param  \Closure  $next
-	 * @return mixed
-	 */
-	public function handle($request, Closure $next)
-	{
+    /**
+     * Handle an incoming request.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Closure  $next
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
         //Don't add the header to acs requests
         if ((strpos($request->path(), 'access-control/') === 0) || ($request->path() === 'acs') || ($request->path() === 'acs/spark')) {
             return $next($request);
         }
 
-		return $next($request)->header('Built-By', 'arthurguy.co.uk');
-	}
+        return $next($request)->header('Built-By', 'arthurguy.co.uk');
+    }
 
 }

Braces +2 added lines, -1 removed lines

@@ -2,7 +2,8 @@
 
 use Closure;
 
-class BrandingHeaders {
+class BrandingHeaders
+{
 
 	/**
 	 * Handle an incoming request.

app/Http/Middleware/RedirectIfAuthenticated.php

Indentation +31 added lines, -31 removed lines

@@ -6,39 +6,39 @@
 
 class RedirectIfAuthenticated {
 
-	/**
-	 * The Guard implementation.
-	 *
-	 * @var Guard
-	 */
-	protected $auth;
+    /**
+     * The Guard implementation.
+     *
+     * @var Guard
+     */
+    protected $auth;
 
-	/**
-	 * Create a new filter instance.
-	 *
-	 * @param  Guard  $auth
-	 * @return void
-	 */
-	public function __construct(Guard $auth)
-	{
-		$this->auth = $auth;
-	}
+    /**
+     * Create a new filter instance.
+     *
+     * @param  Guard  $auth
+     * @return void
+     */
+    public function __construct(Guard $auth)
+    {
+        $this->auth = $auth;
+    }
 
-	/**
-	 * Handle an incoming request.
-	 *
-	 * @param  \Illuminate\Http\Request  $request
-	 * @param  \Closure  $next
-	 * @return mixed
-	 */
-	public function handle($request, Closure $next)
-	{
-		if ($this->auth->check())
-		{
-			return new RedirectResponse(url('/home'));
-		}
+    /**
+     * Handle an incoming request.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Closure  $next
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
+        if ($this->auth->check())
+        {
+            return new RedirectResponse(url('/home'));
+        }
 
-		return $next($request);
-	}
+        return $next($request);
+    }
 
 }

Braces +3 added lines, -3 removed lines

@@ -4,7 +4,8 @@ 
 use Illuminate\Contracts\Auth\Guard;
 use Illuminate\Http\RedirectResponse;
 
-class RedirectIfAuthenticated {
+class RedirectIfAuthenticated
+{
 
 	/**
 	 * The Guard implementation.
@@ -33,8 +34,7 @@ 
 	 */
 	public function handle($request, Closure $next)
 	{
-		if ($this->auth->check())
-		{
+		if ($this->auth->check()) {
 			return new RedirectResponse(url('/home'));
 		}