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

Gelen veriyi şekillendirmek ve kaydetmek !

GkyKrkc

Merhaba arkadaşlar, şimdi şöyle bir sorunla karşı karşıyayım, $request ->all() ile gelen veriaşağıdaki gibidir ,sepet array içindeki bu veriyi siparişler tablosuna kaydetmek istiyorum, bunu

DB::table('siparislers')->insert($request->all());

ile yaptığım zaman seçili ilk ürünleri ilgili tabloya kaydediyor, ama ikinci kez veri göndermeye çalıştığımda

Column not found: 1054 Unknown column '0' in 'field list'' in

hatası alıyorum ve kayıt yapmıyor.

siparisler::create($request->all());

ile katdetmeye çalıştığımda ise

local.ERROR: exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 1364 Field 'kat_id' doesn't have a default value' in

, hatası alıyorum ,bu datayı temiz bir hale mi dönüştürmem gerekiyor bu konuda biraz zayıfım , yardımcı olabilir misiniz ?

[2017-09-30 11:13:45] local.INFO: array (
 'sepet' => 
 array (
 0 => 
 array (
 'id' => 1,
 'kat_id' => 1,
 'name' => 'Kahvaltı Tabağı',
 'fiyat' => '14.00',
 'adet' => 5,
 'detay' => 'Kahvaltı Çeşitleri',
 'aciklama' => 'sd adadasd',
 'created_at' => '2017-09-19 23:26:18',
 'updated_at' => '2017-09-19 23:26:18',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1505863578.jpg" alt="" class="img img-responsive">',
 ),
 1 => 
 array (
 'id' => 2,
 'kat_id' => 2,
 'name' => 'Karışık Izgara',
 'fiyat' => '25.00',
 'adet' => 1,
 'detay' => 'Izgara Çeşitleri',
 'aciklama' => 'sadda dasdasd',
 'created_at' => '2017-09-19 23:26:40',
 'updated_at' => '2017-09-19 23:26:40',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1505863600.jpg" alt="" class="img img-responsive">',
 ),
 2 => 
 array (
 'id' => 3,
 'kat_id' => 1,
 'name' => 'Köy Kahvaltısı',
 'fiyat' => '25.00',
 'adet' => 3,
 'detay' => 'Kahvaltı Çeşitleri, Köy Kahvaltısı',
 'aciklama' => 'Açıklamalar',
 'created_at' => '2017-09-27 21:49:00',
 'updated_at' => '2017-09-27 21:49:00',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1506548940.jpg" alt="" class="img img-responsive">',
 ),
 ),
)

mgsmus

İstek parametreleri çok boyutlu dizi olduğu için create ile oluşturamazsınız.

insert ile ekleme yaptığınız kısım doğru fakat istek parametreleri içerisinde neden id anahtarı ve değeri var? id değerlerini kaldırmanız lazım

$data = array_map(function ($item) {
 unset($item['id']);
 return $item;
}, $request->all());

DB::table('siparislers')->insert($data);

Ayrıca kucuk_resim alanında sadece dosya adını (yani thumb_1506548940.jpg) tutun, resim programsal olarak sonra oluşturun.

GkyKrkc

Teşekkür ederim, deneyip dönecem inşallah

GkyKrkc

Üstadım eline sağlık , sayende sorunu çözdüm , bu haliyle veriyi kayıt yapabiliyorum, fonksiyon parametresi olarak gönderdiğim,$id ile siparisler tablosundan siparis numarasını çekiyorum, ilave olarak bunuda diziye eklemek istedim, aşağıdaki gibi yaptığımda

Route::post('/todos/{id}', function (Request $request, $id) {
 $siparis=Siparis::find($id);
 $siparis_no=$siparis->siparis_no; //burasını array_map içine almak istiyorum

 $data = array_map(function ($item) {
 unset($item['id']);
 unset($item['kucuk_resim']);
 $item['siparis_no']= $siparis_no; // buraya $siparis_no yu eklediğim zaman hata alıyorum
 return $item;
 }, $request->sepet);
 //DB::table('siparisler')->insert($data);
 Log::info($data);
});

$siparis_no yerine elle "siparis_no" gibi bir string ifade yazdığımda ise hata vermiyor, array_map içine dışarıdan yeni bir değeri nasıl ekleriz , array_push felan araştırdım denedim ama olmadı bu konuda yardımcı olabilir misin?

GkyKrkc

bu şekilde sorunu çözdüm ama , farklı bir yöntem var ise tecrübe edinmek için öğrenebiliriz üstadım ; selamlar

Route::post('/todos/{id}', function (Request $request, $id) {
 //Log::info($request->all()->sepet);

 $siparis=Siparis::find($id);
 $siparis_no=$siparis->siparis_no;

 $data = array_map(function ($item) use ($siparis_no) {
 unset($item['id']);
 unset($item['kucuk_resim']);
 //$item['siparis_no']=$siparis_no;
 return $item + array('siparis_no' => $siparis_no );
 }, $request->sepet);
 //DB::table('siparisler')->insert($data);
 Log::info($data);
});

deathisonitsway

$request->all()

derhal bundan uzaklaşmalısın.böyle bir şey çok riskli.tamamen clienta bırakmışsın insiyatifi.

GkyKrkc

nasıl yani tam anlamadım ne yönden riskli ,biraz açar mısın sana zahmet

sineld

Küçük bir bilgi vereyim, unset içine birden fazla anahtar girişi yapabilirsin:

unset($item['id']);
unset($item['kucuk_resim']);

yerine

unset($item['id'], $item['kucuk_resim']);

kullanabilirsin.

mgsmus

Diyelim ki bir tablonuz var, 3 alandan oluşuyor: baslik, icerik, kullanici_id.

Siz formla veri girişi yaptırıyorsunuz,sadece baslik ve icerik gireceksiniz mesela. kullanici_id girilmemesi lazım.

Eğer birileri sizin bu tablodaki alanların isimlerini bir şekilde öğrendiyse, örneğin url parametrelerinden (bazen hızlı olsun diye ?kullanici_id=1 vs deriz mesela), ve siz Makale::create($request->all()) şeklinde kayıt oluşturuyorsanız, formu post ettiğiniz adrese kendileri kullanici_id ekleyip bir istek göndererek istemediğiniz bir kayıt oluşmasını sağlayabilirler.

Bunu engellemek için Model özelliklerinde $fillable ve $guarded var:
https://laravel.com/docs/5.5/eloquent (Mass Assignment bölümünde)

//Makale.php
// Bu şekilde create ile kayıt oluştururken sadece baslik ve icerik alanlarının alınmasını sağlıyoruz. kullanici_id eklense bile kaydedilmeyecek
protected $fillable = [
 'baslik',
 'icerik'
];

Bu model için. Builder kullanırsanız, DB::table... vs diye, burada istek parametrelerini düzenlemek size ait:

$data = $request->only([
 'baslik',
 'icerik'
]);

$data['kullanici_id'] = $request->user()->id;

DB::table('makaleler')->insert($data)

Bu tür istek manipülasyonlarını engellemek için bir çok yol var, araştırabilirsiniz. Farklı tablo alan adları kullanma, bazı değerleri GET parametresi değil de oturum değişkeni olarak tutma vs gibi...

GkyKrkc

sineld yazdıKüçük bir bilgi vereyim, unset içine birden fazla anahtar girişi yapabilirsin:

unset($item['id']);
unset($item['kucuk_resim']);

yerine

unset($item['id'], $item['kucuk_resim']);

kullanabilirsin.

teşekkür ederim sinan hocam ,aynı şey dün benimde aklıma geldi , denedim oldu :)) bu sayede öğrenmiş oldum, eline sağlık teşekkür ederim.

GkyKrkc

mgsmus yazdıDiyelim ki bir tablonuz var, 3 alandan oluşuyor: baslik, icerik, kullanici_id.

Siz formla veri girişi yaptırıyorsunuz,sadece baslik ve icerik gireceksiniz mesela. kullanici_id girilmemesi lazım.

Eğer birileri sizin bu tablodaki alanların isimlerini bir şekilde öğrendiyse, örneğin url parametrelerinden (bazen hızlı olsun diye ?kullanici_id=1 vs deriz mesela), ve siz Makale::create($request->all()) şeklinde kayıt oluşturuyorsanız, formu post ettiğiniz adrese kendileri kullanici_id ekleyip bir istek göndererek istemediğiniz bir kayıt oluşmasını sağlayabilirler.

Bunu engellemek için Model özelliklerinde $fillable ve $guarded var:
https://laravel.com/docs/5.5/eloquent (Mass Assignment bölümünde)
//Makale.php
// Bu şekilde create ile kayıt oluştururken sadece baslik ve icerik alanlarının alınmasını sağlıyoruz. kullanici_id eklense bile kaydedilmeyecek
protected $fillable = [
 'baslik',
 'icerik'
];
Bu model için. Builder kullanırsanız, DB::table... vs diye, burada istek parametrelerini düzenlemek size ait:
$data = $request->only([
 'baslik',
 'icerik'
]);

$data['kullanici_id'] = $request->user()->id;

DB::table('makaleler')->insert($data)
Bu tür istek manipülasyonlarını engellemek için bir çok yol var, araştırabilirsiniz. Farklı tablo alan adları kullanma, bazı değerleri GET parametresi değil de oturum değişkeni olarak tutma vs gibi...

Allah razı olsun üstadım teşekkür ederim

evrend

@mgsmus +1,
Genelde sadece id'yi guarded yapıyorum. Zaten create methodunda gönderdiğim her şeyin kayıt edilmesini istiyorum. O yüzden problem yok. Bazı alanları korumak istiyorsanız, guarded'a eklemeniz yeterli. En az kodla en temiz çözüm.
Uygulamalarımın içerisindeki çoğu store methodu aşağıdaki gibi;

 public function store(OrderRequest $request)
 {
 return Order::create($request->all());
 }

Eğer sadece bazı alanların güncellenmesini istiyorsam;

 public function update(OrderRequest $request, $id)
 {
 Order::find($id)->update($request->only([ 'adress', 'description']));
 return 'true';
 }

deathisonitsway

GkyKrkc yazdınasıl yani tam anlamadım ne yönden riskli ,biraz açar mısın sana zahmet

gayet açık..request->all inputtan gelen datanın alayını database process ediyorsun...


$request->all=[

'username'=>'blabla',
'status'=>0

];

bu kod username ve status alanına insert yapar..sen default olarak status değerinin ilk etapta 0 olmasını istiyorsun.bu haliyle kullanıcı pasif durumda olsun.sen bu alanın email kontrolünden sonra aktif olmasını istiyorsun...sana ben 1 yolladıgımda napıcan dogrudan aktif yapacan beni.yada istenmeyen bir sürü sorunlar cıkar...

database processi request->all yapamazsın..yine de sen bilirsin.
Bütün insiyatifi clienta bırakıyorsun.request-all() var diye hurraaaa kullanayım yok oyle bir şey.

GkyKrkc

mgsmus yazdıİstek parametreleri çok boyutlu dizi olduğu için create ile oluşturamazsınız.

insert ile ekleme yaptığınız kısım doğru fakat istek parametreleri içerisinde neden id anahtarı ve değeri var? id değerlerini kaldırmanız lazım
$data = array_map(function ($item) {
 unset($item['id']);
 return $item;
}, $request->all());

DB::table('siparislers')->insert($data);
Ayrıca kucuk_resim alanında sadece dosya adını (yani thumb_1506548940.jpg) tutun, resim programsal olarak sonra oluşturun.

üstadım php ve diziler konusunda biraz eksiğim var bu konuda zorlanıyorum ; elimde şöyle bir veri var

[2017-10-10 13:30:12] local.INFO: array (
 0 => 
 array (
 'adi' => 'Gökay KARAKOÇ',
 'telefon' => 'xxxxxxxxxxx',
 'adres' => 'Kahramanmaraş',
 'bilgi' => 'Sipariş Notumuz Yok',
 'sepet' => 
 array (
 0 => 
 array (
 'id' => 1,
 'kat_id' => 1,
 'name' => 'Kahvaltı Tabağı',
 'fiyat' => '14.00',
 'adet' => 1,
 'detay' => 'Kahvaltı çeşitleri',
 'aciklama' => 'Kahvaltı Çeşitlerimiz',
 'created_at' => '2017-10-02 07:47:39',
 'updated_at' => '2017-10-02 07:47:39',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1506930459.jpg" alt="" class="img img-responsive">',
 'resim' => 'http://127.0.0.1:8000/uploads/urunler/thumb_1506930459.jpg',
 ),
 1 => 
 array (
 'id' => 2,
 'kat_id' => 2,
 'name' => 'Karışık Izgara',
 'fiyat' => '25.00',
 'adet' => 1,
 'detay' => 'Izgara Çeşitlerimiz',
 'aciklama' => 'asd asd asdasd',
 'created_at' => '2017-10-02 07:48:13',
 'updated_at' => '2017-10-02 07:48:13',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1506930493.jpg" alt="" class="img img-responsive">',
 'resim' => 'http://127.0.0.1:8000/uploads/urunler/thumb_1506930493.jpg',
 ),
 2 => 
 array (
 'id' => 3,
 'kat_id' => 2,
 'name' => 'Adana Kebap',
 'fiyat' => '15.00',
 'adet' => 1,
 'detay' => 'Izgara Çeşitlerimiz',
 'aciklama' => 'Adana kebap',
 'created_at' => '2017-10-02 07:49:59',
 'updated_at' => '2017-10-02 07:49:59',
 'kucuk_resim' => '<img src="http://127.0.0.1:8000/uploads/urunler/thumb_1506930599.jpg" alt="" class="img img-responsive">',
 'resim' => 'http://127.0.0.1:8000/uploads/urunler/thumb_1506930599.jpg',
 ),
 ),
 ),
)

gelen bu datayı parçalayıp 2 ayrı dizi haline getirmek istiyorum kisi_bilgileri, ve sepet şeklinde, kisi_bilgileri dizinde , ilk baştaki adi,telefon,adres, ve bilgiyi tutmak, sepet dizisinde ise sepetteki ürünleri detaydan sonraki kısmı unset ile silerek farklı iki dizi şeklinde çıkartmak istiyorum bunu nasıl yapabilirim. yardımcı olursanız sevinirim selamlar.

mgsmus

$sepet = $data[0]['sepet'];
unset($data[0]['sepet']);
$kisi = $data[0];

GkyKrkc

mgsmus yazdı

$sepet = $data[0]['sepet'];
unset($data[0]['sepet']);
$kisi = $data[0];

teşekkür ederim :)) hızır gibisin maşallah