From ad8a8ae6123146e9318118d4cc3ecc26e4ff876a Mon Sep 17 00:00:00 2001 From: James Cole Date: Sat, 2 Oct 2021 18:43:30 +0200 Subject: [PATCH] Catch value error. --- .../Controllers/Budget/AvailableBudgetController.php | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/app/Http/Controllers/Budget/AvailableBudgetController.php b/app/Http/Controllers/Budget/AvailableBudgetController.php index b2f82e9036..85c8c1b447 100644 --- a/app/Http/Controllers/Budget/AvailableBudgetController.php +++ b/app/Http/Controllers/Budget/AvailableBudgetController.php @@ -36,6 +36,7 @@ use Illuminate\Http\Request; use Illuminate\Routing\Redirector; use Illuminate\View\View; use Log; +use ValueError; /** * @@ -244,7 +245,14 @@ class AvailableBudgetController extends Controller return redirect(route('budgets.index', [$start->format('Y-m-d'), $end->format('Y-m-d')])); } - if (bccomp($amount, '0') <= 0) { + try { + if (bccomp($amount, '0') <= 0) { + session()->flash('error', trans('firefly.invalid_amount')); + + return redirect(route('budgets.index', [$start->format('Y-m-d'), $end->format('Y-m-d')])); + } + } catch (ValueError $e) { + Log::error(sprintf('Value "%s" is not a number: %s', $amount, $e->getMessage())); session()->flash('error', trans('firefly.invalid_amount')); return redirect(route('budgets.index', [$start->format('Y-m-d'), $end->format('Y-m-d')]));