kor-elf/my-projects-website
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 9 Wiki Activity

Replace $request->get() with $request->attributes->get() throughout controllers and middleware for consistency and enhanced reliability.

Browse Source
This commit is contained in:
kor-elf
2026-07-15 22:35:55 +05:00
parent b35f04d220
commit b5258f840e
6 changed files with 20 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/application/app/Http/Controllers/Site/DocumentationController.php
+11 -11
View File
@@ -16,8 +16,8 @@ final class DocumentationController extends Controller
public function defaultVersion(Request $request): RedirectResponse | View public function defaultVersion(Request $request): RedirectResponse | View
{ {
$project = $request->get('project'); $project = $request->attributes->get('project');
$websiteTranslations = $request->get('websiteTranslations'); $websiteTranslations = $request->attributes->get('websiteTranslations');
$result = $this->documentationService->defaultVersion($project, $request->user()); $result = $this->documentationService->defaultVersion($project, $request->user());
if ($result->isError()) { if ($result->isError()) {
@@ -40,9 +40,9 @@ final class DocumentationController extends Controller
public function index(Request $request): View public function index(Request $request): View
{ {
$documentation = new Documentation( $documentation = new Documentation(
project: $request->get('project'), project: $request->attributes->get('project'),
version: $request->get('version'), version: $request->attributes->get('version'),
websiteTranslations: $request->get('websiteTranslations'), websiteTranslations: $request->attributes->get('websiteTranslations'),
user: $request->user(), user: $request->user(),
); );
$result = $this->documentationService->index($documentation); $result = $this->documentationService->index($documentation);
@@ -57,9 +57,9 @@ final class DocumentationController extends Controller
public function category(string $slug, Request $request): View public function category(string $slug, Request $request): View
{ {
$documentation = new Documentation( $documentation = new Documentation(
project: $request->get('project'), project: $request->attributes->get('project'),
version: $request->get('version'), version: $request->attributes->get('version'),
websiteTranslations: $request->get('websiteTranslations'), websiteTranslations: $request->attributes->get('websiteTranslations'),
user: $request->user(), user: $request->user(),
); );
$result = $this->documentationService->category($slug, $documentation); $result = $this->documentationService->category($slug, $documentation);
@@ -77,9 +77,9 @@ final class DocumentationController extends Controller
public function view(string $slug, Request $request): View public function view(string $slug, Request $request): View
{ {
$documentation = new Documentation( $documentation = new Documentation(
project: $request->get('project'), project: $request->attributes->get('project'),
version: $request->get('version'), version: $request->attributes->get('version'),
websiteTranslations: $request->get('websiteTranslations'), websiteTranslations: $request->attributes->get('websiteTranslations'),
user: $request->user(), user: $request->user(),
); );
$result = $this->documentationService->view($slug, $documentation); $result = $this->documentationService->view($slug, $documentation);
app/application/app/Http/Controllers/Site/FeedbackController.php
+4 -4
View File
@@ -17,16 +17,16 @@ final class FeedbackController extends Controller
public function index(Request $request): View public function index(Request $request): View
{ {
return view('site.feedback.index', [ return view('site.feedback.index', [
'project' => $request->get('project'), 'project' => $request->attributes->get('project'),
'websiteTranslations' => $request->get('websiteTranslations'), 'websiteTranslations' => $request->attributes->get('websiteTranslations'),
'captcha' => config('app.captcha', false), 'captcha' => config('app.captcha', false),
]); ]);
} }
public function send(SendRequest $request): RedirectResponse public function send(SendRequest $request): RedirectResponse
{ {
$project = $request->get('project'); $project = $request->attributes->get('project');
$websiteTranslations = $request->get('websiteTranslations'); $websiteTranslations = $request->attributes->get('websiteTranslations');
$user = $request->user(); $user = $request->user();
$data = $request->getDto(); $data = $request->getDto();
app/application/app/Http/Controllers/Site/ProjectsController.php
+2 -2
View File
@@ -15,7 +15,7 @@ final class ProjectsController extends Controller
public function index(Request $request): View public function index(Request $request): View
{ {
$user = $request->user(); $user = $request->user();
$project = $request->get('project'); $project = $request->attributes->get('project');
if (\is_null($project)) { if (\is_null($project)) {
$with = ['storage']; $with = ['storage'];
@@ -27,7 +27,7 @@ final class ProjectsController extends Controller
return \view('site.projects.index', $result->getData()); return \view('site.projects.index', $result->getData());
} }
$websiteTranslations = $request->get('websiteTranslations'); $websiteTranslations = $request->attributes->get('websiteTranslations');
$result = $this->projectService->getAboutByProject($project, $websiteTranslations, $user); $result = $this->projectService->getAboutByProject($project, $websiteTranslations, $user);
if ($result->isError()) { if ($result->isError()) {
$this->errors($result); $this->errors($result);
app/application/app/Http/Middleware/DocumentationVersion.php
+1 -1
View File
@@ -10,7 +10,7 @@ final readonly class DocumentationVersion
{ {
public function handle(Request $request, \Closure $next): Response public function handle(Request $request, \Closure $next): Response
{ {
$project = $request->get('project'); $project = $request->attributes->get('project');
$versionSlug = $request->route()?->parameter('version'); $versionSlug = $request->route()?->parameter('version');
if ($versionSlug === null || $project === null) { if ($versionSlug === null || $project === null) {
\abort(Response::HTTP_NOT_FOUND); \abort(Response::HTTP_NOT_FOUND);
app/application/app/Http/Middleware/IsProject.php
+1 -1
View File
@@ -10,7 +10,7 @@ final class IsProject
{ {
public function handle(Request $request, \Closure $next): Response public function handle(Request $request, \Closure $next): Response
{ {
$project = $request->get('project'); $project = $request->attributes->get('project');
if (\is_null($project)) { if (\is_null($project)) {
\abort(Response::HTTP_NOT_FOUND); \abort(Response::HTTP_NOT_FOUND);
} }
app/application/app/Http/Middleware/IsWebsiteTranslations.php
+1 -1
View File
@@ -10,7 +10,7 @@ final class IsWebsiteTranslations
{ {
public function handle(Request $request, \Closure $next): Response public function handle(Request $request, \Closure $next): Response
{ {
$websiteTranslations = $request->get('websiteTranslations'); $websiteTranslations = $request->attributes->get('websiteTranslations');
if (\is_null($websiteTranslations)) { if (\is_null($websiteTranslations)) {
\abort(Response::HTTP_NOT_FOUND); \abort(Response::HTTP_NOT_FOUND);
} }