From 0cbac4f7706f3e24f98d6f320278c25878ef4b9b Mon Sep 17 00:00:00 2001
From: =?utf8?q?Sven=20Sch=C3=B6ling?= <s.schoeling@linet-services.de>
Date: Mon, 6 Jan 2014 15:37:52 +0100
Subject: [PATCH] all_parts: Warnung wenn versucht wird l_soldtotal ohne bsooqr
 aufzurufen

---
 bin/mozilla/ic.pl                              | 9 ++++++++-
 locale/de/all                                  | 1 +
 templates/webpages/ic/generate_report_top.html | 6 ++++++
 3 files changed, 15 insertions(+), 1 deletion(-)
 create mode 100644 templates/webpages/ic/generate_report_top.html

diff --git a/bin/mozilla/ic.pl b/bin/mozilla/ic.pl
index c7cb71b07..da20bf59f 100644
--- a/bin/mozilla/ic.pl
+++ b/bin/mozilla/ic.pl
@@ -1194,6 +1194,8 @@ sub generate_report {
 
   # soldtotal doesn't make sense with more than one bsooqr option.
   # so reset it to sold (the most common option), and issue a warning
+  # ...
+  # also it doesn't make sense without bsooqr. disable and issue a warning too
   my @bsooqr = qw(sold bought onorder ordered rfq quoted);
   if ($form->{l_subtotal} && 1 < grep { $form->{$_} } @bsooqr) {
     my $enabled       = first { $form->{$_} } @bsooqr;
@@ -1202,6 +1204,11 @@ sub generate_report {
 
     push @options, $::locale->text('Subtotal cannot distinguish betweens record types. Only one of the selected record types will be displayed: #1', $optiontexts{$enabled});
   }
+  if ($form->{l_soldtotal} && !grep { $form->{$_} } @bsooqr) {
+    delete $form->{l_soldtotal};
+
+    flash('warning', $::locale->text('Soldtotal does not make sense without any bsooqr options'));
+  }
 
   IC->all_parts(\%myconfig, \%$form);
 
@@ -1266,7 +1273,7 @@ sub generate_report {
     'assembly' => $locale->text('assembly_list'),
   );
 
-  $report->set_options('top_info_text'         => $locale->text('Options') . ': ' . join(', ', grep $_, @options),
+  $report->set_options('raw_top_info_text'     => $form->parse_html_template('ic/generate_report_top', { options => \@options }),
                        'raw_bottom_info_text'  => $form->parse_html_template('ic/generate_report_bottom'),
                        'output_format'         => 'HTML',
                        'title'                 => $form->{title},
diff --git a/locale/de/all b/locale/de/all
index 038ed2896..6d60fb809 100755
--- a/locale/de/all
+++ b/locale/de/all
@@ -1948,6 +1948,7 @@ $self->{texts} = {
   'Skonto Terms'                => 'Zahlungsziel Skonto',
   'So far you could use one partnumber for severel parts, for example a service and an article.' => 'Bisher war es möglich eine Artikelnummer für mehrere Artikel zu verwenden, zum Beispiel eine Artikelnummer für eine Dienstleistung, eine Ware und ein Erzeugnis.',
   'Sold'                        => 'Verkauft',
+  'Soldtotal does not make sense without any bsooqr options' => 'Option "Menge in gewählten Belegen" ohne gewählte Belege wird ignoriert.',
   'Solution'                    => 'Lösung',
   'Sort By'                     => 'Sortiert nach',
   'Source'                      => 'Beleg',
diff --git a/templates/webpages/ic/generate_report_top.html b/templates/webpages/ic/generate_report_top.html
new file mode 100644
index 000000000..3e757a59f
--- /dev/null
+++ b/templates/webpages/ic/generate_report_top.html
@@ -0,0 +1,6 @@
+[%- USE HTML %]
+[%- USE T8 %]
+[%- USE L %]
+[%- PROCESS 'common/flash.html' %]
+
+[% 'Options' | $T8 %]: [% options.join(', ') %]
-- 
2.20.1