İlişkili tablolarda Güncelleme

Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz? Discord

İlişkili tablolarda Güncelleme

sametatabasch

Merhabalar
Projemde Post ve PostMeta isimli iki tablom var bu tablolar birbirine foregin key ile bağlı(Post.id,PostMeta.postId).Bir post birden fazla postMeta alabiliyor.Modellerde şu şekilde ilişki kurdum: Post modelinde

public function postMeta(){
 return $this->hasMany('PostMeta','postId');
	}

PostMeta model inide :

public function post() {
 return $this->belongsTo( 'Post','postId' );
	}

Bir post güncellendiğinde post metalar ile birlikte güncellenmesini nasıl sağlarım ?

Güncelleme için yazdığım kodlar aşağıdaki şekilde ama bu şekilde PostMeta tablosu güncellenmiyor(hata veriyor).

public function postUpdatePost() {
 $postData = Input::all();
 $id=$postData['id'];
 $rules = array(
 'title' => 'required',
 'content' => 'required'
 );
 
 $messages = array(
 'title.required' => _( 'Başlık boş bırakılamaz' ),
 'content.required' => _( 'İçerik boş bırakılamaz' )
 );
 $validator = Validator::make( $postData, $rules, $messages );

 if ( $validator->fails() ) {
 return Redirect::back()->withInput()->withErrors( $validator->messages() );
 }
 else {
 $post = Post::find( $id );

 $post->author = Auth::user()->id;
 $post->content = $postData['content'];
 $post->title = $postData['title'];
 $post->excerpt = mb_substr( $postData['content'], 0, 450, 'UTF-8' );
 $post->status = $postData['status'];
 $post->type = $postData['type'];
 $post->url = Str::slug( $postData['title'] );
 $post->created_ip = Request::getClientIp();

 if ( isset( $postData['postMeta'] ) ) {
 $postMeta = $postData['postMeta'];
 $modelPostMeta = array();
 foreach ( $postMeta as $key => $value ) {
 $modelPostMeta[] = new PostMeta( array( 'metaKey' => $key, 'metaValue' => $value ) );
 }
 $post->postMeta()->associate( $modelPostMeta );
 }
 $post->push();

 return Redirect::back(); //todo burada bunu kullanmak doğrumu
 }
	}

Projenin tamamı:https://github.com/sametatabasch/ilkLaravel

serturk

Güncelleme yaparken varolan bir post u db den alıp güncelliyorsun ama aşağı ki satırlar da yeni postmeta klası dolayısı ile db ye de yeni postmeta alanı ekliyorsun, eklerken PostMeta.postId satırını kullanmıyorsunuz ($modelPostMeta[] = new PostMeta( array( 'metaKey' => $key, 'metaValue' => $value ) );)
bu da eklenecek alan foreing olduğu için hata verme sebebidir, yni ortada bir foreing key var ama tanımlanmıyor, yani postmeta eklerken, ilgili postmeta'nın hangi post a ait olduğu PostMeta.postId alanına eklenecek $post->id ile tanımlanmalı fakat siz bu eklemeyi yapmadığınızdan foreing key hata veriyor.

ayrıca güncelleme işlemin de metaların da kontrol edilip varolanların düzenlenmesi olmayanların eklenmesi gerekir, siz doğrudan yeni eklemeler yapıyorsunuz bu da aynı metaların tekrar tekrar eklenmesine (unique alan varsa (key, value) hata vermesine) sebep olabilir.

foreach ( $postMeta as $key => $value ) {
 $modelPostMeta[] = new PostMeta( array( 'metaKey' => $key, 'metaValue' => $value ) );
 }

bölümünü kullanmadan önce $post->postMeta() diyerek ilişkili tablodan metaları çekip olanları güncelleme olmayanları ekleme şeklinde düzenlemek sorununuzu çözebilir.

Projenizi incelemedim bunlar sadece ilk izlenimlerim, yanlış anladıysam kusura bakmayın.

sametatabasch

Laravelde modeller ilişkilendiğildiği için id yi otomatik alır diye düşündüm. Yorumunuz daha iyi düşünmemi sağladı birkaç deneme daha yapayım teşekkürler

sametatabasch

public function postUpdatePost() {
 $postData = Input::all();
 $id = $postData['id'];
 $rules = array(
 'title' => 'required',
 'content' => 'required'
 );

 $messages = array(
 'title.required' => _( 'Başlık boş bırakılamaz' ),
 'content.required' => _( 'İçerik boş bırakılamaz' )
 );
 $validator = Validator::make( $postData, $rules, $messages );


 if ( $validator->fails() ) {
 return Redirect::back()->withInput()->withErrors( $validator->messages() );
 }
 else {
 $post = Post::find( $id );

 $post->author = Auth::user()->id;
 $post->content = $postData['content'];
 $post->title = $postData['title'];
 $post->excerpt = mb_substr( $postData['content'], 0, 450, 'UTF-8' );
 $post->status = $postData['status'];
 $post->type = $postData['type'];
 $post->url = Str::slug( $postData['title'] );
 $post->created_ip = Request::getClientIp();

 if ( isset( $postData['postMeta'] ) ) {
 $postMeta = $postData['postMeta'];
 foreach ( $postMeta as $key => $value ) {
 $post->postMeta()->where('metaKey','=',$key)->update(array('metaValue'=>$value));
 }
 }
 $post->push();

 return Redirect::back();
 }
	}

Bu şekilde güncelleme işlemini yaptım ama laravelde bunun daha basit bir yolunun olacağını düşünüyorum

hkan

Yöntem gayet makul gözüktü bana. Veritabanı yapısına bakıp daha optimize bi çözüm üretmek mümkün olabilir.

Laravel Türkiye Forumları, ücretsiz forum uygulaması Flarum ile yapılmıştır.