[Решено] Запрет заказывать больше чем есть в наличии

MGT1, не поможет. Суть вопроса в том, чтоб не дать положить в корзину больше. чем есть на складе, а не чтоб покупатель увидел сколько на складе и сам не положил в корзину ещё единицу.

Надо в корзине при добавлении товара делать проверку остатка и дальше либо добавлять в корзину, либо что-то сообщать покупателю о нехватке и невозможности добавить в корзину ещё.
skiv14, какая версия/сборка?

P.S. @MGT1, следи за стилями при копировании откуда-то, здесь слепых нет (но есть правило про увеличенный шрифт).

 

Ну, так моё решение будет для конкретной, а не для любой.

Я говорил не о странице корзины, а о классе корзины, который обрабатывает добавление в корзину.
Но есть маленький нюанс, который всё очень усложняет - остаток в базе включает и то, что уже есть в корзине (оно не вычитается со склада до завершения оформления заказа).

Итак, файл /system/library/cart.php, добавляем это:

PHP:

public function checkStock($product_id, $qty){
   $canAdd = true;
   $cartProducts = $this->getProducts();

   if(!empty($cartProducts)){
     $products_ids = array();

     foreach ($cartProducts as $pData) {
       if(isset($products_ids[$pData['product_id']])){
         $products_ids[$pData['product_id']]++;
       } else {
         $products_ids[$pData['product_id']] = 1;
       }
     }

     foreach ($products_ids as $p_id => $count) {
       if($product_id == $p_id){
         $stock = $this->db->query("SELECT quantity FROM " . DB_PREFIX . "product p
           WHERE p.product_id = '" . (int)$product_id . "'
           AND p.date_available <= NOW() AND p.status = '1'")->row();

         if($stock - $count - $qty <= 0){
           $canAdd = false;
         }

         break;
       }
     }
   }

   return $canAdd;
}


Файл /catalog/controller/checkout/cart.php, метод add, перед

PHP:

if (!$json) {
                $this->cart->add($this->request->post['product_id'], $quantity, $option);

Добавляем это:

PHP:

$canAdd = $this->cart->checkStock($this->request->post['product_id'], $quantity);
if(!$canAdd){
    $json['error']['stock'] = 'Error message';
}

А дальше просто выводим сообщение из $json['error']['stock'] через js в функции добавления товара в корзину (это будет раздельно в common.js и на странице товара + зависит от шаблона).

P.S. Это решение учитывает, что один товар может быть в корзине несколько раз из-за разных выбранных опций. Все такие товары просуммируются перед рассчётом остатка.

 

Судя по этой ошибке, проблема на стороне js. Мой код никак не влияет и не может влиять на кнопки. Он должен работать так: покупатель кликает на кнопку купить, а дальше товар либо добавляется в корзину, либо выдаётся сообщение, что остаток на складе недостаточен (за эту часть отвечает js, который я не приводил). Сама кнопка при этом активна даже если товара изначально было 0 (при условии, что используется дефолтный ОК без каких-либо модулей, которые что-то делают с кнопкой).

Дай адрес сайта.

 

При втором добавлении - ошибка 500, потому и не добавляется.
Сделай это: https://wmasteru.org/threads/Настройки-оптимизации-фичи-статьи-для-opencart.2219/page-7#post-151032

А потом смотри журнал ошибок.

Я ж этот код из головы писал и не тестировал, там, конечно, могут быть какие-то ошибки, которые нужно отловить.

 

А вот это уже логическая недоработка. Я не учёл, что в пустую корзину могут сразу добавить больше, чем есть на складе. Сейчас допишу.

 

Ошибка примитивнейшая. Вот, что бывает, когда начинаешь писать код на автомате - там просто скобки лишние, ибо row это не метод, а поле.

Обнови метод checkStock:

PHP:

public function checkStock($product_id, $qty){
    $canAdd = true;
    $count = 0;
    $cartProducts = $this->getProducts();
  
    if(!empty($cartProducts)){
        $products_ids = array();

        foreach ($cartProducts as $pData) {
            if(isset($products_ids[$pData['product_id']])){
                $products_ids[$pData['product_id']]++;
            } else {
                $products_ids[$pData['product_id']] = 1;
            }
        }

        if(isset($products_ids[$product_id])){
            $count = $products_ids[$product_id];
        }
    }

    $stock = $this->db->query("SELECT quantity FROM " . DB_PREFIX . "product p
                    WHERE p.product_id = '" . (int)$product_id . "'
                    AND p.date_available <= NOW() AND p.status = '1'")->row;

    if($stock - $count - $qty <= 0){
        $canAdd = false;
    }

    return $canAdd;
}

Здесь и эта ошибка исправлена и проблема с добавлением в корзину сразу большего количества, чем есть на складе.

Я принципиально не использую vQmod, а здесь без него не обойтись, если оформлять в виде модуля.

 

Вот окончательный вариант:

PHP:

public function checkStock($product_id, $qty){
    $canAdd = true;
    $count = 0;
    $cartProducts = $this->getProducts();
   
    if(!empty($cartProducts)){
        $products_ids = array();

        foreach ($cartProducts as $pData) {
            if(isset($products_ids[$pData['product_id']])){
                $products_ids[$pData['product_id']] += (int)$pData['quantity'];
            } else {
                $products_ids[$pData['product_id']] = (int)$pData['quantity'];
            }
        }

        if(isset($products_ids[$product_id])){
            $count = $products_ids[$product_id];
        }
    }

    $stock = $this->db->query("SELECT quantity FROM " . DB_PREFIX . "product p
                    WHERE p.product_id = '" . (int)$product_id . "'
                    AND p.date_available <= NOW() AND p.status = '1'")->row;

    if($stock['quantity'] - $count - $qty < 0){
        $canAdd = false;
    }

    return $canAdd;
}

Вроде, уже всё учёл.

 

Да, на сервере всё работает, как надо. Но надо ж ещё для покупателя сообщение выводить, а то оно просто не добавляется, будто это баг.