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

Laravel Crud Problemi

siradaninsan

mgsmus teşekkür ediyorum $request->input(‘content’) şeklinde kullanınca düzeldi fakat neden öyle oldu?

mgsmus

siradaninsan Laravel'in Illuminate\Http\Request sınıfı Symfony\Component\HttpFoundation\Request sınıfını miras alan bir sınıf. Symfony\Component\HttpFoundation\Request içinde de $content isimli bir sınıf özelliği mevcut. O yüzden her ne kadar sizin query ve form parametreleriniz birleşse de Request kullanırken content, attributes, request, query, server, files, cookies, headers... gibi $request->content gibi sınıf özelliğine erişme şeklinde kullanamayacağınız özel değişkenler var, bunu kara liste gibi düşünebilirsiniz, örneğin formdan gelen parametreler bu isimlere sahip olmamalı. Bunlar sizin aynı isimli parametrelerinizin üzerine yazar. O yüzden daha önce de forumda belirttiğim gibi, bu ve bunun gibi sorunları yaşamamak için daima Request::input() kullanın, böylece isimlendirmeyi düşünmenize gerek kalmaz.

Request sınıfı enjekte edilmemiş ise Request::input() ya da request()->input('content') şeklinde; enjekte edilmiş ise $request->input('content') şeklinde kullanabilirsiniz.

Request::input() ile dizi elemenlarına da dot notation dediğimiz $request->input('products.1.name') şekilde de ulaşabilirsiniz.

Form Request kullandığınızda $request->input('content') yerine $request->validated() kullanmanız lazım. Yani şu şekilde:

$data = $request->validated();

$content = $data['content'];

çünkü form request içindeki bazı değişiklikler validated() yöntemi ile gelen parametrelere yansır ama siz input ile çekerseniz validasyonu hiçe sayıp direkt gelen parametreyi almış oluyorsunuz. Yani form request kullanınca $request->input() ile aldığınız parametre validasyondan geçmemiş hali ile çekiliyor, bu güvenlik sorunlarına ya da istenmeyen durumlara sebep olabilir.

Yani özet olarak:

$request->content;  // Yanlış
$request->get('content'); // Yanlış
$request->post('content'); // Yanlış

$request->input('content'); // ✓ Doğru

BoraN7

mgsmus Normal validate işleminde gerek var mi?
Örneğin;

$request->validate([
  'name' => 'some_rule',
]);
$model = new Model();
$model->name = $request->input('name');

BoraN7

mgsmus

Konuyu diriltiyorum ama zannedersem Laravel 8.x 'in son sürümlerinde yeni bir özellik gelmiş.

Ya da en azından dökümantasyona dahil edilmiş.

https://laravel.com/docs/8.x/validation#creating-form-requests

// Retrieve the validated input data...
    $validated = $request->validated();

    // Retrieve a portion of the validated input data...
    $validated = $request->safe()->only(['name', 'email']);
    $validated = $request->safe()->except(['name', 'email']);

safe() özelliğini ilk defa görüyorum.

$validated = $request->validated();
$validated['name'];

şeklinde de kullanılabilir ama safe'i only ve except ile zincirleyebilmek hoşuma gitti.

mgsmus

BoraN7 Sizin örnekte sorun yok. Benim demek istediğim:

// Sadece validasyona tabi tutulan alanlardan User oluşturur
$user = User::create($request->validated())

// Validasyona tabi olsun olmasın tüm parametreler ile User oluşturur. company_id eklensin 
// istemiyorsunuzdur, rules içinde de yoktur ama $fillable[] içinde company_id varsa, 
// parametreler içine eklenerek istenmeyen bir şekilde girişi yapılabilir.
$user = User::create($request->all())

Bu yapılan en büyük hatalardan biri. Bir de form request içinde passedValidation() kullanarak requesti değiştirme var. Orada da şöyle bir sorun var, o değişiklik validated() dizisine yansımıyor, $request->input() olayına da yansımıyor ama $request->title gibi kullanınca yansıyor. Orada ya bir bug var ya da bir sebepten ötürü yapılmamış, input() olayına yansımaması normal çünkü dediğim gibi her şeyden bağımsız çekiyorsunuz ama bence passedValidation() yöntemi validated() içine yansımalı. Ben bunu engellemek için passedValidation yerine validated yöntemini eziyorum.

siradaninsan

mgsmus kendim zaten validate işleminden geçirip yapıyorum direkt requesti zaten basıp oluşturmak büyük bir sorun dediğiniz gibi. Kara liste olayı aklıma gelmişti aydınlığına kavuşturduğunuz için teşekkürler. 🙂

siradaninsan

mgsmus

BoraN7

mgsmus Form request'te yanlış yapıyormuşum ama, sizden öğrenmiş oldum.

Örneğin;

use App\Http\Requests\AboutUpdateRequest;

  public function update(AboutUpdateRequest $request, About $about)
    {
        abort_if(Gate::denies('admin-action'), 403);
        $about->name = $request->input('name');
        $about->body = $request->input('body');
        $about->slug = $request->input('slug');
        $about->meta_title = $request->input('meta_title');
        $about->meta_description =  $request->input('meta_description');
        $about->meta_keywords =  $request->input('meta_keywords');
        if ($request->hasFile('image')){
            $about->image = Helper::prepareImagePath($request->file('image'), 'about', 300, 250);
            Storage::delete([$about->getOriginal('image')]);
        }
        $about->save();
        return back()->withSuccess('Güncelleme Başarılı!');

    }

Şimdi bu şekilde güncelledim. Doğrusu bu şekilde sanırsam;

public function update(AboutUpdateRequest $request, About $about)
    {
        abort_if(Gate::denies('admin-action'), 403);
        $data = $request->validated();
        $about->name = $data['name'];
        $about->body = $data['body'];
        $about->slug = $data['slug'];
        $about->meta_title = $data['meta_title'];
        $about->meta_description =  $data['meta_description'];
        $about->meta_keywords =  $data['meta_keywords'];
        if ($request->hasFile('image')){
            $about->image = Helper::prepareImagePath($data['image'], 'about', 300, 250);
            Storage::delete([$about->getOriginal('image')]);
        }
        $about->save();
        return back()->withSuccess('Güncelleme Başarılı!');
    }

siradaninsan

public function store(Request $request)
{
    $validator = Validator::make($request->all(), [
        'name' => 'required|max:255',
        'slug' => 'max:255|unique:categories',
        'image' => 'mimes:jpg,png,jpeg|max:1000',
        'parent_category' => 'required|integer',
        'meta_title' => 'max:255',
        'meta_description' => 'max:255',
        'meta_keywords' => 'max:255',
        'bottom_showcase' => 'max:4000',
        'top_showcase' => 'max:4000',
        'desk' => 'min:1|integer',
        'status' => [
            'required',
            'max:1',
            Rule::in([0, 1]),
        ],
    ],[
        'name.required' => 'Kategori adı boş bırakılamaz.',
        'name.max' => 'Kategori adı max 255 karakter olabilir.',
        'slug.max' => 'Kategori url max 255 karakter olabilir.',
        'slug.unique' => 'Bu kategori linki kullanılıyor. Başka url giriniz.',
        'parent_category.required' => 'Bağlı olduğu kategori boş geçilemez.',
        'parent_category.integer' => 'Bağlı olduğu kategori sadece tam sayı alabilir.',
        'image.mimes' => 'Kategori resmine sadece "jpg, png, jpeg" formatları yüklenebilir.',
        'image.max' => 'Kategori resmine max 1mb resim yüklenebilir.',
        'meta_title.max' => 'Meta title max 255 karakter olabilir.',
        'meta_description.max' => 'Meta description max 255 karakter olabilir.',
        'meta_keywords.max' => 'Meta keywords max 255 karakter olabilir.',
        'bottom_showcase.max' => 'Alt vitrin max 4000 karakter olabilir.',
        'top_showcase.max' => 'Üst vitrin max 4000 karakter olabilir.',
        'status.required' => 'Durum boş geçilemez.',
        'status.max' => 'Durum sadece 1 karakterli olabilir.',
        'status.in' => 'Durum sadece 0 veya 1 alabilir.',
        'desk.min' => 'Sıra alanı min 1 karakter olmalıdır.',
        'desk.integer' => 'Sıra alanı tam sayı olmalıdır.',
    ]);

    if ($validator->fails()) {
        $errors = $validator->errors();
        $newErrors = implode('<br>', $errors->all());
        return response([
            'status' => 'info',
            'title' => 'Opps!',
            'message' => $newErrors,
            'orient' => false,
            'location' => null
        ]);
    }

    $category = new Category;
    $category->name = $request->name;

    if(empty($request->slug)){
        $count = Category::where('slug', 'LIKE', "%".Str::slug($request->name, '-')."%")->count();
        if($count!=0){
            $category->slug = Str::slug($request->name, '-')."-".($count+1);
        }else{
            $category->slug = Str::slug($request->name, '-');
        }
    }else{
        $category->slug = $request->slug;
    }


    if ($request->has('image')) {
        $file = $request->file('image');
        $extension = $file->extension();
        //$fileName = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
        if (Storage::disk('public')->exists('categories/'.$category->slug.".".$extension)) {
            return response([
                'status' => 'info',
                'title' => 'Opps!',
                'message' => 'Yüklediğiniz resme ait aynı adda bir dosya zaten var. Farklı adda bir resim yükleyin.',
                'orient' => false,
                'location' => null
            ]);
        }else{
            if($request->image->storeAs('public/categories', $category->slug.".".$extension)){
                $category->image=$category->slug.".".$extension;
            }else{
                $category->image=null;
            }
        }
    }else{
        $category->image=null;
    }

    $category->parent_category = $request->parent_category;
    $category->meta_title = $request->meta_title;
    $category->meta_description = $request->meta_description;
    $category->meta_description = $request->meta_description;
    $category->meta_keywords = $request->meta_keywords;
    $category->bottom_showcase = $request->bottom_showcase;
    $category->top_showcase = $request->top_showcase;
    $category->desk = $request->desk;
    $category->status = $request->status;

    if($category->save()){
        return response([
            'status' => 'success',
            'title' => 'Başarılı',
            'message' => 'Başarılı bir şekilde kategori oluşturuldu.',
            'orient' => true,
            'location' => route('categories.index')
        ]);
    }else{
        return response([
            'status' => 'error',
            'title' => 'Hata',
            'message' => 'Sistemsel bir hata oluştu.. Lütfen durumu bildirin.',
            'orient' => false,
            'location' => null
        ]);
    }
}

böyle bir yazımın yanlış olduğunu söyleyebilir miyiz?

siradaninsan

projenin tüm kodları böyle yazıldı bir güvenlik zafiyeti oluşturur mu?

siradaninsan

ben validate işlemini sayfa içinde yapıyorum direkt uyuşmayan veri varsa returnluyorum

siradaninsan

yukardaki kullanımım doğru mu merak ettim şahsen projenin tüm kodları bu şekilde yazıldı

mgsmus

siradaninsan if ($validator->fails()) { ile kontrol yaptığınız yerde validasyon hatası meydana gelirse HTTP 200 kodu ile hata mesajı dönüyorsunuz, yanlış, 422 hata kodu ile dönmeniz lazım. Diğer hata kısımlarında da yanlış yapmışsınız. Benim tavsiyem controller validation kullanılacaksa (ama tavsiyem form request kullanılması), buralara gereksiz müdahale edilmeyip Request içindeki validator'ün kullanılması:
https://laravel.com/docs/8.x/validation#quick-writing-the-validation-logic

Hata mesajlarıyla çalışırken frontend kısmında status diye alanlara bakılmaz. HTTP kodu, message alanı ve validation exception ise errors alanlarına bakılır. Şunu yapabilirsiniz:

if (Storage::disk('public')->exists('categories/'.$category->slug.".".$extension)) {
    return new ValidationException::withMessages([
        'slug' => 'Yüklediğiniz resme ait aynı adda bir dosya zaten var. Farklı adda bir resim yükleyin.'
    ]);
}

Mevcut slug varsa sonuna sayı ekle şeklindeki slug mantığınız yanlış. Slug zaten yeni-kategori-1 ise sizin sistem yeni-kategori-1-1 şeklinde slug oluşturacak, halbuki yeni-kategori-2 olması gerekiyor. Ben slug arka planda otomatik oluşturmuyorum, kullanıcının ekleme yaptığı formda bir input içinde gösteriyorum, aynı slug varsa hata veriyor, kullanıcı slug'ı elle değiştiriyor. Bunca yıldan sonra bulabildiğim en mantıklı çözüm bu çünkü ayrıca birçok müşterim bazı durumlarda da özel slug belirlemek istiyordu.

if ($request->has('image')) { yanlış if ($request->hasFile('image')) { olması gerekiyor.

if($request->image->storeAs('public/categories', $category->slug.".".$extension)){ burası yanlış, disk adını aşağıdaki gibi ayrı belirtmeniz lazım:

if($request->image->storeAs('categories', $category->slug.".".$extension, 'public')){

Son olarak son aşamada kategoriyi kaydederken hata alırsanız eklenen resim varsa silinmeden sunucuda kalacak. Hata meydana gelirse ve resim eklenmiş ise resmi silmeniz lazım. Böyle yerlerde ben transaction kullanırım:

$category = DB::transaction(function() use ($request) {

    // Önce kategori
    $category = Category::create(...);

    // Sonra resim
    if($request->hasFile('image')) {
        // ...
    }

    return $category;
});

Bu şekilde sadece kategori oluştururken hata verirse exception fırlatır, resim eklemez. Diyelim ki kategori ok ama resim ekleyemedi, kategori de oluşmaz. Her ikisi de hatasız gerçekleşirse bana oluşturulan kategoriyi verecek:
https://laravel.com/docs/8.x/database#database-transactions

siradaninsan

mgsmus yorumlarınız için teşekkürler. fakat slug'ı kullanıcı kendide belirleyebilir dediğiniz mantıkta burda amaç kullanıcı uğraşmadan direkt eklemesi. diğer yorumlarınızı dikkate alacağm. teşekkür ediyorum vakit ayırıp dönüş yaptığınız için

BoraN7

siradaninsan Ben slug için paket kullanıyorum. İncelemenizi öneririm. Basit ve kullanışlı.
https://github.com/cviebrock/eloquent-sluggable

mgsmus

siradaninsan slug otomatik oluşuyor ama kullanıcı kaydetmeden önce kendisi düzenleyebiliyor. Oluşturmak için şu paketi kullanıyorum: https://www.npmjs.com/package/speakingurl

<input id="name" type="text" v-model="form.name" @keyup="generateSlug"/>

generateSlug() {
    this.form.slug = getSlug(this.form.name, {
        lang: 'tr'
    })
},

siradaninsan

mgsmus teşekkürler, peki validate işlemini requests içinde yapıp direkt dönmeden controller içinde errorları almamın imkanı var mı?

deathisonitsway

SQLSTATE[23000]: Integrity constraint violation: 1048 Column ‘content’ cannot be null” hatası alıyorum.

bu sql hatalarını render etmiyorsanız kötü kod yazıyorsunuz demektir.Bunlar validate edilmeden gönderilemez database e..

İşin özü kodunuz kötü.Daha ağır konusabilirim ama gerek yok.

siradaninsan

deathisonitsway ahahha nazik uyarınız için teşekkürler fakat hatanın kötü kod ve validate işleminden değil sınıfın içerisinde kullanından bir propertyden kaynaklı olduğunu vurgulamak isterim. Keşke kodu görüp sonra konuşsaydınız teşekkürler tekrardan...

sineld

AboutUpdateRequest $request

aldığınız kısmı

logger($request->all());

ile log'a basın ve hangi değer geldiğini görün, muhtemelen boş değer geliyor.

BoraN7

sineld Hocam burada size bir şey danışmak istiyorum.

Bu örnekte;

public function update(AboutUpdateRequest $request, About $about)
    {
        abort_if(Gate::denies('admin-action'), 403);
        $data = $request->validated();
        $about->name = $data['name'];
        $about->body = $data['body'];
        $about->slug = $data['slug'];
        $about->meta_title = $data['meta_title'];
        $about->meta_description =  $data['meta_description'];
        $about->meta_keywords =  $data['meta_keywords'];
        if ($request->hasFile('image')){
            $about->image = Helper::prepareImagePath($data['image'], 'about', 300, 300);
            Storage::delete([$about->getOriginal('image')]);
        }
        $about->save();
        return back()->withSuccess('İşlem Başarılı!');
    }

Sadece bir resim alanı için satır satır girmek zorunda kaldım.
$about->update($request->validated());
ile bitirebilirdim ama bu sefer de validation'dan geçen resim direkt yüklenirdi. Ben istiyorum ki resim üzerinde işlem yapayım(isimlendirme,resize gibi)
İkinci olarak acaba üstüne tekrar mı işlem yaptırsam diye düşündüm. Örneğin;

$data = $request->validated();
$about->update($data);
 if ($request->hasFile('image')){
            $about->image = Helper::prepareImagePath($data['image'], 'about', 300, 300);
            Storage::delete([$about->getOriginal('image')]);
            $about->save();
        }
        //return...

Bu sefer de tutarsız davranışa/hataya sebep olur mu diye emin olamayarak, ilk yazdığım örnekte olduğu gibi satır satır işlemeye karar verdim resimli durumlarda. Siz ne önerirdiniz?

deathisonitsway

siradaninsan
Polemik yaratmak için yazmadım kusura bakma.Kaba geldiyse özür dilerim.
Ancak bu tip şeylere hassasiyetim çok.Sadece konu başlığınıza cevap yazıyorum.Diğer mesajları okumadım bile.
Tekrar kusura bakma.

mgsmus

siradaninsan Yazdığınız kodu neden sildiniz?

siradaninsan

mgsmus geri aldım review ettiğim için. peki validate işlemini requests içinde yapıp direkt dönmeden controller içinde errorları almamın imkanı var mı? @sineld

Sonraki Sayfa »