mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #20765
[Bug 1372536] Re: PluginModule-menu items appear in two menus
Right, so although this problem has (in theory) existed for artefact and
interaction plugins for a while, none of the artefact or interaction
plugins in core, includes any top-level menu items, and so the problem
hasn't displayed itself.
However, since modules are meant to be more generic, they should be able
to add top-level menu items. So we do need to fix this for modules, and
we may as well fix it for artefacts and interactions as well.
So I agree, probably the best thing is to create a second method
specifically for the right nav menu items. We'll have to consider
whether that will break backwards-compatibility for any existing
plugins, though. (i.e. are any existing plugins passing back right-nav
menu items in their menu_items() function?)
** Changed in: mahara
Milestone: None => 1.10.0
** Changed in: mahara
Importance: Undecided => High
** Changed in: mahara
Status: New => Confirmed
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1372536
Title:
PluginModule-menu items appear in two menus
Status in Mahara ePortfolio:
Confirmed
Bug description:
With a module-plugin, that overwrites menu_items, the returned menu
items appear in the main navigation and also in the top right corner.
Both are very helpful on their own, but it should be possible to add
an item to either of them without the other, like, have an entry only
in the upper right corner but not in the main menu or vice versa.
The calls are indistinguishable (as far as I see) and originate from
web.php::main_nav, line 2683 and web.php::right_nav, line 2741
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1372536/+subscriptions
References