[Jifty-commit] r1115 - in jifty/trunk: lib/Jifty/Web

[jifty-commit at lists.jifty.org]

Author: jesse
Date: Wed May 24 00:28:20 2006
New Revision: 1115

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Web/Menu.pm

Log:
 r14041 at hualien:  jesse | 2006-05-24 00:17:35 -0400
  * Stringifying links in menus wasn't doing the right thing


Modified: jifty/trunk/lib/Jifty/Web/Menu.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Web/Menu.pm	(original)
+++ jifty/trunk/lib/Jifty/Web/Menu.pm	Wed May 24 00:28:20 2006
@@ -175,7 +175,9 @@
     my @kids = $self->children;
     my $id = Jifty->web->serial;
     Jifty->web->out(
-        qq{<ul class="context_menu">} .qq{<li class="closed toplevel">}.  qq{<span class="title">} . $self->as_link() . qq{</span>}
+        qq{<ul class="context_menu">} .qq{<li class="closed toplevel">}.  qq{<span class="title">});
+     $self->as_link();
+     Jifty->web->out( qq{</span>}
             . (
             @kids
             ? qq{<span class="expand"><a href="#" onClick="Jifty.ContextMenu.hideshow('}.$id.qq{'); return false;">+</a></span>}
@@ -187,7 +189,9 @@
         Jifty->web->out("<li>");
 
         # We should be able to get this as a string.
-        $_->as_link;
+        # Either stringify the link object or output the label
+        # This is really icky. XXX TODO 
+        Jifty->web->out($_->as_link);
         Jifty->web->out("</li>");
     }