Fix variables

This commit is contained in:
James Cole
2026-07-09 06:22:42 +02:00
parent 3b32c61ffe
commit 0524afaf93
77 changed files with 198 additions and 179 deletions
@@ -91,11 +91,11 @@ final class BulkController extends Controller
*/
public function update(BulkEditJournalRequest $request): RedirectResponse
{
$journalIds = $request->get('journals');
$journalIds = $request->input('journals');
$journalIds = is_array($journalIds) ? $journalIds : [];
$ignoreCategory = 1 === (int) $request->get('ignore_category');
$ignoreBudget = 1 === (int) $request->get('ignore_budget');
$tagsAction = $request->get('tags_action');
$ignoreCategory = 1 === (int) $request->input('ignore_category');
$ignoreBudget = 1 === (int) $request->input('ignore_budget');
$tagsAction = $request->input('tags_action');
$collection = new Collection();
$count = 0;
@@ -66,7 +66,7 @@ final class CreateController extends Controller
public function cloneGroup(Request $request): JsonResponse
{
$groupId = (int) $request->get('id');
$groupId = (int) $request->input('id');
if (0 !== $groupId) {
$group = $this->repository->find($groupId);
if ($group instanceof TransactionGroup) {
@@ -81,7 +81,7 @@ final class CreateController extends Controller
session()->flash('success', trans('firefly.stored_journal', ['description' => $title]));
session()->flash('success_url', $link);
if ('edit' === $request->get('redirect')) {
if ('edit' === $request->input('redirect')) {
return response()->json(['redirect' => route('transactions.edit', [$newGroup->id])]);
}
@@ -82,7 +82,7 @@ final class IndexController extends Controller
$subTitleIcon = config('firefly.transactionIconsByType.'.$objectType);
$types = config('firefly.transactionTypesByType.'.$objectType);
$page = (int) $request->get('page');
$page = (int) $request->input('page');
$pageSize = (int) Preferences::get('listPageSize', 50)->data;
if (!$start instanceof Carbon) {
@@ -151,7 +151,8 @@ final class IndexController extends Controller
{
$subTitleIcon = config('firefly.transactionIconsByType.'.$objectType);
$types = config('firefly.transactionTypesByType.'.$objectType);
$page = (int) $request->get('page');
$page = (int) $request->input('page');
$page = min(max(1, $page), 2 ** 16);
$pageSize = (int) Preferences::get('listPageSize', 50)->data;
$path = route('transactions.index.all', [$objectType]);
$first = $this->repository->firstNull();
@@ -140,7 +140,7 @@ final class LinkController extends Controller
*/
public function switchLink(Request $request): RedirectResponse
{
$linkId = (int) $request->get('id');
$linkId = (int) $request->input('id');
$this->repository->switchLinkById($linkId);
return redirect(Steam::getSafePreviousUrl());
@@ -198,7 +198,7 @@ final class MassController extends Controller
private function getDateFromRequest(MassEditJournalRequest $request, int $journalId, string $key): ?Carbon
{
$value = $request->get($key);
$value = $request->input($key);
if (!is_array($value)) {
return null;
}
@@ -220,7 +220,7 @@ final class MassController extends Controller
private function getIntFromRequest(MassEditJournalRequest $request, int $journalId, string $string): ?int
{
$value = $request->get($string);
$value = $request->input($string);
if (!is_array($value)) {
return null;
}
@@ -233,7 +233,7 @@ final class MassController extends Controller
private function getStringFromRequest(MassEditJournalRequest $request, int $journalId, string $string): ?string
{
$value = $request->get($string);
$value = $request->input($string);
if (!is_array($value)) {
return null;
}