Как побороть ошибку Call to a member function GetProperties() on boolean (0)?

Question

[ChertyakaNUB]

Всем привет!

Попробовал создать товар без разделов с ценой 0 и в шаблоне корзины сделал запись:

$Cart = Sale\Basket::loadItemsForFUser(Sale\Fuser::getId(), Bitrix\Main\Context::getCurrent()->getSite());
$item = $Cart->createItem('catalog', 666); 
        $item->setFields(array(
          'PRODUCT_PRICE_ID' => 0,
          'PRICE' => -(int)$Price,
          'CURRENCY' => 'RUB',
          'QUANTITY' => 1,
          'LID' => Bitrix\Main\Context::getCurrent()->getSite(),
          'CUSTOM_PRICE' => 'Y',
          'NAME' => 'Скидон - '.$Price,
        ));
        $Cart->save();

При добавлении товара выскакивает ошибка:

[Error] 
Call to a member function GetProperties() on boolean (0)
/var/www/....php:17

С указанием файла и строки с кодом:

17 строка: $articul= CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()->GetProperties();

Народ, кто может подсказать, как решить сие проблему? При добавлении товара корзина падает и выходит ошибка та что выше указал. Но если навести на корзину в меню, то выпадет список товаров в корзине, и там будет добавленный товар, только без названия, но с ценой.

Comments

[Сергей Сергей]

Такие ошибки можно побороть пошаговой отладкой.
Начать можно с чтения текста ошибки, здесь виден неожиданный возвращённый тип.

[ChertyakaNUB]

Сергей Сергей, Этот пример добавления товара был взят из интернета, он не корректен или что?

[Сергей Сергей]

Очевидно, некорректен.

[PetrPo]

ChertyakaNUB, так ты показываешь вырванный кусок кода, где взаимосвязь первого и второго?
Код добавления в корзину и

$articul= CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()->GetProperties();

разные вещи. На каком сайте ты такой говнокод увидел?

проверки видимо дураки придумали

$articul = [];

$dbRes = CIBlockElement::GetByID($mxResult['ID']);
if($ob = $dbRes->GetNextElement()) {
	$articul = $ob->GetProperties();
}

[ChertyakaNUB]

PetrPo,

так ты показываешь вырванный кусок кода, где взаимосвязь первого и второго?

Я показал на что сослалась ошибка:

foreach ($arResult["GRID"]["ROWS"] as $id )
{
	
$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);

на эту 17 строку ссылается: $articul= CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()->GetProperties();
	//print_r($articul['ARTNUMBER']['VALUE']); echo '<br><br>';
$arFilter = Array("IBLOCK_ID"=>26, "NAME"=>$articul['ARTNUMBER']['VALUE']);
$arSelect = Array("ID", "IBLOCK_ID", "NAME", "code");
$res = CIBlockElement::GetList(Array(), $arFilter, false, Array(), $arSelect);
...

[ChertyakaNUB]

Пробовал официальные примеры добавления товара в корзину(при том, что сначала товар с нулевыми значениями вручную создается в админке) которые представлены в документации 1с, все равно такая же ошибка, значит проблема где-то еще.

[PetrPo]

ChertyakaNUB, что за замес из слов ты устраиваешь, давай по порядку
Что ты пытаешься сделать в фрагменте кода, где foreach ($arResult["GRID"]["ROWS"] as $id ) и ошибка?

[PetrPo]

PetrPo, думаю проблема в том, что какой-то товар не торговое предложение

$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);

после этого поставь var_dump($mxResult) думаю где-то будет false. Я же тебе выше написал про проверки, ты совсем не понимаешь? Ты товар не можешь получить, вот эта часть

CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()

возвращает boolean false, потому что нет у тебя $mxResult['ID'], а затем на этом false ты вызываешь метод GetProperties(). Улавливаешь? Call to a member function GetProperties() on boolean

[Сергей Сергей]

PetrPo, Очевидно, не улавливает.

[ChertyakaNUB]

PetrPo, если в товар добавить торговое предложение, все равно выходит ошибка. var_dump($mxResult) выдает в конце bool false.
Может из-за того что добавляется новый товар в корзину, но не пересчитывается все?

[PetrPo]

ChertyakaNUB,

если в товар добавить торговое предложение

что?

Блин, я не экстрасенс. Вот ты пишешь foreach ($arResult["GRID"]["ROWS"] as $id ) покажи что у тебя в $arResult["GRID"]["ROWS"]

[PetrPo]

ChertyakaNUB, если вот это $id['data']['PRODUCT_ID'] равно id торгового предложения, то CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']); никак не может вернуть false

[ChertyakaNUB]

PetrPo, Вы сказали, что:

думаю проблема в том, что какой-то товар не торговое предложение

Но если добавить торговое предложение в товар, ошибка остается.
Изначально товар был создан как простой без предложений в админке.

Сейчас попробовал добавить товар который имеет торговые предложения в админке, ошибка такая же.

То,что в $arResult["GRID"]["ROWS"]:
https://pastebin.com/QpnR0UhB
2 массив в коде новый товар, который добавил

[PetrPo]

ChertyakaNUB, епт, и че я должен понять в твоем var_dumpe
Короче у тебя несколько товаров в корзине: есть просто товар, есть товар с ТП? правильно

[ChertyakaNUB]

PetrPo,

епт, и че я должен понять в твоем var_dumpe

, а как можно еще показать что в $arResult["GRID"]["ROWS"]? Если показать, что идет после foreach, то после идет сразу тот код:

foreach ($arResult["GRID"]["ROWS"] as $id )
{
$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);
$articul= CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()->GetProperties();
$arFilter = Array("IBLOCK_ID"=>26, "NAME"=>$articul['ARTNUMBER']['VALUE']);
$arSelect = Array("ID", "IBLOCK_ID", "NAME", "code");
$res = CIBlockElement::GetList(Array(), $arFilter, false, Array(), $arSelect);
...

Все товары с ТП, я создал один простой товар, чтобы применять на нем либо скидку либо наценку.
В шаблоне корзины прописано добавление простого товара к товарам которые в корзине. Сейчас в корзине есть товар с ТП, и к нему добавляется простой товар и выходит ошибка.

[PetrPo]

ChertyakaNUB, ну так в этом и ошибка, когда твой цикл доходит до простого товара CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']); возвращает false, я хз как тебе еще объяснить.
Допиши после

$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);
$mxResult['ID'] = isset($mxResult['ID']) ? $mxResult['ID'] : $id['data']['PRODUCT_ID'];

[ChertyakaNUB]

PetrPo, но если я к простому товару в админке добавляю торговые предложения, то ошибка все равно есть.
Если я попробую добавить любой товар который есть в каталоге с ТП, то есть его ID($Cart->createItem('catalog', ANY_ID)), все равно есть ошибка.

Значит проблема не в том, что товар простой?

[PetrPo]

ChertyakaNUB, Ты в инфоблоке ТП вручную создаешь тп без привязки к товару чтоли?

[ChertyakaNUB]

PetrPo, вручную, зашел в простой товар и там поменял тип товара на товар с ТП и на вкладке ТП добавляю предложения.

Но дело то уже даже не в простом товаре, если я просто добавлю товар, который в каталоге уже существует с ценой и ТП в шаблоне корзины, ошибка так же проявляется.

Может надо при добавлении товара в корзину использовать какой-то параметр еще?

[PetrPo]

ChertyakaNUB, напиши так и скинь, не знаю как скринами или еще как-то, что там будет

echo 'First product id = '.current($arResult["GRID"]["ROWS"])['data']['PRODUCT_ID'].'<br>';
foreach ($arResult["GRID"]["ROWS"] as $id )
{
echo '<pre> product id';
print_r($id['data']['PRODUCT_ID']);
echo '</pre>';

$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);

echo '<pre> product info';
print_r($mxResult);
echo '</pre>';
//..............

[ChertyakaNUB]

PetrPo, вышло так с простым товаром:

First product id = 35733
 product id35733
 product infoArray
(
    [ID] => 34989
    [IBLOCK_ID] => 2
    [OFFER_IBLOCK_ID] => 3
    [SKU_PROPERTY_ID] => 28
)
 product id55619
 product infoArray
(
    [ID] => 41383
    [IBLOCK_ID] => 2
    [OFFER_IBLOCK_ID] => 3
    [SKU_PROPERTY_ID] => 28
)
 product id135215
 product info
[Error] 
Call to a member function GetProperties() on boolean (0)

Сейчас попробовал так добавить товар из каталог с ТП и товар добавился в корзину без ошибки, не знаю почему раньше была ошибка тогда.
Значит нужно как то к простому товару добавлять параметр ТП?

[PetrPo]

ChertyakaNUB, ну вот товар с id = 135215 - это простой товар?

[ChertyakaNUB]

PetrPo, Да, это простой товар

[PetrPo]

ChertyakaNUB, ну) давай включай голову

foreach ($arResult["GRID"]["ROWS"] as $id )
{
	$mxResult = CCatalogSku::GetProductInfo($id['data']['PRODUCT_ID']);
	$productId = $mxResult ? $mxResult['ID'] : $id['data']['PRODUCT_ID'];

	$articul = [];
	$dbRes = CIBlockElement::GetByID($productId);
	if($ob = $dbRes->GetNextElement()) {
	  $articul = $ob->GetProperties();
	}

	if($articul) {
		$arFilter = Array("IBLOCK_ID"=>26, "NAME"=>$articul['ARTNUMBER']['VALUE']);
		$arSelect = Array("ID", "IBLOCK_ID", "NAME", "code");
		$res = CIBlockElement::GetList(Array(), $arFilter, false, Array(), $arSelect);
		//..............
	}

[ChertyakaNUB]

PetrPo,
Какие параметры ТП нужно добавить в этот код? ([IBLOCK_ID] => 2 [OFFER_IBLOCK_ID] => 3 [SKU_PROPERTY_ID] => 28) ?

$item = $Cart->createItem('catalog', 135215); //создаём новый товар в корзине 
        $item->setFields(array(
          'CURRENCY' => 'RUB',
          'QUANTITY' => 1,
          'LID' => Bitrix\Main\Context::getCurrent()->getSite(),
	  'PRICE' => -(int)$Price,
          'CUSTOM_PRICE' => 'Y',
          'NAME' => 'Скидон- '.$Price,
        ));

Или простой товар в админке надо делать товаром с ТП?

[PetrPo]

ChertyakaNUB, друг, вопрос звучал "Как побороть ошибку Call to a member function GetProperties() on boolean (0)?", а теперь ты меня спрашиваешь как реализовать твой бизнес-процесс, я не вкурсе. Мы с тобой с трудом добрались до того что ошибка все таки из-за простого товара, а ты хочешь чтобы я тебе какую-то более глобальную задачу помог решить. Нет уж, спасибо. Это уже реальная работа, стоящая денег, а не подсказка на форуме

[ChertyakaNUB]

PetrPo, Спасибо за помощь. Разберусь сам.

[ChertyakaNUB]

Дааа, система блин, проблема крылась в том, что я указывал ID товара, а не ID торгового предложения(у него в систем создается другой ID).

[PetrPo]

ChertyakaNUB, ну так понятное дело, ТП - это такая же запись в БД в той же таблице что и простой товар, со своим ид
Если что, отметь решением, может кому-нибудь поможет ответ

Answer 1

[PetrPo]

В строке

$articul= CIBlockElement::GetByID($mxResult['ID'])->GetNextElement()->GetProperties();

ошибка, так как метод GetNextElement возвращает false, надо добавить проверку

$articul = [];

$dbRes = CIBlockElement::GetByID($mxResult['ID']);
if($ob = $dbRes->GetNextElement()) {
  $articul = $ob->GetProperties();
}

Comments

[ChertyakaNUB]

Спасибо!