Рейтинг: 0.7609  
На голосовании
Предложил Денис 12.04.2016 12:55:43

в компоненте sale.personal.profile.detail не запрашивать все локации из базы, если используются ajax-местоположения

в sale.personal.profile.detail есть кусок кода с говорящим комментарием: // perfomance hole

в нем запрашиваются ВСЕ местоположения независимо от настройки USE_AJAX_LOCATIONS
соответственно при большом количестве локаций и использовании AJAX-выбора страница не открывается, упираясь в лимит памяти


нужно использовать параметр компонента USE_AJAX_LOCATIONS и не запрашивать все местоположения впустую
Рейтинг: 7.6062  
На голосовании
Предложил Борисович Александр 15.08.2013 12:43:56

Раздел критики кода битрикса

Для улучшения кода компонентов и ядра, предлагаю открыть раздел, куда критики битрикса и недоумевающие разработчики могут выкладывать куски кода продукта с вопросами потипу "Боже, за такой говнокод обычно расстреливают"

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

Что это даст:

1) Исправление откровенного говнокода
2) Пристыжение такого кода в будущем
3) Улучшение качества продукта за счет комьюнити. ( Некоторые вообще предложат сами пути лучшего решения)
4) Улучшит отношение со стороны критики.


Прошу голосовать за. Этот раздел жизненно необхоим нам всем как нельзя больше.
Рейтинг: 37.0257  
Новая
Предложил Пупков Семен 02.05.2012 13:30:30

SetPropertyValuesEx и свойство типа файл

Ну почему, почему нельзя сделать так????Оо
$fid = CFile::SaveFile($arFile);
CIBlockElement::SetPropertyValuesEx($elID, false, array('PHOTO' => array("VALUE" => $fid, "DESCRIPTION"=>"Text"));

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

И вот это
if(strlen($value["DESCRIPTION"])>0)
    $val["description"] = $value["DESCRIPTION"];

Ну что за говнокод в ядре??