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

Resim update

yalcin

Resim için Store metodu kolaydı tüm alanlar boş geliyo ve StoreBrandRequest basitti

public function rules(): array
    {
        return [
            'name' => 'required|unique:brands|min:3|max:15',
            'image' => 'required|image|max:2048|mimes:jpeg,png,jpg'
        ];
    }

boş alanları kontrol et , kuralları geçiyor ise resime benzersiz bir ad ver nereye kaydolacagını belirterek

 Brand::create([
                'name' => $validated['name'],
                'slug' => Str::slug($validated['name']),
                'image' => $imageFile->storeAs("brand", $Filename , "public"),
            ]);
            return redirect()->route('brands.index')->banner('Marka Başarıyla Eklendi. ');

işlemi bitir .

Update işlemi zormuş ve haliyle takıldım bende tabi benim için zor sizler sayısız defa yaptığınız için yürür gibi yapıyosunuz böyle zamanlarda yoksa "shipbuilding in a bottle" olayınamı girseydim diyorum neyse

UpdateBrandRequestim

 public function rules(): array
    {
        return [
            'name' => 'required|unique:brands|min:3|max:15',
            'image' => 'image|max:2048|mimes:jpeg,png,jpg'
        ];
    }

isim mecbur olmalı boş geçilmemeli , birde benzersiz olmak zorunda

Controller ise

public function update(UpdateBrandRequest $request, Brand $brand)
    {   
        $validated = $request->validated();

        if ($request->hasFile('image')) {       // bu satırda    => $validated['image']->hasFile   hata verdi

            $imageFile =$validated['image'];
            $originalName = $imageFile->getClientOriginalName();
            $originalExtension = $imageFile->getClientOriginalExtension();
            $explodeName = explode(".", $originalName)[0];
            $Filename = Str::uuid($explodeName). ".".$originalExtension;
        }

        $brand->update([
            'name' => $validated['name'],
            'slug' => Str::slug($validated['name']),
            'image' => $imageFile->storeAs("brand", $Filename , "public"),
        ]);
        return redirect()->route('brands.index')->banner('Marka Başarıyla Eklendi. ');

    }

Öncelikle name alanım dolu geliyo ve UpdateBrandRequestim içindeki unique:brands kuralı yüzünden olduğu gibi bırakamıyorum mutlaka değiştirmek zorunda oluyorum yani biri sadece resmi değiştirmek isterse adıda değiştirmek zorundaki devam edebilsin sanırım bu kuralı update için kaldırmalıyım değilmi ?

Not: Bunun bilimsel bir açıklaması var mıdır bilmiyorum ama yazarak daha iyi düşünebiliyor insan .

peki UpdateBrandRequestim den benzersiz kuralını kaldırdım marka index tablosundan ilgili marka olan (Dünyanın en iyi Portakal markası) SARIÇOBAN için düzenleye bastım ve Name alanım SARIÇOBAN olarak dolu geldi , update edicem tam burda kayıt esnasında yaptığım benzersizlik kontrolu boşa çıkıyor yani update ederken daha önce kaydettiğim bir marka adını tekrar kullanaraka aynı isimden tabloda iki marka olabiliyor

1.sorum
Tıpkı resmin varlığını kontrol ettiğim gibi markanın adının benzersizliğini bir if ile controllerda nasıl kontrol edebilirim.

ikinci soru resim update ile ilgiliydi şu an unttum .

yalcin

public function update(UpdateBrandRequest $request, Brand $brand)
    {
       
        $validated = $request->validated();

       if (Brand::where('name', $validated['name'])->exists()) {
        return "Brand tablosun kontrol ";
       }

böyle bişe uydurdum , eminim daha profesyonel bi çözümü vardır ama bu if içinden devam edicem, peki ya çok fazla markam var ise ve update ederken zaten var olan bir markayı yazar isem ?

çünkü iki senaryo var birincisi
marka adı aynı kalsın sadece resmi güncelle ...

ikincisi ise
marka adı update ederken var olan bir markaya denk gelme durumu

bu galiba 3.yada 4. editim ama sudoku gibi bilmece çözmek gibi çok keyifli . bana şey lazım bir karşılaştırma kullanıcı zaten var olan markayı ellemeden ad aynı kalsın sadece resmi güncellemi demek istiyo yoksa yeni bir marka yazıpmı gönderiyo gibi bişe lazım ki ona göre işlem yapayım

mgsmus

yalcin Dokümanda unique kuralına bakın, orada ignore özelliği var.

sineld

namespace App\Http\Requests\Sample;

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Validation\Rule;

class UpdateRequest extends FormRequest
{
    public function authorize()
    {
        return true;
    }

    public function rules()
    {
        return [
            'type_id' => 'required',
            'name' => [
                'required',
                'max:255',
	            Rule::unique('sample_table')
		            ->ignore($this->id)
		            ->where(fn($query) => $query->where('type_id', $this->type_id)),
            ]
        ];
    }
}

yalcin

bütün pazarımı bunla geçiricem sanırım 🙂
UpdateBrandRequest kafa karıştırıyo bunu kullanmadan yani herhangi bir form request olmadığını varsayarsak

public function update(Request $request, Brand $brand) şeklinde metodum var ve buna

const submit = () => {
    form.post(route('brands.update', props.brand.id), {
      onFinish: () => form.reset('password'),
    });
};

bir brand id submit ediyorum /yolluyorum

name alanım dolu geliyor (required)
resim alanım boş geliyor (input boş)

ve şöyle bir kontrol sağlamam gerekiyo .

name alanına bak eğer $brand->name ile $request->name eşitse bu şu demek kullanıcı name alanını update etmek istemiyor oldugu gibi bırakıyor güncellemeye gerek yok

$brand->name ile $request->name arasında bir fark varsa Brand::where('name' diye veritabanına git kontrolunu yap unique mi değilmi anla ve daha önce bir kayıt varsa uyar yoksa güncelle

Sonra resim alanına geç boş gelmişsee işlem yok doluysa eski resim yerine bunu koy .

yazınca baya bi büyüdü gözümde

mgsmus

yalcin name alanı ile ilgili anlattıklarınız Rule::unique()->ignore() 'un yaptığı şey...

yalcin

peki teşekkürler deniyorum hemen

yalcin

Hayır bu değilmiş ,aslında tam olarak şöyle güncelleme yaptığı satırın adının eşsizliğini görmezden geliyor tüm tabloya bakmıyor aslında yarısına kadar doğru yapıyor sonrası yok
işlem yaptığı satırı değişiklik yoksa görmezden gel devam et değişiklik varsa tablodaki diğer alanlara bak ..

Sanırım geri kalanını Controllerda yapmak gerek

yalcin

Bu şekilde bişe tasarladım , daha sadeleştirilebilir belki yada daha okunabilir veya daha mantıksal bişe olabilir ama elimden gelen budur bilemedim şimdi .
tam olarak istediğimi yapıyor


public function update(UpdateBrandRequest $request, Brand $brand)
    {
        $validated = $request->validated();      

            if ($validated['name'] !== $brand->name) { //Marka eşit değilse

                if (Brand::where('name', $validated['name'])->exists()) { //Marka varlığını kontrol et  

                    return redirect()->route('brands.index')->banner('Bu Marka daha önce kaydedilmiş');

                }else { // Daha önce böyle bir marka yoksa 

                    if ($request->hasFile('image')) {  //Resim varlığını kontrol et varsa marka adını ve resmi kaydet
                        $imageFile =$validated['image'];
                        $originalName = $imageFile->getClientOriginalName();
                        $originalExtension = $imageFile->getClientOriginalExtension();
                        $explodeName = explode(".", $originalName)[0];
                        $Filename = Str::uuid($explodeName). ".".$originalExtension;

                        $brand->update([
                            'name' => $validated['name'],
                            'slug' => Str::slug($validated['name']),
                            'image' => $imageFile->storeAs("brand", $Filename , "public"),
                        ]);

                        return redirect()->route('brands.index')->banner('Marka Başarıyla Güncellendi.');

                    }elseif (!$request->hasFile('image')) { //resim yoksa sade adı kaydet slug ile birlikte
                        $brand->update([
                            'name' => $validated['name'],
                            'slug' => Str::slug($validated['name']),
                        ]);

                        return redirect()->route('brands.index')->banner('MArka Adı Güncellendi ');
                    }                
                
                }
            }else { // /Marka AD  eşit İSE
                if ($request->hasFile('image')) {  //Resim varlığını kontrol et ve resmi kaydet
                    $imageFile =$validated['image'];
                    $originalName = $imageFile->getClientOriginalName();
                    $originalExtension = $imageFile->getClientOriginalExtension();
                    $explodeName = explode(".", $originalName)[0];
                    $Filename = Str::uuid($explodeName). ".".$originalExtension;

                    $brand->update([
                       'image' => $imageFile->storeAs("brand", $Filename , "public"),
                    ]);

                    return redirect()->route('brands.index')->banner('Resim  Güncellendi.');

                }else {
                    return redirect()->route('brands.index')->banner('Hiçbir işlem yapılmadı ');
                }

            }        
 
    }

Şey eksik bi tek resmi güncelledikten sonra önceki resmi de sildirmem gerekiyo sanırım

BoraN7

yalcin Brand modeliniz içerisinde updated eventinde eski resmi silme işlemlerinizi gerçekleştirebilirsiniz.
https://laravel.com/docs/10.x/eloquent#events-using-closures

mgsmus

yalcin Şimdi şöyle bir rotanız var diyelim:

Route::resource('brands', BrandController::class);
// ya da
Route::patch('brands/{brand}', [BrandController::class, 'update']);

Bunun update kısmı PUT|PATCH /brands/5 şeklinde oluyor. Buradaki 5 brand id. Siz model binding kullandığınız için form request içinde ilgili Brand modeline $this->brand şeklinde ulaşabilirsiniz. Segment ismi {brand} olduğu için $this->brand şeklinde ulaşıyorsunuz. Bu durumda kurallar şöyle olacak:

return [
    'name' => [
        'required',
        'min:3',
        'max:15',
       Rule::unique(Brand::class)->ignore($this->brand),
    ]
    'image' => 'nullable|image|max:2048|mimes:jpeg,png,jpg'
];

Böylece controller içinde

$validated = $request->validated();

$updateData['name'] = $validated['name'];
$updateData['slug'] = Str::slug($validated['name']);

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

    $updateData['image'] = $imageFile->storeAs("brand", $Filename , "public");
}

$brand->update($updateData);

return redirect()->route('brands.index')
    ->banner('Marka Başarıyla Güncellendi.');

yapmanız yetecek.

yalcin

Ben daha çok , şurası olmamış burası olmamış veya şurayı yapmışın burayı neden yaptın ki vs gibi eleştriler bekliyodum . ne yalan söyliyim valla çok mutlu oldum . Çok teşekkür ederim

mgsmus

yalcin Aslında çok absürt şeyler gördüğümde ya da özellikle eleştiri bekleyen bir tavır ile karşılaştığımda yapıyorum ama çoğu zaman önemsiz ya da zamanla öğrenilir diyerek bir şey demiyorum. Mesela tüm değişkenleri camelCase yazmışsınız ama $Filename kabak gibi büyük harfle başlıyor. İşi Rule::unique() ile çözmeyip controller içinde yapmanız gözüme onun kadar takılmadı çünkü absürt; $Filename neden $filename değil, arkasına bir türlü geçerli bir sebep koyamıyorum 🙂

yalcin

$Filename hakkında konuşmak istemiyorum 🙂) hayırda neden böyle yapmışım hatırlamadım cidden
Rule::unique() kısmınıda beceremedim. @sineld in verdiği örneği aynen aldım sonuna sanırım $brand->id ekledim sonra gördümki sadece işlem yaptığım id ile ilgili işlem yapıyor , dökümantasyonda tüm tabloya bakıyormu acaba diye baktım ve

Instead of passing the model key's value to the ignore method, you may also pass the entire model instance. Laravel will automatically extract the key from the model:

Rule::unique('users')->ignore($user)

bu kısımda ben böyle anladım Rule::unique('brands')->ignore($brand->id) haliyle sanırım $brand->id gönderince sadece tek bir satıra baktırdım ,
olsun tekrar yaparım ne varki .

çok teşekkür ederim