Skip to content

Commit 5faeac5

Browse files
committed
Clean code
1 parent bce472d commit 5faeac5

File tree

6 files changed

+18
-20
lines changed

6 files changed

+18
-20
lines changed

app/Http/Controllers/API/Auth/LoginController.php

+4-5
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
use App\Http\Requests\AuthRequest;
88
use App\Http\Traits\HttpResponses;
99
use Illuminate\Http\JsonResponse;
10-
use Illuminate\Support\Facades\Auth;
1110

1211
class LoginController extends Controller
1312
{
@@ -19,8 +18,8 @@ class LoginController extends Controller
1918
*/
2019
public function login(AuthRequest $request): JsonResponse
2120
{
22-
if (Auth::attempt($request->all())) {
23-
$user = Auth::user();
21+
if (auth()->attempt($request->all())) {
22+
$user = auth()->user();
2423

2524
$user->tokens()->delete();
2625

@@ -37,7 +36,7 @@ public function login(AuthRequest $request): JsonResponse
3736
*/
3837
public function logout(): JsonResponse
3938
{
40-
$user = Auth::user();
39+
$user = auth()->user();
4140

4241
$user->tokens()->delete();
4342

@@ -49,7 +48,7 @@ public function logout(): JsonResponse
4948
*/
5049
public function details(): JsonResponse
5150
{
52-
$user = Auth::user();
51+
$user = auth()->user();
5352

5453
return $this->success($user, '');
5554
}

app/Http/Middleware/ApiAuthenticate.php

+5-5
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22

33
namespace App\Http\Middleware;
44

5-
use Closure;
6-
use Illuminate\Http\Request;
7-
use Symfony\Component\HttpFoundation\Response;
85
use App\Constants\AuthConstants;
96
use App\Http\Controllers\Controller;
107
use App\Http\Traits\HttpResponses;
11-
use Illuminate\Support\Facades\Auth;
8+
use Closure;
9+
use Illuminate\Http\Request;
10+
use Symfony\Component\HttpFoundation\Response;
1211

1312
class ApiAuthenticate extends Controller
1413
{
@@ -22,7 +21,8 @@ class ApiAuthenticate extends Controller
2221
public function handle(Request $request, Closure $next): Response
2322
{
2423
if ($user = auth('sanctum')->user()) {
25-
Auth::login($user);
24+
auth()->login($user);
25+
2626
return $next($request);
2727
}
2828

app/Http/Middleware/RedirectIfAuthenticated.php

+5-3
Original file line numberDiff line numberDiff line change
@@ -5,22 +5,24 @@
55
use App\Providers\RouteServiceProvider;
66
use Closure;
77
use Illuminate\Http\Request;
8-
use Illuminate\Support\Facades\Auth;
98
use Symfony\Component\HttpFoundation\Response;
109

1110
class RedirectIfAuthenticated
1211
{
1312
/**
1413
* Handle an incoming request.
1514
*
16-
* @param \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response) $next
15+
* @param Request $request
16+
* @param Closure(Request): (Response) $next
17+
* @param string ...$guards
18+
* @return Response
1719
*/
1820
public function handle(Request $request, Closure $next, string ...$guards): Response
1921
{
2022
$guards = empty($guards) ? [null] : $guards;
2123

2224
foreach ($guards as $guard) {
23-
if (Auth::guard($guard)->check()) {
25+
if (auth()->guard($guard)->check()) {
2426
return redirect(RouteServiceProvider::HOME);
2527
}
2628
}

app/Http/Traits/Access.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
namespace App\Http\Traits;
44

55
use Illuminate\Database\Eloquent\Model;
6-
use Illuminate\Support\Facades\Auth;
76

87
trait Access
98
{
@@ -13,6 +12,6 @@ trait Access
1312
*/
1413
protected function canAccess(Model $model): bool
1514
{
16-
return !($model->user_id !== Auth::id());
15+
return !($model->user_id !== auth()->id());
1716
}
1817
}

app/Models/Category.php

+2-3
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
use Illuminate\Database\Eloquent\Model;
99
use Illuminate\Database\Eloquent\Relations\BelongsTo;
1010
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
11-
use Illuminate\Support\Facades\Auth;
1211

1312
class Category extends Model
1413
{
@@ -43,7 +42,7 @@ public function user(): BelongsTo
4342
*/
4443
public function scopeForUser(Builder $query): Builder
4544
{
46-
return $query->where('user_id', Auth::id());
45+
return $query->where('user_id', auth()->id());
4746
}
4847

4948
/**
@@ -53,6 +52,6 @@ public function scopeForUser(Builder $query): Builder
5352
*/
5453
public function scopeForUserByIds(Builder $query, array $ids): Collection
5554
{
56-
return $query->find($ids)->where('user_id', Auth::id());
55+
return $query->find($ids)->where('user_id', auth()->id());
5756
}
5857
}

app/Models/Product.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
use Illuminate\Database\Eloquent\Factories\HasFactory;
77
use Illuminate\Database\Eloquent\Model;
88
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
9-
use Illuminate\Support\Facades\Auth;
109

1110
class Product extends Model
1211
{
@@ -34,6 +33,6 @@ public function categories(): BelongsToMany
3433
*/
3534
public function scopeForUser(Builder $query): Builder
3635
{
37-
return $query->where('user_id', Auth::id());
36+
return $query->where('user_id', auth()->id());
3837
}
3938
}

0 commit comments

Comments
 (0)