# GNU General Public License for more details.
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
-# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+# MA 02110-1335, USA.
#======================================================================
#
# Sales report
use List::Util qw(sum first);
use SL::AM;
+use SL::DB::Employee;
use SL::VK;
use SL::IS;
use SL::ReportGenerator;
use Data::Dumper;
-require "bin/mozilla/arap.pl";
require "bin/mozilla/common.pl";
-require "bin/mozilla/drafts.pl";
require "bin/mozilla/reportgenerator.pl";
use strict;
sub search_invoice {
$main::lxdebug->enter_sub();
- $main::auth->assert('general_ledger | invoice_edit');
+ $main::auth->assert('ar_transactions | ap_transactions | invoice_edit');
my $form = $main::form;
my %myconfig = %main::myconfig;
my $locale = $main::locale;
- my ($customer, $department);
+ my ($customer);
- # setup customer selection
- $form->all_vc(\%myconfig, "customer", "AR");
+ $::request->layout->add_javascripts("autocomplete_project.js");
$form->{title} = $locale->text('Sales Report');
- $form->{jsscript} = 1;
- $form->get_lists("projects" => { "key" => "ALL_PROJECTS", "all" => 1 },
- "departments" => "ALL_DEPARTMENTS",
+ $form->get_lists("departments" => "ALL_DEPARTMENTS",
"business_types" => "ALL_BUSINESS_TYPES",
"salesmen" => "ALL_SALESMEN",
- 'employees' => 'ALL_EMPLOYEES',
- 'partsgroup' => 'ALL_PARTSGROUPS',
- "customers" => "ALL_VC");
+ 'partsgroup' => 'ALL_PARTSGROUPS');
+
+ $form->{ALL_EMPLOYEES} = SL::DB::Manager::Employee->get_all_sorted;
+
$form->{CUSTOM_VARIABLES_IC} = CVar->get_configs('module' => 'IC');
($form->{CUSTOM_VARIABLES_FILTER_CODE_IC},
$form->{CUSTOM_VARIABLES_INCLUSION_CODE_IC}) = CVar->render_search_options('variables' => $form->{CUSTOM_VARIABLES_IC},
$form->{CUSTOM_VARIABLES_INCLUSION_CODE_CT}) = CVar->render_search_options('variables' => $form->{CUSTOM_VARIABLES_CT},
'include_prefix' => 'l_',
'include_value' => 'Y');
- $form->{vc_keys} = sub { "$_[0]->{name}--$_[0]->{id}" };
- $form->{employee_labels} = sub { $_[0]->{"name"} || $_[0]->{"login"} };
- $form->{salesman_labels} = $form->{employee_labels};
-
$form->header;
print $form->parse_html_template('vk/search_invoice', { %myconfig });
sub invoice_transactions {
$main::lxdebug->enter_sub();
- $main::auth->assert('general_ledger | invoice_edit');
+ $main::auth->assert('ar_transactions | ap_transactions | invoice_edit');
my $form = $main::form;
my %myconfig = %main::myconfig;
my ($callback, $href, @columns);
# can't currently be configured from report, empty line between main sortings
- my $addemptylines = '1';
+ my $addemptylines = 1;
+
+ # don't add empty lines between mainsort subtotals when only subtotal_mainsort is selected
+ if ($form->{l_subtotal_mainsort} eq "Y" and not defined $form->{l_headers_mainsort} and not defined $form->{l_headers_subsort} and not defined $form->{l_subtotal_subsort} ) {
+ $addemptylines = 0
+ };
if ( $form->{customer} =~ /--/ ) {
# Felddaten kommen aus Dropdownbox
- ($form->{customername}, $form->{customer_id}) = split(/--/, $form->{customer});
- } elsif ($form->{customer}) {
- # es wurde ein Wert im Freitextfeld übergeben, auf Eindeutigkeit überprüfen
-
- # check_name wird mit no_select => 1 ausgeführt, ist die Abfrage nicht eindeutig kommt ein Fehler
- # und die Abfrage muß erneut ausgeführt werden
-
- # Ohne no_select kommt bei Auswahl des Kunden ein Aufruf von update der ins
- # Nichts führt, daher diese Zwischenlösung
-
- &check_name('customer', no_select => 1);
-
- # $form->{customer_id} wurde schon von check_name gesetzt
- $form->{customername} = $form->{customer};
- };
- # ist $form->{customer} leer passiert hier nichts weiter
+ my $dummy;
+ ($dummy, $form->{customer_id}) = split(/--/, $form->{customer});
+ }
+ # if $form->{customer} is empty nothing further happens here
- # decimalplaces überprüfen oder auf Default 2 setzen
+ # test for decimalplaces or set to default of 2
$form->{decimalplaces} = 2 unless $form->{decimalplaces} > 0 && $form->{decimalplaces} < 6;
my $cvar_configs_ct = CVar->get_configs('module' => 'CT');
VK->invoice_transactions(\%myconfig, \%$form);
-
+
if ( $form->{mainsort} eq 'month' or $form->{subsort} eq 'month' ) {
# Data already comes out of SELECT statement in correct month order, but
$form->{title} = $locale->text('Sales Report');
@columns =
- qw(description invnumber transdate customernumber customername partnumber partsgroup country business transdate qty parts_unit weight sellprice sellprice_total discount lastcost lastcost_total marge_total marge_percent employee salesman);
+ qw(description invnumber transdate shipvia customernumber customername partnumber partsgroup country business transdate qty parts_unit weight sellprice sellprice_total discount lastcost lastcost_total marge_total marge_percent employee salesman);
my @includeable_custom_variables = grep { $_->{includeable} } @{ $cvar_configs_ic }, @{ $cvar_configs_ct };
my @searchable_custom_variables = grep { $_->{searchable} } @{ $cvar_configs_ic }, @{ $cvar_configs_ct };
push @columns, map { "cvar_$_->{name}" } @includeable_custom_variables;
- # hidden variables für pdf/csv export übergeben
- # einmal mit l_ um zu bestimmen welche Spalten ausgegeben werden sollen
- # einmal optionen für die Überschrift (z.B. transdatefrom, partnumber, ...)
- my @hidden_variables = (qw(l_headers_mainsort l_headers_subsort l_subtotal_mainsort l_subtotal_subsort l_total l_parts l_customername l_customernumber transdatefrom transdateto decimalplaces customer customername customer_id department partnumber partsgroup country business description project_id customernumber salesman employee salesman_id employee_id business_id partsgroup_id mainsort subsort),
- "$form->{db}number",
+ # pass hidden variables for pdf/csv export
+ # first with l_ to determine which columns to show
+ # then with the options for headings (such as transdatefrom, partnumber, ...)
+ my @hidden_variables = (qw(l_headers_mainsort l_headers_subsort l_subtotal_mainsort l_subtotal_subsort l_total l_parts l_customername l_customernumber transdatefrom transdateto decimalplaces customer customer_id department_id partnumber partsgroup country business description project_id customernumber salesman employee salesman_id employee_id business_id partsgroup_id mainsort subsort),
+ "$form->{db}number",
map({ "cvar_$_->{name}" } @searchable_custom_variables),
map { "l_$_" } @columns
);
my @hidden_nondefault = grep({ $form->{$_} } @hidden_variables);
- # Variablen werden dann als Hidden Variable mitgegeben, z.B.
+ # variables are passed as hiddens, e.g.
# <input type="hidden" name="report_generator_hidden_transdateto" value="21.05.2010">
$href = build_std_url('action=invoice_transactions', grep { $form->{$_} } @hidden_variables);
my %column_defs = (
'description' => { 'text' => $locale->text('Description'), },
'partnumber' => { 'text' => $locale->text('Part Number'), },
- 'partsgroup' => { 'text' => $locale->text('Group'), },
+ 'partsgroup' => { 'text' => $locale->text('Partsgroup'), },
'country' => { 'text' => $locale->text('Country'), },
'business' => { 'text' => $locale->text('Customer type'), },
'employee' => { 'text' => $locale->text('Employee'), },
'salesman' => { 'text' => $locale->text('Salesperson'), },
'invnumber' => { 'text' => $locale->text('Invoice Number'), },
'transdate' => { 'text' => $locale->text('Invoice Date'), },
+ 'shipvia' => { 'text' => $locale->text('Ship via'), },
'qty' => { 'text' => $locale->text('Quantity'), },
'parts_unit' => { 'text' => $locale->text('Base unit'), },
'weight' => { 'text' => $locale->text('Weight'), },
my %column_alignment = map { $_ => 'right' } qw(lastcost sellprice sellprice_total lastcost_total parts_unit discount marge_total marge_percent qty weight);
-
+
# so now the check-box "Description" is only used as switch for part description in invoice-mode
# always fill the column "Description" if we are in Zwischensummenmode
- if (not defined $form->{"l_parts"}) {
- $form->{"l_description"} = "Y";
- };
+ $form->{"l_description"} = "Y" if not defined $form->{"l_parts"};;
map { $column_defs{$_}->{visible} = $form->{"l_${_}"} ? 1 : 0 } @columns;
my @options;
- if ($form->{description}) {
- push @options, $locale->text('Description') . " : $form->{description}";
- }
- if ($form->{customer}) {
- push @options, $locale->text('Customer') . " : $form->{customername}";
- }
- if ($form->{customernumber}) {
- push @options, $locale->text('Customer Number') . " : $form->{customernumber}";
- }
-# TODO: es wird nur id übergeben
- if ($form->{department}) {
- my ($department) = split /--/, $form->{department};
- push @options, $locale->text('Department') . " : $department";
- }
- if ($form->{invnumber}) {
- push @options, $locale->text('Invoice Number') . " : $form->{invnumber}";
- }
- if ($form->{invdate}) {
- push @options, $locale->text('Invoice Date') . " : $form->{invdate}";
- }
- if ($form->{partnumber}) {
- push @options, $locale->text('Part Number') . " : $form->{partnumber}";
- }
- if ($form->{partsgroup_id}) {
- my $partsgroup = SL::DB::PartsGroup->new(id => $form->{partsgroup_id})->load;
- push @options, $locale->text('Group') . " : $partsgroup->{partsgroup}";
- }
- if ($form->{country}) {
- push @options, $locale->text('Country') . " : $form->{country}";
- }
- if ($form->{employee_id}) {
- my $employee = SL::DB::Employee->new(id => $form->{employee_id})->load;
- push @options, $locale->text('Employee') . ' : ' . $employee->name;
- }
- if ($form->{salesman_id}) {
- my $salesman = SL::DB::Employee->new(id => $form->{salesman_id})->load;
- push @options, $locale->text('Salesman') . ' : ' . $salesman->name;
- }
- if ($form->{business_id}) {
- my $business = SL::DB::Business->new(id => $form->{business_id})->load;
- push @options, $locale->text('Customer type') . ' : ' . $business->description;
- }
- if ($form->{ordnumber}) {
- push @options, $locale->text('Order Number') . " : $form->{ordnumber}";
- }
- if ($form->{notes}) {
- push @options, $locale->text('Notes') . " : $form->{notes}";
- }
- if ($form->{transaction_description}) {
- push @options, $locale->text('Transaction description') . " : $form->{transaction_description}";
- }
- if ($form->{transdatefrom}) {
- push @options, $locale->text('From') . " " . $locale->date(\%myconfig, $form->{transdatefrom}, 1);
- }
- if ($form->{transdateto}) {
- push @options, $locale->text('Bis') . " " . $locale->date(\%myconfig, $form->{transdateto}, 1);
- }
+ push @options, $locale->text('Description') . " : $form->{description}" if $form->{description};
+ push @options, $locale->text('Customer') . " : $form->{customer}" if $form->{customer};
+ push @options, $locale->text('Customer Number') . " : $form->{customernumber}" if $form->{customernumber};
+ # TODO: only customer id is passed
+ push @options, $locale->text('Department') . " : " . SL::DB::Department->new(id => $form->{department_id})->load->description if $form->{department_id};
+ push @options, $locale->text('Invoice Number') . " : $form->{invnumber}" if $form->{invnumber};
+ push @options, $locale->text('Invoice Date') . " : $form->{invdate}" if $form->{invdate};
+ push @options, $locale->text('Ship via') . " : $form->{shipvia}" if $form->{shipvia};
+ push @options, $locale->text('Part Number') . " : $form->{partnumber}" if $form->{partnumber};
+ push @options, $locale->text('Partsgroup') . " : " . SL::DB::PartsGroup->new(id => $form->{partsgroup_id})->load->partsgroup if $form->{partsgroup_id};
+ push @options, $locale->text('Country') . " : $form->{country}" if $form->{country};
+ push @options, $locale->text('Employee') . ' : ' . SL::DB::Employee->new(id => $form->{employee_id})->load->name if $form->{employee_id};
+ push @options, $locale->text('Salesman') . ' : ' . SL::DB::Employee->new(id => $form->{salesman_id})->load->name if $form->{salesman_id};
+ push @options, $locale->text('Customer type') . ' : ' . SL::DB::Business->new(id => $form->{business_id})->load->description if $form->{business_id};
+ push @options, $locale->text('Order Number') . " : $form->{ordnumber}" if $form->{ordnumber};
+ push @options, $locale->text('Notes') . " : $form->{notes}" if $form->{notes};
+ push @options, $locale->text('Transaction description') . " : $form->{transaction_description}" if $form->{transaction_description};
+ push @options, $locale->text('From') . " " . $locale->date(\%myconfig, $form->{transdatefrom}, 1) if $form->{transdatefrom};
+ push @options, $locale->text('Bis') . " " . $locale->date(\%myconfig, $form->{transdateto}, 1) if $form->{transdateto};
my $report = SL::ReportGenerator->new(\%myconfig, $form);
'data' => $form->{AR}
);
+ my $num_visible_columns = scalar $report->get_visible_columns;
+ my %empty_row = (
+ description => {
+ data => '',
+ class => 'listrowempty',
+ colspan => $num_visible_columns,
+ },
+ );
+
# add sort and escape callback, this one we use for the add sub
$form->{callback} = $href .= "&sort=$form->{mainsort}";
$callback = $form->escape($href);
my @subtotal_columns = qw(qty weight sellprice sellprice_total lastcost lastcost_total marge_total marge_percent discount);
- # Gesamtsumme:
- # Summe von sellprice_total, lastcost_total und marge_total
- # Durchschnitt von marge_percent
+ # Total sum:
+ # sum of sellprice_total, lastcost_total and marge_total
+ # average of marge_percent
my @total_columns = qw(sellprice_total lastcost_total marge_total marge_percent );
- my %totals = map { $_ => 0 } @total_columns;
+ my %totals = map { $_ => 0 } @total_columns;
my %subtotals1 = map { $_ => 0 } @subtotal_columns;
my %subtotals2 = map { $_ => 0 } @subtotal_columns;
$basefactor = 1 unless $basefactor;
$ar->{price_factor} = 1 unless $ar->{price_factor};
- # calculate individual sellprice
- # discount was already accounted for in db sellprice
- $ar->{sellprice} = $ar->{sellprice} / $ar->{price_factor} / $basefactor;
- $ar->{lastcost} = $ar->{lastcost} / $ar->{price_factor};
- $ar->{sellprice_total} = $ar->{qty} * ( $ar->{fxsellprice} * ( 1 - $ar->{discount} ) ) / $ar->{price_factor};
- $ar->{lastcost_total} = $ar->{qty} * $ar->{lastcost} * $basefactor;
- # marge_percent wird neu berechnet, da Wert in invoice leer ist (Bug)
+
+ # calculate individual sellprice, discount is already accounted for in column sellprice in db
+
+ # The sellprice total can be calculated from sellprice or fxsellprice (the
+ # value that was actually entered in the sellprice field and is always
+ # stored separately). However, for fxsellprice this method only works when
+ # the tax is not included, because otherwise fxsellprice includes the tax
+ # and there is no simple way to extract the tax rate of the article from
+ # the big query.
+ #
+ # Using fxsellprice is potentially more accurate (certainly for tax
+ # included), because we can use the same method as is used while the
+ # invoice is generated.
+ #
+ # sellprice however has already been converted to the net value (but
+ # rounded in the process, which leads to rounding errors when calculating
+ # the linetotal from the rounded sellprice in the report. These rounding
+ # errors can quickly amount to several cents when qty is large)
+ #
+ # For calculating sellprice_total from fxsellprice, you would use:
+ # sellprice_total_including_tax = qty * fxsellprice * (1-discount) / price_factor * exchangerate
+ # $ar->{sellprice_total_including_tax} = $form->round_amount( $ar->{qty} * ( $ar->{fxsellprice} * ( 1 - $ar->{discount} ) ) / $ar->{price_factor}, $form->{"decimalplaces"});
+
+ my $sellprice_orig = $ar->{sellprice};
+ my $qty_orig = $ar->{qty};
+ # adjust sellprice so it reflects the unit sellprice according to price_factor and basefactor
+ $ar->{sellprice} = $ar->{sellprice} / $ar->{price_factor} / $basefactor;
+ # for sellprice_total use the original amounts
+ $ar->{sellprice_total} = $form->round_amount( $qty_orig * $sellprice_orig / $ar->{price_factor}, $form->{"decimalplaces"});
+
+ my $lastcost_orig = $ar->{lastcost};
+ $ar->{lastcost} = $ar->{lastcost} / $ar->{price_factor} / $basefactor;
+ $ar->{lastcost_total} = $form->round_amount( $qty_orig * $lastcost_orig / $ar->{price_factor}, $form->{"decimalplaces"});
+
+ # marge_percent is recalculated, because the value in invoice used to be empty
$ar->{marge_percent} = $ar->{sellprice_total} ? (($ar->{sellprice_total}-$ar->{lastcost_total}) / $ar->{sellprice_total} * 100) : 0;
- # marge_total neu berechnen
+ # also recalculate marge_total
$ar->{marge_total} = $ar->{sellprice_total} ? $ar->{sellprice_total}-$ar->{lastcost_total} : 0;
- $ar->{discount} *= 100; # für Ausgabe formatieren, 10% stored as 0.1 in db
+ $ar->{discount} *= 100; # format discount value for output, 10% is stored as 0.1 in db
+
+ # adjust qty to reflect the qty of the base unit
+ $ar->{qty} *= $basefactor;
- # Anfangshauptüberschrift
- if ( $form->{l_headers_mainsort} eq "Y" && ( $idx == 0 or $ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'mainsort'} } )) {
- my $headerrow;
+ # now $ar->{sellprice} * $ar->{qty} should be the same as $qty_orig * $ar->{qty}
- # use $emptyname for mainsort header if mainsort is empty
- if ( $ar->{$form->{'mainsort'}} ) {
- $headerrow->{description}->{data} = $ar->{$form->{'mainsort'}};
- } else {
- $headerrow->{description}->{data} = $locale->text('empty');
- };
+ # weight is the still the weight per part, but here we want the total weight
+ $ar->{weight} *= $ar->{qty};
- $headerrow->{description}->{class} = "listmainsortheader";
- my $headerrow_set = [ $headerrow ];
- $report->add_data($headerrow_set);
+ # Main header
+ if ( $form->{l_headers_mainsort} eq "Y" && ( $idx == 1 or $ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'mainsort'} } )) {
+ my $headerrow = {
+ # use $emptyname for mainsort header if mainsort is empty
+ data => $ar->{$form->{'mainsort'}} || $locale->text('empty'),
+ class => "listmainsortheader",
+ colspan => $num_visible_columns,
+ };
+ $report->add_data([ { description => $headerrow } ]);
# add empty row after main header
# my $emptyheaderrow->{description}->{data} = "";
# $report->add_data($emptyheaderrow_set) if $form->{l_headers} eq "Y";
};
- # subsort überschriften
- if ( $idx == 0
- or $ar->{ $form->{'subsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'subsort'} }
- or $ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'mainsort'} }
+ # subsort headers
+ # special case: subsort headers only makes (aesthetical) sense if we show individual parts
+ if (( $idx == 0
+ or $ar->{ $form->{'subsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'subsort'} }
+ or $ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx - 1]->{ $form->{'mainsort'} })
+ && ($form->{l_headers_subsort} eq "Y")
+ && $form->{l_parts}
) {
- my $headerrow;
-
- # if subsort name is defined, use that name in header, otherwise use $emptyname
- if ( $ar->{$form->{'subsort'}} ) {
- $headerrow->{description}->{data} = $ar->{$form->{'subsort'}};
- } else {
- $headerrow->{description}->{data} = $locale->text('empty');
+ my $headerrow = {
+ # if subsort name is defined, use that name in header, otherwise use $emptyname
+ data => $ar->{$form->{'subsort'}} || $locale->text('empty'),
+ class => "listsubsortheader",
+ colspan => $num_visible_columns,
};
- $headerrow->{description}->{class} = "listsubsortheader";
- my $headerrow_set = [ $headerrow ];
- # special case: subsort headers only makes (aesthetical) sense if we show individual parts
- $report->add_data($headerrow_set) if $form->{l_headers_subsort} eq "Y" and $form->{l_parts};
+ $report->add_data([ { description => $headerrow } ]);
};
map { $subtotals1{$_} += $ar->{$_};
# calculate averages for subtotals1 and subtotals2
# credited positions reduce both total and qty and thus don't influence average prices
$subtotals1{sellprice} = $subtotals1{sellprice_total} / $subtotals1{qty};
- $subtotals1{lastcost} = $subtotals1{lastcost_total} / $subtotals1{qty};
+ $subtotals1{lastcost} = $subtotals1{lastcost_total} / $subtotals1{qty};
} else {
# qty is zero, so we have a special case where each position in subtotal
# group has a corresponding credit note so that the total qty is zero in
# rather than leaving the last value in sellprice/lastcost
$subtotals1{sellprice} = 0;
- $subtotals1{lastcost} = 0;
+ $subtotals1{lastcost} = 0;
};
if ( $subtotals2{qty} != 0 ) {
$subtotals2{sellprice} = $subtotals2{sellprice_total} / $subtotals2{qty};
- $subtotals2{lastcost} = $subtotals2{lastcost_total} / $subtotals2{qty};
+ $subtotals2{lastcost} = $subtotals2{lastcost_total} / $subtotals2{qty};
} else {
$subtotals2{sellprice} = 0;
- $subtotals2{lastcost} = 0;
+ $subtotals2{lastcost} = 0;
};
- # Ertrag prozentual in den Summen: (summe VK - summe Ertrag) / summe VK
+ # marge percent as sums: (sum VK - sum Ertrag) / sum VK
$subtotals1{marge_percent} = $subtotals1{sellprice_total} ? (($subtotals1{sellprice_total} - $subtotals1{lastcost_total}) / $subtotals1{sellprice_total}) * 100 : 0;
$subtotals2{marge_percent} = $subtotals2{sellprice_total} ? (($subtotals2{sellprice_total} - $subtotals2{lastcost_total}) / $subtotals2{sellprice_total}) *100 : 0;
- # Ertrag prozentual: (Summe VK betrag - Summe EK betrag) / Summe VK betrag
- # wird laufend bei jeder Position neu berechnet
+ # total marge percent: (sum VK betrag - sum EK betrag) / sum VK betrag
+ # is recalculated after each position
$totals{marge_percent} = $totals{sellprice_total} ? ( ($totals{sellprice_total} - $totals{lastcost_total}) / $totals{sellprice_total} ) * 100 : 0;
- #passt die qty an die gewählte Einheit an
- #qty wurde bisher noch für andere Berechnungen benötigt und daher erst am Schluss überschrieben
- $ar->{qty} *= $basefactor;
-
- #weight is the still the weight per part, but here we want the total weight
- $ar->{weight} *= $ar->{qty};
-
- map { $ar->{$_} = $form->format_amount(\%myconfig, $ar->{$_}, 2) } qw(marge_total marge_percent);
- map { $ar->{$_} = $form->format_amount(\%myconfig, $ar->{$_}, $form->{"decimalplaces"} )} qw(lastcost sellprice sellprice_total lastcost_total);
+ map { $ar->{$_} = $form->format_amount(\%myconfig, $ar->{$_}, 2) } qw(marge_percent qty);
+ map { $ar->{$_} = $form->format_amount(\%myconfig, $ar->{$_}, 3) } qw(weight);
+ map { $ar->{$_} = $form->format_amount(\%myconfig, $ar->{$_}, $form->{"decimalplaces"} )} qw(lastcost sellprice sellprice_total lastcost_total marge_total);
- my $row = { };
+ # only show individual lines when l_parts is set, this is useful, if you
+ # only want to see subtotals and totals
- foreach my $column (@columns) {
- $row->{$column} = {
- 'data' => $ar->{$column},
- 'align' => $column_alignment{$column},
- };
- }
+ if ($form->{l_parts}) {
+ my %row = (
+ map { ($_ => { data => $ar->{$_}, align => $column_alignment{$_} }) } @columns
+ );
- $row->{description}->{class} = 'listsortdescription';
+ $row{invnumber}->{link} = build_std_url("script=is.pl", 'action=edit') . "&id=" . E($ar->{id}) . "&callback=${callback}";
- $row->{invnumber}->{link} = build_std_url("script=is.pl", 'action=edit')
- . "&id=" . E($ar->{id}) . "&callback=${callback}";
+ # use partdescription according to invoice in article mode
+ $row{description}->{data} = $ar->{invoice_description};
- # Einzelzeilen nur zeigen wenn l_parts gesetzt ist, nützlich, wenn man nur
- # Subtotals und Totals sehen möchte
- my $row_set = $form->{l_parts} ? [ $row ] : [ ];
+ $report->add_data(\%row);
+ }
- # hier wird bei l_subtotal nicht differenziert zwischen mainsort und subsort
- # macht man l_subtotal_mainsort aus wird l_subtotal_subsort auch nicht ausgeführt
- if (($form->{l_subtotal_mainsort} eq 'Y')
+ # choosing l_subtotal doesn't make a distinction between mainsort and subsort
+ # if l_subtotal_mainsort is not selected l_subtotal_subsort isn't run either
+ if ( ($form->{l_subtotal_mainsort} eq 'Y')
+ && ($form->{l_subtotal_subsort} eq 'Y')
&& (($idx == (scalar @{ $form->{AR} } - 1)) # last element always has a subtotal
|| ($ar->{ $form->{'subsort'} } ne $form->{AR}->[$idx + 1]->{ $form->{'subsort'} })
|| ($ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx + 1]->{ $form->{'mainsort'} })
)) { # if value that is sorted by changes, print subtotal
- if ($form->{l_subtotal_subsort} eq 'Y') {
- push @{ $row_set }, create_subtotal_row_invoice(\%subtotals2, \@columns, \%column_alignment, \@subtotal_columns, 'listsubsortsubtotal', $ar->{ $form->{'subsort'} }) ;
- push @{ $row_set }, insert_empty_row() if $form->{l_parts} and $addemptylines;
- };
+ $report->add_data(create_subtotal_row_invoice(\%subtotals2, \@columns, \%column_alignment, \@subtotal_columns, $form->{l_parts} ? 'listsubtotal' : undef, $ar->{ $form->{'subsort'} }));
+ $report->add_data({ %empty_row }) if $form->{l_parts} and $addemptylines;
}
# if last mainsort is reached or mainsort has changed, add mainsort subtotal and empty row
- if (($form->{l_subtotal_mainsort} eq 'Y')
+ if ( ($form->{l_subtotal_mainsort} eq 'Y')
+ && ($form->{l_subtotal_mainsort} eq 'Y')
+ && ($form->{mainsort} ne $form->{subsort})
&& (($idx == (scalar @{ $form->{AR} } - 1)) # last element always has a subtotal
|| ($ar->{ $form->{'mainsort'} } ne $form->{AR}->[$idx + 1]->{ $form->{'mainsort'} })
)) { # if value that is sorted by changes, print subtotal
- if ($form->{l_subtotal_mainsort} eq 'Y' and $form->{mainsort} ne $form->{subsort} ) {
- # subtotal is overriden if mainsort and subsort are equal, don't print
+ # subtotal is overridden if mainsort and subsort are equal, don't print
# subtotal line even if it is selected
- push @{ $row_set }, create_subtotal_row_invoice(\%subtotals1, \@columns, \%column_alignment, \@subtotal_columns, 'listmainsortsubtotal', $ar->{$form->{mainsort}});
- push @{ $row_set }, insert_empty_row() if $addemptylines; # insert empty row after mainsort
- };
+ $report->add_data(create_subtotal_row_invoice(\%subtotals1, \@columns, \%column_alignment, \@subtotal_columns, 'listsubtotal', $ar->{$form->{mainsort}}));
+ $report->add_data({ %empty_row }) if $addemptylines; # insert empty row after mainsort
}
- $report->add_data($row_set);
-
$idx++;
}
if ( $form->{l_total} eq "Y" ) {
$main::lxdebug->leave_sub();
}
-
-sub insert_empty_row {
- my $dummyrow;
- $dummyrow->{description}->{data} = "";
- my $dummyrowset = [ $dummyrow ];
- return $dummyrow;
-};
-
-
-
sub create_subtotal_row_invoice {
$main::lxdebug->enter_sub();
my %myconfig = %main::myconfig;
my $locale = $main::locale;
- my $row = { map { $_ => { 'data' => '', 'class' => $class, 'align' => $column_alignment->{$_}, } } @{ $columns } };
+ my $row = { map { $_ => { data => '', class => $class, align => $column_alignment->{$_}, } } @{ $columns } };
# set name as "empty" if no value is given, except if we are dealing with the
# absolute total, then just write "Total sum"
$row->{description}->{data} = $locale->text('Total') . ' ' . $name;
};
- map { $row->{$_}->{data} = $form->format_amount(\%myconfig, $totals->{$_}, 2) } qw(marge_total marge_percent weight);
- map { $row->{$_}->{data} = $form->format_amount(\%myconfig, $totals->{$_}, 0) } qw(qty);
+ map { $row->{$_}->{data} = $form->format_amount(\%myconfig, $totals->{$_}, 2) } qw(marge_total marge_percent qty);
+ map { $row->{$_}->{data} = $form->format_amount(\%myconfig, $totals->{$_}, 3) } qw(weight);
map { $row->{$_}->{data} = $form->format_amount(\%myconfig, $totals->{$_}, $form->{decimalplaces}) } qw(lastcost sellprice sellprice_total lastcost_total);
}
1;
-