ArthurGuy / BBMembershipSystem

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

Doc Comments +12 added lines, -1 removed lines

@@ -196,6 +196,9 @@ 
         $this->subscriptionChargeRepository->cancelOutstandingCharges($userId);
     }
 
+    /**
+     * @param integer $userId
+     */
     public function recordGoCardlessMandateDetails($userId, $subscriptionId)
     {
         /** @var User $user */
@@ -205,6 +208,11 @@ 
         $user->save();
     }
 
+    /**
+     * @param integer $userId
+     * @param string $paymentMethod
+     * @param integer $paymentDay
+     */
     public function updateUserPaymentMethod($userId, $paymentMethod, $paymentDay = null)
     {
         /** @var User $user */
@@ -216,6 +224,9 @@ 
         $user->save();
     }
 
+    /**
+     * @param integer $userId
+     */
     public function recordGoCardlessSubscription($userId, $subscriptionId, $paymentDay = null)
     {
         /** @var User $user */
@@ -230,7 +241,7 @@ 
     /**
      * Record the fact that the user has agreed to the member induction and the rules
      *
-     * @param $userId
+     * @param integer $userId
      */
     public function recordInductionCompleted($userId)
     {

app/Http/Controllers/AccountController.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use BB\Events\MemberGivenTrustedStatus;
 use BB\Events\MemberPhotoWasDeclined;
 use BB\Exceptions\ValidationException;
-use BB\Validators\InductionValidator;
 
 class AccountController extends Controller
 {

Spacing +3 added lines, -3 removed lines

@@ -324,12 +324,12 @@ 
         if (\Input::has('experimental_dd_subscription')) {
             $subscription = $this->goCardless->createSubscription($user->mandate_id, $user->monthly_subscription * 100, $user->payment_day, 'NEW-BBSUB' . $user->id);
 
-            $this->userRepository->recordGoCardlessSubscription($user->id,  $subscription->id);
+            $this->userRepository->recordGoCardlessSubscription($user->id, $subscription->id);
         }
         if (\Input::has('cancel_experimental_dd_subscription')) {
             $this->goCardless->cancelSubscription($user->subscription_id);
 
-            $this->userRepository->recordGoCardlessSubscription($user->id,  null);
+            $this->userRepository->recordGoCardlessSubscription($user->id, null);
         }
 
 
@@ -413,7 +413,7 @@ 
 
         if ($amount < 5) {
             throw new ValidationException('The minimum subscription is 5 GBP');
-        } elseif (!\Auth::user()->isAdmin() && ($amount < 20)) {
+        } elseif ( ! \Auth::user()->isAdmin() && ($amount < 20)) {
             throw new ValidationException('The minimum subscription is 20 GBP, please contact the trustees for a lower amount. trustees@buildbrighton.com');
         }
 

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/Helpers/UserImage.php

Doc Comments +4 added lines, -1 removed lines

@@ -12,6 +12,9 @@ 
 
     }
 
+    /**
+     * @param string $userId
+     */
     public function uploadPhoto($userId, $filePath, $newImage = false)
     {
         $tmpFilePath = storage_path('app') . '/' . $userId . '.png';
@@ -46,7 +49,7 @@ 
 
     /**
      * Delete an old profile image and replace it with a new one.
-     * @param $userId
+     * @param string $userId
      */
     public function approveNewImage($userId)
     {

Unused Use Statements -1 removed lines

@@ -1,6 +1,5 @@
 <?php namespace BB\Helpers;
 
-use BB\Exceptions\UserImageFailedException;
 use Illuminate\Support\Facades\Storage;
 use Intervention\Image\Facades\Image;
 

app/Http/Controllers/InductionController.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace BB\Http\Controllers;
 
 use BB\Entities\Induction;
-use BB\Exceptions\PaymentException;
 use BB\Repo\PaymentRepository;
 
 class InductionController extends Controller

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/Repo/InductionRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     }
 
     /**
-     * @param $deviceId
+     * @param string $deviceId
      * @return array
      */
     public function getTrainersForEquipment($deviceId)

app/Entities/Settings.php

Doc Comments +3 added lines

@@ -39,6 +39,9 @@
         self::create(['key' => $key, 'value' => $value]);
     }
 
+    /**
+     * @param string $key
+     */
     public static function get($key)
     {
         $setting = self::findOrFail($key);