Joomline / jlsitemap

JL Sitemap - Component sitemap for Joomla
https://joomline.ru/rasshirenija/komponenty/jlsitemap.html
GNU General Public License v3.0
27 stars 9 forks source link

Ошибка в приоритете частоты изменений #55

Closed atramentus closed 4 years ago

atramentus commented 4 years ago

Столкнулся с проблемой. В общих настройках компонента выставлен 1 приоритет, в плагине другой. Настройки компонента image

Настройки плагина image image

На скриншоте видно что частота изменений берется с компонента, а приоритет с плагина image

zikkuratvk commented 4 years ago

Если мы говорим про эти пункты?

image То это не материалы, по этому приоритет отдается настройкам из компонента.

atramentus commented 4 years ago

по идее везде, где приоритет изменился на 0,3 (это настройка пришла из плагина), также должна была измениться и частота изменений (тоже из плагина "monthly").

zikkuratvk commented 4 years ago

Вот теперь яснее стало :-) Это действительно баг.

Septdir commented 4 years ago

Ну вот приотиет надо будет проверить ибо 1приорететнее 0.3 в остальном верно. delaly паритетнее mounthsly, а т.к это пункт меню то и соответственно берутся настройки компонента.

Septdir commented 4 years ago

Строка для проверки, чтобы не потерять https://github.com/Joomline/jlsitemap/blob/master/com_jlsitemap/site/models/sitemap.php#L580

atramentus commented 4 years ago

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

Прикладная задача расставить частоту и приоритеты страницам. Главная -- daily 1,0 Товары (не являются пунктами меню) -- weekly 0,7 Категории товаров (являются пунктами меню) -- weekly 0,8 Материалы/категории сайта (не важно являются они пунктами меню или нет) -- monthly 0,4

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

Возможно я не прав и не до конца разобрался с настройками.

Septdir commented 4 years ago

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

atramentus commented 4 years ago

Было бы великолепно. В любом случае спасибо за компонент

Septdir commented 4 years ago

Для каждого отдельного итема сделал отдельный issue

56