Произвольные ссылки в дизайн-меню

[savage4pro]

Версия 2.1.0.1.2
 
Дизайн -> Меню
добавление "Произвольных ссылок" приводит к небольшому вороху ошибок
в итоге js-ом создается пункт, который можно, конечно, редактировать и уже сохранить, но это как-то неправильно
 
 

PHP Notice:  Undefined index: menu_desc in [...]/admin/model/design/menu.php on line 110
PHP Warning:  Invalid argument supplied for foreach() in [...]/admin/model/design/menu.php on line 110
PHP Notice:  Undefined index: menu_desc in [...]/admin/model/design/menu.php on line 117
PHP Notice:  Undefined offset: 70 in [...]/admin/controller/design/menu.php on line 238
PHP Notice:  Undefined offset: 70 in [...]/admin/controller/design/menu.php on line 238
PHP Notice:  Undefined offset: 70 in [...]/admin/controller/design/menu.php on line 248
PHP Notice:  Undefined offset: 70 in [...]/admin/controller/design/menu.php on line 248

 
при этом создается новая запись menu_id в таблице menu, но не добавляется запись в таблицу menu_description
метод add класса ModelDesignMenu, по всей видимости писали разные люди, или один, но с перерывом на какое-нибудь радостное событие
 
вариант решения

/admin/model/design/menu.php:77-82
заменить

            $query = new stdClass();            $query->rows = array();            foreach ($languages as $language) {                $query->rows[] = array('name' => $data['name'], 'language_id' => $language['language_id']);            }

на

            $data['menu_desc'] = array();            foreach ($languages as $language) {                $data['menu_desc'][] = array('name' => $data['name'], 'language_id' => $language['language_id']);            }

[admin]

про оба варианта угадали :-) и не один и радостное событие тоже было :-)

доберемся еще до меню

и учтем ваши правки

[Samoil]

 

 

для меня это набор символов, но это сработало )

[Pelmen]

наткунлся на другую ошибку: по какой-то причине сбросился признак "Использовать в качестве основного меню?" и в "Да" не сохраняется.

[admin]

А что вы для этого сделали?

[Pelmen]

не уверен на 100%, но по-моему я создал новую статью, добавил новую схему макета, навесил на статью макет и добавил ссылку на неё в меню.

и вместо моего почти сделанного меню стало меню категорий.

[admin]

Сейчас проверить не смогу

Но как я понял вы его включить не можете?

Вы рядом с Да нажимаете кнопку сохранить?

[Pelmen]

да, если необходимо, могу дать доступ.

в журнале ошибок нет. в консоли браузера тоже.

[admin]

Со следующего понедельника дадите сейчас нет доступа к десктопу

[Pelmen]

Проблему нашёл: совпало SEO имя у макета и у статьи.

 

Скажите, как правильно создавать ссылки в меню ?

если вставляю это же имя, например "auto", то формируется ссылка "/index.php?route=auto" и она не работает

[savage4pro]

Скажите, как правильно создавать ссылки в меню ?

указывать вместе с доменным именем сайта, что не всегда приемлимо

 

с произвольными ссылками в меню пока, к сожалению, все не совсем так, как хотелось бы

поэтому временно (до выхода обновления по меню) для себя решил данную задачу изменением шаблона, в котором оно выводится

например, в дефолтном шаблоне оно выводится в /catalog/view/theme/default/template/common/header.tpl

мое решение такое

заменить

        <?php foreach ($categories as $category) { ?>        <?php if ($category['children']) { ?>        <li class="dropdown"><a href="<?php echo $category['href']; ?>" class="dropdown-toggle" data-toggle="dropdown"><?php echo $category['name']; ?></a>          <div class="dropdown-menu">            <div class="dropdown-inner">              <?php foreach (array_chunk($category['children'], ceil(count($category['children']) / $category['column'])) as $children) { ?>              <ul class="list-unstyled">                <?php foreach ($children as $child) { ?>                <li><a href="<?php echo $child['href']; ?>"><?php echo $child['name']; ?></a></li>                <?php } ?>              </ul>              <?php } ?>            </div>            <a href="<?php echo $category['href']; ?>" class="see-all"><?php echo $text_all; ?> <?php echo $category['name']; ?></a> </div>        </li>        <?php } else { ?>        <li><a href="<?php echo $category['href']; ?>"><?php echo $category['name']; ?></a></li>        <?php } ?>        <?php } ?>

на

        <?php foreach ($categories as $category) { ?>        <?php            $showHref = false;            $tmpHref = explode('index.php?route=', $category['href']);            if (isset($tmpHref[1]) && !empty($tmpHref[1])) {                $showHref = true;                if (substr($tmpHref[1], 0,1)=='/') {                    $category['href'] = $tmpHref[1];                }            } else if (isset($tmpHref[0])) {                $tmpHrefPath = str_replace('/', '', parse_url($tmpHref[0],PHP_URL_PATH));                if (!empty($tmpHrefPath)) {                    $showHref = true;                    $category['href'] = $tmpHref[0];                }                unset($tmpHrefPath);            }            $category['href'] = html_entity_decode($category['href']);        ?>        <?php if ($category['children']) { ?>        <li class="dropdown"><a href="<?php echo ($showHref)?$category['href']:''; ?>" class="dropdown-toggle" data-toggle="dropdown"><?php echo $category['name']; ?></a>            <div class="dropdown-menu">                <div class="dropdown-inner">                    <?php foreach (array_chunk($category['children'], ceil(count($category['children']) / $category['column'])) as $children) { ?>                    <ul class="list-unstyled">                        <?php foreach ($children as $child) { ?>                        <?php                            $showHrefChild = false;                            $tmpHrefChild = explode('index.php?route=', $child['href']);                            if (isset($tmpHrefChild[1]) && !empty($tmpHrefChild[1])) {                                $showHrefChild = true;                                if (substr($tmpHrefChild[1], 0,1)=='/') {                                    $child['href'] = $tmpHrefChild[1];                                }                            } else if (isset($tmpHrefChild[0])) {                                $tmpHrefChildPath = str_replace('/', '', parse_url($tmpHrefChild[0],PHP_URL_PATH));                                if (!empty($tmpHrefChildPath)) {                                    $showHrefChild = true;                                    $child['href'] = $tmpHrefChild[0];                                }                                unset($tmpHrefChildPath);                            }                            $child['href'] = html_entity_decode($child['href']);                        ?>                        <li><a href="<?php echo ($showHrefChild)?$child['href']:''; ?>"><?php echo $child['name']; ?></a></li>                        <?php                            unset($tmpHrefChild);                            unset($showHrefChild);                        ?>                        <?php } ?>                        <?php                            if ($showHref) {                        ?>                                <li role="separator" class="divider"></li>                                <li><a href="<?php echo $category['href']; ?>" class="see-all"><?php echo $text_all; ?> <?php echo $category['name']; ?></a></li>                        <?php                            }                        ?>                        </ul>                    <?php } ?>                </div>            </div>        </li>        <?php } else { ?>        <li><a href="<?php echo $category['href']; ?>"><?php echo $category['name']; ?></a></li>        <?php } ?>        <?php            unset($tmpHref);            unset($showHref);        ?>        <?php } ?>

после этого можно использовать произвольные ссылки в меню в более удобоваримом виде, т.е. добавлять как сео-ссылки типа "/auto" (такие ссылки должны начинаться на слэш), так и относительные типа "product/product&path=20&product_id=35" (они будут в итоге слеплены со стандартным началом: "сайт.ру/index.php?route=", или сразу можно в полном виде добавлять)

 

но если где-то еще меню выводить, то либо надо и там похожую канитель выписывать, либо решать уже по-человечески - правкой контроллера

 

ах да, чуть не забыл, при такой замене кода шаблона, можно вообще ссылки не указывать для верхних пунктов меню, в этом случае они будут работать как табы и после дочерних пунктов также не будет пункта "Смотреть Все ..."

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

[Pelmen]

да, как временное решение пока прописываю полный путь.

[terhant]

Дизайн-Меню сломано получается? Раньше работал переход в категорию. Сегодня доделывал структуру меню (через конструктор Дизайн-Меню), лепит ошибки (ругается на menu.php).

 

2015-11-01 18:54:18 - PHP Notice: Undefined index: sort_order in /admin/model/design/menu.php on line 300

2015-11-01 18:54:18 - PHP Notice: Undefined index: menu_type in /admin/model/design/menu.php on line 300
2015-11-01 18:54:21 - PHP Notice: Undefined index: sort_order in /admin/model/design/menu.php on line 273
2015-11-01 18:54:21 - PHP Notice: Undefined index: menu_type in /admin/model/design/menu.php on line 273
2015-11-01 18:54:21 - PHP Notice: Undefined index: sort_order in /admin/model/design/menu.php on line 300
2015-11-01 18:54:21 - PHP Notice: Undefined index: menu_type in /admin/model/design/menu.php on line 300

 

Сами кнопки при добавлении в админке, в меню появляются и подменю тоже. Но не переходит в соотв. раздел (категорию) на сайте. При наведении мыши в статусной строке браузера отображается ссылка на гл. страницу сайта. Где эту ссылку поправить в самих пунктах меню в разделе Дизайн-Меню не понял (соответствующей строки со ссылкой нет в настройках пункта меню). По идее, она должна генерироваться сама.

 

Тем не менее в демке OCSHOPa все работает и ссылка отображается правильно и переходит все верно. Файлы движка по адресу /admin/model/design имеют дату изменения ту же, что и остальные, т.е. установленные дополнения сломать ничего не должны были..

 

В ветке упоминалось, что про этот баг знают, но там говорилось про ошибки с произвольными ссылками. А как обстоят дела с пунктами меню - Категории?

 

И второй вопрос. Обратил внимание, что пункты меню "из коробки" имеют в настройках в конструкторе строку для ссылки. Это и есть результат добавления пункта меню "произвольная ссылка", которая пока не работает? Т.е. новые такие же пункты добавить пока нельзя.

 

Апдейт.

Поставил на локалку последнюю версию движка, указанная проблема отсутствует. Буду копать что я мог натворить. Из установленных дополнений - Автом. обраб. прайс-листов, Связанные опции и Batch Edit..

 

Апдейт.

На локалке попытался создать пункт меню "Произвольные ссылки", вывалил ту же ошибку об неисправностях в menu.php. Но по категориям из гл. меню переходит. На веб версии, нет)) Сдается мне, что-то я в настройках Макетов напортачил.

[admin]

пока из вашего объяснения все понятно что ничего не понятно :-)

ждем следующее обновление ваших изысканий :-)

[DronSF]

OCSHOP PRO 2.1.0.1.4.

Уже было два обновления с момента обнаружения данной ошибки, но пока она не устранена (хотя решение уже дали). Когда же?

[terhant]

Апдейт. 

 

Следующим изысканием причины была установка версии окшоп 2.1.0.1.4 из коробки(без приблуд). Для чистоты эксперимента из модулей установил на сервер только модуль МеджикЗоомПлюс. Потыкал по менюшке, вроде все ок.

 

Сношу все. 

 

Снова накатываю окшоп из коробки (без доп модулей), версия последняя, 21014.

 

Восстанавливаю бекап БД (только данные,  через ПхпМуАдмин) сами внутренности двигла остаются без изменений.

 

После чего,  в момент установки модуля Меджик Зоом...  менюшка ломается))

 

Думаю попробовать на окшоп 2 из коробки поставить сначала модули,  потом не восстанавливая данные из бекапа БД сделать ручной перенос данных.  Т.е. создать руками  заново категории,  настроить сам магазин вручную (через админку,  настройки,  мой магазин). Импортировать модулем автоматич. обработка прайсов из прайса тестовые товары. И посмотреть что будет с меню.

 

Есть подозрение, что из старого бекапа (он от версии 21013), через ПхПМуАдмин импортируется что-то , что при установке модуля MagicZoom портит гл. меню.

 

Вопрос к знающим,  это вообще возможно?  чтобы данные из таблиц БД "ломали" ссылки из гл. меню на соотв.  раздела сайта?  я нуб в этом деле,  соображаю пока на уровне принципов работы) 

 

Цель- выявить либо:

1)модуль кривой -  написать разрабам модуля, 

 

2)гл. меню кривое - написать сюда) 

 

3)криво что-то навосстанавливал я -  экспериментально устранить ошибку и отписаться сюда.

 

Такие дела.. 

 

ЗЫ. Надеюсь, понятно объяснил. 

[admin]

попробуйте восстановить без таблиц menu

[wivern]

Ошибку исправили?

[admin]

Ошибку исправили?

какую именно ошибку ?

сейчас нужно вставлять обсалютную ссылку начиная с http

[wivern]

воспользовался решением автора топика, всё работает с прямыми ссылками)

[terhant]

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

 

Засунул свои изыскания под спойлер. Кому интересно прочитает.

 

Пока пришел к выводу, что все портит модуль MagicZoom+. Придется отказаться от него, а жаль - с ним очень эффектно выглядят даже убогие (на телефон  ) фотографии товаров .

 

Апдейт.

 

На оригинальном Опенкарте (с офсайта) ошибка не воспроизводится. Все работает без ошибок. У них там нет отдельного раздела по Дизайн - Меню)) Видимо только с нашей сборкой конфликтует.

 

Но я все равно написал разрабам модуля (письма писать не дают, все общение только через форму обратной связи с сайта..)

 

Не знаю, правомерно ли просить, но гляньте как будет время совместимость своей сборки с модулем MZ+    Я был бы очень признателен, если бы удалось подружить)

 

1) Накатываю чистый Опенкарт.про (2.1.0.1.5). Модули не устанавливаю.

2) Создаю вручную Родительскую категорию, подкатегорию. (Демонстрационные пункты не удаляю)

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

4) Дизайн-Меню. Добавляю Главную категорию и подкатегорию. скрин. Журнал ошибок пуст (здесь и далее имеется ввиду Журнал ошибок Опенкарта (Система-Инструменты-Журнал ошибок)).

5) Ставлю модуль Автом. обраб. прайс-листов (АОПЛ). В логе установки упоминается про внесение изменений в файл menu.php. скрин. Журнал ошибок пуст.

6) Импортирую модулем АОПЛ несколько товаров из тестового прайса. Главное меню работает. При наведении на пункт меню в строке состояния отображается ссылка на категорию. скрин Журнал ошибок пуст.

7) Устанавливаю модуль MZ+. Все. Ссылки больше не работают. Все пункты главного меню теперь ссылаются на главную страницу сайта. скрин. Даже те пункты, что были изначально в демонстрационном магазине. Тем не менее, журнал ошибок пуст. Но он пуст до тех пор, пока не поворошишь через конструктор Дизайн-Меню пункты меню. Пункт меню Категория добавляется без вызова ошибок (журнал ошибок пуст). Но стоит создать пункт меню Произвольная ссылка, вываливается ошибка. 

2015-11-08 16:14:36 - PHP Notice:  Undefined index: menu_desc in Мой_магазин/admin/model/design/menu.php on line 108

2015-11-08 16:14:36 - PHP Warning:  Invalid argument supplied for foreach() in Мой_магазин/admin/model/design/menu.php on line 108

2015-11-08 16:14:36 - PHP Notice:  Undefined index: menu_desc in Мой_магазин/admin/model/design/menu.php on line 115

2015-11-08 16:14:36 - PHP Notice:  Undefined offset: 67 in Мой_магазин/admin/controller/design/menu.php on line 236

2015-11-08 16:14:36 - PHP Notice:  Undefined offset: 67 in Мой_магазин/admin/controller/design/menu.php on line 246

 

Провел аналогичный пошаговый эксперимент с единственным модулем MZ+. Ошибка воспроизводится. 

 

Жаль логов установки от MZ+ нет, чтобы посмотреть лазит ли он в файл menu.php.

 

Напишу разрабам модуля MZ+. Обидно, столько интерактива этот модуль доставлял.. Придется отказаться пока от него, может допилят.

 

ЗЫ. Блин, с закрытыми глазами теперь могу поднимать двигло и настраивать после таких тренировок..

menu.php(дефолтный).php

menu.php_после_создания_произв.ссылки.php

menu.php_после_установкиMZplus.php

[savage4pro]

terhant

в двух словах

модуль MZ+ окмодит catalog/controller/common/header.php

нам интересен вот этот кусочек

		<operation>

			<search><![CDATA[return]]></search>

			<add  position="replace"><![CDATA[

			    $contents = ]]>

			</add>

		</operation>

это означает буквально заменить в нашем контроллере все "return" на "$contents = ", прекрасно

смотрим на сам контроллер и видим, что в отличие от оригинальной сборки, контроллер имеет метод getMenuLink, который так же возвращает некое значение:

return $link;

и этот return так же весело заменяется модулем MZ+, что приводит к тому, что метод ничего не возвращает и меню ссылок не имеет

возьмите ваш окмод mod_opencart_magiczoomplus.zip/magiczoomplus/module/system/magiczoomplus.ocmod.xml (или откуда вы его берете) и замените вышеприведенный кусок на, например, такой:

		<operation>

			<search regex="true" limit="2"><![CDATA[#return \$this\->load\->view\(#i]]></search>

			<add  position="replace"><![CDATA[

			    $contents = $this->load->view(]]>

			</add>

		</operation>

надеюсь, это прекратит ваши мучения

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

[savage4pro]

модуль не ставил, поэтому проверить не могу, сами сами

[terhant]

Спасибо за ответ)

 

Я понял) Временный костыль)) (Без сарказма).

 

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

 

Как меру, максимально приближенную к результатам работы модуля MZ+, в настройках выставил побольше размер картинок. Больше 1200 точек (или около того) выставлять не имеет смысла. Похоже максимальное значение для фрейма. (Возможно максимальное для моего монитора - 1920*1080точек).

 

А вообще, как говорят оценивающие мою любительскую работу со стороны, все эти красивости - лишние заморочки и совершенно не нужны. Особенно на начальном этапе работы магазина) Имеющегося функционала движка вполне достаточно.

 

Буду обходиться пока без "хотелки", в будущем в планах улучшения функционала, возможно, рассмотрю замену модулю (уж больно он красиво работает ).

[BuslikDrev]

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

• Notice: Undefined offset: 0 in /var/www/... .../admin/model/design/menu.php on line 114
• Notice: Undefined offset: 69 in  /var/www/... .../admin/model/design/menu.php on line 237
• Notice: Undefined offset: 69 in  /var/www/... .../admin/model/design/menu.php on line 237