Minor fixes

This commit is contained in:
David Bomba 2021-09-21 22:45:28 +10:00
parent 5837d544ea
commit cdd6bee046
2 changed files with 15 additions and 19 deletions

View File

@ -279,7 +279,7 @@ class Statement
private function getAgingAmount($range) private function getAgingAmount($range)
{ {
$ranges = $this->calculateDateRanges($range); $ranges = $this->calculateDateRanges($range);
nlog($ranges);
$from = $ranges[0]; $from = $ranges[0];
$to = $ranges[1]; $to = $ranges[1];
@ -287,8 +287,10 @@ class Statement
$amount = Invoice::where('company_id', $this->client->company->id) $amount = Invoice::where('company_id', $this->client->company->id)
->where('client_id', $client->id) ->where('client_id', $client->id)
->where('company_id', $this->client->company_id)
->whereIn('status_id', [Invoice::STATUS_SENT, Invoice::STATUS_PARTIAL]) ->whereIn('status_id', [Invoice::STATUS_SENT, Invoice::STATUS_PARTIAL])
->where('balance', '>', 0) ->where('balance', '>', 0)
->where('is_deleted', 0)
->whereBetween('date', [$from, $to]) ->whereBetween('date', [$from, $to])
->sum('balance'); ->sum('balance');
@ -307,35 +309,29 @@ class Statement
switch ($range) { switch ($range) {
case '30': case '30':
$ranges[0] = now(); $ranges[0] = now()->startOfDay();
$ranges[1] = now()->subDays(30); $ranges[1] = now()->startOfDay()->subDays(30);
return $ranges; return $ranges;
break;
case '60': case '60':
$ranges[0] = now()->subDays(30); $ranges[0] = now()->startOfDay()->subDays(30);
$ranges[1] = now()->subDays(60); $ranges[1] = now()->startOfDay()->subDays(60);
return $ranges; return $ranges;
break;
case '90': case '90':
$ranges[0] = now()->subDays(60); $ranges[0] = now()->startOfDay()->subDays(60);
$ranges[1] = now()->subDays(90); $ranges[1] = now()->startOfDay()->subDays(90);
return $ranges; return $ranges;
break;
case '120': case '120':
$ranges[0] = now()->subDays(90); $ranges[0] = now()->startOfDay()->subDays(90);
$ranges[1] = now()->subDays(120); $ranges[1] = now()->startOfDay()->subDays(120);
return $ranges; return $ranges;
break;
case '120+': case '120+':
$ranges[0] = now()->subDays(120); $ranges[0] = now()->startOfDay()->subDays(120);
$ranges[1] = now()->subYears(40); $ranges[1] = now()->startOfDay()->subYears(40);
return $ranges; return $ranges;
break;
default: default:
$ranges[0] = now()->subDays(0); $ranges[0] = now()->startOfDay()->subDays(0);
$ranges[1] = now()->subDays(30); $ranges[1] = now()->subDays(30);
return $ranges; return $ranges;
break;
} }
} }

View File

@ -428,7 +428,7 @@ class Design extends BaseDesign
public function statementPaymentTableTotals(): array public function statementPaymentTableTotals(): array
{ {
if (!$this->payments->first() || $this->type !== self::STATEMENT) { if (is_null($this->payments) || !$this->payments->first() || $this->type !== self::STATEMENT) {
return []; return [];
} }