Процесс работ над релизом ocStore 1.5.5.1.2

[Bogdan1975]

Вы так и не раскрыли суть и я, если честно, ничего не понял из этого сообщения.  :ugeek:

Ну да ладно, если в 1.5.4.1

- сортировка делалась по другому, то что ж вы не написали пост, прочитав который, было бы понятно как работал код сортировки категорий в 1.5.4.1 ?

В 1.5.4.1 категории забирались из модели, формировался массив $categories, затем этот массив сортировался рекурсивной функцией getAllCategories();

$this->data['categories'] = $this->getAllCategories($categories);

В 1.5.5.1.1 этой сортировки уже не было, было просто

$this->data['categories'] = $this->model_catalog_category->getCategories(0);

Собственно, я и предложил сортировать рекурсивной функцией, которая как я увидел позже просто делает примерно тоже самое, что функция в getAllCategories() в версии 1.5.4.1

А затем увидел, что в мастер версии использование getAllCategories() восстановлено, правда позже результат затирается строкой

$this->data['categories'] = $this->model_catalog_category->getCategories(0);

Поэтому и говорю, что решение сводится к удалению этой строки из мастер-версии

[Alexey]

Сегодня мой Pull-реквест 

https://github.com/myopencart/ocStore/pull/26

 

был отклонён, так как оказалось что 

проблема с index.php?route=common/home

решается SQL-запросом:

INSERT INTO oc_url_alias (QUERY, keyword) VALUES('common/home','');

 

Я отредактировал все свои прошлые сообщения относительно этой проблемы.

[GotchaMabuka]

Приветствую. Скажите, а где можно будет взглянуть на лог всех изменений в коде osStore   по сравнению с "родным" opencart?

Наверняка, какие-то запросы к базе данных тоже, да?

Допустим, чтобы была возможность людям "в ручную" перейти с оригинала opencart на OcStore.

Минуя необходимость сносить уже настроенное одно - заново ставить другое.

Ведь  шаблоны часто продаются в поставке "full package" только в связке с родным opencart,

с внесенными изменениями в код отдельных файлов. И просто так поверх, как я понял, OcStore на такое не положишь.

И на OсStore такой шаблон не оденешь в связи с той же необходимостью перезаписи сист. файлов.

Я конечно перелопатил вручную выискивая разницу методикой сравнения 70 файлов в папках Admin Controller и Model  шаблона от opencart 1.5.5.1 чтобы "одеть" его на новый OcStore, но прилично утомился)

С уважением к энтузиастам.

[Zeppelin]

отличия описаны в wiki http://wiki.myopencart.ru/index.php?title=%D0%97%D0%B0%D0%B3%D0%BB%D0%B0%D0%B2%D0%BD%D0%B0%D1%8F_%D1%81%D1%82%D1%80%D0%B0%D0%BD%D0%B8%D1%86%D0%B0#.D0.A0.D0.B0.D0.B7.D0.BD.D0.B8.D1.86.D0.B0_.D0.BC.D0.B5.D0.B6.D0.B4.D1.83_ocStore_v1.5.5.1.1_.D0.B8_.D0.BE.D1.80.D0.B8.D0.B3.D0.B8.D0.BD.D0.B0.D0.BB.D1.8C.D0.BD.D1.8B.D0.BC_OpenCart_v1.5.5.1:

 

про полный лог отличий не скажу, про WinMerge и подобные, ты уже написал.

Змінено 9 листопада 2013 користувачем Zeppelin

[sitecreator]

Правильно ли я понимаю, что сборка 1.5.5.1.2  в том виде в каком она есть сейчас уже предпочтительнее для создания магазина с нуля чем сборка 1.5.5.1.1?

 

Насколько я понимаю фатальных ошибок в 1.5.5.1.2 нет?

но зато отсутствуют ошибки и неудобства, которые есть в 1.5.5.1.1.

 

сужу по тому списку исправлений, которые уже сделаны и опубликованы на первой странице.

 

или я не все уловил?

[smind]

я дико извиняюсь, посмотрел на гитхабе репо https://github.com/myopencart/ocStore там почему то ни одного тестового бранча нет, если не секрет, скажите у вас есть ещё дополнительные репозитарии гдето вне github-а или вы прямо наживую master правите?

[freelancer]

"тестового" бранча нет

[smind]

"тестового" бранча нет

ну дефакто это как бы master получается и есть "тестовый"...

[smind]

можно я ещё немного покритикую?

вы большие молодцы что используете git и github но влитие (мердж) веток происходило не совсем идеологически правильно... без предварительного ребейза... это хорошо и наглядно видно вот тут https://github.com/myopencart/ocStore/network  11 августа, 13 сент. и т.п. там где ветки переплетаются... по-хорошему такого быть не должно бы...

[freelancer]

merge - слиянее скорее.

на то были свои причины

[markimax]

В новой версии убрали баг opencart?

 

Т.е. строка $controller->addPreAction(new Action('common/maintenance'));
должна находиться сразу после $controller = new Front($registry);
и перед вызовом "seo url".

[smind]

merge - слиянее скорее.

на то были свои причины

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

если вкратце то

$ git checkout master

$ git reset --hard origin/master

$ git pull

$ git merge --log --no-ff origin/123_branch_name

позволяет сделать историю мерджей, мягкой и шёлковистой... т.е. не даст сделать мердж с неотребейженным бранчем.

[freelancer]

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

баг с maintenance не понмю если честно, но о нем известно

[smind]

спасибо, мы обратимся

 

намёк понял :), офтопить прекращаю...

[putnik]

Прочитал всю ветку, не нашел, что приняли проблему которая здесь: https://opencartforum.com/topic/25354-релиз-ocstore-v15511/?do=findComment&comment=216141

 

Спасибо за проделанную работу!

[sitecreator]

Прочитал всю ветку, не нашел, что приняли проблему которая здесь: https://opencartforum.com/topic/25354-релиз-ocstore-v15511/?do=findComment&comment=216141

 

Спасибо за проделанную работу!

 

делать имена файлов с пробелами - это очень дурной тон (на Linux), и к тому же это вызывает массу других проблем, напрямую не связанных с движком.

 

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

 

А с дефисом хорошо бы исправить проблемку.

[Alexey]

В новой версии убрали баг opencart?

 

Т.е. строка $controller->addPreAction(new Action('common/maintenance'));

должна находиться сразу после $controller = new Front($registry);

и перед вызовом "seo url".

 

Нет, исправлено не было.

Отправил Pull-реквест в master-branch

https://github.com/myopencart/ocStore/pull/29/files

[Alexey]

Правильно ли я понимаю, что сборка 1.5.5.1.2  в том виде в каком она есть сейчас уже предпочтительнее для создания магазина с нуля чем сборка 1.5.5.1.1?

 

Насколько я понимаю фатальных ошибок в 1.5.5.1.2 нет?

но зато отсутствуют ошибки и неудобства, которые есть в 1.5.5.1.1.

 

сужу по тому списку исправлений, которые уже сделаны и опубликованы на первой странице.

 

или я не все уловил?

Да, всё верно.

 

Кроме тех исправлений которые уже отмечены как исправленные, есть ещё несколько критических исправлений, 

которые не успели ни внести в список ни "одобрить" Pull-реквесты, которые их делают.

Список не принятых Pull-реквестов можно посмотреть тут:

https://github.com/myopencart/ocStore/pulls

 

Уговаривайте Dinox-а чтобы он их рассмотрел, и если всё ОК, добавил в мастер-версию:

https://github.com/myopencart/ocStore/archive/master.zip

:-)

[sitecreator]

Уговаривайте Dinox-а чтобы он их рассмотрел, и если всё ОК, добавил в мастер-версию:

https://github.com/myopencart/ocStore/archive/master.zip

:-)

А по какой причине они до сих пор не рассматриваются?

Я имею ввиду те из них, которые сделаны 23 дня тому назад и 10 дней тому назад.

срок то вроде бы немалый прошел.

 

Администрация ведь заинтересована в скорейшем выходе новой версии?

Может быт я каких то нюансов не улавливаю?

[Alexey]

Прочитал всю ветку, не нашел, что приняли проблему которая здесь: https://opencartforum.com/topic/25354-релиз-ocstore-v15511/?do=findComment&comment=216141

 

Спасибо за проделанную работу!

 

Ага, такая проблема есть и она до сих пор не решена.  :(

Предлагаю добавить её под номером 1) в новый to-do list:

 

1)

Проблемы с избражениями товара:

 

Проблема№1:

 

Есть 2 картинки товара:

1. model-11.jpg
2. model-11-.jpg

В папку cashe добавляется 2 разных изображения но с одним именем model-11-70x70.jpg . Тоесть не учитывается дефис в конце названия. Та же беда и с картинками у которых разница в названии это пробел. К примеру :

1. model-11.jpg
2. model-11 .jpg

Ранее стояла сборка 1.5.1.3 и в папку cashe добавлялись картинки с учетом тире и пробелов

 

Проблема№2:

 

Раньше( в сборке 1.5.1.3)названия изображений в папке кэш имели вид /image-70x70.jpg . Теперь названия картинок /название-папки-image-70x70.jpg .

Впринципе на работу магазина это не влияет, но есть одно НО!

Мой магазин существует 2 года и за это время люди постили картинки моего товара на разных форумах, блогах и прочих сайтах. Теперь же оказалось что старые ссылки на картинку битые и не отображаются в сообщениях

 

 

А также добавить в to-do эту проблему:

2) https://opencartforum.com/topic/25354-релиз-ocstore-v15511/?do=findComment&comment=216599

Полистал ветку, вроде этот баг ещё не описывали.

 

В связях при добавлении Сопутствующие товары

 

Например я создаю 3 товара, у каждого по 4 сопутствующих товара, после сохранения у одного 2, у другого 6, у третьего 4.

 

 

3) И просьбу ко всем участникм сообщества ocStore протестировать решение проблемы с мультиязычным Seo-pro

https://opencartforum.com/topic/27002-процесс-работ-над-релизом-ocstore-15512/?do=findComment&comment=213622

Протестировал мультиязычный SeoPro от cdm&freelancer.

На предмет ajax он четко отрабатывает, во всяком случае не наткнулся на несрабатывание.

Споткнулся только в случае, если какой-нибудь модуль втягивает контент в iFrame из своего контроллера.

Немного иземнил, теперь срабатывает и на iFrame.

Желательно, конечно, что бы еще кто-нибудь потестил на всякий случай.

https://opencartforum.com/index.php?app=core&module=attach&section=attach&attach_id=10143

 

4) А также просьбу ко всем разработчикам у которых есть полчаса времени,

глянуть имеют ли смысл следующие предложения по оптимизации:

Возможно некоторые из этих вещей можно внести в сборку 1.5.5.1.2

http://opencartsoftware.blogspot.com/2013/03/opencart.html

http://opencartsoftware.blogspot.com/2013/02/opencart.html

http://opencartsoftware.blogspot.com/2013/05/revision-opencart.html

http://opencartsoftware.blogspot.com/2013/05/opencart-performance.html

 

Вот, вроде из обеих топиков всё собрал что осталось :-)

[markimax]

Нет, исправлено не было.

Отправил Pull-реквест в master-branch

https://github.com/myopencart/ocStore/pull/29/files

Спасибо, потому что вроде и "мелочь", но очень неприятная.

[Aurora]

Здравствуйте. (если что, не пинайте сильно, я новичок в этом деле)

Подскажите пожалуйста, это баг, недочет разработчиков или так задумано?

Если я указываю минимальное количество товара для покупки, к примеру 5, то в карточке товара оно так и отображается и при нажатии кнопки "Купить" в корзине появляется 5 единиц товара, но если в категории нажать кнопку купить, то в корзину отправляется только 1 единица этого товара, ну а при оформлении заказа, соответственно выходит сообщение: "Минимальное количество для заказа товара ХХХ составляет 5!", что на мой взгляд неправильно.

Возможно ли устранить в сборке этот недостаток?

[Alexey]

В 1.5.4.1 все нормально сортировалось (рекурсивно, кстати, а не как принято)

В 1.5.5.1.1 ничего этого уже нет, рекурсивная функция отсутствует, категории забираются скопом и без сортировки отдаются в мир.

В мастер-версии все восстановленно, но одна строчка все портит.

А по сему, изобретенные мною с Вами велосипеды нужно выкинуть на помойку (а лучше сжечь), а вместо этого в мастер-версии фрагмент:

        // Categories
        $this->load->model('catalog/category');
        $this->data['categories'] = $this->model_catalog_category->getCategories(0);

нужно отправить вслед за велосипедами.

 

Внёс правки и отправил Pull-реквест в мастер версию

https://github.com/myopencart/ocStore/pull/30/files

[kisluk]

http://www.opencart.com/index.php?route=extension/extension/info&extension_id=6210

/mens/
/mens/sportswear/
/mens/sportswear/footwear/
/womens/
/womens/sportswear/
/womens/sportswear/footwear/

 

Можно реализовать вот такой фикс, который бы позволил делать категории с одинаковыми названиями категорий в seo_url?

[ingenerks]

http://www.opencart.com/index.php?route=extension/extension/info&extension_id=6210

/mens/

/mens/sportswear/

/mens/sportswear/footwear/

/womens/

/womens/sportswear/

/womens/sportswear/footwear/

 

Можно реализовать вот такой фикс, который бы позволил делать категории с одинаковыми названиями категорий в seo_url?

https://opencartforum.com/topic/26722-%D0%BC%D0%BE%D1%8F-%D0%BD%D0%B5%D0%B1%D0%BE%D0%BB%D1%8C%D1%88%D0%B0%D1%8F-%D0%B4%D0%BE%D1%80%D0%B0%D0%B1%D0%BE%D1%82%D0%BA%D0%B0/