AT
написание кода конечно хромает, но логика то правильная?
Да, работать будет правильно, но запрос в базу будет очень затратный
Size: a a a
2020 August 20
AT
В этом случае он сначала берет все данные с базы, а потом из этого берет последний
J
написание кода конечно хромает, но логика то правильная?
$orderItem->id = $orderId; - у тебя не инкремент разве?А
дело в том что я беру последнюю запись из заказов
AP
Дело в том, что это может быть не та запись которая тебе нужна
AP
$order = Order::create($request->all());
вот так сделай
А
делаю
AP
дело в том что я беру последнюю запись из заказов
но в твоем случае ты берешь все записи и только потом уже из выбранных последнюю
А
да согласен
AT
Александр я делаю вот так
public function checkoutComplete(Request $request)
{
Order::create($request->all());
$orderId = Order::all()->last();
foreach (session('cart') as $item):
$orderItem = new OrderItem();
$orderItem->id = $orderId;
$orderItem->name = $item['name'];
$orderItem->price = $item['price'];
$orderItem->qty = $item['qty'];
$orderItem->save();
endforeach;
}
public function checkoutComplete(Request $request)
{
Order::create($request->all());
$orderId = Order::all()->last();
foreach (session('cart') as $item):
$orderItem = new OrderItem();
$orderItem->id = $orderId;
$orderItem->name = $item['name'];
$orderItem->price = $item['price'];
$orderItem->qty = $item['qty'];
$orderItem->save();
endforeach;
}
Также, тут скорее всего в строке
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
DA
Также, тут скорее всего в строке
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
Вообще не надо так получать id здесь - есть вероятность, что это будет id чужого заказа
ДК
Order::last() же
Gt
Также, тут скорее всего в строке
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
$orderId = Order::all()->last()
Нужно в конец id добавить
$orderId = Order::all()->last()->id
А если там 100500 заказов будет, будем сразу все выбирать чтобы получить ПОСЛЕДНИЙ?
AT
Order::latest()->first()->id ?
AT
Order::last() же
А так можно?
DA
Блин, вы прикалываетесь что ли:) Тут в принципе не надо выбирать last
А
да понял свою ошибку, он всегда будет брать последний заказ даже если это не мой заказ
AP
зачем делать такое?
R#
А так можно?
я в этом чате уже нормально всяких премудростей узнал
видать да )))
видать да )))