[inny]Laravel problem z rolami użytkownika, nie bierze pod uwagę warunku Opcje

[casperii]

Panowie, przejdę może od razu do rzeczy, mam kod:

Role.php :

[PHP] pobierz, plaintext 
<?php
 
namespace App;
 
use Illuminate\Database\Eloquent\Model;
 
class Role extends Model
{
 
	public $guarded = []; 
    public $timestamps = false;
 
    public function users()
    {
        return $this->belongsToMany('App\User');
    }
}
[PHP] pobierz, plaintext 

oraz RegisterController.php

[PHP] pobierz, plaintext 
if(!Role::where('name','type')->exists())
        {
            Role::create(['name'=>'person']);
            Role::create(['name'=>'firm']);
        }
 
        //sprawdzamy czy typ = 0 czyli równy 1 - jeżeli tak dodaje Role firm w innym przypadku dodaje person w tabeli roles
 
        if($data['type']  0)
            $user->roles()->attach( Role::where('name','firm')->first()->id );
        else
            $user->roles()->attach( Role::where('name','person')->first()->id );
 
[PHP] pobierz, plaintext 

czyli przy każdej rejestracji użytkownika dodaje także do tabeli role_user typ konta (firma / os. fiz) , oraz dodatkowo mam stworzony warunek , który nie działa zgodnie z jego logiką, tzn. sprawdzam czy istnieje w tabeli roles jakieś pola , jeżeli nie to dopisuje 2 rekordy person oraz firm. Problem w tym, że warunek ten działa zawsze bez znaczenia czy ów wartości znajdują się już w tabeli czy też nie.

[SmokAnalog]

Jakiś dziwny ten Twój kod. Sprawdzasz, czy istnieje rola z kolumną *name* o wartości *type*? Tak po prostu, bez względu na przypisania do użytkownika?

Po drugie nie pokazałeś tego $data i nie powiedziałeś o którym warunku mówisz. Napisz jaśniej i pokaż więcej kodu.

Poza tym, firma to po angielsku company, a nie firm

Ten post edytował SmokAnalog 27.09.2020, 16:46:32

[casperii]

@SmokAnalog odnośnie twojej uwagi:
They are almost the same, but "firm" is often used for legal type businesses. "Company" is more common.

co do kodu :

[PHP] pobierz, plaintext 
    protected function create(array $data)
    {
        $user =  User::create(
            [
            'user' => $data['user'],
            'email' => $data['email'],
            'password' => bcrypt($data['password']),
            'ip' => request()->ip(),
            'key' => Uuid::generate()->string,
            ]
        );
 
 
        if(!Role::where('name','type')->exists())
        {
            Role::create(['name'=>'person']);
            Role::create(['name'=>'firm']);
        }
 
        //sprawdzamy czy typ = 0 czyli równy 1 - jeżeli tak dodaje Role firm w innym przypadku dodaje person w tabeli roles
 
        if($data['type']  0)
            $user->roles()->attach( Role::where('name','firm')->first()->id );
        else
            $user->roles()->attach( Role::where('name','person')->first()->id );
 
 
        return $user;
    }
 
 
    public function register(Request $request)
    {
        $this->validator($request->all())->validate();
        $user = $this->create($request->all());
 
        event(new Registered($user));
        $this->guard()->login($user);
        UserVerification::generate($user);
        UserVerification::send($user, 'Potwierdzenie rejestracji konta.', env('MAIL_FROM_ADDRESS'), env('APP_NAME'));
 
        /* wiadomość znajduje się w isVerified*/
 
        return $this->registered($request, $user)
            ?: redirect($this->redirectPath());
    }
[PHP] pobierz, plaintext 

Mam 3 tabele : users , roles, role_user,

users - wiadomo
tabela roles:
id, name
1, person
2, firm

oraz trzecią tabele role_user
user_id, role_id
1, 1
2, 1
3, 2
4, 1
5, 2

czyli zamierzeniem / zabezpieczeniem kodu:

[PHP] pobierz, plaintext 
if(!Role::where('name','type')->exists())
[PHP] pobierz, plaintext 

powinno być jeśli nie ma danych w tabeli roles , uzupełnij je danymi :

[PHP] pobierz, plaintext 
 Role::create(['name'=>'person']);
            Role::create(['name'=>'firm']);
[PHP] pobierz, plaintext 

Niestety każde dodanie nowego użytkownika powoduje, że w tabeli roles dane mi się dopisują tzn:
1, person
2, firm
3, peron
4, firm
5, person
6, firm

itd.

[SmokAnalog]

Ale ta linia:

[PHP] pobierz, plaintext 
if(!Role::where('name','type')->exists())
[PHP] pobierz, plaintext 

Nie oznacza, że nie ma danych w tabeli Roles, tylko że nie ma rekordu w tabeli roles, gdzie name = 'type'. Trochę bez sensu chyba?

[casperii]

inaczej, przy stworzeniu pierwszego użytkownika zabezpieczam się przed tym by tabela roles miała już jakieś wartości.
kurde czemu dałem tam type a nie person (?)

[PHP] pobierz, plaintext 
if(!Role::where('name','person')->exists())
[PHP] pobierz, plaintext 

i bangla

[SmokAnalog]

A nie lepiej sprawdzić czy w ogóle są jakiekolwiek, zamiast zakładać, że ma być ta jedna? Moim zdaniem taki kod prosi się o trudne do znalezienia błędy w przyszłości.

[casperii]

jakaś podpowiedź ?

[SmokAnalog]

Moim zdaniem lepiej byłoby po prostu tego nie sprawdzać, tylko mieć seedy do wstawiania tych podstawowych ról. Logika nie powinna się sypać, kiedy nie masz ról. Ewentualnie dałbym do panelu admina ostrzeżenie, gdy role są puste. Wiesz, warto unikać takich sprawdzajek, bo to niewiele wnosi, a dodaje warstwę logiki, o której się zapomina.

[casperii]

oczywiście na dalszym planie jest możliwość dodawania / edytowania / usuwania ról z poziomu panelu administracyjnego. Nie mniej jednak dziękuje za sugestię

Ten post edytował casperii 27.09.2020, 21:47:15

[SmokAnalog]

No tak szczerze mówiąc to wstawianie ról przy wstawianiu użytkownika prosi się o problemy To nie jest miejsce w kodzie, w którym chcesz cichaczem dodawać rekordy do innego modelu.

Gdybyś pytał, to żeby sprawdzić czy tabela zawiera jakiekolwiek rekordy, możesz dać exists() od razu, czyli:

[PHP] pobierz, plaintext 
if (Role::exists()) {
[PHP] pobierz, plaintext 

[casperii]

@SmokAnalog jeszcze mam pytanie odnośnie dodatkowych danych usera (imie, nazwisko , płeć, data ur, adres ... ) te dane będzie można dopiero dopisać po weryfikacji maila ( nie chcę usera zanudzać wypełnieniem pól)
czy lepiej dodać powyższe pola do tabeli user, czy np utworzyć inną tabelę i tam wstawiać te pola ?

[SmokAnalog]

Wiesz co, tak naprawdę to i tak, i tak będzie w porządku. Osobna tabela będzie miała taką zaletę, że możesz z poziomu bazy zadbać o integralność danych.

Konkretniej, to jest dobry przykład na użycie relacji 1:1. Dane byłyby połączone z użytkownikiem i user_id byłby unikalny. Kolumny w tej tabeli nie musiałyby być nullable. Mógłbyś też w jednoznaczny sposób się dowiedzieć, czy dany użytkownik wypełnił już swoje dane, czy nie. Świadczy o tym istnienie (lub brak) rekordu w tej dodatkowej tabeli.

Wadą takiego rozwiązania jest to, że przy domyślnym użyciu Eloquent, pojawiłby się oczywiście obiekt pośredni, czyli zamiast np. $user->name miałbyś np. $user->data->name. Trochę upierdliwe.

Ja bym chyba poszedł na skróty i dodał po prostu te kilka pól bezpośrednio do Usera w formie nullable. Integralność nie jest tak dobra, ale jest łatwiej ogarnąć dostęp do tych danych. Wtedy możesz wybrać jedną kolumnę jako dowód, że dane zostały już wypełnione, lub też wiele kolumn. Ja bym wstawił np. pole z datą wypełnienia danych (nullable) i tym się posługiwał przy ustalaniu czy ktoś wypełnił dane.

Ten post edytował SmokAnalog 27.09.2020, 21:56:27