Alexey

Пожалуйста, опишите это баг подробнее, в этой ветке: https://opencartforum.com/topic/27002-%D0%BF%D1%80%D0%BE%D1%86%D0%B5%D1%81%D1%81-%D1%80%D0%B0%D0%B1%D0%BE%D1%82-%D0%BD%D0%B0%D0%B4-%D1%80%D0%B5%D0%BB%D0%B8%D0%B7%D0%BE%D0%BC-ocstore-15512/page-9

Сегодня мой Pull-реквест https://github.com/myopencart/ocStore/pull/26 был отклонён, так как оказалось что проблема с index.php?route=common/home решается SQL-запросом: INSERT INTO oc_url_alias (QUERY, keyword) VALUES('common/home',''); Я отредактировал все свои прошлые сообщения относительно этой проблемы.

Вы так и не раскрыли суть и я, если честно, ничего не понял из этого сообщения. :ugeek: Насчёт холивара - то никакого холивара не было. Ваше первоначальное решение - было мягко говоря "кривоватым", и я поправил сортировку в SQL а вы с невероятным усердием начали доказывать "право на жизнь" своего метода. Ну да ладно, если в 1.5.4.1 - сортировка делалась по другому, то что ж вы не написали пост, прочитав который, было бы понятно как работал код сортировки категорий в 1.5.4.1 ? Из хороших новостей, проблему по которой были вот эти все сообщения: Я исправил сегодня вот этим Pull-реквестом https://github.com/myopencart/ocStore/pull/28/files Баг был в том, что создание переменной-итератор цикла $limit - перезатирало внешнюю переменную $limit. В special.php, я исправил переменную-итератор на $value (по аналогии с оригинальным Opencart) И, хоть бага не было, но для порядка отрефакторил $limits на $value, в этих файлах: category.php manufacturer.php search.php Прикреплённый архив делать не буду. Уговаривайте Dinox-a, чтобы он наконец-то принял мои правки, уже 6 Pull реквестов висит. https://github.com/myopencart/ocStore/pulls После того, как Dinox примет мои правки, пре-релиз ocStore со всеми багфиксами можно будет скачать тут: https://github.com/myopencart/ocStore/archive/master.zip

Стандартно, нужно сортировать по sort_order. Это общепринято. Если, как вы утверждаете, рекурсивная сортировка имеет право на жизнь - предлагаю вам сделать её в виде дополнительного модуля. Если кому то будет нужна именно такая, нестандартная "рекурсивная" сортировка категорий - будут пользоваться вашим модулем.

Согласитесь, что совместимость с оригинальным Opencart - для нас важнее, чем совместимость с шаблонами для старых версий. В оригинальном Opencart-е нет тех переменных, которые вы указали. Соответственно и в ocStore должно быть точно так же. А шаблоны для старых версий - пускай авторы шаблонов обновляют. Это их сфера ответственности и не зря же, они берут деньги и обещают поддержку совместимости с будущими версиями.

Неправильный алгоритм. В той ситуации что вы описываете, нужно сделать так. 1.Добавляем один язык, стартуем магазин. 2.Пришло время добавить второй язык - добавляем его (при этом тексты автоматически копируются из старого языка в новый) 3.Отключаем новый язык в админке. Он не показывается, но при редактировании товара есть табы с текстами нового языка 4.Редактируем в админке названия и описания товаров для нового языка для всех товаров. Внешне этого не видно, работа идёт "внутри", то есть в админке. 5.Когда всё отредактировали - включаем новый язык и он начинает показыватся на сайте.

Мнение такое, что поддержка совместимости шаблона с новой версией - это сфера отвественности автора шаблона. Иначе, следуя вашей логике можно дойти до того, что нужно поддерживать шаблоны разработанные для Opencart 1.5.3.1 и т.д. Всего один раз "попарившись" - чтобы правильно задать sort_order (задача для первоклассника, если честно) У нас прямо в результате запроса к БД - сразу получаются категории в нужном порядке. А так как это делается на уровне SQL и без рекурсии - то это выигрыш в производительности. Пускай незначительный выигрыш, но из таких вот небольших составляющих - складывается быстродействие всей системы в целом.

Ну да, в админке не будет табов с отключенными языками, а дальше что? Вы включите язык, который раньше был отключен и как будут показыватся товары, для которых вы не прописали тексты? Если вам точно не нужен язык ни сейчас ни в будущем - тогда удаляйте его и не будет у вас лишних табов для него. А если вы всего лишь временно отключили язык - то всё там правильно, эти табы нужны, так как в таком случае тексты обязательно нужно прописывать. Так что, ничего не трогайте, в этом плане там всё правильно сделано.

Так а чтобы не ломать себе голову как бы так "структурировать по уровням вложенности", всего то нужно в категориях правильный sort_order проставить. У меня родительские категории первого уровня имеют sort_order 100 200 ... 800 для родительской категории с sort_order 200 вложенные имеют sort_order: 210 220 .. 270 Третий уровень вложенности (если есть) для категории с sort_order 210: 211 212 213 ... 219 Вообще, есть sort_order - по нему и нужно выдавать список категорий. Везде так и есть, а в том месте отсортировать забыли. Вот я и пофиксил.

Пардон, но это очень бредовое предложение! :( Я согласен с afwollis: и с freelancer: Потому что всё правильно они говорят! В ядро нужно вносить минимальные изменения — только то, что действительно очень критично и очень важно. Одному будет удобней переименовать красный цвет в зелёный, другому будет удобней переименовать красный цвет в табуретки, третьему будет удобней переименовать красный цвет в "плюшевые мишки". Ну бред же! :ugeek: Если кому то надо дополнительные поля - то во всех отношениях будет лучше самостоятельно их добавить для своего магазина, причём с соответствующими названиями полей в базе данных, соответствующими названиями переменных и т.д. Это очень поможет не путаться при дальнейшей доработке своего магазина. Я вот каждый раз скрываю (не удаляю для совместимости) везде отображаемое поле "количество" и не переименовываю какое-то из существующих, а добавляю новое "цена производителя" и ничего, не жалуюсь. Изменения минимальны и понятны, обновлятся легко. :-) Потому что: каждая кастомизация — мешает обновлять движок ocStore при выходе новых версий оригинального Opencart. И если эта кастомизация нужна не всем, то проще её сделать в ввиде отдельного модуля чем добавлять в ядро. Поскольку добавление в ядро не всем нужной фичи — создаёт лишний гемморой для всего сообщества при разработке всех следующих версий ocStore на базе новых версии оригинального Opencart.

Как там говорят: "Критикуешь - предлагай. Предлагаешь - выполняй" :ugeek: Bogdan1975 - респект вам за инициативу, но ваше решение мягко говоря "не очень". Не нужно писать дополнительную функцию сортировки и сортировать массив данных на стороне PHP, когда достаточно лишь правильно поменять сортировку в SQL запросе. Я сегодня наконец то добрался до этого бага, пофиксил и сделал Pull-реквест. Вся суть изменений вот здесь: https://github.com/myopencart/ocStore/pull/27/files До одобрения Pull-реквеста Dinox-ом, для тех кто хочет уже сегодня получить багфикс - нужно извлечь содержимое прикреплённого архива в корень сайта. fixSortingCategories.zip

Кроме мультиязычного seoPro, было ещё 2 вещи которые не успели внести в to-do list: 1) Расмотреть предложения по оптимизации 2) Исправить некритичный баг

Ага! :-) Правду говорят: "В жизни как и в театре, самым требовательным бывает тот, кто не заплатил за билет." :-D Нет, не плохо а очень даже хорошо. А плохо то, что эти самые "многие" - ничего для этого не делают! О, ну это уже лучше. Конечно же протестируйте пожалуйста, если это и будет решением - это будет супер.

Какой смысл в этом посте? Да, конечно, было бы полезно это выяснить, но нет смысла вести пустые разговоры об этом. Есть смысл, например, почитать посты cmd или спросить у него в ЛС, и привести здесь его ответ или то, что удалось найти по этому вопросу. Вот такой пост, был бы полезен.

Может вы базу данных из старой версии переносили? Попробуйте нажать на кнопочку "Починить" в разделе Категории (она есть у вас на скриншоте). Вдруг поможет :-)

Так конечно же есть! Я бы не исправлял, если б не было. Я скачал с https://github.com/myopencart/ocStore/archive/master.zip установил себе локально и ещё установил на хостинге, и на обеих версиях сайта ссылки на главную были mysite.com/index.php?route=common/home проблема с index.php?route=common/home решается SQL-запросом INSERT INTO oc_url_alias (QUERY, keyword) VALUES('common/home',''); seopro_multilang - если я правильно понимаю именно он и был удалён из сборки по этой причине: https://opencartforum.com/topic/25354-%D1%80%D0%B5%D0%BB%D0%B8%D0%B7-ocstore-v15511/page-5#entry196020 а подходит или нет, это расширение http://www.opencart.com/index.php?route=extension/extension/info&extension_id=8897 я понятия не имею. Bogdan1975 - если вы его предлагаете, почему сами не проверили на работоспособность?

На самом деле да, почти весь to-do list выполнен, но... Dinox принял мой Pull-реквест который фиксит: 17) При пагинации ссылки на первую страницу выглядят не правильно - присутствует переменная page=1 что создает дубляж для первой страницы категории. http://opencartforum...я/?hl=пагинация Поэтому последняя девелоперская версия: https://github.com/myopencart/ocStore/archive/master.zip - уже содержит фикс для 17) Остальные мои 5 Pull-реквестов, которые делают то, что не успели внести в to-do list пока висят непринятыми: https://github.com/myopencart/ocStore/pulls Но можно скачать последнюю девелоперскую версию: https://github.com/myopencart/ocStore/archive/master.zip - и накатить сверху файлы из прикреплённого архива, в который я собрал все свои правки из первых 3-х Pull-реквестов. Будет, почти что Release Candidate :-) Однако, остаётся ещё: 1) Расмотреть предложения по оптимизации 2) починить и вернуть мультиязычный Seo-Pro или найти какую-то альтернативу для корректного SEO-продвижения мультиязычных сайтов. Это, пожалуй, самый важный пункт из того что осталось. Лично я сделал всё возможное, чтобы релиз вышел как можно скорее. Я и пожертвования делал, и Dinox-а уговаривал, и в конце концов сам начал делать правки. Но я не знаю как сделать то, что осталось. В особенности третий пункт. Передаю эстафету в ваши руки. Попробуйте или самостоятельно найти решение или мотивировать разработчиков просьбами и пожертвованиями! :geek: Как показала практика, это работает! Но, к сожалению или к счастью, я на этом всё, завтра мне нужно переключится на другой проект и поэтому теперь всё в ваших руках. Дерзайте! :-) FixManufacturerCategoryDescription_UpdateFilemanager.zip

Сделал сегодня ещё один Pull-реквест который исправляет "детскую болячку" index.php?route=common/home https://github.com/myopencart/ocStore/pull/26 Пока ждём одобрения Pull-реквеста Dinox-ом, особо нетерпеливые могут получить фикс, распаковав в корень сайта содержимое прикреплённого архива Мой Pull-реквест был отклонён, так как проблема с index.php?route=common/home решается SQL-запросом INSERT INTO oc_url_alias (QUERY, keyword) VALUES('common/home','');

Нашёл баг. Может быть его и легко исправить, но я не знаю как. При редактировании категории, выпадающий список значений для "Родительская категория:" сортируется по алфавиту, а не по заданному "Порядку сортировки"

Обновил сегодня свои правки filemanager, увеличил лимит загружаемых файлов с 300 000 до 3 200 000 байтов. Вот новый pull-реквест https://github.com/myopencart/ocStore/pull/25/ И в прикреплённом файле - обновлённый архив. Старое сообщение тоже отредактировал. filemanagerUpdated2.zip

Ну что ж вы невнимательно читаете сообщения? Я ж написал, что сделал Pull-реквест, как только Dinox его примет, эти исправления будут в текущей девелоперской версии 1.5.5.1.2: https://github.com/myopencart/ocStore/archive/master.zip и, если всё будет хорошо, в дальнейшем войдут в финальный релиз. А прикреплённые файлы - это так, чтобы получить эти исправления себе сразу, не дожидаясь одобрения Pull-реквеста Dinox-ом.

sql_mode это название режима а не номер версии Что он обозначает и для чего нужен можно почитать вот здесь: filemanagerUpdated2.zip filemanagerUpdated2.zip filemanagerUpdated2.zip

Если примут мои pull-реквесты, обсуждаемый здесь фикс войдёт в сборку: https://opencartforum.com/topic/27002-процесс-работ-над-релизом-ocstore-15512/?do=findComment&comment=210395 :-)

Если примут мои pull-реквесты, этот обсуждаемый здесь фикс войдёт в сборку: https://opencartforum.com/topic/27002-процесс-работ-над-релизом-ocstore-15512/?do=findComment&comment=210395

Смотрел Discovery :ugeek: :-D https://github.com/myopencart/ocStore/archive/master.zip плюс скачайте из моего предыдущего сообщения прикреплённый архив и залейте его содержимое в корень сайта и тогда у вас будет самый свежачок! :wink: Ну и всё проверить, пожалуй не будет лишним: :-)