lagdo / tontine

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/Meeting/Credit/ProfitService.php

Spacing +8 added lines, -8 removed lines

@@ -55,26 +55,26 @@ 
     private function setDistributions(Session $session, Collection $fundings): Collection
     {
         // Set fundings durations and distributions
-        foreach($fundings as $funding)
+        foreach ($fundings as $funding)
         {
             $funding->duration = $this->getFundingDuration($session, $funding);
             $funding->distribution = $funding->amount * $funding->duration;
         }
         // Reduce the distributions
-        $distributionGcd = (int)$fundings->reduce(function($gcd, $funding) {
-            if($gcd === 0)
+        $distributionGcd = (int) $fundings->reduce(function($gcd, $funding) {
+            if ($gcd === 0)
             {
                 return $funding->distribution;
             }
-            if($funding->duration === 0)
+            if ($funding->duration === 0)
             {
                 return $gcd;
             }
             return gmp_gcd($gcd, $funding->distribution);
         }, $fundings->first()->distribution);
-        if($distributionGcd > 0)
+        if ($distributionGcd > 0)
         {
-            foreach($fundings as $funding)
+            foreach ($fundings as $funding)
             {
                 $funding->distribution /= $distributionGcd;
             }
@@ -101,7 +101,7 @@ 
             ->orderBy('members.name', 'asc')
             ->orderBy('sessions.start_at', 'asc')
             ->with(['session', 'member'])->get();
-        if($fundings->count() === 0)
+        if ($fundings->count() === 0)
         {
             return $fundings;
         }
@@ -148,7 +148,7 @@ 
             'amount' => $profitAmount,
         ];
         $round = $this->tenantService->round();
-        if($round->property !== null)
+        if ($round->property !== null)
         {
             $content = $round->property->content;
             $content['profit'] = $profit;

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;

src/Service/Planning/SubscriptionService.php

Spacing +10 added lines, -10 removed lines

@@ -113,13 +113,13 @@ 
     {
         // When the remitments are planned, don't create a subscription
         // if receivables already exist on the pool.
-        if($pool->remit_planned &&
+        if ($pool->remit_planned &&
             $pool->subscriptions()->whereHas('receivables')->count() > 0)
         {
             throw new MessageException(trans('tontine.subscription.errors.create'));
         }
         // Enforce unique subscription per member in pool with libre deposit amount.
-        if(!$pool->deposit_fixed &&
+        if (!$pool->deposit_fixed &&
             $pool->subscriptions()->where('member_id', $memberId)->count() > 0)
         {
             throw new MessageException(trans('tontine.subscription.errors.create'));
@@ -149,18 +149,18 @@ 
     {
         // When the remitments are planned, don't delete a subscription
         // if receivables already exist on the pool.
-        if($pool->remit_planned &&
+        if ($pool->remit_planned &&
             $pool->subscriptions()->whereHas('receivables')->count() > 0)
         {
             throw new MessageException(trans('tontine.subscription.errors.delete'));
         }
         $subscription = $pool->subscriptions()->where('member_id', $memberId)
             ->with(['payable', 'payable.remitment'])->first();
-        if(!$subscription)
+        if (!$subscription)
         {
             throw new MessageException(trans('tontine.subscription.errors.not_found'));
         }
-        if($subscription->payable->remitment !== null)
+        if ($subscription->payable->remitment !== null)
         {
             throw new MessageException(trans('tontine.subscription.errors.delete'));
         }
@@ -204,7 +204,7 @@ 
      */
     public function unsetPayableSession(Subscription $subscription)
     {
-        if(($subscription->payable->session_id))
+        if (($subscription->payable->session_id))
         {
             $subscription->payable->session()->dissociate();
             $subscription->payable->save();
@@ -226,10 +226,10 @@ 
     {
         $session = $this->tenantService->getSession($sessionId);
         $currSubscription = null;
-        if($currSubscriptionId > 0)
+        if ($currSubscriptionId > 0)
         {
             $currSubscription = $pool->subscriptions()->find($currSubscriptionId);
-            if(($currSubscription !== null && $currSubscription->payable !== null &&
+            if (($currSubscription !== null && $currSubscription->payable !== null &&
                 $currSubscription->payable->remitment !== null) || $session->closed)
             {
                 // Can't chage the beneficiary if the session is closed or if
@@ -242,12 +242,12 @@ 
 
         DB::transaction(function() use($session, $currSubscription, $nextSubscription) {
             // If the beneficiary already has a session assigned, first remove it.
-            if($currSubscription !== null)
+            if ($currSubscription !== null)
             {
                 $this->unsetPayableSession($currSubscription);
             }
             // If there is a new session assigned to the beneficiary, then save it.
-            if($nextSubscription !== null)
+            if ($nextSubscription !== null)
             {
                 $this->setPayableSession($nextSubscription, $session);
             }

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