From 789fd31c2d483a8f20db3708d79171ec7267dd84 Mon Sep 17 00:00:00 2001
From: Mike <mikerthomsen@gmail.com>
Date: Sat, 4 Dec 2010 21:02:07 -0500
Subject: [PATCH] [#641 state:resolved] Fixed an error which caused the
 widgetset loop tag to misbehave with the last widget.

---
 lib/MT/Template/ContextHandlers.pm | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/lib/MT/Template/ContextHandlers.pm b/lib/MT/Template/ContextHandlers.pm
index 5d76daf90..bd9019104 100644
--- a/lib/MT/Template/ContextHandlers.pm
+++ b/lib/MT/Template/ContextHandlers.pm
@@ -22031,19 +22031,21 @@ sub _hdlr_widget_loop {
     $ctx->stash('widgetset', $tmpl);
     local $vars->{__size__} = scalar(@selected);
     my $glue = $args->{glue} || '';
-    for (my $index = 0; $index <= $#selected; $index++) {
+    my $size = scalar(@selected);
+    for (my $index = 0; $index < $size; $index++) {
         my $widget = MT->model('template')->load({ id => $selected[$index] });
         local $vars->{__first__} = ($index == 0);
-        local $vars->{__last__} = ($index == $#selected-1);
+        local $vars->{__last__} = ($index == ($size-1));
         local $vars->{__index__} = $index;
         local $vars->{__odd__} = $index % 2 == 1;
         local $vars->{__even__} = $index % 2 == 0;
         $ctx->stash('widget', $widget);
         my $res = $builder->build($ctx, $tokens, $cond);
         return $ctx->error($builder->errstr) unless defined $res;
-        
-        $out .= ($index < $#selected and $glue ? sprintf('%s%s', $res, $glue) : $res);
+        $res .= $glue if ($glue && $index < $size - 1);
+        $out .= $res;
     }
+
     
     $ctx->stash('widgeset', undef);