[LR] Zmienna (obiekt) globalny Opcje

[markonix]

W aplikacji oprócz standardowej sesji użytkownika jest tez wybór odpowiedniego zasobu (coś jak w gmail można się przełączać między subkontami).
Zmienna $activeAccess jest mi potrzebna w wielu miejscach, jest to obiekt zawierający istotne szczegóły.

Generalnie to działa, korzystam ze sposobu znanego mi z CI - korzystam z BaseControllera po którym dziedziczą kontrolery.
I to fajnie działa, w tym kontrolerze mogę też zmienną udostępnić do widoków za pomocą share().

Jest tylko z tym podejściem jeden problem - widoczność zaczyna się i kończy na kontrolerach. Ta zmienna przydałaby się mi się w innych np. w Requests, Middleware'ach, a także w Policies (choć tu ją mogę przekazywać w metodzie za pomocą helpera przykładowo $this->authorize('view', [$resource, $this->activeAccess]);. Teraz np. przy walidacji muszę kombinować i przekazywać to za pomocą ukrytych pól. Wydawało by się, że sesja była dobrym rozwiązaniem ale nie widzę tego, nie lubię takich rzeczy tam pchać, zwłaszcza całych obiektów.

Jakieś sugestie na lepsze umiejscowienie?

[Pyton_000]

A czemu nie DI?

[markonix]

Cóż, nie myślałem o tym zupełnie, DI raczej mi się kojarzy z wstrzykiwaniem potrzebnych klas do działania konkretnego fragmentu kodu np. w kontrolerze ładujemy repozytorium.
Nie myślałem o tym activeAccess jako o takie klasie, która jest jakim takim "state" aplikacji (czyli inaczej zmienną sesyjną).

Ale z ciekawości spróbuje.

No nie powiem, chyba to działa. Bałem się, że w takiej wstrzykniętej klasie nie będę miał dostępu do tego czego potrzebuje (sesji, zalogowanego użytkownika itp.) ale nie ma z tym problemu.
Plusem jest, że rozdzieliłem troszkę logikę ustalania tego aktywnego dostępu (bo to się opiera o kilka rzeczy - sesje, ciacho, db + jakaś wstępna autoryzacja) do tej osobnej klasy.

Ostatecznie to wygląda tak:

To sobie zostawiłem w moim bazowym kontrolerze aby zachować kompatybilność z obecnym kodem:
<!--Geshi:480696:php--><pre class="php-brief" style="font-family:monospace;"><div class="head">

[PHP] pobierz, plaintext 
 [topic=0]ype</span><span class="sy0">;</span></div><li class="li1"><div class="de1">    <span class="br0">}</span></div><li class="li2"><div class="de2"> </div><li class="li1"><div class="de1">    <span class="coMULTI">/**</span></div><li class="li2"><div class="de2"><span class="coMULTI">     * @return int</span></div><li class="li1"><div class="de1"><span class="coMULTI">     */</span></div><li class="li2"><div class="de2">    <span class="kw2">public</span> <span class="kw2">function</span> getResourceId<span class="br0">(</span><span class="br0">)</span></div><li class="li1"><div class="de1">    <span class="br0">{</span></div><li class="li2"><div class="de2">        <span class="kw1">return</span> <span class="co1">// tu id w zależności od typu zasobu;</span></div><li class="li1"><div class="de1">    <span class="br0">}</span></div><li class="li2"><div class="de2"> </div><li class="li1"><div class="de1">    <span class="coMULTI">/**</span></div><li class="li2"><div class="de2"><span class="coMULTI">     * @return string</span></div><li class="li1"><div class="de1"><span class="coMULTI">     */</span></div><li class="li2"><div class="de2">    <span class="kw2">public</span> <span class="kw2">function</span> getAccessId<span class="br0">(</span><span class="br0">)</span></div><li class="li1"><div class="de1">    <span class="br0">{</span></div><li class="li2"><div class="de2">        <span class="kw1">return</span> <span class="re0">$this</span><span class="sy0">-></span><span class="me1">accessObject</span><span class="sy0">-></span><span class="me1">access_id</span><span class="sy0">;</span></div><li class="li1"><div class="de1">    <span class="br0">}</span></div><li class="li2"><div class="de2"> </div><li class="li1"><div class="de1">    <span class="coMULTI">/**</span></div><li class="li2"><div class="de2"><span class="coMULTI">     * @return string</span></div><li class="li1"><div class="de1"><span class="coMULTI">     */</span></div><li class="li2"><div class="de2">    <span class="kw2">public</span> <span class="kw2">function</span> getAccessCode<span class="br0">(</span><span class="br0">)</span></div><li class="li1"><div class="de1">    <span class="br0">{</span></div><li class="li2"><div class="de2">        <span class="kw1">return</span> <span class="re0">$this</span><span class="sy0">-></span><span class="me1">accessObject</span><span class="sy0">-></span><span class="me1">access_code</span><span class="sy0">;</span></div><li class="li1"><div class="de1">    <span class="br0">}</span></div><li class="li2"><div class="de2"> </div><li class="li1"><div class="de1"><span class="br0">}</span></div>
[/list]<div class="foot">[PHP] [url="./Pobierz-Plik-480697.html"]plaintext[/url] </div></pre><!--/Geshi:480697:php-->
 
no i magiczne DI 
 
[php]    /**
     * Get the validation rules that apply to the request.
     *
     * @param ActiveAccess $activeAccess
     * @return array
     */
    public function rules(ActiveAccess $activeAccess)
    {
        $activeAccess->getResourceId();
        // i można użyć bieżącego ID
    }
[PHP] pobierz, plaintext 

Minus jak widzę to póki co podwójny narzut na to samo - tzn. wykonanie zapytań które są w ActiveAccess. Nie wiem czy Singleton by mógł tu pomóc
Z Singletonem w LR jakoś nie idzie:

[PHP] pobierz, plaintext 
        $this->app->singleton('Session\ActiveAccess', function ($app) {
            return new ActiveAccess();
        });
[PHP] pobierz, plaintext 

Kod się generalnie nie wykonuje gdy wstrzykuję klasę ActiveAccess :/

edit: Masakra z tym forum, rozpierdzieliło cały kod... Nie mam ochoty tego 3 raz już poprawiać..

Ten post edytował markonix 28.11.2017, 17:47:03

[Pyton_000]

Ale co się nie wykonuje. Ustaliłes jako singleton swoją klasę i ją zainicjowałeś. Więc kod powinien się 1x wykonać

[markonix]

[PHP] pobierz, plaintext 
$this->app->singleton('Session\ActiveAccess', function ($app) {
    dd('tutaj');
    return new ActiveAccess();
});
[PHP] pobierz, plaintext 

Tzn. nie działa ten "hook". Być, może ten kod daje w złym miejscu, próbuje w AppServiceProvider.

Zrobiłem sobie dump w konstruktorze tej nowej klasy dump(rand(0,999)) i póki co wykonuje się 2 razy - w tym moim bazowym kontrolerze i przy walidacji requestu (no i będzie się wykonywał w innych miejscach, gdziekolwiek to dodam przy obsłudze requestu więc chciałbym to zoptymalizować).

Chyba mam:

[PHP] pobierz, plaintext 
$this->app->singleton(ActiveAccess::class, ActiveAccess::class);
[PHP] pobierz, plaintext 

Ten post edytował markonix 28.11.2017, 18:08:19

[Pyton_000]

Zrób tak:

[PHP] pobierz, plaintext 
$this->app->singleton(ActiveSession::class, function() {
            return new ActiveSession();
        });
[PHP] pobierz, plaintext 

importując namespace

PS. Możesz też i tak:

Kod

$this->app->singleton(ActiveSession::class);

Ten post edytował Pyton_000 28.11.2017, 18:31:38

[markonix]

Moje krótsze ale generalnie o to mniej więcej się rozchodziło

Dobra, wygrałeś. Twój edit jest jeszcze krótszy i też działa

Ten post edytował markonix 28.11.2017, 18:32:54