[Jifty-commit] r7305 - in plugins: Jifty-Plugin-Chart/lib/Jifty/Plugin/Chart

Jifty commits jifty-commit at lists.jifty.org
Fri Jul 10 16:23:35 EDT 2009


Author: trs
Date: Fri Jul 10 16:23:34 2009
New Revision: 7305

Modified:
   plugins/   (props changed)
   plugins/Jifty-Plugin-Chart/lib/Jifty/Plugin/Chart/View.pm

Log:
 r48321 at zot:  tom | 2009-07-10 16:23:13 -0400
 Whether we have a label or not, include it as a <string> tag.  If there's no tag for the label, xmlswf will be off by one when it displays the data as it takes the first tag to be a label.
 
 This fixes the off by one issue r5850 introduced, but may break the issue r5850 was trying to fix.


Modified: plugins/Jifty-Plugin-Chart/lib/Jifty/Plugin/Chart/View.pm
==============================================================================
--- plugins/Jifty-Plugin-Chart/lib/Jifty/Plugin/Chart/View.pm	(original)
+++ plugins/Jifty-Plugin-Chart/lib/Jifty/Plugin/Chart/View.pm	Fri Jul 10 16:23:34 2009
@@ -121,8 +121,8 @@
         my $label = $args->{legend}[$i];
 
         push @{$chart{'chart_data'}{'row'}}, {
+            string => [ defined $label ? $label : {} ],
             number => $args->{data}[$i],
-            defined($label) ? (string => [ $label ]) : (),
         };
     }
 


More information about the Jifty-commit mailing list