Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

Yetki Kontrolünü Yaptığım Bu Şekil Doğru mu Sizce ?

koti42

Yetki kontrolünü bu şekilde yapıyorum ama acaba doğru mu çalışıyor fakat çalıştığı için çalışsın istemiyorum sade ve temiz kod yazmak adına acaba fazla yazmış olduğum bir kod bloğu gözünüze çarpıyor mu şu şekilde kısaltılabilirdi diye Ayrıca bu kodları bir çok yerde kullanmam gerekecek kontrolleri yapabilmem için de Helper kullanmalı mıyım sizce acaba?

$roles = auth()->user()->roles()->get();
        $cont="";
        foreach ($roles as $rol) {
            $role = Role::find($rol->id)->load('permissions');
            foreach ($rol->permissions as $rols) {
                $cont= $rols->name;
                if($cont==='edit-user')
                {
                    try {
                        $user->name = $request->name;
                        $user->email = $request->email;
                        $user->save();
                        return redirect()->route('users.index')->with('success', 'Kayıt Günceleme İşlemi Başarıyla Tamamlandı!');
                    } catch (Throwable $exception) {
                        return redirect()->route('users.index')->with('error', 'Güncelleme İşlemi Şuanda Çalışmıyor, Lütfen Sistem Yöneticisine Başvurunuz!');
                    }
                }

            }
            return redirect()->route('users.index')->with('error', 'Bunun İçin Yetkiniz Yok!, Lütfen Sistem Yöneticisine Başvurunuz!');
        }

mgsmus

koti42 https://laravel.com/docs/8.x/authorization

sineld

foreach ($roles as $rol) {
            $role = Role::find($rol->id)->load('permissions');

döngü içinde sorgu kullanmışsınız, bu düpedüz intihardır.

koti42

sineld Bir altında ki veriye nasıl erişebileceğimi bilemedim açıkcası yetkilendirme ile ilk kez uğraşıyorum da.

Bu yorumdan sonra ufak bir düzeltme ile intihara son verdiğimi düşünüyorum 😁

@mgsmus'un göndermiş olduğu yetki sistemini öğrenene kadar acaba başka bir patlak vermiş olabilir miyim

   try {
            $roles = auth()->user()->roles()->first();
            $role = Role::findOrFail($roles->id)->load('permissions');

            foreach ($role->permissions as $rol) {
                    if($rol->name==='edit-user')
                    {
                        try {
                            $user->name = $request->name;
                            $user->email = $request->email;
                            $user->save();
                            return redirect()->route('users.index')->with('success', 'Kayıt Günceleme İşlemi Başarıyla Tamamlandı!');
                        } catch (Throwable $exception) {
                            return redirect()->route('users.index')->with('error', 'Güncelleme İşlemi Şuanda Çalışmıyor, Lütfen Sistem Yöneticisine Başvurunuz!');
                        }
                    }
            }
            return redirect()->route('users.index')->with('error', 'Bunun İçin Yetkiniz Yok!, Lütfen Sistem Yöneticisine Başvurunuz!');
        }
       catch (ModelNotFoundException $exception)
       {
           return redirect()->route('users.index')->with('error', 'Güncelleme İşlemi Şuanda Çalışmıyor, Lütfen Sistem Yöneticisine Başvurunuz!');
       }

mgsmus

koti42 Test etmedim ama şöyle bir şey deneyebilirsiniz:

// Kullanıcı birden fazla role sahip ise tüm yetkileri derlemeniz lazım
if(!$request->user()->roles->flatMap->permissions->contains('name', 'edit-user')) {
    return redirect()
        ->route('users.index')
        ->with('error', 'Bunun için yetkiniz yok!, Lütfen sistem yöneticisine başvurunuz!');
}

$user->name = $request->input('name');
$user->email = $request->input('email');
$user->save();

return redirect()
    ->route('users.index')
    ->with('success', 'Kayıt günceleme işlemi başarıyla tamamlandı!');

Ayrıca:

$roles = auth()->user()->roles()->first();

Scope içinde Request varsa kullanıcıya $request->user() şeklinde ulaşın.

$role = Role::findOrFail($roles->id)->load('permissions');

Bu kullanımı sizde daha önce de gördüm. Bu yanlış, size daha önce de söyledim. Buradaki load yöntemi daha önce yüklenmiş bir modelin ilişkisini sonradan yüklemek için. Doğru kullanım şekli şöyle:

if($condition) {
    $model->load('relation');
}

Yani daha önce yüklenmemiş bir ilişkiyi bir şarta göre sonradan dönen collection'a eklemek için kullanmanız lazım. Eğer şart yoksa, direkt ilişkiyi de alarak modeli istiyorsanız with kullanacaksınız. Eğer tek bir model yüklüyorsanız ve yüklediğiniz ilişki de nested değilse with kullanmanın da anlamı yok çünkü mecburen o bir sorgu atılacak. Ayrıca kullanmanız gerekirse load yerine loadMissing kullanın, zaten yüklenmiş ise ilişkiyi tekrar yüklemenizi engeller.

$request->name;

Request içerisinde content, headers, body... gibi kendine ait public değişkenler olduğu için isimden dolayı sizin değişkenlerle karışma riski var (forumda bir arkadaş $request->content kullandığı için işin içinden çıkamamıştı, yardımcı olduk), o yüzden $request->input('name'); şeklinde kullanın. Bu şekilde iki özelliği de elde etmiş oluyorsunuz:

// Gelen değişken array ise dot-nation dediğimiz yolla içerisindeki elemana nokta ile ulaşabilirsiniz.
$photo = $request->input('user.photo'); // <input name="user[photo]"/>

// Değişken null ise ikinci parametreyi kullanarak default bir değer verebilirsiniz.
$startsAt = $request->input('starts_at', now());

Rol yetki sistemi (RBAC) için tavsiye ettiğim paket:
https://github.com/spatie/laravel-permission

koti42

mgsmus Çok teşekkür ederim sizleri de yoruyorum bu konu da. Ben bir önce ki soruda bu işlemin collectionlar da sadece paginate ile alakalı olduğunu düşünmüştüm sadece, Şimdi doğru kullanımını anladım.

mgsmus

koti42

// Şu an $roles bir Eloquent\Builder objesi. Bundan sonraki yöntemler Builder yöntemi olacak.
// Sorgu henüz yapılmadı.
$roles = $user->roles();

 // Sorgu yapıldı, roller çekildi ve roles özelliğine Eloquent\Collection olarak eklendi. 
 // $roles artık bir Eloquent\Collection.
$roles = $user->roles;

// Sorgu yapıldı. Sonuçlar çekildi.
// $roles artık bir Eloquent\Collection
$roles = $user->roles()->get();

// get() ile dönen Collection'a Support\Collection içindeki pluck yöntemi uygulandı. toArray()
// ile diziye dönüştürüldü. $roles artık bir array, artık Fluent arayüz kullanamaz. 
$roles = $user->roles()->get()->pluck('name')->toArray();

koti42

mgsmus Teşekkür ederim açıklamanız için bunları kaydedeceğimm