ocStore 2.3 - финальное тестирование перед релизом

[dinox]

Всем добрый вечер!

Сообщаю о том что команда которая работала над релизом ocStore 2.3 закончила работы и так как все кто работал над проектом уже ошибок там не видит :) У меня просьба всех кто желает протестировать и сообщить об ошибках, и если умеете сделать pull request на эти ошибки

Вот репозиторий где последний актуальный код 

https://github.com/myopencart/ocStore/tree/dev

Кто не умеет пользоваться Git вот Вам ссылка на скачивание архива https://github.com/myopencart/ocStore/archive/dev.zip

Тестирование будет происходит одну неделю после чего мы выпустим релиз исправив ошибки если найдутся

Спасибо за участие в тестировании

[aridius]

1. Не сохраняет email в Дополнительные E-Mail адреса, ну и переименовать Additional Alert Mail ->Дополнительные E-Mail адреса и проверить что бы работало потом)

2. В почтовой рассылке, если нажать отправить TypeError: $(...).code is not a function $('textarea[name=\'message\']').val($('#input-message').code()); если использовать Summer Note

3. в товаре пропала возможность добавить изображение) ТУТ ПОНЯЛ, ЕГО ПЕРЕНЕСЛИ ВО ВКЛАДКУ ИЗОБРАЖЕНИЯ

4. при добавлении нового изображения с ПК в менеджере изображений SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

Змінено 24 жовтня 2016 користувачем shoputils
Добавлена нумерация пунктов

[shoputils]

1 - Исправлено https://github.com/myopencart/ocStore/commit/e66085d77240682acaa32989d4c80636cea12975

4 - Не могу воспроизвести. Возможно, у вас старый файл admin/controller/common/filemanager.php

[aridius]

4 - Не могу воспроизвести. Возможно, у вас старый файл admin/controller/common/filemanager.php

Я залил на домен именно эту сборку на "чистый" домен, старому filemanager.php там неоткуда взяться

это когда я хочу загрузить новую фотку с компа

Змінено 24 жовтня 2016 користувачем mrz

[kamar87]

Я залил на домен именно эту сборку на "чистый" домен, старому filemanager.php там неоткуда взяться

это когда я хочу загрузить новую фотку с компа

у меня такая же ошибка

[shoputils]

2. В почтовой рассылке, если нажать отправить TypeError: $(...).code is not a function $('textarea[name=\'message\']').val($('#input-message').code()); если использовать Summer Note

 

Исправлено https://github.com/myopencart/ocStore/commit/395c4d16722ff7fae309863703c1297d021308e9

[Skiphog]

В файле filemanager.php, 277 строка, $this->request->files['file']['tmp_name'] - это массив, а is_file ждет строку. Отсюда и ошибка у всех.

И вообще, в методе upload начиная с 276 строки и заканчивая 348 написана полная белиберда.

Проверьте пожалуйста  :-)

[shoputils]

Менеджер изображений исправлен.

[AlexDW]

баг с изображениями при их вставке непосредственно в тексты (статьи/товары/производители/категории)

 

в самом summernote код поменялся с версии 0.7.0 (opencart 2200 и выше)

и наличие в tpl конструкции вида (пример взят из admin\view\template\catalog\information_form.tpl)

$('#input-description<?php echo $language['language_id']; ?>').summernote({
    height: 300,
    lang:'<?php echo $lang; ?>'
});

приводит к вставке изображения ПРЯМО в описание в формате base64

(data:image/jpeg;base64,/9j/4AAQSkZJRgABA...)

 

решается вырезанием подобных конструкций из tpl

[kamar87]

Как побороть права доступа к модулям? Приходится постоянно выключать и включать модули...

[Skiphog]

решается вырезанием подобных конструкций из tpl

 

Да, но тогда при вставке картинки из менеджера изображений происходит дублирование и вставляется два изображения вместо одного.

Связано это с тем, что сам менеджер тянет с собой скрипт, который конфликтует со скриптом редактора.

 

Строка 50: $(element).summernote('insertImage', $(this).attr('href')); в файле  /admin/viewjavascript/summernote/opencart.js

и 

Строка 91: range.insertNode(img); в файле /admin/view/template/common/filemanager.tpl

 

Если закомментировать первую строку, то картинки вставляются корректно.

Но это всего лишь очередной костыль...

[AlexDW]

Да, но тогда при вставке картинки из менеджера изображений происходит дублирование и вставляется два изображения вместо одного.

Связано это с тем, что сам менеджер тянет с собой скрипт, который конфликтует со скриптом редактора.

 

Строка 50: $(element).summernote('insertImage', $(this).attr('href')); в файле  /admin/viewjavascript/summernote/opencart.js

и 

Строка 91: range.insertNode(img); в файле /admin/view/template/common/filemanager.tpl

 

Если закомментировать первую строку, то картинки вставляются корректно.

Но это всего лишь очередной костыль...

дублирование только там, где уже были попытки добавить изображение с .summernote

после удаления кода из tpl и пересохранения - все нормально

 

upd:

т.е. фактически изображение в формате base64 в описании остается (видно в БД), но summernote его не отображает

лечится через ctrl+a -> ctrl+c -> shift+ins и сохранение

 

а при изначальном отсутствии изображений в формате base64 в описании - и проблемы не будет

 

точно, есть такое - не обратил внимания на изменения в filemanager

Змінено 24 жовтня 2016 користувачем AlexDW

[SiteMix]

Сам проверить не смогу, но кто уже скачал, проверьте, плиз, нет ли бага с удалением из корзины при включенном SEO_PRO

[ArtemPitov]

Вспомнил ! Майл ругается же 

$mail->setTo($this->config->get('config_email'));
$mail->setFrom($this->config->get('config_email'));

https://github.com/myopencart/ocStore/blob/dev/upload/catalog/controller/information/contact.php

 

И в само тело добавляем от кого 

[Skiphog]

Сам проверить не смогу, но кто уже скачал, проверьте, плиз, нет ли бага с удалением из корзины при включенном SEO_PRO

 

Нет. Не исправили. Корзина после удаления не обновляется сама. Только после перезагрузки страницы.

[AlexDW]

при выборе в качестве редактора ckeditor - видно и окно ckeditor, и окно summernote

в header.tpl зря добавили подключение summernote

 

в 2302 он подключается только там где нужен (в information_form.tpl и тд), желательно эти места обернуть условиями проверки используемого редактора (через $ckeditor)

[Skiphog]

Удаление любого языка из админки вызывает Fatal error из-за отсутствия таблицы banner_image_description

 

Fatal error: Uncaught Exception: Error: Table 'ocstore.banner_image_description' doesn't exist<br />Error No: 1146<br />DELETE FROM banner_image_description WHERE language_id = '6' in W:\domains\ocStore\system\library\db\mysqli.php on line 40

[AlexDW]

странно, потому что эта таблица у меня есть :ugeek:

[thentru]

Ошибка в файле

admin/language/ru-ru/marketing/contact.php

в переменной $_['text_sent']

не хватает аргументов строки

Если получателей рассылки больше одной страницы, письма отправятся только с первой

[AlexDW]

не то чтобы мне стыдно - но с гитхабом не сталкивался :ugeek:

вроде как исправления внес, но - без понятия как их объединить с основным репозиторием, оставлю ссылки

 

исправления для правильной работы редакторов summernote/ckeditor

 

admin/view/template/catalog/information_form.tpl

admin/view/template/common/header.tpl

admin/view/template/common/filemanager.tpl

admin/view/template/catalog/category_form.tpl

admin/view/template/catalog/manufacturer_form.tpl

admin/view/template/catalog/product_form.tpl

[SiteMix]

Нет. Не исправили. Корзина после удаления не обновляется сама. Только после перезагрузки страницы.

 

Спасибо!

 

Хочется верить, что разработчики обратят внимание на этот баг...

[ZKostik]

В Админке при редактирование заказа в "Деталях оплаты" не фиксируется закладка "Выбрать адрес". При нажатии "Продолжить" выпадает ошибка.

Змінено 25 жовтня 2016 користувачем ZKostik

[AlexDW]

Все ваши изменения (пул-реквесты) я влил в продакшн. Спасибо!

Не обязательно каждый отдельный файл в отдельном комите править. В веб-интерфейсе изменения вносились?

да, через веб

опыта работы с гитом нет - так и не въехал как изменения в один коммит впихнуть :ugeek:

[AlexDW]

что характерно

 

вижу обсуждения "совета депутатов", вспоминаю как бурно обсуждалась тема фиаско нового каталога

зато в теме с просьбой потестить бету ocStore 2302 - практически никого

 

то ли все баги уже нашли и побороли, то ли это показатель..

[vasilev86]

что характерно

 

вижу обсуждения "совета депутатов", вспоминаю как бурно обсуждалась тема фиаско нового каталога

зато в теме с просьбой потестить бету ocStore 2302 - практически никого

 

то ли все баги уже нашли и побороли, то ли это показатель..

за это не платят, вот лясы и точат. :-D