Unrelated code cleanup.

This commit is contained in:
James Cole 2017-12-16 19:48:31 +01:00
parent 8176356509
commit 1d58d519a0
No known key found for this signature in database
GPG Key ID: C16961E655E74B5E
19 changed files with 52 additions and 50 deletions

View File

@ -41,7 +41,6 @@ use Navigation;
use Preferences; use Preferences;
use Response; use Response;
use Session; use Session;
use View;
/** /**
* Class ReconcileController. * Class ReconcileController.
@ -147,9 +146,14 @@ class ReconcileController extends Controller
} }
$difference = bcadd(bcadd(bcsub($startBalance, $endBalance), $clearedAmount), $amount); $difference = bcadd(bcadd(bcsub($startBalance, $endBalance), $clearedAmount), $amount);
$diffCompare = bccomp($difference, '0'); $diffCompare = bccomp($difference, '0');
$return = [ $return = [
'post_uri' => $route, 'post_uri' => $route,
'html' => view('accounts.reconcile.overview', compact('account', 'start', 'diffCompare', 'difference', 'end', 'clearedIds', 'transactionIds', 'clearedAmount', 'startBalance', 'endBalance', 'amount', 'route', 'countCleared'))->render(), 'html' => view(
'accounts.reconcile.overview', compact(
'account', 'start', 'diffCompare', 'difference', 'end', 'clearedIds', 'transactionIds', 'clearedAmount', 'startBalance', 'endBalance', 'amount',
'route', 'countCleared'
)
)->render(),
]; ];
return Response::json($return); return Response::json($return);
@ -204,7 +208,11 @@ class ReconcileController extends Controller
$overviewUri = route('accounts.reconcile.overview', [$account->id, '%start%', '%end%']); $overviewUri = route('accounts.reconcile.overview', [$account->id, '%start%', '%end%']);
$indexUri = route('accounts.reconcile', [$account->id, '%start%', '%end%']); $indexUri = route('accounts.reconcile', [$account->id, '%start%', '%end%']);
return view('accounts.reconcile.index', compact('account', 'currency', 'subTitleIcon', 'start', 'end', 'subTitle', 'startBalance', 'endBalance', 'transactionsUri', 'overviewUri', 'indexUri')); return view(
'accounts.reconcile.index', compact(
'account', 'currency', 'subTitleIcon', 'start', 'end', 'subTitle', 'startBalance', 'endBalance', 'transactionsUri', 'overviewUri', 'indexUri'
)
);
} }
/** /**

View File

@ -51,7 +51,7 @@ class LinkController extends Controller
return $next($request); return $next($request);
} }
); );
$this->middleware(IsLimitedUser::class)->except(['index','show']); $this->middleware(IsLimitedUser::class)->except(['index', 'show']);
} }
/** /**

View File

@ -52,7 +52,7 @@ class UserController extends Controller
return $next($request); return $next($request);
} }
); );
$this->middleware(IsLimitedUser::class)->except(['index','show']); $this->middleware(IsLimitedUser::class)->except(['index', 'show']);
} }
/** /**

View File

@ -118,7 +118,7 @@ class LoginController extends Controller
/** /**
* Show the application's login form. * Show the application's login form.
* *
* @param Request $request * @param Request $request
* *
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
*/ */

View File

@ -229,7 +229,7 @@ class BillController extends Controller
public function show(Request $request, BillRepositoryInterface $repository, Bill $bill) public function show(Request $request, BillRepositoryInterface $repository, Bill $bill)
{ {
/** @var Carbon $date */ /** @var Carbon $date */
$date = session('start'); $date = session('start');
/** @var Carbon $end */ /** @var Carbon $end */
$end = session('end'); $end = session('end');
$year = $date->year; $year = $date->year;

View File

@ -85,7 +85,7 @@ class BudgetController extends Controller
$start = Carbon::createFromFormat('Y-m-d', $request->get('start')); $start = Carbon::createFromFormat('Y-m-d', $request->get('start'));
$end = Carbon::createFromFormat('Y-m-d', $request->get('end')); $end = Carbon::createFromFormat('Y-m-d', $request->get('end'));
$budgetLimit = $this->repository->updateLimitAmount($budget, $start, $end, $amount); $budgetLimit = $this->repository->updateLimitAmount($budget, $start, $end, $amount);
if (bccomp($amount,'0') === 0) { if (bccomp($amount, '0') === 0) {
$budgetLimit = null; $budgetLimit = null;
} }

View File

@ -227,7 +227,7 @@ class CategoryController extends Controller
$start = null; $start = null;
$end = null; $end = null;
$periods = new Collection; $periods = new Collection;
$path = route('categories.show', [$category->id]); $path = route('categories.show', [$category->id]);
// prep for "all" view. // prep for "all" view.
if ('all' === $moment) { if ('all' === $moment) {
@ -236,7 +236,7 @@ class CategoryController extends Controller
/** @var Carbon $start */ /** @var Carbon $start */
$start = null === $first ? new Carbon : $first; $start = null === $first ? new Carbon : $first;
$end = new Carbon; $end = new Carbon;
$path = route('categories.show', [$category->id,'all']); $path = route('categories.show', [$category->id, 'all']);
} }
// prep for "specific date" view. // prep for "specific date" view.
@ -249,7 +249,7 @@ class CategoryController extends Controller
'start' => $start->formatLocalized($this->monthAndDayFormat), 'end' => $end->formatLocalized($this->monthAndDayFormat),] 'start' => $start->formatLocalized($this->monthAndDayFormat), 'end' => $end->formatLocalized($this->monthAndDayFormat),]
); );
$periods = $this->getPeriodOverview($category); $periods = $this->getPeriodOverview($category);
$path = route('categories.show', [$category->id, $moment]); $path = route('categories.show', [$category->id, $moment]);
} }
// prep for current period // prep for current period

View File

@ -26,9 +26,6 @@ use FireflyIII\Exceptions\FireflyException;
use FireflyIII\Http\Controllers\Controller; use FireflyIII\Http\Controllers\Controller;
use FireflyIII\Repositories\ImportJob\ImportJobRepositoryInterface; use FireflyIII\Repositories\ImportJob\ImportJobRepositoryInterface;
use FireflyIII\Support\Import\Prerequisites\PrerequisitesInterface; use FireflyIII\Support\Import\Prerequisites\PrerequisitesInterface;
use Illuminate\Http\Request;
use Log;
use Session;
class BankController extends Controller class BankController extends Controller
{ {
@ -61,5 +58,4 @@ class BankController extends Controller
} }
} }

View File

@ -82,7 +82,7 @@ class ConfigurationController extends Controller
if ($configurator->isJobConfigured()) { if ($configurator->isJobConfigured()) {
return redirect(route('import.status', [$job->key])); return redirect(route('import.status', [$job->key]));
} }
$data = $request->all(); $data = $request->all();
$configurator->configureJob($data); $configurator->configureJob($data);
// get possible warning from configurator: // get possible warning from configurator:

View File

@ -29,7 +29,6 @@ use FireflyIII\Import\Configurator\ConfiguratorInterface;
use FireflyIII\Import\Routine\ImportRoutine; use FireflyIII\Import\Routine\ImportRoutine;
use FireflyIII\Models\ImportJob; use FireflyIII\Models\ImportJob;
use FireflyIII\Repositories\ImportJob\ImportJobRepositoryInterface; use FireflyIII\Repositories\ImportJob\ImportJobRepositoryInterface;
use FireflyIII\Repositories\Tag\TagRepositoryInterface;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Response as LaravelResponse; use Illuminate\Http\Response as LaravelResponse;
use Log; use Log;
@ -176,7 +175,6 @@ class FileController extends Controller
} }
/** /**
* Step 4. Save the configuration. * Step 4. Save the configuration.
* *

View File

@ -31,6 +31,7 @@ class StatusController extends Controller
} }
); );
} }
/** /**
* @param ImportJob $job * @param ImportJob $job
* *
@ -89,6 +90,7 @@ class StatusController extends Controller
$result['started'] = true; $result['started'] = true;
$result['running'] = true; $result['running'] = true;
} }
// TODO cannot handle 'errored' // TODO cannot handle 'errored'
return Response::json($result); return Response::json($result);

View File

@ -62,7 +62,7 @@ class ProfileController extends Controller
return $next($request); return $next($request);
} }
); );
$this->middleware(IsLimitedUser::class)->except(['confirmEmailChange','index', 'undoEmailChange']); $this->middleware(IsLimitedUser::class)->except(['confirmEmailChange', 'index', 'undoEmailChange']);
} }
/** /**

View File

@ -246,7 +246,6 @@ class ExpenseController extends Controller
} }
public function topIncome(Collection $accounts, Collection $expense, Carbon $start, Carbon $end) public function topIncome(Collection $accounts, Collection $expense, Carbon $start, Carbon $end)
{ {
// Properties for cache: // Properties for cache:

View File

@ -88,8 +88,9 @@ class ReportController extends Controller
View::share( View::share(
'subTitle', trans( 'subTitle', trans(
'firefly.report_default', ['start' => $start->formatLocalized($this->monthFormat), 'end' => $end->formatLocalized($this->monthFormat),] 'firefly.report_default',
) ['start' => $start->formatLocalized($this->monthFormat), 'end' => $end->formatLocalized($this->monthFormat),]
)
); );
$generator = ReportGeneratorFactory::reportGenerator('Account', $start, $end); $generator = ReportGeneratorFactory::reportGenerator('Account', $start, $end);

View File

@ -32,7 +32,6 @@ use Log;
use Preferences; use Preferences;
use Session; use Session;
use URL; use URL;
use View;
/** /**
* Class LinkController. * Class LinkController.

View File

@ -22,7 +22,6 @@ declare(strict_types=1);
namespace FireflyIII\Http\Middleware; namespace FireflyIII\Http\Middleware;
use Closure;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Session\Middleware\StartSession; use Illuminate\Session\Middleware\StartSession;

View File

@ -22,8 +22,6 @@ declare(strict_types=1);
namespace FireflyIII\Http\Requests; namespace FireflyIII\Http\Requests;
use Carbon\Carbon;
/** /**
* Class PiggyBankFormRequest. * Class PiggyBankFormRequest.
*/ */

View File

@ -45,28 +45,6 @@ class ReportFormRequest extends Request
return auth()->check(); return auth()->check();
} }
/**
* @return Collection
*/
public function getExpenseList(): Collection
{
// fixed
/** @var AccountRepositoryInterface $repository */
$repository = app(AccountRepositoryInterface::class);
$set = $this->get('exp_rev');
$collection = new Collection;
if (is_array($set)) {
foreach ($set as $accountId) {
$account = $repository->find(intval($accountId));
if (null !== $account->id) {
$collection->push($account);
}
}
}
return $collection;
}
/** /**
* @return Collection * @return Collection
*/ */
@ -152,6 +130,28 @@ class ReportFormRequest extends Request
return $date; return $date;
} }
/**
* @return Collection
*/
public function getExpenseList(): Collection
{
// fixed
/** @var AccountRepositoryInterface $repository */
$repository = app(AccountRepositoryInterface::class);
$set = $this->get('exp_rev');
$collection = new Collection;
if (is_array($set)) {
foreach ($set as $accountId) {
$account = $repository->find(intval($accountId));
if (null !== $account->id) {
$collection->push($account);
}
}
}
return $collection;
}
/** /**
* @return Carbon * @return Carbon
* *

View File

@ -882,7 +882,9 @@ Breadcrumbs::register(
'accounts.reconcile.edit', 'accounts.reconcile.edit',
function (BreadCrumbGenerator $breadcrumbs, TransactionJournal $journal) { function (BreadCrumbGenerator $breadcrumbs, TransactionJournal $journal) {
$breadcrumbs->parent('transactions.show', $journal); $breadcrumbs->parent('transactions.show', $journal);
$breadcrumbs->push(trans('breadcrumbs.edit_reconciliation', ['description' => $journal->description]), route('accounts.reconcile.edit', [$journal->id])); $breadcrumbs->push(
trans('breadcrumbs.edit_reconciliation', ['description' => $journal->description]), route('accounts.reconcile.edit', [$journal->id])
);
} }
); );