Ошибки в 2.3.5 (Исправить в 2.3.6)

[makki]

Ошибка №1
В админке в заказе не подтягивается Ед. изм.товаров
Исправьте пожалуйста в следующем обновлении 2-й версии

В файле backend/design/html/order.tpl
1) в строке 188 вместо

Код: Выделить всё

{$purchase->amount} {if $purchase->units}{$purchase->units|escape}{else}{$settings->units|escape}{/if}</span>

должно быть

Код: Выделить всё

{$purchase->amount} {if $purchase->variant->units}{$purchase->variant->units|escape}{else}{$settings->units|escape}{/if}</span>

2) в строке 220 вместо

Код: Выделить всё

{if $purchase->units}{$purchase->units|escape}{else}{$settings->units|escape}{/if}</span>

должно быть

Код: Выделить всё

{if $purchase->variant->units}{$purchase->variant->units|escape}{else}{$settings->units|escape}{/if}</span>

[makki]

Ошибка №2
В хлебных крошках при переключенном языке, в конце ссылки на Главную стоит лишний знак /
например demookay.com/en/ вместо demookay.com/en

Добавлено спустя 16 часов 18 минут:
Ошибка №3
Не правильный hreflang на всех страницах списка товаров (каталога товаров) с выбранным в фильтре брендом, что вызывает вопросы у google

Ошибка из-за того, что в функции filter_chpu_sort_brands идет обращение к переменной $this->category_brands, которая определяется позже. Нужно перенести в __construct код

Код: Выделить всё

foreach ($this->brands->get_brands(array('category_id' => $this->category->children, 'visible' => 1, 'visible_brand' => 1)) as $b) {
                    $this->category_brands[$b->id] = $b;
                }

В 3-й версии исправили, но в 2-й оставили.

[виктор]

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

в третьей версии таже фигня

[makki]

Ошибка №4
Переименуйте таблицу ok_options_aliases_values в ok_features_values_aliases_values как в 3-й версии. ok_options насколько я помню убрали еще 2.3.0

Добавлено спустя 1 час 18 минут:
Ошибка №5
В разделе Синонимы свойств, забыли вывести в шаблон сохраненные значения синонимов свойства после нажатия кнопки Применить, что приводит при повторном сохранении к дублям в базе данных и другим последствиям. Интересно, что до сих пор никто не сообщил о такой грубой ошибке.
Решение ошибки:
в файле backend/core/FeaturesAliasesAdmin.php после

Код: Выделить всё

$features_aliases = array();
                    foreach ($this->features_aliases->get_features_aliases() as $f) {
                        $features_aliases[$f->id] = $f;
                    }

нужно добавить следующий код

Код: Выделить всё

foreach ($this->features_aliases->get_feature_aliases_values(array('feature_id'=>$feature->id)) as $fv) {
                        $features_aliases[$fv->feature_alias_id]->value = $fv;
                    }

[makki]

Ошибка №6
В файле /backend/core/DeliveryAdmin.php строка 11 опечатка в слове integer

Код: Выделить всё

$delivery->id               = $this->request->post('id', 'intgeger');

[trainracing]

Еще не работает "другие фильтры" other filters по товарам со скидкой, рекоменд, в категории на сайте

[makki]

Исправьте пожалуйста в будущем обновлении 2-й версии стили autocomplete в файле design/okay_shop/css/libs.css, чтобы была полоса прокрутки, а то сейчас выпадает простыня на весь экран.

Например так

Код: Выделить всё

/* Autocomplete CSS */
.autocomplete-suggestions {
    border: 1px solid #e0e0e0;
    background: #fff;
    overflow: auto;
}
.autocomplete-suggestion {
    overflow: hidden;
    display: flex;
    align-items: center;
    cursor: pointer;
    border-bottom: 1px solid #e8e8e1;
}
.autocomplete-selected {
    background: #f0f0f0;
}
.autocomplete-suggestions strong {
    font-weight: normal;
    color: #ffb400;
}
.autocomplete-suggestion div {
    width: 45px !important;
    text-align: center;
}
.autocomplete-suggestion a {
    padding: 5px 10px;
    text-decoration: none;
}
.autocomplete-suggestion span {
    white-space: nowrap;
    padding: 5px 10px;
    margin-left: auto;
}

[sevswat]

Еще не работает "другие фильтры" other filters по товарам со скидкой, рекоменд, в категории на сайте

и еще не работают бренды в фильтре так же не выводятся

[makki]

Еще не работает "другие фильтры" other filters по товарам со скидкой, рекоменд, в категории на сайте

Это будет Ошибка 7
Лечится так:
В файле view/ProductsView.php вместо строки (строка номер 720)

Код: Выделить всё

if (!empty($this->page->url) && !in_array($this->page->url, array('all-products', 'discounted', 'bestsellers'))) {

написать

Код: Выделить всё

if (empty($this->page->url) || (!empty($this->page->url) && !in_array($this->page->url, array('all-products', 'discounted', 'bestsellers')))) {

Ошибка 8
Файл design/okay_shop/html/cart_deliveries.tpl строка 13 в условии

Код: Выделить всё

{if $delivery_id==$delivery->id || $delivery@first} checked{/if}

переменная $delivery_id не определена. Наверное достаточно так

Код: Выделить всё

{if $delivery@first} checked{/if}

[makki]

Ошибка 9 актуально и для OkayCMS 3
Ошибка возникает при изменении варианта в оформленном заказе со статусом, который списывает товар со склада, например Принят.
Допустим у нас в заказе 5 шт товара и у него есть вариант-2, которого на складе 3 шт. Если мы поменяем в заказе вариант товара на вариант-2 и оставим количество 5 шт, после списания у варинта-2 количество на складе -2 шт (минус 2 шт).

[makki]

Исправьте пожалуйста название строчки меню в админке Импорт, Лог Импорта, Экспорт на Импорт товаров, Лог Импорта товаров и Экспорт товаров соответственно. Ведь это не единственный возможный импорт и экспорт. Надоело в доработках каждый раз менять, и пользователям понятнее такое название.

Добавлено спустя 3 часа 4 минуты:
Ошибка 10 актуально и для OkayCMS 3
На странице Импорт в админке на мобильных разрешениях, блок Загрузка файла полностью пропадает.
Забыли заключить блок в

Код: Выделить всё

<div class="toggle_body_wrap on fn_card">... </div>

И почему-то в форме страницы добавлен стиль hidden-xs-down

[makki]

Ошибка 11
Ошибка в запросе на странице бренда при сортировке по названию. Соответсвенно товары не отображаются
Лечится так. В файле api/Products.php строка 138 вместо

Код: Выделить всё

$order = 'p.name ASC';

пишем

Код: Выделить всё

$order = $px.'.name ASC';

и строка 141 вместо

Код: Выделить всё

$order = 'p.name DESC';

пишем

Код: Выделить всё

$order = $px.'.name DESC';

[makki]

Ошибка 12
Серьезные 2 ошибки в SQL запросах в файле api/Features.php. Могу только предположить к каким проблемам со свойствами приводят эти ошибки, особенно при импорте товаров.
в строке 199 вместо

Код: Выделить всё

$this->db->query("SELECT `pv`.`product_id`

должно быть

Код: Выделить всё

$this->db->query("SELECT `pf`.`product_id`

в строке 246 вместо

Код: Выделить всё

$this->db->query("SELECT `pv`.`product_id`

должно быть

Код: Выделить всё

$this->db->query("SELECT `pf`.`product_id`

Ждем update 2-й версии!

[makki]

Ошибка 13
При дублировании товара, получаем ошибку в логах

Код: Выделить всё

Error (1064) You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ', last_modify=NOW() WHERE id in ('118') LIMIT 1' at line 1
UPDATE ok_products SET , last_modify=NOW() WHERE id in ('118') LIMIT 1

Исправляется в файле api/Products.php в функции update_product переносом строки

Код: Выделить всё

$result = $this->languages->get_description($product, 'product');

в начало тела условия ниже

Код: Выделить всё

if($this->db->query($query)) {

чтобы получилось так

Код: Выделить всё

public function update_product($id, $product) {
        $product = (object)$product;

        $query = $this->db->placehold("UPDATE __products SET ?%, last_modify=NOW() WHERE id in (?@) LIMIT ?", $product, (array)$id, count((array)$id));
        if($this->db->query($query)) {
            $result = $this->languages->get_description($product, 'product');
            if(!empty($result->description)) {
                $this->languages->action_description($id, $result->description, 'product', $this->languages->lang_id());
            }.......

[OkayCMS]

Спасибо за ваш пулреквест. Планируем сделать 2.3.6 с учётом ваших пулреквестов в следующем месяце.