lagdo / tontine

app/helpers.php

Spacing +2 added lines, -3 removed lines

@@ -35,12 +35,11 @@
 {
     $icon = '<i class="fa fa-toggle-off"></i>';
     $linkClass = "btn-add-$name";
-    if(($payment))
+    if (($payment))
     {
         $icon = '<i class="fa fa-toggle-on"></i>';
         $linkClass = "btn-del-$name";
     }
 
-    return $disableLink ? $icon :
-        '<a href="javascript:void(0)" class="' . $linkClass . '">' . $icon . '</a>';
+    return $disableLink ? $icon : '<a href="javascript:void(0)" class="' . $linkClass . '">' . $icon . '</a>';
 }

app/Ajax/Web/Meeting/Pool/Deposit/Pool.php

Spacing +14 added lines, -16 removed lines

@@ -68,7 +68,7 @@ 
             $this->target()->args()[0] : $this->bag('meeting')->get('pool.id');
         $this->pool = $this->depositService->getPool($poolId);
 
-        if(!$this->session || !$this->pool || $this->session->disabled($this->pool))
+        if (!$this->session || !$this->pool || $this->session->disabled($this->pool))
         {
             $this->notify->error(trans('tontine.session.errors.disabled'), trans('common.titles.error'));
             $this->pool = null;
@@ -134,7 +134,7 @@ 
      */
     public function addDeposit(int $receivableId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -153,21 +153,20 @@ 
      */
     public function editAmount(int $receivableId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
         }
         $receivable = $this->depositService->getReceivable($this->pool, $this->session, $receivableId);
-        if(!$receivable || !$receivable->deposit)
+        if (!$receivable || !$receivable->deposit)
         {
             return $this->page();
         }
 
         $html = $this->view()->render('tontine.pages.meeting.deposit.libre.edit', [
             'id' => $receivable->id,
-            'amount' => !$receivable->deposit ? '' :
-                $this->localeService->getMoneyValue($receivable->deposit->amount),
+            'amount' => !$receivable->deposit ? '' : $this->localeService->getMoneyValue($receivable->deposit->amount),
         ]);
         $fieldId = 'receivable-' . $receivable->id;
         $this->response->html($fieldId, $html);
@@ -188,22 +187,21 @@ 
      */
     public function saveAmount(int $receivableId, string $amount)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
         }
         $amount = str_replace(',', '.', trim($amount));
-        if($amount !== '' && filter_var($amount, FILTER_VALIDATE_FLOAT) === false)
+        if ($amount !== '' && filter_var($amount, FILTER_VALIDATE_FLOAT) === false)
         {
             $this->notify->error(trans('meeting.errors.amount.invalid', ['amount' => $amount]));
             return $this->response;
         }
-        $amount = $amount === '' ? 0 : $this->localeService->convertMoneyToInt((float)$amount);
+        $amount = $amount === '' ? 0 : $this->localeService->convertMoneyToInt((float) $amount);
 
         $amount > 0 ?
-            $this->depositService->saveDepositAmount($this->pool, $this->session, $receivableId, $amount):
-            $this->depositService->deleteDeposit($this->pool, $this->session, $receivableId);
+            $this->depositService->saveDepositAmount($this->pool, $this->session, $receivableId, $amount) : $this->depositService->deleteDeposit($this->pool, $this->session, $receivableId);
 
         return $this->page();
     }
@@ -215,7 +213,7 @@ 
      */
     public function delDeposit(int $receivableId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -231,12 +229,12 @@ 
      */
     public function addAllDeposits()
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
         }
-        if(!$this->pool->deposit_fixed)
+        if (!$this->pool->deposit_fixed)
         {
             return $this->response;
         }
@@ -251,12 +249,12 @@ 
      */
     public function delAllDeposits()
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
         }
-        if(!$this->pool->deposit_fixed)
+        if (!$this->pool->deposit_fixed)
         {
             return $this->response;
         }

src/Service/Traits/EventTrait.php

Spacing +15 added lines, -15 removed lines

@@ -97,13 +97,13 @@ 
      */
     protected function chargeCreated(Tontine $tontine, Charge $charge)
     {
-        if(!$charge->period_once)
+        if (!$charge->period_once)
         {
             return;
         }
         $today = now();
         // Create a tontine bill for each member
-        foreach($tontine->members()->get() as $member)
+        foreach ($tontine->members()->get() as $member)
         {
             $this->createTontineBill($charge, $member, $today);
         }
@@ -119,7 +119,7 @@ 
     {
         $today = now();
         // Create a tontine bill for each charge
-        foreach($tontine->charges()->active()->once()->get() as $charge)
+        foreach ($tontine->charges()->active()->once()->get() as $charge)
         {
             $this->createTontineBill($charge, $member, $today);
         }
@@ -142,14 +142,14 @@ 
         ])->get();
         $roundCharges = $tontine->charges()->active()->round()->get();
         // Create a round bill for each member
-        foreach($members as $member)
+        foreach ($members as $member)
         {
-            foreach($roundCharges as $charge)
+            foreach ($roundCharges as $charge)
             {
                 $count = $member->round_bills->filter(function($bill) use($charge) {
                     return $bill->charge_id = $charge->id;
                 })->count();
-                if($count === 0)
+                if ($count === 0)
                 {
                     $this->createRoundBill($charge, $member, $round, $today);
                 }
@@ -157,14 +157,14 @@ 
         }
         $tontineCharges = $tontine->charges()->active()->once()->get();
         // Create a tontine bill for each member
-        foreach($members as $member)
+        foreach ($members as $member)
         {
-            foreach($tontineCharges as $charge)
+            foreach ($tontineCharges as $charge)
             {
                 $count = $member->tontine_bills->filter(function($bill) use($charge) {
                     return $bill->charge_id = $charge->id;
                 })->count();
-                if($count === 0)
+                if ($count === 0)
                 {
                     $this->createTontineBill($charge, $member, $today);
                 }
@@ -192,14 +192,14 @@ 
         $sessionCharges = $tontine->charges()->active()->session()->get();
 
         // Sync the session bills for each member and each session charge
-        foreach($sessionCharges as $charge)
+        foreach ($sessionCharges as $charge)
         {
-            foreach($members as $member)
+            foreach ($members as $member)
             {
                 $count = $member->session_bills->filter(function($bill) use($charge) {
                     return $bill->charge_id = $charge->id;
                 })->count();
-                if($count === 0)
+                if ($count === 0)
                 {
                     $this->createSessionBill($charge, $member, $session, $today);
                 }
@@ -207,15 +207,15 @@ 
         };
 
         // Sync the receivables for each subscription on each pool
-        foreach($session->round->pools as $pool)
+        foreach ($session->round->pools as $pool)
         {
-            if($session->enabled($pool))
+            if ($session->enabled($pool))
             {
                 $subscriptions = $pool->subscriptions()
                     ->whereDoesntHave('receivables', function(Builder $query) use($session) {
                         return $query->where('session_id', $session->id);
                     });
-                foreach($subscriptions->get() as $subscription)
+                foreach ($subscriptions->get() as $subscription)
                 {
                     $subscription->receivables()->create(['session_id' => $session->id]);
                 }

src/Service/Meeting/Charge/BillService.php

Spacing +2 added lines, -5 removed lines

@@ -93,9 +93,7 @@ 
     private function getBillRelation(Charge $charge): string
     {
         // The intermediate relation to reach the member model.
-        return $charge->is_variable ? 'libre_bill' :
-            ($charge->period_session ? 'session_bill' :
-            ($charge->period_round ? 'round_bill' : 'tontine_bill'));
+        return $charge->is_variable ? 'libre_bill' : ($charge->period_session ? 'session_bill' : ($charge->period_round ? 'round_bill' : 'tontine_bill'));
     }
 
     /**
@@ -111,8 +109,7 @@ 
         string $search = '', ?bool $onlyPaid = null, int $page = 0): Collection
     {
         $billRelation = $this->getBillRelation($charge);
-        $billRelations = !$charge->is_variable ? [$billRelation . '.member', 'settlement'] :
-            ['libre_bill.session', 'libre_bill.member', 'settlement'];
+        $billRelations = !$charge->is_variable ? [$billRelation . '.member', 'settlement'] : ['libre_bill.session', 'libre_bill.member', 'settlement'];
         $query = $this->getBillsQuery($charge, $session, $billRelation, $search, $onlyPaid);
 
         return $query->with($billRelations)

src/Service/Meeting/SessionService.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
      */
     public function checkPoolsSubscriptions()
     {
-        if($this->tenantService->round()->pools()->whereDoesntHave('subscriptions')->count() > 0)
+        if ($this->tenantService->round()->pools()->whereDoesntHave('subscriptions')->count() > 0)
         {
             throw new MessageException(trans('tontine.errors.action') .
                 '<br/>' . trans('tontine.pool.errors.no_subscription'));
@@ -129,7 +129,7 @@ 
      */
     public function openSession(Session $session)
     {
-        if($session->opened)
+        if ($session->opened)
         {
             return;
         }

app/Ajax/Web/Meeting/Charge/Libre/Settlement.php

Spacing +2 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
      */
     public function addSettlement(int $billId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -135,7 +135,7 @@ 
      */
     public function delSettlement(int $billId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;

app/Ajax/Web/Meeting/Charge/Fixed/Settlement.php

Spacing +4 added lines, -4 removed lines

@@ -122,7 +122,7 @@ 
      */
     public function addSettlement(int $billId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -144,7 +144,7 @@ 
      */
     public function delSettlement(int $billId)
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -164,7 +164,7 @@ 
      */
     public function addAllSettlements()
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;
@@ -184,7 +184,7 @@ 
      */
     public function delAllSettlements()
     {
-        if($this->session->closed)
+        if ($this->session->closed)
         {
             $this->notify->warning(trans('meeting.warnings.session.closed'));
             return $this->response;

app/Ajax/CallableClass.php

Spacing +3 added lines, -3 removed lines

@@ -55,12 +55,12 @@
     protected function pageNumber(int $pageNumber, int $itemCount, string $bagName, string $attrName = 'page'): array
     {
         $perPage = 10;
-        $pageCount = (int)floor($itemCount / $perPage) + ($itemCount % $perPage > 0 ? 1 : 0);
-        if($pageNumber < 1)
+        $pageCount = (int) floor($itemCount / $perPage) + ($itemCount % $perPage > 0 ? 1 : 0);
+        if ($pageNumber < 1)
         {
             $pageNumber = $this->bag($bagName)->get($attrName, 1);
         }
-        if($pageNumber > $pageCount)
+        if ($pageNumber > $pageCount)
         {
             $pageNumber = $pageCount;
         }

src/Service/Meeting/Credit/RefundService.php

Spacing +11 added lines, -11 removed lines

@@ -80,7 +80,7 @@ 
                         $query->where('session_id', $sessionId);
                     });
                 });
-                if($prevSessions->count() === 0)
+                if ($prevSessions->count() === 0)
                 {
                     return;
                 }
@@ -168,11 +168,11 @@ 
         $debt = Debt::whereHas('loan', function(Builder $query) use($sessionIds) {
             $query->whereIn('session_id', $sessionIds);
         })->with(['partial_refunds'])->find($debtId);
-        if(!$debt || $debt->refund)
+        if (!$debt || $debt->refund)
         {
             throw new MessageException(trans('tontine.loan.errors.not_found'));
         }
-        if(!$this->debtCalculator->debtIsEditable($session, $debt))
+        if (!$this->debtCalculator->debtIsEditable($session, $debt))
         {
             throw new MessageException(trans('meeting.refund.errors.cannot_refund'));
         }
@@ -183,7 +183,7 @@ 
         DB::transaction(function() use($session, $debt, $refund) {
             $refund->save();
             // For simple or compound interest, also save the final amount.
-            if($debt->is_interest && !$debt->loan->fixed_interest)
+            if ($debt->is_interest && !$debt->loan->fixed_interest)
             {
                 $debt->amount = $this->debtCalculator->getDebtAmount($session, $debt);
                 $debt->save();
@@ -203,15 +203,15 @@ 
     {
         $refund = Refund::with('debt')->where('session_id', $session->id)
             ->where('debt_id', $debtId)->first();
-        if(!$refund)
+        if (!$refund)
         {
             throw new MessageException(trans('meeting.refund.errors.not_found'));
         }
-        if((!$refund->editable))
+        if ((!$refund->editable))
         {
             throw new MessageException(trans('meeting.refund.errors.cannot_delete'));
         }
-        if(!$this->debtCalculator->debtIsEditable($session, $refund->debt))
+        if (!$this->debtCalculator->debtIsEditable($session, $refund->debt))
         {
             throw new MessageException(trans('meeting.refund.errors.cannot_refund'));
         }
@@ -287,12 +287,12 @@ 
         $debt = Debt::whereHas('loan', function(Builder $query) use($sessionIds) {
             $query->whereIn('session_id', $sessionIds);
         })->find($debtId);
-        if(!$debt || $debt->refund)
+        if (!$debt || $debt->refund)
         {
             throw new MessageException(trans('meeting.refund.errors.not_found'));
         }
         // A partial refund must not totally refund a debt
-        if($amount >= $debt->due_amount)
+        if ($amount >= $debt->due_amount)
         {
             throw new MessageException(trans('meeting.refund.errors.pr_amount'));
         }
@@ -316,11 +316,11 @@ 
     {
         $refund = PartialRefund::where('session_id', $session->id)
             ->with(['debt.refund'])->find($refundId);
-        if(!$refund)
+        if (!$refund)
         {
             throw new MessageException(trans('meeting.refund.errors.not_found'));
         }
-        if($refund->debt->refund !== null || !$refund->editable)
+        if ($refund->debt->refund !== null || !$refund->editable)
         {
             throw new MessageException(trans('meeting.refund.errors.cannot_delete'));
         }