dinox

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

212 сообщений в этой теме

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

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

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

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

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

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

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

1

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


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

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
Добавлена нумерация пунктов
2

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


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

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

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

1

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


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

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

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

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

eccd9e9c4de0.png

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

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


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

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

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

eccd9e9c4de0.png

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

0

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


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

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

 

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

1

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


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

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

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

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

0

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


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

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

1

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


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

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

 

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

0

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


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

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

0

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


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

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

 

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

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

 

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

и 

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

 

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

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

1

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


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

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

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

 

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

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


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

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

0

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


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

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

$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

 

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

0

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


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

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

 

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

0

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


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

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

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

 

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

0

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


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

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

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


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

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

0

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


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

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

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

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

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

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

0

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


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

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

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


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

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

 

Спасибо!

 

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

0

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


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

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

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

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


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

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

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

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

0

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


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

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

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

да, через веб

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

0

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


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

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

 

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

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

 

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

2

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


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

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

    Ни один зарегистрированный пользователь не просматривает эту страницу.