From 8b4a7d4e3c8beb66aea8c7c4dc94a9c22465f9c0 Mon Sep 17 00:00:00 2001 From: Your Name Date: Mon, 7 Sep 2026 19:33:15 +0930 Subject: [PATCH] fix: correct SiteController view return types, drop ViewContract alias An automated return-type pass aliased the view contract as ViewContract to avoid clashing with the imported View facade, but left four methods (curatedOnboarding, language, redirectUrl, followIntent) typed against the facade instead of the contract. That threw a TypeError on /auth/sign_up. Convert the four View::make() calls to the view() helper, drop the facade import, and use a single Illuminate\Contracts\View\View import for all return types. --- app/Http/Controllers/SiteController.php | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/app/Http/Controllers/SiteController.php b/app/Http/Controllers/SiteController.php index bc09a16a5..1ef10496d 100644 --- a/app/Http/Controllers/SiteController.php +++ b/app/Http/Controllers/SiteController.php @@ -10,11 +10,10 @@ use App\Services\FollowerService; use App\Services\StatusService; use App\Util\ActivityPub\Helpers; use App\Util\Localization\Localization; -use Illuminate\Contracts\View\View as ViewContract; +use Illuminate\Contracts\View\View; use Illuminate\Http\RedirectResponse; use Illuminate\Http\Request; use Illuminate\Support\Facades\Cache; -use Illuminate\Support\Facades\View; use Illuminate\Support\Str; class SiteController extends Controller @@ -30,12 +29,12 @@ class SiteController extends Controller } } - public function homeGuest(): ViewContract + public function homeGuest(): View { return view('site.index'); } - public function homeTimeline(Request $request): RedirectResponse|ViewContract + public function homeTimeline(Request $request): RedirectResponse|View { if ($request->has('force_old_ui')) { return view('timeline.home', ['layout' => 'feed']); @@ -82,7 +81,7 @@ class SiteController extends Controller $slug = '/site/kb/community-guidelines'; $page = Page::whereSlug($slug)->whereActive(true)->first(); - return View::make('site.help.community-guidelines')->with(compact('page'))->render(); + return view('site.help.community-guidelines', compact('page'))->render(); }); } @@ -92,7 +91,7 @@ class SiteController extends Controller return $this->cachedPage('/site/privacy'); }); - return View::make('site.privacy')->with(compact('page'))->render(); + return view('site.privacy', compact('page'))->render(); } public function terms(Request $request) @@ -101,7 +100,7 @@ class SiteController extends Controller return $this->cachedPage('/site/terms'); }); - return View::make('site.terms')->with(compact('page'))->render(); + return view('site.terms', compact('page'))->render(); } public function redirectUrl(Request $request): View @@ -174,7 +173,7 @@ class SiteController extends Controller }); abort_if(! $page, 404); - return View::make('site.legal-notice')->with(compact('page'))->render(); + return view('site.legal-notice', compact('page'))->render(); } public function curatedOnboarding(Request $request): RedirectResponse|View