dinox

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

Рекомендуемые сообщения

dinox    639

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

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

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

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

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
mrz    102

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

Изменено пользователем shoputils
Добавлена нумерация пунктов

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
shoputils    227

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
mrz    102

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

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

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

eccd9e9c4de0.png

Изменено пользователем mrz

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
kamar87    58

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

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

eccd9e9c4de0.png

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
shoputils    227

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
Skiphog    2

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
shoputils    227

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

 

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
Skiphog    2

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

 

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

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

 

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

и 

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

 

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

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

 

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

Изменено пользователем AlexDW

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
SiteMix    141

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

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

$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    2

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
Skiphog    2

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
thentru    171

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

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

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

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

 

Спасибо!

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ZKostik    2

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

Изменено пользователем ZKostik

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
toporchillo    426

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

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

да, через веб

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

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

 

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

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

 

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
Гость
Эта тема закрыта для публикации ответов.

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу