Форум

17.0.8

но еще раз уточню эту ситуацию поймать проблематично. Примерно на 50-100 заказов проиходится 1-2 таких.

Подскажите, вдруг кто сталкивался, как бороться с такой проблемой.

Для добавления в корзину используется Add2BasketByProductID($id, $_REQUEST['qty'] ? $_REQUEST['qty'] : 1, array(), $arParams)

Товар добавляется, но по какой-то причине переодически товары оказываются без PRODUCT_XML_ID

[WEIGHT] => 0.00
           [QUANTITY] => 1.0000
           [DELAY] => N
           [CAN_BUY] => Y
           [MODULE] => catalog
           [PRODUCT_PROVIDER_CLASS] => CCatalogProductProvider
           [NOTES] => Интернет магазин
           [DETAIL_PAGE_URL] =>
           [DISCOUNT_PRICE] => 250
           [CATALOG_XML_ID] => 8d6cef00-3fdb-4dba-be75-d32bceaa348e#
          [COLOR=#FF0000] [PRODUCT_XML_ID] => [/COLOR]
           [DISCOUNT_NAME] =>
           [DISCOUNT_VALUE] =>
           [DISCOUNT_COUPON] =>
           [VAT_RATE] => 0.0000

из-за этого когда заказы падают в 1С они там не прогружаются и получается на сайте заказ на 1000 рублей, а в 1С на 100(только сумма доставки)

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

[QUOTE]Сергей Цыбульский написал:
не вводите людей в заблуждение. Как писалось выше данный функционал доступен только в редакции "Бизнес". В редакции "Малый бизнес" такого нет.[/QUOTE]
[URL=https://yadi.sk/i/WTgsnDhpuX-Qbw]скрин[/URL]

да последний вариант уже первоначальный код не в 17 раз памяти больше использует, в 2
но и время возросло до 5-8 секунд

[QUOTE]Я дал вам ссылку на документацию. Там четко написано - при передаче 4-го параметра меняется только значение свойства с этим кодом.[/QUOTE]
сейчас увидел
[QUOTE]Используйте CIBlockElement::GetPropertyValuesArray для получения свойств набора элементов.[/QUOTE]
как раз 2 вариант
[QUOTE]Если второй версии, то почему бы сразу их не перечислить те свойства, которые необходимо получить?[/QUOTE]
в первом варианте нужны дополнительные данные по свойствам, такие заначения и описания свойства поэтому использовался
[CODE] while (($product = $products->GetNextElement()) !== false) {
           $productFields = $product->GetFields();
           $productProps = $product->GetProperties();[/CODE]

[QUOTE]Зачем в том же цикле сразу делать CFile::GetFileArray и изменение элемента? Можно собрать массив из этих данных, за 1 запрос получить данные и дальше уже изменить их в БД.
Выйдет быстрее.[/QUOTE]
так как мне нужны описания всех картинок, то и собираю их в цикле и чтобы в дальнейшем не плодить циклов прямо тут же произвожу изменния
Если можете подсказать более правильный вариант, я буду только за

и еще момент - я привел 2 решения, но складывается ощущение, что видите только одно

первый вариант был написан давно, я его и изменяю из-за его скорости - 2 вариант(работает быстро, но памяти потребляет очень много, превышает лимит установленный в настройках сервера ).
[QUOTE]если достаточно цикла по $products->Fetch() без получения свойств?[/QUOTE]
свойства нужны и если пользоваться в первом варианте fetch то их не получишь

[QUOTE]Кроме того, вы используете \CIBlockElement::SetPropertyValuesEx, хотя меняете только одно свойство. В этом случае быстрее использовать SetPropertyValues с указанием символьного кода свойства. [/QUOTE]
да изменяется 1 свойство, и насколько я знаю как раз для этого и нужно SetPropertyValuesEx, а SetPropertyValues требует перечисления всех свойств не зависимо нужно их менять или нет

Всем здравствуйте.
Есть 2 варианта одного и того же - заполнение свойства по дескрипшину картинки.

1 вариант - работает очень долго(20-28 секунд), но потребяет памяти 94980808

[CODE] $products = \CIBlockElement::GetList(
           [],
           [
               'IBLOCK_ID' => 26,
               'ACTIVE' => 'Y',
           ],
           false,
           false,
           [
               'ID',
               'IBLOCK_ID',
'DETAIL_PICTURE',
           ]
       );

       while (($product = $products->GetNextElement()) !== false) {
           $productFields = $product->GetFields();
           $productProps = $product->GetProperties();

$arFile = \CFile::GetFileArray($productFields["DETAIL_PICTURE"]);
if ($arFile['DESCRIPTION'] == 'color') {
                   if ($productProps['COLOR_IMG']['VALUE'] != $productFields["DETAIL_PICTURE"]) {
                       \CIBlockElement::SetPropertyValuesEx(
                           $productFields['ID'],
                           $productFields['IBLOCK_ID'],
                           [
                               'COLOR_IMG' => $productFields["DETAIL_PICTURE"],
                           ]
                       );
                   }
           }else{
foreach ($productProps['MORE_PHOTO']['VALUE'] as $key => $value) {
if ($productProps['MORE_PHOTO']['DESCRIPTION'][$key] == 'color') {
if ($productProps['COLOR_IMG']['VALUE'] != $value) {
\CIBlockElement::SetPropertyValuesEx(
$productFields['ID'],
$productFields['IBLOCK_ID'],
[
'COLOR_IMG' => $value,
]
);
}
}
}
}
       }[/CODE]

2 вариант(2-4 секунды), но памяти потребляет 1651298448(что в 15-17 раз больше предыдущего варианта)

[CODE]
$products = \CIBlockElement::GetList(
           [],
           [
               'IBLOCK_ID' => 26,
               'ACTIVE' => 'Y',
           ],
           false,
           false,
           [
               'ID',
               'IBLOCK_ID',
'DETAIL_PICTURE',
           ]
       );
$rprArr=[];
       while ($fields = $products->Fetch()) {
           $items[$fields['ID']] = [
               'ID' => $fields['ID'],
               'IBLOCK_ID' => $fields['IBLOCK_ID'],
               'DETAIL_PICTURE' => \CFile::GetFileArray($fields["DETAIL_PICTURE"])
           ];
           
           $rprArr[$fields['ID']] = $fields['ID'];
       }
\CIBlockElement::GetPropertyValuesArray($rprArr, 26, array(
           'ID' => $rprArr,
           'IBLOCK_ID' => 26,
       ));
foreach($rprArr as $kl=>$vl){
           if ($items[$kl]['DETAIL_PICTURE']['DESCRIPTION'] == 'color') {
                   if ($vl['COLOR_IMG']['VALUE'] != $items[$kl]['DETAIL_PICTURE']["ID"]) {
                       \CIBlockElement::SetPropertyValuesEx(
                           $kl,
                           $items[$kl]['IBLOCK_ID'],
                           [
                               'COLOR_IMG' => $items[$kl]['DETAIL_PICTURE']["ID"],
                           ]
                       );
                   }
           }else{
foreach ($vl['MORE_PHOTO']['VALUE'] as $key => $value) {
if ($vl['MORE_PHOTO']['DESCRIPTION'][$key] == 'color') {
if ($vl['COLOR_IMG']['VALUE'] != $value) {
\CIBlockElement::SetPropertyValuesEx(
$kl,
$items[$kl]['IBLOCK_ID'],
[
'COLOR_IMG' => $value,
]
);
}
}
}
}
       }
[/CODE]

Получается хочешь уложиться во время выполнения скрипта 2 вариант лучше, но упираешься в нехватку памяти
пытаешься уложиться в память упираешься в предел времени выполнения

как быть? и тот вариант плох и другой

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

[QUOTE]require($_SERVER["DOCUMENT_ROOT"]."/bitrix/modules/main/include/prolog_before.php");
use \Bitrix\Main\SystemException;
array_map('CModule::IncludeModule', ['iblock', 'catalog', 'sale']);

$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => 10]);[/QUOTE]

больше в нем ничего нет. при попытке вызвать этот файл получаю ошибку указанную выше.

по похожей ошибке нашел тему [URL=https://dev.1c-bitrix.ru/support/forum/forum32/topic97193/]https://dev.1c-bitrix.ru/support/forum/forum32/topic97193/[/URL]

но пока никак не пойму как с этим разобраться установил ulimit -n 10240.
Если в консоле из под рута с таким лимитом запустить, скрипт отрабатывает, без лимита нет

Максим, вы только ulimit изменяли?

я пытаюсь обновить количество товара
[QUOTE]$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => 90]);[/QUOTE]
а в ответ ошибка как у вас

Евгений Жуков, можете подсказать, указываю обновить количество в карточке товара по сумме количества торгового предложения

[QUOTE]
require($_SERVER["DOCUMENT_ROOT"]."/bitrix/modules/main/include/prolog_before.php");
use \Bitrix\Main\SystemException;
array_map('CModule::IncludeModule', ['iblock', 'catalog', 'sale']);
/* ... */
$obProduct = new CCatalogProduct();
$obProduct->Update(369109, ['QUANTITY' => $items[$id]['QUANTITY']]);[/QUOTE]
но в ответ получаю
[QUOTE]Fatal error: Uncaught Error: Class 'Bitrix\Main\Diag\ExceptionHandlerLog' not found in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php:266 Stack trace: #0 [internal function]: Bitrix\Main\Diag\ExceptionHandler->handleException(Object(Error)) #1 {main} thrown in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php on line 266

Fatal error: Uncaught Error: Class 'Bitrix\Main\Diag\ExceptionHandlerLog' not found in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhand­ler.php:350 Stack trace: #0 [internal function]: Bitrix\Main\Diag\ExceptionHandler->handleFatalError() #1 {main} thrown in /var/www/shop/www/bitrix/modules/main/lib/diag/exceptionhandler.php on line 350[/QUOTE]