Добрый день.
У меня Bitrix 8.5.1 Исследовал стандартный компонент bitrix/catalog.section и обнаружил, что в файле .description.php судя по всему строчка 29
является лишней.
Мне кажется, что по задумке массив $arProperty_LNS уже собран несколькими строчками выше и нет необходимости его очищать, иначе смысл предыдущей сборки данных теряется. Могу ошибаться, возможно уже исправлено в более старших версиях, но на всякий случай хочу обратить внимание.
Вот как выглядит код с 13 по 33 строчки:
У меня Bitrix 8.5.1 Исследовал стандартный компонент bitrix/catalog.section и обнаружил, что в файле .description.php судя по всему строчка 29
Код |
---|
$arProperty_LNS = array(); |
является лишней.
Мне кажется, что по задумке массив $arProperty_LNS уже собран несколькими строчками выше и нет необходимости его очищать, иначе смысл предыдущей сборки данных теряется. Могу ошибаться, возможно уже исправлено в более старших версиях, но на всякий случай хочу обратить внимание.
Вот как выглядит код с 13 по 33 строчки:
Код |
---|
$arProperty_LNS = array(); $arProperty_N = array(); $rsProp = CIBlockProperty::GetList(Array("sort"=>"asc", "name"=>"asc"), Array("ACTIVE"=>"Y", "IBLOCK_ID"=>$arCurrentValues["IBLOCK_ID"])); while ($arr=$rsProp->Fetch()) { $arProperty[$arr["CODE"]] = "[".$arr["CODE"]."] ".$arr["NAME"]; if(in_array($arr["PROPERTY_TYPE"], array("L", "N", "S"))) { $arProperty_LNS[$arr["CODE"]] = "[".$arr["CODE"]."] ".$arr["NAME"]; } if($arr["PROPERTY_TYPE"]=="N") { $arProperty_N[$arr["CODE"]] = "[".$arr["CODE"]."] ".$arr["NAME"]; } } $arProperty_LNS = array(); // [NOTICE] Эта строчка, если не ошибаюсь, тут не нужна... $arUserFields = $GLOBALS["USER_FIELD_MANAGER"]->GetUserFields("IBLOCK_".$arCurrentValues["IBLOCK_ID"]."_SECTION"); foreach($arUserFields as $FIELD_NAME=>$arUserField) if($arUserField["USER_TYPE"]["BASE_TYPE"]=="string") $arProperty_LNS[$FIELD_NAME] = $arUserField["LIST_COLUMN_LABEL"]? $arUserField["LIST_COLUMN_LABEL"]: $FIELD_NAME; |