Bug #15182
closed
new similar Option to HMENU includeNotInMenu
Added by Marc Bastian Heinrichs over 18 years ago.
Updated almost 18 years ago.
Description
We have the "Hide in Menu" property for any page type, but there is no option for HMENU to include menu-hidden pages in a special menu, like includeNotInMenu for the pagetype 'Not in Menu'.
(issue imported from #M1794)
Files
I think as well that this can be done like tob outlined. If not, please reopen the bug report.
Greets, Sebastian
I don't want to exclude other doktypes from menus, which is done by excludeDoktypes.
I think, it's useful to have the TS-option for an HMENU to include pages, where the option "Hide in menu" is set.
This is currently done by the TS-option includeNotInMenu for pages with the pagetype 'Not in menu', but don't work with option "Hide in menu".
ok confirmed..
following menu shows only pages with the doctype "not in menu", but not "hide in menu" (hide in menu sets nav_hide=1 in pages and this is hardcodet in class.tslib_menu.php
eg:
$extraWhere = ' AND pages.nav_hide=0'.$this->getDoktypeExcludeWhere();
)
temp.menu=HMENU
temp.menu.includeNotInMenu=1
temp.menu.excludeDoktypes=5
temp.menu.1=TMENU
temp.menu.1.NO {
allWrap = | >>
}
I also wondered why there are 2 possibilities to hide a page .. maybe for eg. mailform confirm/thank you pages which you really dont want to have in a menu
basheinr: Yes, you are right. The pagetype "not in menu" will most likely be removed in 4.0 and replaced by the nav_hide setting during the update.
Greets, Sebastian
Hi,
the current patch modifies the behavior in a way that the pages with the "hide in menu" property are shown when includeNotInMenu is set.
Does this make sense?
It would be great if you could test the patch,
Greets, Sebastian
Hi Sebastian,
I've tested tes patch and it doesn't work, sorry.
I looked at the source and I think your changes applies only to HMENU special 'updated' and 'keywords'. In function filterMenuPages pages with 'nav_hide' set don't get into the menu.
In function generate_level of class tslib_jsmenu the nav_hide options is also used.
greets, Marc Bastian
I think you are right - however I don't have time to provide a new patch at the moment. Will look at this later.
Greets, Sebastian
Also available in: Atom
PDF