Новости Joomla

0 Пользователей и 1 Гость просматривают эту тему.
  • 10 Ответов
  • 1111 Просмотров
*

Stweet

  • Новичок
  • 6
  • 0 / 0
Прошу помощи у гуру "джомладелов". Не могу понять, ошибка это или идея автором CMS.
В классе "ContentViewArticle" компонента "com_content" есть такой метод "_prepareDocument". Так вот там есть условие построения цепочки навигации.
Проблема в том, что не могу понять почему автор кода сравнивает ID записи с ID категории. Ошибка это или я чего то недопонимаю?
Заранее спасибо!
Код: php
if ($menu && ($menu->query['option'] != 'com_content' || $menu->query['view'] != 'article' || $id != $this->item->id))
{
// If this is not a single article menu item, set the page title to the article title
if ($this->item->title)
{
$title = $this->item->title;
}
$path = array(array('title' => $this->item->title, 'link' => ''));
$category = JCategories::getInstance('Content')->get($this->item->catid);

// Вот тут условие $id != $category->id где ранее в переменную задаётся ID записи ($id = (int) @$menu->query['id'])
// или $menu->query['id'] - может быть как ID категории так и ID записи?!...
while ($category && ($menu->query['option'] != 'com_content' || $menu->query['view'] == 'article' || $id != $category->id) && $category->id > 1)
{
$path[] = array('title' => $category->title, 'link' => ContentHelperRoute::getCategoryRoute($category->id));
$category = $category->getParent();
}
$path = array_reverse($path);

foreach ($path as $item)
{
$pathway->addItem($item['title'], $item['link']);
}
}

Что то я в конец перестаю улавливать логику авторов кода.
« Последнее редактирование: 07.01.2015, 21:16:31 от Stweet »
*

Shustry

  • Гуру
  • 6434
  • 745 / 3
А в чём суть вопроса? Вполне себе код.
*

Stweet

  • Новичок
  • 6
  • 0 / 0
А в чём суть вопроса? Вполне себе код.
Прошу прощения, опубликовал раньше чем дописал, перечитайте пжалста.
*

Shustry

  • Гуру
  • 6434
  • 745 / 3
Здесь id из строки и есть id категории при условии, что строка вообще про категорию. Всё тут верно.
*

Stweet

  • Новичок
  • 6
  • 0 / 0
Здесь id из строки и есть id категории при условии, что строка вообще про категорию. Всё тут верно.
Хорошо, тогда почему в первом условии сравнивается (в вашем случае ID категории с ID записи) $id != $this->item->id
Код: php
if ($menu && ($menu->query['option'] != 'com_content' || $menu->query['view'] != 'article' || $id != $this->item->id)) { /* ... */ }
*

Stweet

  • Новичок
  • 6
  • 0 / 0
Т.е. я не могу понять где и на что автор пытается проверить?

Код: php
// тут он проверяет отношение пункта меню к записи
if ($menu && ($menu->query['option'] != 'com_content' || $menu->query['view'] != 'article' || $id != $this->item->id)) {
// а тут отношение к категории
while ($category && ($menu->query['option'] != 'com_content' || $menu->query['view'] == 'article' || $id != $category->id) && $category->id > 1) { /* ... */ }
}

// тогда вопрос, $menu->query['id'] - может быть как ID категории так и ID записи? Или в данном случае, при явном обращении к записи ID категории не передаётся?
// У меня мозг начинает плавиться потихоньку.

Просто, что бы понять идею автором данного движка, я разбираю детали которые и не дают покоя.
Ещё раз спасибо за разъяснения!
*

robert

  • Живу я здесь
  • 4974
  • 457 / 20
Читайте comment
Код: php-brief
// if the menu item does not concern this article
if ($menu && ($menu->query['option'] != 'com_content' || $menu->query['view'] != 'article' || $id != $this->item->id))
Это значит: если есть $menu + (не com_content или com_content, но не article, или com_content+article, но не с этим $id) - дальше действуй. Что тут не так?
Не будь паразитом, сделай что-нибудь самостоятельно!
*

Stweet

  • Новичок
  • 6
  • 0 / 0
Добавлю, когда поправил малость код, всё заработало адекватно:

Код: php

        // Добавил ещё одну переменную.
        $catid = (int) @$menu->query['catid'];
        $id = (int) @$menu->query['id'];
        
        // trace($menu);
        
        if ($menu && ($menu->query['option'] != 'com_catalog' || $menu->query['view'] != 'record' || $id != $this->item->id)) {

            // и тут её сравниваю.
            while ($category && ($menu->query['option'] != 'com_catalog' || $menu->query['view'] == 'record' || $catid != $category->id) && $category->id > 1) { }

Перепробовал все возможные варианты, работает корректно. А вот при работе старого решения, категории в цепочке дублируются, например:
Home > First > Next
Если из категории First обратиться к ней ещё раз передав catid категории First, результат:
Home > First > First > Next

Это нормально?
*

robert

  • Живу я здесь
  • 4974
  • 457 / 20
Home > First > First > Next
ненормально. А оригинал что, некорректно работает?
Не будь паразитом, сделай что-нибудь самостоятельно!
*

Stweet

  • Новичок
  • 6
  • 0 / 0
ненормально. А оригинал что, некорректно работает?

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

Ещё раз спасибо, буду копать далее.
*

b2z

  • Глобальный модератор
  • 7287
  • 778 / 0
  • Разраблю понемногу
Re: Баги или недопонимание? (ContentViewArticle)
« Ответ #10 : 08.01.2015, 11:23:04 »
У Вас два раза повторяется навзание категории в цепочке при переходе на запись? По идее такого быть не должно, так как это проверяется кодом $catid != $category->id. Попробуйте сделать дамп этих переменных, чтобы убедиться, что не в этом проблема.
Чтобы оставить сообщение,
Вам необходимо Войти или Зарегистрироваться