COVID2019 и это вот все. Друзья, вся эта история начинает плохо пахнет. Мойте руки, не ходите в люди. Отложите все плановые покупки и положите в носок заначку. Заприте ваших родителей, бабушек-дедушек на даче. Лучше перебдеть чем недобдеть. Берегите себя!

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

savage4pro

Автор: ,
в Сообщения об ошибках

 Share

Recommended Posts

  • Это популярное сообщение.
savage4pro Newbie

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']);            }
Ссылка на комментарий
Поделиться на других сайтах
Pelmen Newbie

Pelmen

Опубликовано:
Опубликовано:

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


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

Pelmen

Опубликовано:
Опубликовано:

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


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


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

Pelmen

Опубликовано:
Опубликовано:

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


 


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


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


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

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 надо для этого менять, а что нет, не смогу

Ссылка на комментарий
Поделиться на других сайтах
  • 2 weeks later...
terhant Newbie

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 Collaborator

admin

Опубликовано:
  • Это популярное сообщение.
Опубликовано:

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

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

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

DronSF

Опубликовано:
Опубликовано:

OCSHOP PRO 2.1.0.1.4.


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


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

terhant

Опубликовано:
Опубликовано:
Апдейт. 

 

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

 

Сношу все. 

 

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

 

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

 

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

 

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

 

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

 

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

 

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

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

 

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

 

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

 

Такие дела.. 

 

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

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

terhant

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

 

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

 

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

 

Апдейт.

 

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

 

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

 

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

 


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 Newbie

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>

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


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


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

terhant

Опубликовано:
Опубликовано:

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


 


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


 


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


 


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


 


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


 


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

Ссылка на комментарий
Поделиться на других сайтах
  • 1 year later...
BuslikDrev Enthusiast

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
Ссылка на комментарий
Поделиться на других сайтах
  • RHCk changed the title to Произвольные ссылки в дизайн-меню

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Гость
Ответить в тему...

×   Вы вставили отформатированное содержимое.   Удалить форматирование

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Перейти к списку тем

  • Сейчас на странице   0 пользователей

    • Нет пользователей, просматривающих эту страницу