Почему в __construct перестал определятся аутентифицированный пользователь?

Question

[ettychel]

Сделал приватную переменную, определил в конструкторе значение, сначала работало, потом ни с того ни с сего перестало, может что то не так написал?
Причем если в методе прописываю чему равно, то всё прекрасно работает

class PostController extends BaseController{
    private $auth;
    
    public function __construct()    {
        parent::__construct();

        $this->auth = Auth::user();
}

public function index()    {
        // $this->auth = Auth::user();
        dd($this->auth);

в dd null
снимаю коммент и всё прекрасно, пользователь есть

Answer 1

[ettychel]

Решил таким способом

В контроллере от которого наследуется PostController контроллер добавил

use App\Http\Controllers\Controller;
use Illuminate\Support\Facades\Auth;

abstract class BaseController extends Controller
{
    public function authUser()
    {
        return Auth::user();
    }
}

потом можно вызывать $this->authUser() со всеми вытекающими

Comments

[Alex Wells]

А не проще ли доставать юзера самому? Зачем этот метод вообще?

[ettychel]

Alex Wells, а если он у меня вызывается в нескольких методах? Почему бы 1 раз не прописать?

[Alex Wells]

ettychel, ну и пусть вызывается, вам то что? А еще у вас вызывается Arr::get по всему проекту, или еще какая хрень. Что будете делать?

Если делаете такой метод - называйте authUser(), потому что от user() нихрена не понятно, откуда он берется. А если назвать authUser(), но не проще ли делать auth()->user(), $this->auth->user() или Auth::user()?

[ettychel]

Alex Wells, эмм... мы будем спорить о личных предпочтениях?
Назови как хочешь, кто сказал, что прям только так и никак иначе?
Нравится прописывать каждый раз прописывать, так пожалуйста, я тебе что, руки связал?
А если уже не понятно и ты прям вот не знаешь где искать что оно делает или из контекста не понял что оно делается там, поставь PhpStorm и не парься

[ettychel]

Alex Wells, Тебе так спокойнее будет?

[Alex Wells]

ettychel, в смысле личные предпочтения? Это качество кода. Мне (ровно как и любому отличному от тебя человеку) не понятно, что за юзер такой, а значит приставка auth обязательна.

А если она обязательна, то какая н разница, делать authUser или три варианта, предложенных вверху?

Я не хочу гадать, что оно делает. И не хочу кликать по методам, что бы смотреть, что они делают. Я хочу читаемый код. Это - не читаемо.

PS: добавлю ПС, а то чую что придолбаешься к словам: "я" - это все нормальные люди.

Answer 2

[JhaoDa]

Вангую, что ты обновился, а руководство по обновлению читать не стал. Раздел «Session In The Constructor» — https://laravel.com/docs/5.3/upgrade

Comments

[ettychel]

Не обновлялся, как была 5.8.17(или 14) так и осталась

[JhaoDa]

ettychel, ну как видишь, работать это не должно, а если и работало, то считай это багофичей — делать надо так, как в документации.

[ettychel]

Сделал в точности как в документации, не помогло(

class PostController extends BaseController
{
    protected $auth;

    public function __construct()
    {
        parent::__construct();
        
        $this->middleware(function ($request, $next) {
            $this->auth = Auth::user()->auth;

            return $next($request);
        });
}
 public function index()
    {
        // $this->auth = Auth::user();
        dd($this->auth);

В итоге снова null

[Дмитрий]

ettychel, расхождения в собственном коде не заметил?
$this->auth = Auth::user()->auth;

и

// $this->auth = Auth::user();

Что за Auth::user()->auth такой?

[ettychel]

Дмитрий, то по документации пример
а то что в комменте работает

[JhaoDa]

ettychel, не может быть в документации такого примера, не надо врать.

[Дмитрий]

ettychel, вы же понимаете, что делает инструкция Auth::user()->auth и почему на выходе null?

[ettychel]

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Support\Facades\Auth;
use App\Http\Controllers\Controller;

class ProjectController extends Controller
{
    /**
     * All of the current user's projects.
     */
    protected $projects;

    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware(function ($request, $next) {
            $this->projects = Auth::user()->projects;

            return $next($request);
        });
    }
}

Это документация

[JhaoDa]

ettychel, ясно.

Дмитрий, нет он не понимает.

[ettychel]

Дмитрий, даже если не auth, менял на пример как в документации толку ноль

[ettychel]

JhaoDa, я ж не с потолка это взял, ссылку кстати вы и дали

[JhaoDa]

ettychel, да всё уже, говнокодь дальше, не парься.

[ettychel]

JhaoDa, возможно dd не так, но __construct в точности

[ettychel]

JhaoDa, ну может всё таки подскажешь, что не так? может и перестану

[ettychel]

Дмитрий, JhaoDa, спасибо, догнал наконец то, что это связь
коммент над переменной не прочитал
Ниче, со временем научусь, в начале нужно говнокодить, чтобы потом понимать на сколько ты вырос и почему должно быть так, а не подругому