Change to ternary

tighten · Sep 27, 2024 · f2174ac · f2174ac
1 parent 7ab5d6e
commit f2174ac
Showing 1 changed file with 36 additions and 40 deletions.
diff --git a/app/Http/Controllers/PreferenceController.php b/app/Http/Controllers/PreferenceController.php
@@ -10,55 +10,51 @@
 
 class PreferenceController extends Controller
 {
-    public function index(): View
-    {
-        return view('preferences', [
-            'currentResourceLanguagePreference' => Preferences::get('resource-language'),
-            'resourceLanguagePreferences' => (new ResourceLanguagePreference)->options(),
-            'preferredLocale' => Preferences::get('locale'),
-        ]);
-    }
+	public function index(): View
+	{
+		return view('preferences', [
+			'currentResourceLanguagePreference' => Preferences::get('resource-language'),
+			'resourceLanguagePreferences' => (new ResourceLanguagePreference)->options(),
+			'preferredLocale' => Preferences::get('locale'),
+		]);
+	}
 
-    public function update(Request $request)
-    {
-        Preferences::set(
-            collect($request->only(Preferences::getValidKeys()))->filter()
-        );
+	public function update(Request $request)
+	{
+		Preferences::set(
+			collect($request->only(Preferences::getValidKeys()))->filter()
+		);
 
-        $user = auth()->user();
+		$user = auth()->user();
 
-        if ($user) {
-            if ($request->filled('track')) {
-                $user->track_id = $request->input('track');
-            }
+		if ($user) {
+			if ($request->filled('track')) {
+				$user->track_id = $request->input('track');
+			}
 
-            $request->validate([
-                'is_subscriber' => 'nullable|boolean',
-            ]);
+			$request->validate([
+				'is_subscriber' => 'nullable|boolean',
+			]);
 
-            $user->is_subscriber = $request->has('digest-subscriber');
+			$user->is_subscriber = $request->has('digest-subscriber');
 
-            if ($user->is_subscriber) {
-                $user->unsubscribe_token = Str::random(60);
-            } else {
-                $user->unsubscribe_token = null;
-            $user->unsubscribe_token = $user->is_subscriber ? Str::random(60) : null;
+			$user->unsubscribe_token = $user->is_subscriber ? Str::random(60) : null;
 
-            $user->save();
-        }
+			$user->save();
+		}
 
-        session()->flash('toast', 'Your preferences were saved.');
+		session()->flash('toast', 'Your preferences were saved.');
 
-        if ($request->wantsJson()) {
-            return response()->json(['status' => 'success']);
-        }
+		if ($request->wantsJson()) {
+			return response()->json(['status' => 'success']);
+		}
 
-        if ($request->input('locale') !== locale()) {
-            return redirect(
-                str_replace('/' . locale() . '/', '/' . $request->input('locale') . '/', back()->getTargetUrl())
-            );
-        }
+		if ($request->input('locale') !== locale()) {
+			return redirect(
+				str_replace('/' . locale() . '/', '/' . $request->input('locale') . '/', back()->getTargetUrl())
+			);
+		}
 
-        return back();
-    }
+		return back();
+	}
 }