[Jifty-commit] r6610 - in jifty/trunk: lib/Jifty/Web
Jifty commits
jifty-commit at lists.jifty.org
Tue Mar 17 10:58:45 EDT 2009
Author: alexmv
Date: Tue Mar 17 10:58:45 2009
New Revision: 6610
Modified:
jifty/trunk/ (props changed)
jifty/trunk/lib/Jifty/Web/Menu.pm
Log:
r43249 at kohr-ah: chmrr | 2009-03-17 10:32:55 -0400
Avoid lots of calls to Jifty->web by caching Jifty::Web object
Modified: jifty/trunk/lib/Jifty/Web/Menu.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Web/Menu.pm (original)
+++ jifty/trunk/lib/Jifty/Web/Menu.pm Tue Mar 17 10:58:45 2009
@@ -285,14 +285,15 @@
@_
);
my @kids = $self->children;
- my $id = Jifty->web->serial;
- Jifty->web->out( qq{<li class="toplevel }
+ my $web = Jifty->web;
+ my $id = $web->serial;
+ $web->out( qq{<li class="toplevel }
. ( $self->active ? 'active' : 'closed' ) .' '.($self->class||"").' '. qq{">}
. qq{<span class="title">} );
- Jifty->web->out( $self->as_link );
- Jifty->web->out(qq{</span>});
+ $web->out( $self->as_link );
+ $web->out(qq{</span>});
if (@kids) {
- Jifty->web->out(
+ $web->out(
qq{<span class="expand"><a href="#" onclick="Jifty.ContextMenu.hideshow('}
. $id
. qq{'); return false;"> </a></span>}
@@ -300,17 +301,17 @@
. $id
. qq{">} );
for (@kids) {
- Jifty->web->out(qq{<li class="submenu }.($_->active ? 'active' : '' ).' '. ($_->class || "").qq{">});
+ $web->out(qq{<li class="submenu }.($_->active ? 'active' : '' ).' '. ($_->class || "").qq{">});
# We should be able to get this as a string.
# Either stringify the link object or output the label
# This is really icky. XXX TODO
- Jifty->web->out( $_->as_link );
- Jifty->web->out("</li>");
+ $web->out( $_->as_link );
+ $web->out("</li>");
}
- Jifty->web->out(qq{</ul>});
+ $web->out(qq{</ul>});
}
- Jifty->web->out(qq{</li>});
+ $web->out(qq{</li>});
'';
}
More information about the Jifty-commit
mailing list