Существует ли товар в корзине Bitrix?

Question

[Mikhail Kim]

Дано:
Редакция Интернет-магазин + CRM.
Все обновления актуальны.

Задача:
Проверить - существует ли товар в корзине.
Тип товара - SKU с 1 свойством, добавляемым в корзину.

Имею код:

$obBasket = Basket::loadItemsForFUser(
	FUser::getId(),
	's3'
);

$productId = $_REQUEST['id'];
$obItem = $obBasket->getExistsItem('catalog', $productId, ['CODE' => 'PROP1', 'VALUE' => 'electron']);
var_dump($obItem); // null

Товар в корзине точно есть, но по-какой-то причине возвращается null.

Answer 1

[PetrPo]

Ждал красивого решения от битрикса? )) А вот и хрен. Чтобы получить твой \Bitrix\Sale\BasketItem $obItem придется передать все его $properties в 3-ий параметр getExistsItem, типа

$obItem = $obBasket->getExistsItem('catalog', $productId, [
	0 => ['CODE' => 'PROP1', 'VALUE' => 'electron'],
	//......
]);

вообще не понятно зачем этот метод public, если им хрен воспользуешься, надо знать все свойства товара в корзине, у меня это выглядело так

$obItem = $obBasket->getExistsItem('catalog', $productId, [
	0 => ['CODE' => 'ARTNUMBER', 'VALUE' => '235-81-03'],
	1 => ['CODE' => 'COLOR_REF', 'VALUE' => 'Черный'],
	2 => ['CODE' => 'SIZES_CLOTHES', 'VALUE' => 'XS'],
	3 => ['CODE' => 'CATALOG.XML_ID', 'VALUE' => 'clothes_offers_s1'],
	4 => ['CODE' => 'PRODUCT.XML_ID', 'VALUE' => '332#337'],
]);

Вообщем решение конечно будет не таким красивым, как просто методом воспользоваться, но если метод getExistsItem посмотреть, там тоже самое делается, только еще кое-что

\Bitrix\Main\Loader::includeModule('sale');

$siteId = 's1';
$fUserId = \Bitrix\Sale\FUser::getId();
$productId = 139;
$productByBasketItem = null;
$bProductInBasket = false;

$basket = \Bitrix\Sale\Basket::loadItemsForFUser($fUserId, $siteId);
$basketItems = $basket->getBasketItems();

if($basketItems) {
	foreach($basketItems as $basketItem) {
		if($basketItem->getField('PRODUCT_ID') == $productId) {
			$productByBasketItem = $basketItem;
			$bProductInBasket = true;
			break;
		}
	}
}

var_dump($bProductInBasket);
var_dump($productByBasketItem);

В примере написал булево $bProductInBasket и записал сам объект \Bitrix\Sale\BasketItem в $productByBasketItem, там сам по ситуации выбирай как надо

Comments

[Mikhail Kim]

Формально конечно интересовал метод getExistsItem, даже с передачей всех свойств возвращается null.

Способ с циклом подошел, хотя не комильфо.

[PetrPo]

Mikhail K, ты не прав. Все нормально возвращается, если передать все свойства

Напиши так и увидешь все свойства товара в корзине, если все их передать в формате, который я указывал в ответе, все будет работать

\Bitrix\Main\Loader::includeModule('sale');

$siteId = 's1';
$fUserId = \Bitrix\Sale\FUser::getId();
$productId = 139;
$productByBasketItem = null;
$bProductInBasket = false;

$basket = \Bitrix\Sale\Basket::loadItemsForFUser($fUserId, $siteId);
$basketItems = $basket->getBasketItems();

if($basketItems) {
	foreach($basketItems as $basketItem) {
		if($basketItem->getField('PRODUCT_ID') == $productId) {
			$basketPropertyCollection = $basketItem->getPropertyCollection();
			print_r($basketPropertyCollection->getPropertyValues());
		}
	}
}

Чем тебя не устраивает цикл, говорю тебе в этом методе тот же самый цикл


Если ты хочешь много раз использовать этот код, сделай свой класс, тут можно по-разному написать, я самый очевидный, но не самый оптимальный, вариант приведу

class Basket {
	protected $siteId;
	protected $fUserId;
	
	protected $basket;
	protected $basketItems;
	
	public function __construct($siteId = 's1', $fUserId = 0) {
		$this->siteId = $siteId;
		$this->setFUserId($fUserId);
		$this->setBasket();
		$this->setBasketItems();
	}
	
	protected function setFUserId($fUserId) {
		$this->fUserId = $fUserId ? $fUserId : \Bitrix\Sale\FUser::getId();
	}
	
	protected function setBasket() {
		$this->basket = \Bitrix\Sale\Basket::loadItemsForFUser($this->fUserId, $this->siteId);
	}
	
	protected function getBasket() {
		return $this->basket;
	}
	
	protected function setBasketItems() {
		$this->basketItems = $this->basket->getBasketItems();
	}
	
	public function isProductInBasket(int $productId) {
		$basketItem = $this->getBasketItemByProductId($productId);
		
		return $basketItem ? true : false;
	}
	
	public function getBasketItemByProductId(int $productId) {
		$basketItem = null;
		
		if($this->basketItems) {
			foreach($this->basketItems as $item) {
				if($item->getField('PRODUCT_ID') == $productId) {
					$basketItem = $item;
					break;
				}
			}
		}
		
		return $basketItem;
	}
}

$siteId = 's1';
$fUserId = \Bitrix\Sale\FUser::getId();
$productId = 139;

$basket = new \Basket($siteId, $fUserId);
$basketItem = $basket->getBasketItemByProductId($productId);
$bProductInBasket = $basket->isProductInBasket($productId);

[Mikhail Kim]

Благодарю за развернутый ответ.
Проверю, отпишусь.

Вопрос по свойствам:
CATALOG.XML_ID
PRODUCT.XML_ID
это что за свойства?
Если с PRODUCT.XML_ID понятно, это свойство, полагаю, берется из таблицы b_catalog_product . Верно?

[Mikhail Kim]

В догонку к свойствам.
Я проверяю инфоблок торговых предложений.
Соответственно - не хватает ссылки на родительский элемент инфоблока.
Ее тоже добавлять в формате ['CML2_LINK' => ] ?

[PetrPo]

Mikhail K, В этом примере тоже торговое предложение

$obItem = $obBasket->getExistsItem('catalog', $productId, [
  0 => ['CODE' => 'ARTNUMBER', 'VALUE' => '235-81-03'],
  1 => ['CODE' => 'COLOR_REF', 'VALUE' => 'Черный'],
  2 => ['CODE' => 'SIZES_CLOTHES', 'VALUE' => 'XS'],
  3 => ['CODE' => 'CATALOG.XML_ID', 'VALUE' => 'clothes_offers_s1'],
  4 => ['CODE' => 'PRODUCT.XML_ID', 'VALUE' => '332#337'],
]);

CATALOG.XML_ID - это XML_ID инфоблока торговых предложений
PRODUCT.XML_ID - состоит из XML_ID товара и XML_ID торгового предложения.
332 = XML_ID товара, 337 = XML_ID ТП

Про CML2_LINK не знаю. Выше код есть, над скриншотом про метод getExistsItem, запусти его и увидишь все свойства товара в корзине

[awizard]

Спасибо! Потратил пол дня на эту хрень.