Clean up some code.

app/Http/Controllers/Chart/BudgetController.php

@@ -310,7 +310,7 @@ class BudgetController extends Controller
         $cache->addProperty('budget');
         $cache->addProperty('year');
         if ($cache->has()) {
-            //return Response::json($cache->get()); // @codeCoverageIgnore
+            return Response::json($cache->get()); // @codeCoverageIgnore
         }
 
         $budgetInformation = $repository->getBudgetsAndExpensesPerMonth($accounts, $start, $end);
@@ -328,9 +328,6 @@ class BudgetController extends Controller
             $row           = [clone $start];
             $dateFormatted = $start->format('Y-m');
 
-            //echo $start,' '.$month.'<br>';
-
-
             // each budget, check if there is an entry for this month:
             /** @var array $row */
             foreach ($budgetInformation as $budgetRow) {

app/Validation/FireflyValidator.php

@@ -54,7 +54,7 @@ class FireflyValidator extends Validator
      *
      * @return bool
      */
-    public function validate2faCode($attribute, $value, $parameters): bool
+    public function validate2faCode($attribute, $value): bool
     {
         if (!is_string($value) || is_null($value) || strlen($value) <> 6) {
             return false;

tests/TestCase.php

@@ -36,8 +36,6 @@ class TestCase extends Illuminate\Foundation\Testing\TestCase
             // set period to match?
 
         }
-        // custom is a weird range
-        // (20 days):
         if ($range === 'custom') {
             $this->session(
                 [