ArthurGuy / BBMembershipSystem

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'));
 		}
 

app/Http/Middleware/SSLOnly.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function handle($request, Closure $next)
     {
-        if( ! $request->isSecure() && env('FORCE_SECURE', 'true')) {
+        if ( ! $request->isSecure() && env('FORCE_SECURE', 'true')) {
             if ((strpos($request->path(), 'access-control/') !== 0) && ($request->path() !== 'acs') && ($request->path() !== 'acs/spark')) {
                 //return redirect()->secure($request->path());
             }

app/Http/Middleware/VerifyCsrfToken.php

Indentation +11 added lines, -11 removed lines

@@ -5,16 +5,16 @@
 
 class VerifyCsrfToken extends BaseVerifier {
 
-	/**
-	 * Handle an incoming request.
-	 *
-	 * @param  \Illuminate\Http\Request  $request
-	 * @param  \Closure  $next
-	 * @return mixed
-	 */
-	public function handle($request, Closure $next)
-	{
-		return parent::handle($request, $next);
-	}
+    /**
+     * Handle an incoming request.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Closure  $next
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
+        return parent::handle($request, $next);
+    }
 
 }

Braces +2 added lines, -1 removed lines

@@ -3,7 +3,8 @@
 use Closure;
 use Illuminate\Foundation\Http\Middleware\VerifyCsrfToken as BaseVerifier;
 
-class VerifyCsrfToken extends BaseVerifier {
+class VerifyCsrfToken extends BaseVerifier
+{
 
 	/**
 	 * Handle an incoming request.

app/Http/Requests/Request.php

Indentation +1 added lines, -1 removed lines

@@ -4,6 +4,6 @@
 
 abstract class Request extends FormRequest {
 
-	//
+    //
 
 }

Braces +2 added lines, -1 removed lines

@@ -2,7 +2,8 @@
 
 use Illuminate\Foundation\Http\FormRequest;
 
-abstract class Request extends FormRequest {
+abstract class Request extends FormRequest
+{
 
 	//
 

app/Http/routes.php

Spacing +17 added lines, -17 removed lines

@@ -61,7 +61,7 @@ 
 
 Route::post('account/{account}/payment', ['uses' => 'PaymentController@store', 'as' => 'account.payment.store', 'middleware' => 'role:admin']);
 
-Route::group(array('middleware' => 'role:finance'), function() {
+Route::group(array('middleware' => 'role:finance'), function () {
     Route::resource('payments', 'PaymentController', ['only' => ['index', 'destroy', 'update']]);
     Route::get('payments/overview', ['uses'=>'PaymentOverviewController@index', 'as'=>'payments.overview']);
     Route::get('payments/sub-charges', ['as' => 'payments.sub-charges', 'uses' => 'SubscriptionController@listCharges']);
@@ -80,13 +80,13 @@ 
 
 
 //Cash
-Route::group(array('middleware' => 'role:finance'), function() {
+Route::group(array('middleware' => 'role:finance'), function () {
     Route::post('account/{account}/payment/cash/create', ['as'=>'account.payment.cash.create', 'uses' => 'CashPaymentController@store']);
     Route::delete('account/{account}/payment/cash', ['as'=>'account.payment.cash.destroy', 'uses' => 'CashPaymentController@destroy']);
 });
 
 # Statements
-Route::group(array('middleware' => 'role:finance'), function() {
+Route::group(array('middleware' => 'role:finance'), function () {
     Route::resource('statement-import', 'StatementImportController', ['except' => ['index', 'show', 'edit', 'update', 'destroy']]);
 });
 
@@ -99,7 +99,7 @@ 
 # Inductions
 ##########################
 
-Route::group(array('middleware' => 'role:admin'), function() {
+Route::group(array('middleware' => 'role:admin'), function () {
     Route::post('equipment_training/update', ['uses'=>'InductionController@update', 'as'=>'equipment_training.update']);
     Route::resource('account.induction', 'InductionController', ['only' => ['update', 'destroy']]);
 });
@@ -110,7 +110,7 @@ 
 # Equipment
 ##########################
 
-Route::group(array('middleware' => 'role:member'), function() {
+Route::group(array('middleware' => 'role:member'), function () {
     Route::resource('equipment', 'EquipmentController');
     Route::post('equipment/{id}/photo', ['uses'=>'EquipmentController@addPhoto', 'as'=>'equipment.photo.store']);
     Route::delete('equipment/{id}/photo/{key}', ['uses'=>'EquipmentController@destroyPhoto', 'as'=>'equipment.photo.destroy']);
@@ -133,7 +133,7 @@ 
 # Key fobs
 ##########################
 
-Route::group(array('middleware' => 'role:admin'), function() {
+Route::group(array('middleware' => 'role:admin'), function () {
     Route::resource('keyfob', 'KeyFobController', ['only' => ['index', 'store', 'update', 'destroy']]);
 });
 
@@ -165,11 +165,11 @@ 
 
 Route::post('camera/event/store', ['uses' => 'CCTVController@store']);
 
-Route::group(array('middleware' => 'role:admin'), function() {
+Route::group(array('middleware' => 'role:admin'), function () {
     Route::resource('detected_devices', 'DetectedDevicesController');
 });
 
-Route::group(array('middleware' => 'role:acs'), function() {
+Route::group(array('middleware' => 'role:acs'), function () {
     Route::resource('devices', 'DeviceController');
 });
 
@@ -245,11 +245,11 @@ 
 # Roles
 ##########################
 
-Route::group(array('middleware' => 'role:admin'), function() {
+Route::group(array('middleware' => 'role:admin'), function () {
     Route::resource('roles', 'RolesController', []);
     Route::resource('roles.users', 'RoleUsersController', ['only' => ['destroy', 'store']]);
 });
-Route::group(array('middleware' => 'role:member'), function() {
+Route::group(array('middleware' => 'role:member'), function () {
     Route::resource('groups', 'GroupsController', ['only' => ['index', 'show']]);
 });
 
@@ -268,7 +268,7 @@ 
 # Expenses
 ##########################
 
-Route::group(array('middleware' => 'role:member'), function() {
+Route::group(array('middleware' => 'role:member'), function () {
     Route::resource('expenses', 'ExpensesController');
 });
 
@@ -285,10 +285,10 @@ 
 
 
 
-Route::any('api-docs.json', function() {
+Route::any('api-docs.json', function () {
     $filePath = Config::get('swagger.doc-dir') . "/api-docs.json";
 
-    if (!File::Exists($filePath)) {
+    if ( ! File::Exists($filePath)) {
         App::abort(404, "Cannot find {$filePath}");
     }
 
@@ -298,13 +298,13 @@ 
     ));
 });
 
-Route::get('api-docs', function() {
+Route::get('api-docs', function () {
     if (Config::get('swagger.generateAlways')) {
-        $appDir = base_path()."/".Config::get('swagger.app-dir');
+        $appDir = base_path() . "/" . Config::get('swagger.app-dir');
         $docDir = Config::get('swagger.doc-dir');
 
 
-        if (!File::exists($docDir) || is_writable($docDir)) {
+        if ( ! File::exists($docDir) || is_writable($docDir)) {
             // delete all existing documentation
             if (File::exists($docDir)) {
                 File::deleteDirectory($docDir);
@@ -314,7 +314,7 @@ 
 
             $excludeDirs = Config::get('swagger.excludes');
 
-            $swagger =  \Swagger\scan($appDir, [
+            $swagger = \Swagger\scan($appDir, [
                 'exclude' => $excludeDirs
             ]);
 

app/Listeners/RecordMemberActivity.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 namespace BB\Listeners;
 
 use BB\Events\MemberActivity;
-use BB\Repo\ActivityRepository;use Illuminate\Queue\InteractsWithQueue;
+use BB\Repo\ActivityRepository; use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Contracts\Queue\ShouldQueue;
 
 class RecordMemberActivity

app/Listeners/SlackMemberNotification.php

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
         if (\App::environment('production')) {
 
             //If the user doesn't have a slack username registered there is nothing we can do
-            if (empty($event->notification->user()->slack_username))
-            {
+            if (empty($event->notification->user()->slack_username)) {
                 return;
             }