Fix up context menus a bit.

- Update digibug_event::context_menu() to take the actual item
- Change gallery::context_menu() to not require the page_type as an argument

2 files changed, 6 insertions, 6 deletions

diff --git a/modules/gallery/helpers/gallery.php b/modules/gallery/helpers/gallery.php
index 0bf180c1..279f2013 100644
--- a/modules/gallery/helpers/gallery.php
+++ b/modules/gallery/helpers/gallery.php
@@ -197,7 +197,8 @@ class gallery_Core {
     return $menu;
   }
 
-  static function context_menu($menu, $theme, $item, $page_type) {
+  static function context_menu($menu, $theme, $item) {
+    $page_type = $theme->page_type();
     switch ($item->type) {
     case "movie":
       $edit_title = t("Edit this movie");
@@ -218,9 +219,9 @@ class gallery_Core {
     $move_title = t("Move to another album");
 
     $csrf = access::csrf_token();
-    
+
     $menu->append($options_menu = Menu::factory("submenu")
-                  ->id("options")
+                  ->id("options_menu")
                   ->label(t("Options"))
                   ->css_class("ui-icon-carat-1-n"));
 
diff --git a/modules/gallery/libraries/Theme_View.php b/modules/gallery/libraries/Theme_View.php
index 24dea729..a5f5db03 100644
--- a/modules/gallery/libraries/Theme_View.php
+++ b/modules/gallery/libraries/Theme_View.php
@@ -118,9 +118,8 @@ class Theme_View_Core extends Gallery_View {
                ->label(t("Options")))
       ->css_class("gContextMenu");
 
-    $page_type = Input::instance()->get("page_type");
-    gallery::context_menu($menu, $this, $item, $page_type);
-    module::event("context_menu", $menu, $this, $item, $page_type);
+    gallery::context_menu($menu, $this, $item);
+    module::event("context_menu", $menu, $this, $item);
     return $menu->compact();
   }