Revert the change that removed adding the click handler for the digibug print

functionality.  Just return false when using javascript:digibug_popup was not
enough from having the browser switch pages and show a page with the word false.
Using an event handler allows us to call event.preventDefault which cancels the
page switch.

1 files changed, 2 insertions, 4 deletions

diff --git a/modules/digibug/helpers/digibug_menu.php b/modules/digibug/helpers/digibug_menu.php
index 4b8db5a2..f1d4fc1d 100644
--- a/modules/digibug/helpers/digibug_menu.php
+++ b/modules/digibug/helpers/digibug_menu.php
@@ -32,8 +32,7 @@ class digibug_menu {
       Menu::factory("link")
       ->id("digibug")
       ->label(t("Print with Digibug"))
-      ->url("javascript:digibug_popup('" .
-            url::site("digibug/print_photo/$item->id?csrf=$theme->csrf") . "')")
+      ->url(url::site("digibug/print_photo/$item->id?csrf=$theme->csrf"))
       ->css_id("gDigibugLink"));
   }
 
@@ -43,8 +42,7 @@ class digibug_menu {
         Menu::factory("link")
         ->id("digibug")
         ->label(t("Print with Digibug"))
-        ->url("javascript:digibug_popup('" .
-              url::site("digibug/print_photo/$item->id?csrf=$theme->csrf") . "')")
+        ->url(url::site("digibug/print_photo/$item->id?csrf=$theme->csrf"))
         ->css_id("gDigibugLink"));
     }
   }