Hi Stacey

It looks more like we need to tidy up the code a bit as we have in
group_get_menu_tabs() both

            safe_require('interaction', $plugin->name);
            $plugin_menu = 
call_static_method(generate_class_name('interaction', $plugin->name), 
'group_menu_items', $group);
            $menu = array_merge($menu, $plugin_menu);

and

            safe_require('artefact', $plugin->name);
            $plugin_menu = 
call_static_method(generate_class_name('artefact',$plugin->name), 'group_tabs', 
$group->id);
            $menu = array_merge($menu, $plugin_menu);

calling either 'group_menu_items' or 'group_tabs'

We should make this more generic and have all plugins, maybe only from
an array eg (artefact,interaction,module) to be checked for 'group_tabs'
functions


** Changed in: mahara
   Importance: Undecided => Wishlist

-- 
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/1718480

Title:
  Allow modules to extend group menu items

Status in Mahara:
  New

Bug description:
  Similar to the interaction plugin types allow modules to
  add/edit/remove items within group instance navigations.

  This is a useful extension for adding contributed modules.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1718480/+subscriptions

_______________________________________________
Mailing list: https://launchpad.net/~mahara-contributors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp

Reply via email to