ArthurGuy / BBMembershipSystem

app/Helpers/GoCardlessHelper.php

Doc Comments +12 added lines

@@ -30,6 +30,9 @@ 
         ]);
     }
 
+    /**
+     * @return double
+     */
     public function getPayment($paymentId)
     {
         return $this->client->payments()->get($paymentId);
@@ -56,6 +59,11 @@ 
     }
 
 
+    /**
+     * @param string|null $mandate
+     * @param integer $amount
+     * @param string $subscriptionNumber
+     */
     public function createSubscription($mandate, $amount, $dayOfMonth, $subscriptionNumber)
     {
         $subscription = $this->client->subscriptions()->create([
@@ -75,6 +83,10 @@ 
 
         return $subscription;
     }
+
+    /**
+     * @param string $id
+     */
     public function cancelSubscription($id)
     {
         return $this->client->subscriptions()->cancel($id);

app/Repo/UserRepository.php

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

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 < 15)) {
+        } elseif ( ! \Auth::user()->isAdmin() && ($amount < 15)) {
             throw new ValidationException('The minimum subscription is 15 GBP, please contact the trustees for a lower amount. trustees@buildbrighton.com');
         }