Обнаруженные ошибки ViArt Shop v.4.07 RE 19_07_utf8

[nemo]

В управлении заказами, при просмотре какого-либо заказа вываливается сообщение:
Пользователь, который оформил этот заказ, сделал ещё заказы.
Всего заказано на сумму -
даже если этот пользователь сделал только один заказ.

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

[Aleksey]

В файле admin_order_sms.php в строке 81 отсутствуют кавычки вокруг sms_sent. Должно быть так:

$oe->set_value("event_type", "sms_sent"); 

[Aleksey]

admin/styles/admin.css, строка 432:

.popup-descr {padding: 0px 35px 20px 30px; font-size: 14px, sans-serif; text-align:justify;}

Должно быть:

.popup-descr {padding: 0px 35px 20px 30px; font-size: 14px; font-family: sans-serif; text-align:justify;}

[Aleksey]

Не определена константа SHIPPING_CONTROL_TYPE_MSG, из-за этого в админке в настройке оформления заказа там, где выбирается вид отображения типов доставки - пусто:


Нужно прописать в admin_messages.php что-то типа:

define("SHIPPING_CONTROL_TYPE_MSG", "Показ типа доставки"); 

[Aleksey]

В файле block_categories_list.php в самом верху есть строка:

header("Content-Type: text/html; charset=" . CHARSET); 

CHARSET равен utf-8.

У меня из-за нее неправильно отображается страница с результатами поиска - видны только колонтитулы и меню. Остальное содержимое не видно, хотя при просмотре исходного кода все есть. Выяснил, что это происходит из-за 3-х байт EFBBBF в начале страницы.
Такой глюк происходит в FF и IE9 на локальном компе. На реальном сайте все нормально.
При удалении этой строки на локальном компе все отображается нормально.

Нужна ли вообще эта строка и зачем?

[Aleksey]

При расширенном поиске по значениям опций и с указанием ценового диапазона скриптом не учитываются цены значений опций. В случае, если цена товара формируется за счет цен значений опций, а основная цена продажи товара равна 0 - не будет найден ни один товар.

Строки выборки товара из БД по ценам формируются в файле block_products_list.php в районе строки 340 этим кодом:

if (strlen($lprice)) {
    if (strlen($sql_where)) $sql_where .= " AND ";
    $conv_price = $lprice / $currency["rate"];
    $sql_where .= " ( ";
    $sql_where .= " (i.is_sales=1 AND (i." . $sales_field . "+i.".$properties_field.")>=" . $db->tosql($conv_price, NUMBER) . ") ";
    $sql_where .= " OR ((i.is_sales<>1 OR i.is_sales IS NULL) AND (i." . $price_field . "+i.".$properties_field.")>= " . $db->tosql($conv_price, NUMBER) . ") ";
    $sql_where .= ") ";
}
if (strlen($hprice)) {
    if (strlen($sql_where)) $sql_where .= " AND ";
    $conv_price = $hprice / $currency["rate"];
    $sql_where .= " ( ";
    $sql_where .= " (i.is_sales=1 AND (i." . $sales_field . "+i.".$properties_field.")<=" . $db->tosql($conv_price, NUMBER) . ") ";
    $sql_where .= " OR ((i.is_sales<>1 OR i.is_sales IS NULL) AND (i." . $price_field . "+i.".$properties_field.")<= " . $db->tosql($conv_price, NUMBER) . ") ";
    $sql_where .= ") ";
} 

Из него видно, что поле additional_price таблицы va_items_properties_values вообще не участвует в выборке.

Для решения проблемы и сохранения существующего функционала нужно вышеприведенной код заменить на следующий:

Скрытый текст (вы должны войти под своим логином или зарегистрироваться и иметь 50 сообщение(ий)):

У вас нет прав чтобы видеть скрытый текст, содержащийся здесь.

Этот код подходит для случая, когда поиск осуществляется только по одной опции, например "Размер". Для реализации поиска одновременно по нескольким опциям код усложнится.

-------------------------

Также не реализован поиск значений опций с указанием диапазона веса.
Код выборки по весу расположен в этом же файле сразу за кодом выборки по ценам, и в нем учитывается только вес самого товара (i.weight), а вес значений опций additional_weight не учитывается.

[Aleksey]

После добавления фильтра ("Default filter" в дистрибутиве) на страницу (например, "Список товаров") вываливается ошибка при обращении к этой странице:

Fatal error: Call to undefined function get_filter_sql() in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\ViartShop407RE_utf8_original\blocks\block_filter.php on line 156

При добавлении

include_once("./includes/filter_functions.php"); 

в начало файла block_filter.php вываливается куча сообщений:

Notice: Undefined variable: show_sub_products in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\ViartShop407RE_utf8_original\blocks\block_filter.php on line 156
Notice: Undefined variable: category_path in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\ViartShop407RE_utf8_original\blocks\block_filter.php on line 156
Notice: Undefined variable: show_sub_products in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\ViartShop407RE_utf8_original\blocks\block_filter.php on line 195
Notice: Undefined variable: category_path in C:\Program Files\Apache Software Foundation\Apache2.2\htdocs\ViartShop407RE_utf8_original\blocks\block_filter.php on line 195

Вставляем в начало файла block_filter.php

error_reporting(0); 

и фильтр таки появляется на странице.

[Aleksey]

В файле admin_cms_block_properties.php в строке 100 вместо

$sql  =