Jump to content
Search In
  • More options...
Find results that contain...
Find results in...
  • Sign Up

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


Recommended Posts

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

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

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

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

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

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

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

  • +1 1
Link to post
Share on other sites

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

Edited by shoputils
Добавлена нумерация пунктов
  • +1 2
Link to post
Share on other sites

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

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

  • +1 1
Link to post
Share on other sites

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

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

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

eccd9e9c4de0.png

Edited by mrz
Link to post
Share on other sites

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

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

eccd9e9c4de0.png

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

Link to post
Share on other sites

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

 

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

  • +1 1
Link to post
Share on other sites

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

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

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

Link to post
Share on other sites

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

  • +1 1
Link to post
Share on other sites

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

 

в самом 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

Link to post
Share on other sites

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

Link to post
Share on other sites

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

 

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

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

 

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

и 

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

 

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

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

  • +1 1
Link to post
Share on other sites

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

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

 

Строка 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

Edited by AlexDW
  • +1 1
Link to post
Share on other sites

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

$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

 

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

Link to post
Share on other sites

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

 

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

Link to post
Share on other sites

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

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

 

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

Link to post
Share on other sites

Удаление любого языка из админки вызывает 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

  • +1 1
Link to post
Share on other sites

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

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

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

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

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

Link to post
Share on other sites

не то чтобы мне стыдно - но с гитхабом не сталкивался :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

  • +1 1
Link to post
Share on other sites

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

 

Спасибо!

 

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

Link to post
Share on other sites

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

Edited by ZKostik
Link to post
Share on other sites

не то чтобы мне стыдно - но с гитхабом не сталкивался :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

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

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

Link to post
Share on other sites

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

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

да, через веб

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

Link to post
Share on other sites

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

 

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

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

 

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

  • +1 2
Link to post
Share on other sites
Guest
This topic is now closed to further replies.
  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.