Update multiple rows with loop not working in Laravel

Question

Hi I have simple problem in my project. I want to update data in db. The sku codes sent with ajax. Like this A-1, A-2, A-3

I tried:

$sku2 = $request['sku'];
$sku = explode(',', $sku2);
foreach ($sku as $key => $value){
     $content = ProductNew::where('sku', $value)->update(['price' => 
     $price]);
}

But it updated only first record (A-1) Where is the problem?

This is full fuction

    public function setdiscount(Request $request)
    {
    $discount = Discount::where('id', $request['id'])->first();
    $d_price = $discount['price'];
    $d_type = $discount['type'];
    $price = 17;
    $sku2 = $request['sku'];
    $sku = explode(',', $sku2);
    foreach ($sku as $key => $value){
        ProductNew::whereIn('sku', $sku)->update(['price' => $price]);
    }



    }

ScriptsTown · Accepted Answer · 2019-06-01 16:00:25Z

3

You can try using wherein() along with update() to update multiple records:

$sku2 = $request['sku'];
$sku = explode(',', $sku2);

$sku = array_map('trim', $sku);

ProductNew::whereIn('sku', $sku)->update(['price' => $price]);

edited Jun 1, 2019 at 16:00

answered Jun 1, 2019 at 15:47

ScriptsTown

464 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

ScriptsTown Over a year ago

what's inside of $sku? is this an array with more than one value? check it: dd($sku);

Energy40 Over a year ago

array:5 [ 0 => "NHD-F070" 1 => " NHD-F070-1" 2 => " NHD-F070-2" 3 => " NHD-F070-3" 4 => " NHD-F070-4" ]

ScriptsTown Over a year ago

it seems to be no problem with this code, the issue might be somewhere else.

ScriptsTown Over a year ago

I notice there is space in other elements in your array. That is the issue. Look at this " NHD-F070-2" (there is a space at the start).

Energy40 Over a year ago

i added full function where the problem?

|

Collectives™ on Stack Overflow

Update multiple rows with loop not working in Laravel

1 Answer 1

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related