1. $form->{title} wird nicht mehr zwangsweise umgeschrieben und nach $form->header...
authorMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 14 Jun 2007 15:33:44 +0000 (15:33 +0000)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 14 Jun 2007 15:33:44 +0000 (15:33 +0000)
2. Kosmetik: lokale Variable $form anstelle von $self->{form}.

SL/ReportGenerator.pm

index dbdbf72..4a48dfd 100644 (file)
@@ -161,9 +161,10 @@ sub set_export_options {
 sub generate_with_headers {
   my $self   = shift;
   my $format = lc $self->{options}->{output_format};
+  my $form   = $self->{form};
 
   if (!$self->{columns}) {
-    $self->{form}->error('Incorrect usage -- no columns specified');
+    $form->error('Incorrect usage -- no columns specified');
   }
 
   my $filename =  $self->{options}->{attachment_basename} || 'report';
@@ -171,8 +172,11 @@ sub generate_with_headers {
   $filename    =~ s|.*/||;
 
   if ($format eq 'html') {
-    $self->{form}->{title} = $self->{title};
-    $self->{form}->header();
+    my $title      = $form->{title};
+    $form->{title} = $self->{title} if ($self->{title});
+    $form->header();
+    $form->{title} = $title;
+
     print $self->generate_html_content();
 
   } elsif ($format eq 'csv') {
@@ -184,7 +188,7 @@ sub generate_with_headers {
     $self->generate_pdf_content();
 
   } else {
-    $self->{form}->error('Incorrect usage -- unknown format (supported are HTML, CSV, PDF)');
+    $form->error('Incorrect usage -- unknown format (supported are HTML, CSV, PDF)');
   }
 }