[freeside-commits] freeside/httemplate/search 477partIA_detail.html, 1.3, 1.4

Jeff Finucane,420,, jeff at wavetail.420.am
Wed Aug 25 00:34:28 PDT 2010


Update of /home/cvs/cvsroot/freeside/httemplate/search
In directory wavetail.420.am:/tmp/cvs-serv28780/httemplate/search

Modified Files:
	477partIA_detail.html 
Log Message:
stricter than docs suggest: no 0 values for partIA RT#9721

Index: 477partIA_detail.html
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/search/477partIA_detail.html,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -w -d -r1.3 -r1.4
--- 477partIA_detail.html	24 Aug 2010 17:30:34 -0000	1.3
+++ 477partIA_detail.html	25 Aug 2010 07:34:26 -0000	1.4
@@ -9,6 +9,7 @@
                   'disable_total'    => 1,
                   'header'           => [ '', @column_option_name ],
                   'xml_elements'     => [ @xml_elements ],
+                  'xml_omit_empty'   => 1,
                   'fields'           => [  @fields ],
               )
 %>
@@ -70,6 +71,9 @@
 my $query = 'SELECT '. join(' UNION ALL SELECT ', at row_option);
 my $count_query = 'SELECT '. scalar(@row_option);
 
+my $xml_element = 'OOPS, I was never set';
+my $rowchar = 101; # 'e' -- rows are columns! (pi/2)
+
 my $value = sub {
   my ($rowref, $column) = (shift, shift);
   my $row = $rowref->[0];
@@ -78,7 +82,7 @@
     return $row_option_name{$row} || 'no such report option';
   } elsif ( $column =~ /^(\d+)$/ ) {
     my @report_option = ( $row || '',
-                          $column_option[$1 - 2] || '',
+                          $column_option[$column] || '',
                           $technology_option[$tech_code] || '',
                         );
 
@@ -88,54 +92,35 @@
 
     my $percentage = sprintf('%.2f', $count ? 100 * $residential / $count : 0);
     my $return = $count;
-    $return .= "<BR>$percentage% residential"
-      unless $cgi->param('_type') eq 'xml';
+
+    if ($cgi->param('_type') eq 'xml') {
+      $rowchar++ if $column == 1;
+      $xml_element = $xml_prefix. chr($rowchar). ($column+1);
+      $return = '' if $count == 0 and $cgi->param('_type') eq 'xml';
+    } else {
+      $return .= "<BR>$percentage% residential";
+    }
+
     return $return;
   } else {
     return '<FONT SIZE="+1" COLOR="#ff0000">Bad call to column_value</FONT>';
   }
 };
 
-my @fields = (
-  sub { &{$value}(shift, 'name');},
-  sub { &{$value}(shift, 2);},
-  sub { &{$value}(shift, 3);},
-  sub { &{$value}(shift, 4);},
-  sub { &{$value}(shift, 5);},
-  sub { &{$value}(shift, 6);},
-  sub { &{$value}(shift, 7);},
-  sub { &{$value}(shift, 8);},
-  sub { &{$value}(shift, 9);},
- );
+my @fields = map { my $ci = $_; sub { &{$value}(shift, $ci); } }
+            ( 'name', (0 .. $#column_option) );
 shift @fields if $cgi->param('_type') eq 'xml';
 
-my $rowchar = 102; # 'f' -- rows are columns! (pi/2)
-my $opentag = 0;
-my $xml_element = sub {
-  my ($rowref, $column) = (shift, shift);
-  my $row = chr($rowchar);
-
-  if ($column == 8) {
-    $opentag++;
-    if ($opentag > 1) { # a new row
-      $rowchar++;
-      $opentag = 0;
-    }
-  }
-
-  $xml_prefix. $row. $column;
-
-};
-
 my @xml_elements = (  # -- columns are rows! (pi/2)
-  sub { &{$xml_element}(shift, 1) },
-  sub { &{$xml_element}(shift, 2) },
-  sub { &{$xml_element}(shift, 3) },
-  sub { &{$xml_element}(shift, 4) },
-  sub { &{$xml_element}(shift, 5) },
-  sub { &{$xml_element}(shift, 6) },
-  sub { &{$xml_element}(shift, 7) },
-  sub { &{$xml_element}(shift, 8) },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
+  sub { return $xml_element; },
 );
 
 </%init>



More information about the freeside-commits mailing list