ArthurGuy / BBMembershipSystem

app/Repo/PaymentRepository.php

Spacing +2 added lines, -2 removed lines

@@ -188,7 +188,7 @@ 
     {
         $payment = $this->getById($paymentId);
 
-        if (!empty($payment->user_id)) {
+        if ( ! empty($payment->user_id)) {
             throw new PaymentException('Payment already assigned to user');
         }
 
@@ -266,7 +266,7 @@ 
      */
     public function getEquipmentFeePayments($referencePrefix)
     {
-        return $this->model->where('reason', 'equipment-fee')->get()->filter(function($payment) use($referencePrefix) {
+        return $this->model->where('reason', 'equipment-fee')->get()->filter(function ($payment) use($referencePrefix) {
             return strpos($payment->reference, ':' . $referencePrefix) !== false;
         });
     }

app/Services/Registrar.php

Indentation +28 added lines, -28 removed lines

@@ -6,34 +6,34 @@
 
 class Registrar implements RegistrarContract {
 
-	/**
-	 * Get a validator for an incoming registration request.
-	 *
-	 * @param  array  $data
-	 * @return \Illuminate\Contracts\Validation\Validator
-	 */
-	public function validator(array $data)
-	{
-		return Validator::make($data, [
-			'name' => 'required|max:255',
-			'email' => 'required|email|max:255|unique:users',
-			'password' => 'required|confirmed|min:6',
-		]);
-	}
+    /**
+     * Get a validator for an incoming registration request.
+     *
+     * @param  array  $data
+     * @return \Illuminate\Contracts\Validation\Validator
+     */
+    public function validator(array $data)
+    {
+        return Validator::make($data, [
+            'name' => 'required|max:255',
+            'email' => 'required|email|max:255|unique:users',
+            'password' => 'required|confirmed|min:6',
+        ]);
+    }
 
-	/**
-	 * Create a new user instance after a valid registration.
-	 *
-	 * @param  array  $data
-	 * @return User
-	 */
-	public function create(array $data)
-	{
-		return User::create([
-			'name' => $data['name'],
-			'email' => $data['email'],
-			'password' => bcrypt($data['password']),
-		]);
-	}
+    /**
+     * Create a new user instance after a valid registration.
+     *
+     * @param  array  $data
+     * @return User
+     */
+    public function create(array $data)
+    {
+        return User::create([
+            'name' => $data['name'],
+            'email' => $data['email'],
+            'password' => bcrypt($data['password']),
+        ]);
+    }
 
 }

Braces +2 added lines, -1 removed lines

@@ -4,7 +4,8 @@
 use Validator;
 use Illuminate\Contracts\Auth\Registrar as RegistrarContract;
 
-class Registrar implements RegistrarContract {
+class Registrar implements RegistrarContract
+{
 
 	/**
 	 * Get a validator for an incoming registration request.

app/Repo/UserRepository.php

Spacing +2 added lines, -2 removed lines

@@ -109,7 +109,7 @@ 
 
         $memberData['hash'] = str_random(30);
 
-        $memberData['rules_agreed'] = $memberData['rules_agreed']? Carbon::now(): null;
+        $memberData['rules_agreed'] = $memberData['rules_agreed'] ? Carbon::now() : null;
 
         $user = $this->model->create($memberData);
         $this->profileDataRepository->createProfile($user->id);
@@ -218,7 +218,7 @@ 
 
         $user->induction_completed = true;
 
-        $user->rules_agreed = $user->rules_agreed? $user->rules_agreed: Carbon::now();
+        $user->rules_agreed = $user->rules_agreed ? $user->rules_agreed : Carbon::now();
 
         $user->save();
     }

app/Presenters/PaymentPresenter.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
             case 'manual':
                 return 'Manual';
             case 'cash':
-                return 'Cash' . ($this->entity->source_id? ' (' . $this->entity->source_id . ')':'');
+                return 'Cash' . ($this->entity->source_id ? ' (' . $this->entity->source_id . ')' : '');
             case 'other':
                 return 'Other';
             case 'balance':

app/Services/Credit.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
 
     public function recalculate()
     {
-        if (! $this->user instanceof User) {
+        if ( ! $this->user instanceof User) {
             throw new InvalidDataException("User not set");
         }
         $runningTotal = 0;
@@ -96,7 +96,7 @@ 
      */
     public function getBalanceSign()
     {
-        return ( (int) $this->getBalance() >= 0 ? 'positive' : 'negative' );
+        return ((int)$this->getBalance() >= 0 ? 'positive' : 'negative');
     }
 
     public function getBalanceFormatted()

app/Http/Controllers/MembersController.php

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 
     public function index()
     {
-        $users = $this->userRepository->getActivePublicList(!\Auth::guest());
+        $users = $this->userRepository->getActivePublicList( ! \Auth::guest());
         return \View::make('members.index')->withUsers($users);
     }
 
@@ -44,7 +44,7 @@ 
             return \Response::make('', 404);
         }
 
-        if (!$user->active) {
+        if ( ! $user->active) {
             return \Response::make('', 404);
         }
 

app/Http/Controllers/SettingsController.php

Indentation +8 added lines, -8 removed lines

@@ -5,19 +5,19 @@
 class SettingsController extends Controller
 {
 
-	/**
-	 * Store a newly created resource in storage.
-	 *
-	 * @return \Illuminate\Http\RedirectResponse
-	 */
-	public function update()
-	{
+    /**
+     * Store a newly created resource in storage.
+     *
+     * @return \Illuminate\Http\RedirectResponse
+     */
+    public function update()
+    {
         $input = \Input::only('key', 'value');
 
         Settings::change($input['key'], $input['value']);
 
         \Notification::success("Setting updated");
         return redirect()->back()->withInput();
-	}
+    }
 
 }

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/routes.php

Spacing +17 added lines, -17 removed lines

@@ -72,7 +72,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']);
@@ -92,13 +92,13 @@ 
 
 
 //Cash
-Route::group(array('middleware' => 'role:admin'), function() {
+Route::group(array('middleware' => 'role:admin'), 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']]);
 });
 
@@ -111,7 +111,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']]);
 });
@@ -122,7 +122,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']);
@@ -145,7 +145,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']]);
 });
 
@@ -178,11 +178,11 @@ 
 Route::post('camera/event/store', ['uses' => 'CCTVController@store']);
 Route::post('camera/store', ['uses' => 'CCTVController@storeSingle']);
 
-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');
 });
 
@@ -259,11 +259,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']]);
 });
 
@@ -282,7 +282,7 @@ 
 # Expenses
 ##########################
 
-Route::group(array('middleware' => 'role:member'), function() {
+Route::group(array('middleware' => 'role:member'), function () {
     Route::resource('expenses', 'ExpensesController');
 });
 
@@ -305,10 +305,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}");
     }
 
@@ -318,13 +318,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);
@@ -334,7 +334,7 @@ 
 
             $excludeDirs = Config::get('swagger.excludes');
 
-            $swagger =  \Swagger\scan($appDir, [
+            $swagger = \Swagger\scan($appDir, [
                 'exclude' => $excludeDirs
             ]);