kenmirrors/firefly-iii
1
0
Fork 0
mirror of https://github.com/firefly-iii/firefly-iii.git synced 2025-09-18 18:44:16 +00:00
Code Issues Projects Releases Wiki Activity

Make method nullable.

Browse Source
This commit is contained in:
James Cole
2018-08-07 19:29:10 +02:00
parent 229d033e1a
commit c5051b3e46
4 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/Http/Controllers/Account/ShowController.php
View File

@@ -145,7 +145,7 @@ class ShowController extends Controller
}
$end = new Carbon;
$today = new Carbon;
$start = $this->repository->oldestJournalDate($account);
$start = $this->repository->oldestJournalDate($account) ?? Carbon::now()->startOfMonth();
$subTitleIcon = config('firefly.subIconsByIdentifier.' . $account->accountType->type);
$page = (int)$request->get('page');
$pageSize = (int)app('preferences')->get('listPageSize', 50)->data;
@@ -190,7 +190,7 @@ class ShowController extends Controller
private function getPeriodOverview(Account $account, ?Carbon $date): Collection
{
$range = app('preferences')->get('viewRange', '1M')->data;
$start = $this->repository->oldestJournalDate($account);
$start = $this->repository->oldestJournalDate($account) ?? Carbon::now()->startOfMonth();
$end = $date ?? new Carbon;
if ($end < $start) {
[$start, $end] = [$end, $start]; // @codeCoverageIgnore

6
app/Http/Controllers/Chart/AccountController.php
View File

@@ -166,7 +166,7 @@ class AccountController extends Controller
*/
public function expenseBudgetAll(AccountRepositoryInterface $repository, Account $account): JsonResponse
{
$start = $repository->oldestJournalDate($account);
$start = $repository->oldestJournalDate($account) ?? Carbon::now()->startOfMonth();
$end = Carbon::now();
return $this->expenseBudget($account, $start, $end);
@@ -229,7 +229,7 @@ class AccountController extends Controller
*/
public function expenseCategoryAll(AccountRepositoryInterface $repository, Account $account): JsonResponse
{
$start = $repository->oldestJournalDate($account);
$start = $repository->oldestJournalDate($account) ?? Carbon::now()->startOfMonth();
$end = Carbon::now();
return $this->expenseCategory($account, $start, $end);
@@ -320,7 +320,7 @@ class AccountController extends Controller
*/
public function incomeCategoryAll(AccountRepositoryInterface $repository, Account $account): JsonResponse
{
$start = $repository->oldestJournalDate($account);
$start = $repository->oldestJournalDate($account) ?? Carbon::now()->startOfMonth();
$end = Carbon::now();
return $this->incomeCategory($account, $start, $end);

6
app/Repositories/Account/AccountRepository.php
View File

@@ -410,11 +410,11 @@ class AccountRepository implements AccountRepositoryInterface
*
* @param Account $account
*
* @return Carbon
* @return Carbon|null
*/
public function oldestJournalDate(Account $account): Carbon
public function oldestJournalDate(Account $account): ?Carbon
{
$result = new Carbon;
$result = null;
$journal = $this->oldestJournal($account);
if (null !== $journal) {
$result = $journal->date;

4
app/Repositories/Account/AccountRepositoryInterface.php
View File

@@ -184,9 +184,9 @@ interface AccountRepositoryInterface
*
* @param Account $account
*
* @return Carbon
* @return Carbon|null
*/
public function oldestJournalDate(Account $account): Carbon;
public function oldestJournalDate(Account $account): ?Carbon;
/**
* @param User $user