Мы все периодически копаемся в коде
Над движком работают постоянно, работает команда
Тут очень просто не увидеть, в результате обязательно будут появляться дублирующиеся вызовы, условия, присваивания. Отследить их специально очень сложно, и времязатратно.
ПРЕДЛАГАЮ
мы все копаемся в коде, как я уже написал, и наверняка периодически замечаем такие мелочи
Вот допустим сегодня я увидел, что в файле
app/addons/rus_exim_1c/Tygh/Commerceml/RusEximCommerceml.php
в функции getDataOrderUser
строка 2972
if (!isset($order_data[‘phone’])) {
$order_data[‘phone’] = ‘-’;
}
и строка 3012
$phone = (!empty($order_data['phone'])) ? $order_data['phone'] : '-';
делают одно и тоже, поэтому второе написание на 3012 можно заменить на $phone = $order_data[‘phone’];
Про одну такую мелочовку которая даст пару микросекунд выигрыша за минусом лишнего условия в коде - писать в багтрекер пустое. Но если мы будем писать в эту тему о подобных замеченных в процессе своей работы неочевидных местах, которые позволят оптимизировать выполнение кода - думаю движок сможет работать еще быстрее ))
7 лайков
Здравствуйте, alex_vp,
Спасибо за замечание и отличное предложение, но у него есть один недостаток — при наплыве сообщений на баг-трекер, данная тема просто утонет в них и в итоге потеряется. Я могу попробовать сделать прикреплённую тему, в которой пользователи будут оставлять подобные замечания. А пока что, рекомендую создавать новую тему под каждый новый баг.
С наилучшими пожеланиями,
Сергей Маркин
Ну я то точно не потеряю, буду тут всяких жучков (багами не назовешь ведь) бросившихся в глаза, писать
вот еще задвоение в файле config.php
// FIXME: backward compatibility
// Updates server address
$config['updates_server'] = 'http://updates.cs-cart.com';
// external resources, related to product
$config['resources'] = array(
'knowledge_base' => 'http://docs.cs-cart.com/4.7.x/install/index.html',
'updates_server' => 'http://updates.cs-cart.com',
updates_server два раза присваивается
3 лайка
сегодня копался в fn.catalog.php
строка 8626
/**
* @deprecated In favour of use fn_get_product_details_view function.
* @since 4.3.1
* @TODO: remove in 4.3.2
*/
function fn_get_product_details_layout($product_id)
{
return fn_get_product_details_view($product_id);
}
хотели в 4.3.2 изъять, а уже 4.7.4, забыли? (за чистоту кода! )
2 лайка