Перейти до вмісту
Пошук в
  • Детальніше...
Шукати результати, які ...
Шукати результати в ...

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


dinox

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
Надіслати
Поділитися на інших сайтах


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
Добавлена нумерация пунктов
  • +1 2
Надіслати
Поділитися на інших сайтах

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

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

  • +1 1
Надіслати
Поділитися на інших сайтах

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

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

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

eccd9e9c4de0.png

Змінено користувачем mrz
Надіслати
Поділитися на інших сайтах

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

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

eccd9e9c4de0.png

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

Надіслати
Поділитися на інших сайтах


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
Надіслати
Поділитися на інших сайтах

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

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

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

Надіслати
Поділитися на інших сайтах


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

  • +1 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

Надіслати
Поділитися на інших сайтах

решается вырезанием подобных конструкций из 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
Надіслати
Поділитися на інших сайтах


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

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

 

Строка 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 1
Надіслати
Поділитися на інших сайтах

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

$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

 

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

Надіслати
Поділитися на інших сайтах

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

 

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

Надіслати
Поділитися на інших сайтах


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

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

 

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

Надіслати
Поділитися на інших сайтах

Удаление любого языка из админки вызывает 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
Надіслати
Поділитися на інших сайтах


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

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

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

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

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

Надіслати
Поділитися на інших сайтах


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

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

 

Спасибо!

 

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

Надіслати
Поділитися на інших сайтах


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

Змінено користувачем ZKostik
Надіслати
Поділитися на інших сайтах


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

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

да, через веб

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

Надіслати
Поділитися на інших сайтах

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

 

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

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

 

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

  • +1 2
Надіслати
Поділитися на інших сайтах

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

 

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

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

 

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

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

  • +1 1
Надіслати
Поділитися на інших сайтах


Гість
Ця тема закрита для публікації повідомлень.
  • Зараз на сторінці   0 користувачів

    • Ні користувачів, які переглядиють цю сторінку
×
×
  • Створити...

Important Information

На нашому сайті використовуються файли cookie і відбувається обробка деяких персональних даних користувачів, щоб поліпшити користувальницький інтерфейс. Щоб дізнатися для чого і які персональні дані ми обробляємо перейдіть за посиланням . Якщо Ви натиснете «Я даю згоду», це означає, що Ви розумієте і приймаєте всі умови, зазначені в цьому Повідомленні про конфіденційність.