Плюс-минус товаров в корзине. Как сделать удаление?

Question

[sdgs4s4 .]

Всем привет!

Визуально вот такая штука)


Есть вывод товаров в модальном окне

<?php foreach($session['cart'] as $id => $item):?>
    <tr>
        <td><?= \yii\helpers\Html::img($item['img'], ['alt' => $item['name'], 'height' => 50]) ?></td>
        <td><?= $item['name']?></td>
        <td>
            <span data-id="<?= $id ?>" type="submit" class="del-n"><span class="button__inc">-</span></span>
            <?= $item['qty']?>
            <span data-id="<?= $id ?>" type="submit" class="add-to-cart"><span class="button__inc">+</span></span></td>
        <td><?= $item['price']?></td>
        <td><span data-id="<?= $id?>" class="glyphicon glyphicon-remove text-danger del-item" aria-hidden="true"></span></td>
    </tr>
<?php endforeach?>

.del-n - должен удалять 1 единицу товара, но не удаляет
.add-to-cart - прибавляет 1 ед. товара, сделал, работает

А это js, контроллеры, модели на удаление, не могу понять что тут надо сделать
Проблема в том, что при удалении он удаляет позицию товара полностью, вне зависимости 4 шт указано или 1

JS

$('#cart .modal-body').on('click', '.del-n', function () {
    var id = $(this).data('id'),
        qty = $(this).prev('.js-qty').val();
    $.ajax({
        url: '/cart/del-pr',
        data: {id: id, qty: qty},
        type: 'GET',
        success: function(res){
            if(!res) alert('Ошибка!');
            showCart(res);
        },
        error: function(){
            alert('Error!');
        }
    });
});

Контроллер

public function actionDelPr(){
        $id = Yii::$app->request->get('id');
        $qty = (int)Yii::$app->request->get('qty');

        $session =Yii::$app->session;
        $session->open();
        $cart = new Cart();
        $cart->recalcp($id, $qty);
        $this->layout = false;
        return $this->render('cart-modal', compact('session'));
    }

Модель

public function recalcp($id){
        if(!isset($_SESSION['cart'][$id])) return false;
        $qtyMinus = $_SESSION['cart'][$id]['qty'] === 1;
        $sumMinus = $_SESSION['cart'][$id]['price'];
        $_SESSION['cart.qty'] -= $qtyMinus;
        $_SESSION['cart.sum'] -= $sumMinus;
        unset($_SESSION['cart'][$id]);
    }

Comments

[Arman]

1. GET не должен менять состояние, меняйте на POST
2. вроде session->open() - лишнее
3. зачем лезть в $_SESSION если уже начали использовать Yii session?
4. В контроллере запускаете с двумя аргументами, а сам метод recalcp с одним?

[sdgs4s4 .]

Arik,

Model

public function recalcp($id, $qty){
        if(!isset($_SESSION['cart'][$id])) return false;
        $qtyMinus = $_SESSION['cart'][$id][$qty] === 1;
        $sumMinus = $_SESSION['cart'][$id][$qty] * $_SESSION['cart'][$id]['price'];
        $_SESSION['cart.qty'] -= $qtyMinus;
        $_SESSION['cart.sum'] -= $sumMinus;
        unset($_SESSION['cart'][$id]);
    }

Controller

public function actionDelPr(){
        $id = Yii::$app->request->post('id');
        $qty = (int)Yii::$app->request->post('qty');

        $session =Yii::$app->session;
        $cart = new Cart();
        $cart->recalcp($id, $qty);
        $this->layout = false;
        return $this->render('cart-modal', compact('session'));
    }

js

$('#cart .modal-body').on('click', '.del-n', function () {
    var id = $(this).data('id'),
        qty = $(this).prev('.js-qty').val();
    $.ajax({
        url: '/cart/del-pr',
        data: {id: id, qty: qty},
        type: 'POST',
        success: function(res){
            if(!res) alert('Ошибка!');
            showCart(res);
        },
        error: function(){
            alert('Error!');
        }
    });
});

В итоге все тоже самое, удаляет товар полностью, а не его кол-во

[Arman]

lipatkin ., а что делает unset($_SESSION['cart'][$id]);? там храните наличие товара в корзине?

[sdgs4s4 .]

Arik, да

Шибко много не понимаю, когда то поставил все с уроков на ютубе ( https://goo.gl/PmPsSr )

[sdgs4s4 .]

Как я понял unset "разрушает" id товара, типо удалить товар и все

Answer 1

[Максим Тимофеев]

<span type="submit"
Это новое слово в web дизайне?

$(this).prev('.js-qty').val();
вы проверяли что там? Там undefined, потому что нет ничего перед ним на том же уровне. А скорее даже ошибка в консоле.

ПС: Работаете с js - консоль всегда открыта, смотрите что улетает ajax-ом. Тогда бы Вы сразу заметили проблему