From 532b0d4cc8ceb1af4489c0233d7bbad10739b5d6 Mon Sep 17 00:00:00 2001 From: Moritz Bunkus Date: Fri, 4 May 2007 12:50:13 +0000 Subject: [PATCH] Alle Queries zur Vermeidung von SQL injections auf die Verwendung von Parametern bzw. ordentliches Quoten umgestellt. --- SL/IS.pm | 1551 ++++++++++++++++++++++++++---------------------------- 1 file changed, 749 insertions(+), 802 deletions(-) diff --git a/SL/IS.pm b/SL/IS.pm index 402df0d28..205cbcbaf 100644 --- a/SL/IS.pm +++ b/SL/IS.pm @@ -44,19 +44,14 @@ sub invoice_details { my ($self, $myconfig, $form, $locale) = @_; - $form->{duedate} = $form->{invdate} unless ($form->{duedate}); + $form->{duedate} ||= $form->{invdate}; # connect to database my $dbh = $form->dbconnect($myconfig); + my $sth; - my $query = qq|SELECT date '$form->{duedate}' - date '$form->{invdate}' - AS terms - FROM defaults|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($form->{terms}) = $sth->fetchrow_array; - $sth->finish; + my $query = qq|SELECT date | . conv_dateq($form->{duedate}) . qq| - date | . conv_dateq($form->{invdate}) . qq| AS terms|; + ($form->{terms}) = selectrow_query($form, $dbh, $query); my (@project_ids, %projectnumbers); @@ -178,16 +173,11 @@ sub invoice_details { if ($form->{lizenzen}) { if ($form->{"licensenumber_$i"}) { - $query = - qq|SELECT l.licensenumber, l.validuntil FROM license l WHERE l.id = $form->{"licensenumber_$i"}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($licensenumber, $validuntil) = $sth->fetchrow_array; + $query = qq|SELECT licensenumber, validuntil FROM license WHERE id = ?|; + ($licensenumber, $validuntil) = selectrow_query($form, $dbh, $query, conv_i($form->{"licensenumber_$i"})); push(@{ $form->{licensenumber} }, $licensenumber); - push(@{ $form->{validuntil} }, - $locale->date($myconfig, $validuntil, 0)); - $sth->finish; + push(@{ $form->{validuntil} }, $locale->date($myconfig, $validuntil, 0)); + } else { push(@{ $form->{licensenumber} }, ""); push(@{ $form->{validuntil} }, ""); @@ -269,7 +259,7 @@ sub invoice_details { push(@{ $form->{projectnumber} }, $projectnumbers{$form->{"project_id_$i"}}); - @taxaccounts = split / /, $form->{"taxaccounts_$i"}; + @taxaccounts = split(/ /, $form->{"taxaccounts_$i"}); $taxrate = 0; $taxdiff = 0; @@ -326,16 +316,13 @@ sub invoice_details { $sortorder = qq|ORDER BY a.$oid{$myconfig->{dbdriver}}|; } - $query = qq|SELECT p.partnumber, p.description, p.unit, a.qty, - pg.partsgroup - FROM assembly a - JOIN parts p ON (a.parts_id = p.id) - LEFT JOIN partsgroup pg ON (p.partsgroup_id = pg.id) - WHERE a.bom = '1' - AND a.id = '$form->{"id_$i"}' - $sortorder|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = + qq|SELECT p.partnumber, p.description, p.unit, a.qty, pg.partsgroup + FROM assembly a + JOIN parts p ON (a.parts_id = p.id) + LEFT JOIN partsgroup pg ON (p.partsgroup_id = pg.id) + WHERE (a.bom = '1') AND (a.id = ?) $sortorder|; + $sth = prepare_execute_query($form, $dbh, $query, conv_i($form->{"id_$i"})); while (my $ref = $sth->fetchrow_hashref(NAME_lc)) { if ($form->{groupitems} && $ref->{partsgroup} ne $sameitem) { @@ -374,7 +361,7 @@ sub invoice_details { for my $i (1 .. $form->{paidaccounts}) { if ($form->{"paid_$i"}) { push(@{ $form->{payment} }, $form->{"paid_$i"}); - my ($accno, $description) = split /--/, $form->{"AR_paid_$i"}; + my ($accno, $description) = split(/--/, $form->{"AR_paid_$i"}); push(@{ $form->{paymentaccount} }, $description); push(@{ $form->{paymentdate} }, $form->{"datepaid_$i"}); push(@{ $form->{paymentsource} }, $form->{"source_$i"}); @@ -415,15 +402,8 @@ sub project_description { my ($self, $dbh, $id) = @_; - my $query = qq|SELECT p.description - FROM project p - WHERE p.id = $id|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($_) = $sth->fetchrow_array; - - $sth->finish; + my $query = qq|SELECT description FROM project WHERE id = ?|; + my ($description) = selectrow_query($form, $dbh, $query, conv_i($id)); $main::lxdebug->leave_sub(); @@ -441,20 +421,25 @@ sub customer_details { # get contact id, set it if nessessary $form->{cp_id} *= 1; - $contact = ""; + my @values; + + my $where = ""; if ($form->{cp_id}) { - $contact = "and cp.cp_id = $form->{cp_id}"; + $where = qq| AND (cp.cp_id = ?) |; + push(@values, conv_i($form->{cp_id})); } # get rest for the customer - my $query = qq|SELECT ct.*, cp.*, ct.notes as customernotes, ct.phone AS customerphone, ct.fax AS customerfax, ct.email AS customeremail - FROM customer ct - LEFT JOIN contacts cp on ct.id = cp.cp_cv_id - WHERE ct.id = $form->{customer_id} $contact order by cp.cp_id limit 1|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - $ref = $sth->fetchrow_hashref(NAME_lc); + my $query = + qq|SELECT ct.*, cp.*, ct.notes as customernotes, + ct.phone AS customerphone, ct.fax AS customerfax, ct.email AS customeremail + FROM customer ct + LEFT JOIN contacts cp on ct.id = cp.cp_cv_id + WHERE (ct.id = ?) $where + ORDER BY cp.cp_id + LIMIT 1|; + push(@values, conv_i($form->{customer_id})); + my $ref = selectfirst_hashref_query($form, $dbh, $query, @values); # remove id and taxincluded before copy back delete @$ref{qw(id taxincluded)}; @@ -467,31 +452,26 @@ sub customer_details { } map { $form->{$_} = $ref->{$_} } keys %$ref; - $sth->finish; if ($form->{delivery_customer_id}) { - my $query = qq|SELECT ct.*, ct.notes as customernotes - FROM customer ct - WHERE ct.id = $form->{delivery_customer_id} limit 1|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - $ref = $sth->fetchrow_hashref(NAME_lc); + $query = + qq|SELECT *, notes as customernotes + FROM customer + WHERE id = ? + LIMIT 1|; + $ref = selectfirst_hashref_query($form, $dbh, $query, conv_i($form->{delivery_customer_id})); - $sth->finish; map { $form->{"dc_$_"} = $ref->{$_} } keys %$ref; } if ($form->{delivery_vendor_id}) { - my $query = qq|SELECT ct.*, ct.notes as customernotes - FROM customer ct - WHERE ct.id = $form->{delivery_vendor_id} limit 1|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - $ref = $sth->fetchrow_hashref(NAME_lc); + $query = + qq|SELECT *, notes as customernotes + FROM customer + WHERE id = ? + LIMIT 1|; + $ref = selectfirst_hashref_query($form, $dbh, $query, conv_i($form->{delivery_vendor_id})); - $sth->finish; map { $form->{"dv_$_"} = $ref->{$_} } keys %$ref; } $dbh->disconnect; @@ -507,53 +487,28 @@ sub post_invoice { # connect to database, turn off autocommit my $dbh = $form->dbconnect_noauto($myconfig); - my ($query, $sth, $null, $project_id, $deliverydate); + my ($query, $sth, $null, $project_id, $deliverydate, @values); my $exchangerate = 0; - ($null, $form->{employee_id}) = split /--/, $form->{employee}; + ($null, $form->{employee_id}) = split(/--/, $form->{employee}); unless ($form->{employee_id}) { $form->get_employee($dbh); } - $form->{payment_id} *= 1; - $form->{language_id} *= 1; - $form->{taxzone_id} *= 1; - $form->{delivery_customer_id} *= 1; - $form->{delivery_vendor_id} *= 1; - $form->{storno} *= 1; - $form->{shipto_id} *= 1; - - ($null, $form->{department_id}) = split(/--/, $form->{department}); - $form->{department_id} *= 1; - - my $service_units = AM->retrieve_units($myconfig,$form,"service"); - my $part_units = AM->retrieve_units($myconfig,$form,"dimension"); - + my $all_units = AM->retrieve_units($myconfig, $form); if ($form->{id}) { &reverse_invoice($dbh, $form); } else { - my $uid = rand() . time; + $query = qq|SELECT nextval('glid')|; + ($form->{"id"}) = selectrow_query($form, $dbh, $query); - $uid .= $form->{login}; - - $uid = substr($uid, 2, 75); - - $query = qq|INSERT INTO ar (invnumber, employee_id) - VALUES ('$uid', $form->{employee_id})|; - $dbh->do($query) || $form->dberror($query); - - $query = qq|SELECT a.id FROM ar a - WHERE a.invnumber = '$uid'|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($form->{id}) = $sth->fetchrow_array; - $sth->finish; + $query = qq|INSERT INTO ar (id, invnumber) VALUES (?, ?)|; + do_query($form, $dbh, $query, $form->{"id"}, $form->{"id"}); if (!$form->{invnumber}) { $form->{invnumber} = @@ -562,9 +517,6 @@ sub post_invoice { } } - map { $form->{$_} =~ s/\'/\'\'/g } - (qw(invnumber shippingpoint shipvia notes intnotes message)); - my ($netamount, $invoicediff) = (0, 0); my ($amount, $linetotal, $lastincomeaccno); @@ -583,10 +535,12 @@ sub post_invoice { $form->{expense_inventory} = ""; + my %baseunits; + foreach my $i (1 .. $form->{rowcount}) { if ($form->{type} eq "credit_note") { $form->{"qty_$i"} = $form->parse_amount($myconfig, $form->{"qty_$i"}) * -1; - $form->{shipped} = 1; + $form->{shipped} = 1; } else { $form->{"qty_$i"} = $form->parse_amount($myconfig, $form->{"qty_$i"}); } @@ -598,35 +552,25 @@ sub post_invoice { } if ($form->{"id_$i"}) { + my $item_unit; - # get item baseunit - $query = qq|SELECT p.unit - FROM parts p - WHERE p.id = $form->{"id_$i"}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - my ($item_unit) = $sth->fetchrow_array(); - $sth->finish; - - if ($form->{"inventory_accno_$i"}) { - if (defined($part_units->{$item_unit}->{factor}) && $part_units->{$item_unit}->{factor} ne '' && $part_units->{$item_unit}->{factor} ne '0') { - $basefactor = $part_units->{$form->{"unit_$i"}}->{factor} / $part_units->{$item_unit}->{factor}; - } else { - $basefactor = 1; - } - $baseqty = $form->{"qty_$i"} * $basefactor; + if (defined($baseunits{$form->{"id_$i"}})) { + $item_unit = $baseunits{$form->{"id_$i"}}; } else { - if (defined($service_units->{$item_unit}->{factor}) && $service_units->{$item_unit}->{factor} ne '' && $service_units->{$item_unit}->{factor} ne '0') { - $basefactor = $service_units->{$form->{"unit_$i"}}->{factor} / $service_units->{$item_unit}->{factor}; - } else { - $basefactor = 1; - } - $baseqty = $form->{"qty_$i"} * $basefactor; + # get item baseunit + $query = qq|SELECT unit FROM parts WHERE id = ?|; + ($item_unit) = selectrow_query($form, $dbh, $query, conv_i($form->{"id_$i"})); + $baseunits{$form->{"id_$i"}} = $item_unit; } - map { $form->{"${_}_$i"} =~ s/\'/\'\'/g } - (qw(partnumber description unit)); + if (defined($all_units->{$item_unit}->{factor}) + && ($all_units->{$item_unit}->{factor} ne '') + && ($all_units->{$item_unit}->{factor} != 0)) { + $basefactor = $all_units->{$form->{"unit_$i"}}->{factor} / $all_units->{$item_unit}->{factor}; + } else { + $basefactor = 1; + } + $baseqty = $form->{"qty_$i"} * $basefactor; # undo discount formatting $form->{"discount_$i"} = @@ -650,8 +594,8 @@ sub post_invoice { $form->{"sellprice_$i"} = $fxsellprice - $discount; # add tax rates - map { $taxrate += $form->{"${_}_rate"} } split / /, - $form->{"taxaccounts_$i"}; + map({ $taxrate += $form->{"${_}_rate"} } split(/ /, + $form->{"taxaccounts_$i"})); # round linetotal to 2 decimal places $linetotal = @@ -671,7 +615,7 @@ sub post_invoice { map { $form->{amount}{ $form->{id} }{$_} += $taxamount * $form->{"${_}_rate"} / $taxrate - } split / /, $form->{"taxaccounts_$i"}; + } split(/ /, $form->{"taxaccounts_$i"}); } # add amount to income, $form->{amount}{trans_id}{accno} @@ -703,12 +647,12 @@ sub post_invoice { if ($form->{"assembly_$i"}) { # do not update if assembly consists of all services - $query = qq|SELECT sum(p.inventory_accno_id) - FROM parts p - JOIN assembly a ON (a.parts_id = p.id) - WHERE a.id = $form->{"id_$i"}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = + qq|SELECT sum(p.inventory_accno_id) + FROM parts p + JOIN assembly a ON (a.parts_id = p.id) + WHERE a.id = ?|; + $sth = prepare_execute_query($form, $dbh, $query, conv_i($form->{"id_$i"})); if ($sth->fetchrow_array) { $form->update_balance($dbh, "parts", "onhand", qq|id = ?|, @@ -728,46 +672,36 @@ sub post_invoice { } } - $deliverydate = - ($form->{"deliverydate_$i"}) - ? qq|'$form->{"deliverydate_$i"}'| - : "NULL"; - # get pricegroup_id and save it - ($null, my $pricegroup_id) = split /--/, $form->{"sellprice_pg_$i"}; + ($null, my $pricegroup_id) = split(/--/, $form->{"sellprice_pg_$i"}); $pricegroup_id *= 1; my $subtotal = $form->{"subtotal_$i"} * 1; # save detail record in invoice table - $query = qq|INSERT INTO invoice (trans_id, parts_id, description,longdescription, qty, - sellprice, fxsellprice, discount, allocated, assemblyitem, - unit, deliverydate, project_id, serialnumber, pricegroup_id, - ordnumber, transdate, cusordnumber, base_qty, subtotal) - VALUES ($form->{id}, $form->{"id_$i"}, - '$form->{"description_$i"}', '$form->{"longdescription_$i"}', $form->{"qty_$i"}, - $form->{"sellprice_$i"}, $fxsellprice, - $form->{"discount_$i"}, $allocated, 'f', - '$form->{"unit_$i"}', $deliverydate, | . conv_i($form->{"project_id_$i"}, 'NULL') . qq|, - '$form->{"serialnumber_$i"}', '$pricegroup_id', - '$form->{"ordnumber_$i"}', '$form->{"transdate_$i"}', '$form->{"cusordnumber_$i"}', $baseqty, '$subtotal')|; - $dbh->do($query) || $form->dberror($query); - - if ($form->{lizenzen}) { - if ($form->{"licensenumber_$i"}) { - $query = - qq|SELECT i.id FROM invoice i WHERE i.trans_id=$form->{id} ORDER BY i.oid DESC LIMIT 1|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($invoice_row_id) = $sth->fetchrow_array; - $sth->finish; - - $query = - qq|INSERT INTO licenseinvoice (trans_id, license_id) VALUES ($invoice_row_id, $form->{"licensenumber_$i"})|; - $dbh->do($query) || $form->dberror($query); - } + $query = + qq|INSERT INTO invoice (trans_id, parts_id, description, longdescription, qty, + sellprice, fxsellprice, discount, allocated, assemblyitem, + unit, deliverydate, project_id, serialnumber, pricegroup_id, + ordnumber, transdate, cusordnumber, base_qty, subtotal) + VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)|; + + @values = (conv_i($form->{id}), conv_i($form->{"id_$i"}), + $form->{"description_$i"}, $form->{"longdescription_$i"}, $form->{"qty_$i"}, + $form->{"sellprice_$i"}, $fxsellprice, + $form->{"discount_$i"}, $allocated, 'f', + $form->{"unit_$i"}, conv_date($form->{"deliverydate_$i"}), conv_i($form->{"project_id_$i"}), + $form->{"serialnumber_$i"}, conv_i($pricegroup_id), + $form->{"ordnumber_$i"}, conv_date($form->{"transdate_$i"}), + $form->{"cusordnumber_$i"}, $baseqty, $subtotal); + do_query($form, $dbh, $query, @values); + + if ($form->{lizenzen} && $form->{"licensenumber_$i"}) { + $query = + qq|INSERT INTO licenseinvoice (trans_id, license_id) + VALUES ((SELECT id FROM invoice WHERE trans_id = ? ORDER BY oid DESC LIMIT 1), ?)|; + @values = (conv_i($form->{"id"}), conv_i($form->{"licensenumber_$i"})); + do_query($form, $dbh, $query, @values); } - } } @@ -795,7 +729,7 @@ sub post_invoice { $diff += $amount - $netamount * $form->{exchangerate}; $netamount = $amount; - foreach my $item (split / /, $form->{taxaccounts}) { + foreach my $item (split(/ /, $form->{taxaccounts})) { $amount = $form->{amount}{ $form->{id} }{$item} * $form->{exchangerate}; $form->{amount}{ $form->{id} }{$item} = $form->round_amount($amount, 2); $tax += $form->{amount}{ $form->{id} }{$item}; @@ -812,7 +746,7 @@ sub post_invoice { $amount = $form->round_amount($netamount * $form->{exchangerate}, 2); $diff = $amount - $netamount * $form->{exchangerate}; $netamount = $amount; - foreach my $item (split / /, $form->{taxaccounts}) { + foreach my $item (split(/ /, $form->{taxaccounts})) { $form->{amount}{ $form->{id} }{$item} = $form->round_amount($form->{amount}{ $form->{id} }{$item}, 2); $amount = @@ -850,13 +784,12 @@ sub post_invoice { $form->round_amount($form->{amount}{$trans_id}{$accno}, 2) ) != 0 ) { - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, - transdate, taxkey, project_id) - VALUES ($trans_id, (SELECT c.id FROM chart c - WHERE c.accno = '$accno'), - $form->{amount}{$trans_id}{$accno}, '$form->{invdate}', - (SELECT taxkey_id FROM chart WHERE accno = '$accno'), ?)|; - do_query($form, $dbh, $query, $project_id); + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, taxkey, project_id) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; + @values = (conv_i($trans_id), $accno, $form->{amount}{$trans_id}{$accno}, conv_date($form->{invdate}), $accno, conv_i($project_id)); + do_query($form, $dbh, $query, @values); $form->{amount}{$trans_id}{$accno} = 0; } } @@ -867,13 +800,12 @@ sub post_invoice { $form->round_amount($form->{amount}{$trans_id}{$accno}, 2) ) != 0 ) { - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, - transdate, taxkey, project_id) - VALUES ($trans_id, (SELECT id FROM chart - WHERE accno = '$accno'), - $form->{amount}{$trans_id}{$accno}, '$form->{invdate}', - (SELECT taxkey_id FROM chart WHERE accno = '$accno'), ?)|; - do_query($form, $dbh, $query, $project_id); + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, taxkey, project_id) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; + @values = (conv_i($trans_id), $accno, $form->{amount}{$trans_id}{$accno}, conv_date($form->{invdate}), $accno, conv_i($project_id)); + do_query($form, $dbh, $query, @values); } } } @@ -887,85 +819,77 @@ sub post_invoice { } } - # force AR entry if 0 - # $form->{amount}{$form->{id}}{$form->{AR}} = 1 if ($form->{amount}{$form->{id}}{$form->{AR}} == 0); - # record payments and offsetting AR if (!$form->{storno}) { for my $i (1 .. $form->{paidaccounts}) { - - if ($form->{"paid_$i"} != 0) { - my ($accno) = split /--/, $form->{"AR_paid_$i"}; - $form->{"datepaid_$i"} = $form->{invdate} - unless ($form->{"datepaid_$i"}); - $form->{datepaid} = $form->{"datepaid_$i"}; - - $exchangerate = 0; - - if ($form->{currency} eq $form->{defaultcurrency}) { - $form->{"exchangerate_$i"} = 1; - } else { - $exchangerate = - $form->check_exchangerate($myconfig, $form->{currency}, - $form->{"datepaid_$i"}, 'buy'); - - $form->{"exchangerate_$i"} = - ($exchangerate) - ? $exchangerate + + next if ($form->{"paid_$i"} == 0); + + my ($accno) = split(/--/, $form->{"AR_paid_$i"}); + $form->{"datepaid_$i"} = $form->{invdate} + unless ($form->{"datepaid_$i"}); + $form->{datepaid} = $form->{"datepaid_$i"}; + + $exchangerate = 0; + + if ($form->{currency} eq $form->{defaultcurrency}) { + $form->{"exchangerate_$i"} = 1; + } else { + $exchangerate = + $form->check_exchangerate($myconfig, $form->{currency}, + $form->{"datepaid_$i"}, 'buy'); + + $form->{"exchangerate_$i"} = + $exchangerate ? $exchangerate : $form->parse_amount($myconfig, $form->{"exchangerate_$i"}); - } - - # record AR - $amount = - $form->round_amount($form->{"paid_$i"} * $form->{exchangerate} + $diff, - 2); - - if ($form->{amount}{ $form->{id} }{ $form->{AR} } != 0) { - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, - transdate, taxkey, project_id) - VALUES ($form->{id}, (SELECT c.id FROM chart c - WHERE c.accno = ?), - $amount, '$form->{"datepaid_$i"}', - (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; - do_query($form, $dbh, $query, $form->{AR}, $form->{AR}, $project_id); - } - - # record payment - $form->{"paid_$i"} *= -1; - - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, - source, memo, taxkey, project_id) - VALUES ($form->{id}, (SELECT c.id FROM chart c - WHERE c.accno = ?), - $form->{"paid_$i"}, '$form->{"datepaid_$i"}', - '$form->{"source_$i"}', '$form->{"memo_$i"}', - (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; - do_query($form, $dbh, $query, $accno, $accno, $project_id); - - # exchangerate difference - $form->{fx}{$accno}{ $form->{"datepaid_$i"} } += - $form->{"paid_$i"} * ($form->{"exchangerate_$i"} - 1) + $diff; - - # gain/loss - $amount = - $form->{"paid_$i"} * $form->{exchangerate} - $form->{"paid_$i"} * - $form->{"exchangerate_$i"}; - if ($amount > 0) { - $form->{fx}{ $form->{fxgain_accno} }{ $form->{"datepaid_$i"} } += - $amount; - } else { - $form->{fx}{ $form->{fxloss_accno} }{ $form->{"datepaid_$i"} } += - $amount; - } - - $diff = 0; - - # update exchange rate - if (($form->{currency} ne $form->{defaultcurrency}) && !$exchangerate) { - $form->update_exchangerate($dbh, $form->{currency}, - $form->{"datepaid_$i"}, - $form->{"exchangerate_$i"}, 0); - } + } + + # record AR + $amount = $form->round_amount($form->{"paid_$i"} * $form->{exchangerate} + $diff, 2); + + if ($form->{amount}{ $form->{id} }{ $form->{AR} } != 0) { + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, taxkey, project_id) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; + @values = (conv_i($form->{"id"}), $form->{AR}, $amount, $form->{"datepaid_$i"}, $form->{AR}, $project_id); + do_query($form, $dbh, $query, @values); + } + + # record payment + $form->{"paid_$i"} *= -1; + + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, source, memo, taxkey, project_id) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; + @values = (conv_i($form->{"id"}), $accno, $form->{"paid_$i"}, $form->{"datepaid_$i"}, + $form->{"source_$i"}, $form->{"memo_$i"}, $accno, $project_id); + do_query($form, $dbh, $query, @values); + + # exchangerate difference + $form->{fx}{$accno}{ $form->{"datepaid_$i"} } += + $form->{"paid_$i"} * ($form->{"exchangerate_$i"} - 1) + $diff; + + # gain/loss + $amount = + $form->{"paid_$i"} * $form->{exchangerate} - $form->{"paid_$i"} * + $form->{"exchangerate_$i"}; + if ($amount > 0) { + $form->{fx}{ $form->{fxgain_accno} }{ $form->{"datepaid_$i"} } += + $amount; + } else { + $form->{fx}{ $form->{fxloss_accno} }{ $form->{"datepaid_$i"} } += + $amount; + } + + $diff = 0; + + # update exchange rate + if (($form->{currency} ne $form->{defaultcurrency}) && !$exchangerate) { + $form->update_exchangerate($dbh, $form->{currency}, + $form->{"datepaid_$i"}, + $form->{"exchangerate_$i"}, 0); } } } @@ -979,14 +903,12 @@ sub post_invoice { ) != 0 ) { - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, - transdate, cleared, fx_transaction, taxkey, project_id) - VALUES ($form->{id}, - (SELECT c.id FROM chart c - WHERE c.accno = ?), - $form->{fx}{$accno}{$transdate}, '$transdate', '0', '1', - (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; - do_query($form, $dbh, $query, $accno, $accno, $project_id); + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, cleared, fx_transaction, taxkey, project_id) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, '0', '1', + (SELECT taxkey_id FROM chart WHERE accno = ?), ?)|; + @values = (conv_i($form->{"id"}), $accno, $form->{fx}{$accno}{$transdate}, conv_date($transdate), $accno, $project_id); + do_query($form, $dbh, $query, @values); } } } @@ -994,12 +916,9 @@ sub post_invoice { $amount = $netamount + $tax; # set values which could be empty to 0 - $form->{terms} *= 1; - $form->{taxincluded} *= 1; - my $datepaid = ($form->{paid}) ? qq|'$form->{datepaid}'| : "NULL"; - my $duedate = ($form->{duedate}) ? qq|'$form->{duedate}'| : "NULL"; - my $deliverydate = - ($form->{deliverydate}) ? qq|'$form->{deliverydate}'| : "NULL"; + my $datepaid = conv_date($form->{paid}); + my $duedate = conv_date($form->{duedate}); + $deliverydate = conv_date($form->{deliverydate}); # fill in subject if there is none $form->{subject} = qq|$form->{label} $form->{invnumber}| @@ -1021,59 +940,69 @@ Message: $form->{message}\r| if $form->{message}; # save AR record $query = qq|UPDATE ar set - invnumber = '$form->{invnumber}', - ordnumber = '$form->{ordnumber}', - quonumber = '$form->{quonumber}', - cusordnumber = '$form->{cusordnumber}', - transdate = '$form->{invdate}', - orddate = | . conv_dateq($form->{orddate}) . qq|, - quodate = | . conv_dateq($form->{quodate}) . qq|, - customer_id = $form->{customer_id}, - amount = $amount, - netamount = $netamount, - paid = $form->{paid}, - datepaid = $datepaid, - duedate = $duedate, - deliverydate = $deliverydate, - invoice = '1', - shippingpoint = '$form->{shippingpoint}', - shipvia = '$form->{shipvia}', - terms = $form->{terms}, - notes = '$form->{notes}', - intnotes = '$form->{intnotes}', - taxincluded = '$form->{taxincluded}', - curr = '$form->{currency}', - department_id = $form->{department_id}, - payment_id = $form->{payment_id}, - type = '$form->{type}', - language_id = $form->{language_id}, - taxzone_id = $form->{taxzone_id}, - shipto_id = $form->{shipto_id}, - delivery_customer_id = $form->{delivery_customer_id}, - delivery_vendor_id = $form->{delivery_vendor_id}, - employee_id = $form->{employee_id}, - salesman_id = | . conv_i($form->{salesman_id}, 'NULL') . qq|, - storno = '$form->{storno}', - globalproject_id = | . conv_i($form->{"globalproject_id"}, 'NULL') . qq|, - cp_id = | . conv_i($form->{"cp_id"}, 'NULL') . qq| - WHERE id = $form->{id} - |; - $dbh->do($query) || $form->dberror($query); + invnumber = ?, + ordnumber = ?, + quonumber = ?, + cusordnumber = ?, + transdate = ?, + orddate = ?, + quodate = ?, + customer_id = ?, + amount = ?, + netamount = ?, + paid = ?, + datepaid = ?, + duedate = ?, + deliverydate = ?, + invoice = '1', + shippingpoint = ?, + shipvia = ?, + terms = ?, + notes = ?, + intnotes = ?, + taxincluded = ?, + curr = ?, + department_id = ?, + payment_id = ?, + type = ?, + language_id = ?, + taxzone_id = ?, + shipto_id = ?, + delivery_customer_id = ?, + delivery_vendor_id = ?, + employee_id = ?, + salesman_id = ?, + storno = ?, + globalproject_id = ?, + cp_id = ? + WHERE id = ?|; + @values = ($form->{"invnumber"}, $form->{"ordnumber"}, $form->{"quonumber"}, $form->{"cusordnumber"}, + conv_date($form->{"invdate"}), conv_date($form->{"orddate"}), conv_date($form->{"quodate"}), + conv_i($form->{"customer_id"}), $amount, $netamount, $form->{"paid"}, + conv_date($form->{"datepaid"}), conv_date($form->{"duedate"}), conv_date($form->{"deliverydate"}), + $form->{"shippingpoint"}, $form->{"shipvia"}, conv_i($form->{"terms"}), + $form->{"notes"}, $form->{"intnotes"}, $form->{"taxincluded"} ? 't' : 'f', + $form->{"currency"}, conv_i($form->{"department_id"}), conv_i($form->{"payment_id"}), + $form->{"type"}, conv_i($form->{"language_id"}), conv_i($form->{"taxzone_id"}), + conv_i($form->{"shipto_id"}), + conv_i($form->{"delivery_customer_id"}), conv_i($form->{"delivery_vendor_id"}), + conv_i($form->{"employee_id"}), conv_i($form->{"salesman_id"}), + $form->{"storno"} ? 't' : 'f', conv_i($form->{"globalproject_id"}), + conv_i($form->{"cp_id"}), + conv_i($form->{"id"})); + do_query($form, $dbh, $query, @values); if ($form->{storno}) { - $query = qq| update ar set paid=paid+amount where id=$form->{storno_id}|; - $dbh->do($query) || $form->dberror($query); - $query = qq| update ar set storno='$form->{storno}' where id=$form->{storno_id}|; - $dbh->do($query) || $form->dberror($query); - $query = qq§ update ar set intnotes='Rechnung storniert am $form->{invdate} ' || intnotes where id=$form->{storno_id}§; - $dbh->do($query) || $form->dberror($query); - - $query = qq| update ar set paid=amount where id=$form->{id}|; - $dbh->do($query) || $form->dberror($query); + $query = + qq!UPDATE ar SET + paid = paid + amount, + storno = 't', + intnotes = ? || intnotes + WHERE id = ?!; + do_query($form, $dbh, $query, "Rechnung storniert am $form->{invdate} ", conv_i($form->{"storno_id"})); + do_query($form, $dbh, qq|UPDATE ar SET paid = amount WHERE id = ?|, conv_i($form->{"id"})); } - $form->{pago_total} = $amount; - # add shipto $form->{name} = $form->{customer}; $form->{name} =~ s/--$form->{customer_id}//; @@ -1104,11 +1033,11 @@ sub post_payment { $form->{datepaid} = $form->{invdate}; # total payments, don't move we need it here - for my $i (1 .. $form->{paidaccounts}) { + for my $i ( 1 .. $form->{paidaccounts} ) { $form->{"paid_$i"} = $form->parse_amount($myconfig, $form->{"paid_$i"}); - $form->{"paid_$i"} *= -1 if ($form->{type} eq "credit_note"); + $form->{"paid_$i"} *= -1 if $form->{type} eq "credit_note"; $form->{"paid"} += $form->{"paid_$i"}; - $form->{"datepaid"} = $form->{"datepaid_$i"} if ($form->{"datepaid_$i"}); + $form->{"datepaid"} = $form->{"datepaid_$i"} if $form->{"datepaid_$i"}; } $form->{exchangerate} = $form->get_exchangerate($dbh, $form->{currency}, $form->{invdate}, "buy"); @@ -1132,20 +1061,33 @@ sub post_payment { # record AR $amount = $form->round_amount($form->{"paid_$i"} * $form->{"exchangerate"}, 2); - $query = qq|DELETE FROM acc_trans WHERE trans_id = ? AND chart_id = (SELECT c.id FROM chart c WHERE c.accno = ?) AND amount = ? AND transdate = ?|; - do_query($form, $dbh, $query, $form->{id}, $form->{AR}, $amount, $form->{"datepaid_$i"}); - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, project_id, taxkey) - VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, ?, (SELECT taxkey_id FROM chart WHERE accno = ?))|; + $query = + qq|DELETE FROM acc_trans + WHERE (trans_id = ?) + AND (chart_id = (SELECT id FROM chart WHERE accno = ?)) + AND (amount = ?) AND (transdate = ?)|; + do_query($form, $dbh, $query, $form->{id}, $form->{AR}, $amount, conv_date($form->{"datepaid_$i"})); + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, project_id, taxkey) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?))|; do_query($form, $dbh, $query, $form->{id}, $form->{AR}, $amount, $form->{"datepaid_$i"}, conv_i($form->{"globalproject_id"}), $accno); # record payment $form->{"paid_$i"} *= -1; - $query = qq|DELETE FROM acc_trans WHERE trans_id = ? AND chart_id = (SELECT c.id FROM chart c WHERE c.accno = ?) AND amount = ? AND transdate = ? AND source = ? AND memo = ?|; + $query = + qq|DELETE FROM acc_trans + WHERE (trans_id = ?) + AND (chart_id = (SELECT id FROM chart WHERE accno = ?)) + AND (amount = ?) AND (transdate = ?) AND (source = ?) AND (memo = ?)|; do_query($form, $dbh, $query, $form->{id}, $accno, $form->{"paid_$i"}, $form->{"datepaid_$i"}, $form->{"source_$i"}, $form->{"memo_$i"}); - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, source, memo, project_id, taxkey) - VALUES (?, (SELECT c.id FROM chart c WHERE c.accno = ?), ?, ?, ?, ?, ?, (SELECT taxkey_id FROM chart WHERE accno = ?))|; - do_query($form, $dbh, $query, $form->{id}, $accno, $form->{"paid_$i"}, $form->{"datepaid_$i"}, $form->{"source_$i"}, $form->{"memo_$i"}, + + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, source, memo, project_id, taxkey) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, ?, ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?))|; + do_query($form, $dbh, $query, $form->{id}, $accno, $form->{"paid_$i"}, $form->{"datepaid_$i"}, $form->{"source_$i"}, $form->{"memo_$i"}, conv_i($form->{"globalproject_id"}), $accno); # gain/loss @@ -1167,11 +1109,16 @@ sub post_payment { foreach my $transdate (keys %{ $form->{fx}{$accno} }) { if ($form->{fx}{$accno}{$transdate} = $form->round_amount($form->{fx}{$accno}{$transdate}, 2)) { # '=' is no typo, it's an assignment - $query = qq|DELETE FROM acc_trans WHERE trans_id = ? AND chart_id = (SELECT c.id FROM chart c WHERE c.accno = ?) - AND amount = ? AND transdate = ? AND cleared = ? AND fx_transaction = ?|; + $query = + qq|DELETE FROM acc_trans + WHERE (trans_id = ?) + AND (chart_id = (SELECT c.id FROM chart c WHERE c.accno = ?)) + AND (amount = ?) AND (transdate = ?) AND (cleared = ?) AND (fx_transaction = ?)|; do_query($form, $dbh, $query, $form->{id}, $accno, $form->{fx}{$accno}{$transdate}, $transdate, 0, 1); - $query = qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, cleared, fx_transaction, project_id, taxkey) - VALUES (?, (SELECT c.id FROM chart c WHERE c.accno = ?), ?, ?, ?, ?, ?, (SELECT taxkey_id FROM chart WHERE accno = ?))|; + $query = + qq|INSERT INTO acc_trans (trans_id, chart_id, amount, transdate, cleared, fx_transaction, project_id, taxkey) + VALUES (?, (SELECT id FROM chart WHERE accno = ?), ?, ?, ?, ?, ?, + (SELECT taxkey_id FROM chart WHERE accno = ?))|; do_query($form, $dbh, $query, $form->{id}, $accno, $form->{fx}{$accno}{$transdate}, $transdate, 0, 1, conv_i($form->{"globalproject_id"}), $accno); } @@ -1181,8 +1128,8 @@ sub post_payment { # save AR record delete $form->{datepaid} unless $form->{paid}; - my $query = qq|UPDATE ar set paid = ?, datepaid = ? WHERE id = ?|; - do_query($form, $dbh, $query, $form->{paid}, $form->{datepaid}, $form->{id}); + my $query = qq|UPDATE ar SET paid = ?, datepaid = ? WHERE id = ?|; + do_query($form, $dbh, $query, $form->{"paid"}, conv_date($form->{"datepaid"}), conv_i($form->{"id"})); my $rc = $dbh->commit; $dbh->disconnect; @@ -1195,15 +1142,13 @@ sub process_assembly { my ($dbh, $form, $id, $totalqty) = @_; - my $query = qq|SELECT a.parts_id, a.qty, p.assembly, - p.partnumber, p.description, p.unit, - p.inventory_accno_id, p.income_accno_id, - p.expense_accno_id - FROM assembly a - JOIN parts p ON (a.parts_id = p.id) - WHERE a.id = $id|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + my $query = + qq|SELECT a.parts_id, a.qty, p.assembly, p.partnumber, p.description, p.unit, + p.inventory_accno_id, p.income_accno_id, p.expense_accno_id + FROM assembly a + JOIN parts p ON (a.parts_id = p.id) + WHERE (a.id = ?)|; + my $sth = prepare_execute_query($form, $dbh, $query, conv_i($id)); while (my $ref = $sth->fetchrow_hashref(NAME_lc)) { @@ -1212,8 +1157,6 @@ sub process_assembly { $ref->{inventory_accno_id} *= 1; $ref->{expense_accno_id} *= 1; - map { $ref->{$_} =~ s/\'/\'\'/g } (qw(partnumber description unit)); - # multiply by number of assemblies $ref->{qty} *= $totalqty; @@ -1227,13 +1170,11 @@ sub process_assembly { } # save detail record for individual assembly item in invoice table - $query = qq|INSERT INTO invoice (trans_id, description, parts_id, qty, - sellprice, fxsellprice, allocated, assemblyitem, unit) - VALUES - ($form->{id}, '$ref->{description}', - $ref->{parts_id}, $ref->{qty}, 0, 0, $allocated, 't', - '$ref->{unit}')|; - $dbh->do($query) || $form->dberror($query); + $query = + qq|INSERT INTO invoice (trans_id, description, parts_id, qty, sellprice, fxsellprice, allocated, assemblyitem, unit) + VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)|; + @values = (conv_i($form->{id}), $ref->{description}, conv_i($ref->{parts_id}), $ref->{qty}, 0, 0, $allocated, 't', $ref->{unit}); + do_query($form, $dbh, $query, @values); } @@ -1247,21 +1188,32 @@ sub cogs { my ($dbh, $form, $id, $totalqty, $basefactor, $row) = @_; $form->{taxzone_id} *=1; - my $transdate = ($form->{invdate}) ? "'$form->{invdate}'" : "current_date"; - my $query = qq|SELECT i.id, i.trans_id, i.base_qty, i.allocated, i.sellprice, - c1.accno AS inventory_accno, c1.new_chart_id AS inventory_new_chart, date($transdate) - c1.valid_from as inventory_valid, - c2.accno AS income_accno, c2.new_chart_id AS income_new_chart, date($transdate) - c2.valid_from as income_valid, - c3.accno AS expense_accno, c3.new_chart_id AS expense_new_chart, date($transdate) - c3.valid_from as expense_valid - FROM invoice i, parts p - LEFT JOIN chart c1 ON ((select inventory_accno_id from buchungsgruppen where id=p.buchungsgruppen_id) = c1.id) - LEFT JOIN chart c2 ON ((select income_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c2.id) - LEFT JOIN chart c3 ON ((select expense_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c3.id) - WHERE i.parts_id = p.id - AND i.parts_id = $id - AND (i.base_qty + i.allocated) < 0 - ORDER BY trans_id|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + my $transdate = $form->{invdate} ? $dbh->quote($form->{invdate}) : "current_date"; + my $taxzone_id = $form->{"taxzone_id"} * 1; + my $query = + qq|SELECT i.id, i.trans_id, i.base_qty, i.allocated, i.sellprice, + + c1.accno AS inventory_accno, + c1.new_chart_id AS inventory_new_chart, + date($transdate) - c1.valid_from AS inventory_valid, + + c2.accno AS income_accno, + c2.new_chart_id AS income_new_chart, + date($transdate) - c2.valid_from AS income_valid, + + c3.accno AS expense_accno, + c3.new_chart_id AS expense_new_chart, + date($transdate) - c3.valid_from AS expense_valid + + FROM invoice i, parts p + LEFT JOIN chart c1 ON ((SELECT inventory_accno_id FROM buchungsgruppen WHERE id = p.buchungsgruppen_id) = c1.id) + LEFT JOIN chart c2 ON ((SELECT income_accno_id_${taxzone_id} FROM buchungsgruppen WHERE id = p.buchungsgruppen_id) = c2.id) + LEFT JOIN chart c3 ON ((select expense_accno_id_${taxzone_id} FROM buchungsgruppen WHERE id = p.buchungsgruppen_id) = c3.id) + WHERE (i.parts_id = p.id) + AND (i.parts_id = ?) + AND ((i.base_qty + i.allocated) < 0) + ORDER BY trans_id|; + my $sth = prepare_execute_query($form, $dbh, $query, conv_i($id)); my $allocated = 0; my $qty; @@ -1271,8 +1223,7 @@ sub cogs { $qty = $totalqty; } - $form->update_balance($dbh, "invoice", "allocated", qq|id = $ref->{id}|, - $qty); + $form->update_balance($dbh, "invoice", "allocated", qq|id = $ref->{id}|, $qty); # total expenses and inventory # sellprice is the cost of the item @@ -1290,7 +1241,7 @@ sub cogs { } # add allocated - $allocated += -$qty; + $allocated -= $qty; last if (($totalqty -= $qty) <= 0); } @@ -1308,13 +1259,12 @@ sub reverse_invoice { my ($dbh, $form) = @_; # reverse inventory items - my $query = qq|SELECT i.id, i.parts_id, i.qty, i.assemblyitem, p.assembly, - p.inventory_accno_id - FROM invoice i - JOIN parts p ON (i.parts_id = p.id) - WHERE i.trans_id = $form->{id}|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + my $query = + qq|SELECT i.id, i.parts_id, i.qty, i.assemblyitem, p.assembly, p.inventory_accno_id + FROM invoice i + JOIN parts p ON (i.parts_id = p.id) + WHERE i.trans_id = ?|; + my $sth = prepare_execute_query($form, $dbh, $query, conv_i($form->{"id"})); while (my $ref = $sth->fetchrow_hashref(NAME_lc)) { @@ -1324,61 +1274,50 @@ sub reverse_invoice { if (!$ref->{assemblyitem}) { # adjust onhand in parts table - $form->update_balance($dbh, "parts", "onhand", - qq|id = $ref->{parts_id}|, - $ref->{qty}); + $form->update_balance($dbh, "parts", "onhand", qq|id = $ref->{parts_id}|, $ref->{qty}); } # loop if it is an assembly next if ($ref->{assembly}); # de-allocated purchases - $query = qq|SELECT i.id, i.trans_id, i.allocated - FROM invoice i - WHERE i.parts_id = $ref->{parts_id} - AND i.allocated > 0 - ORDER BY i.trans_id DESC|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - while (my $inhref = $sth->fetchrow_hashref(NAME_lc)) { + $query = + qq|SELECT i.id, i.trans_id, i.allocated + FROM invoice i + WHERE (i.parts_id = ?) AND (i.allocated > 0) + ORDER BY i.trans_id DESC|; + my $sth2 = prepare_execute_query($form, $dbh, $query, conv_i($ref->{"parts_id"})); + + while (my $inhref = $sth2->fetchrow_hashref(NAME_lc)) { $qty = $ref->{qty}; if (($ref->{qty} - $inhref->{allocated}) > 0) { $qty = $inhref->{allocated}; } # update invoice - $form->update_balance($dbh, "invoice", "allocated", - qq|id = $inhref->{id}|, - $qty * -1); + $form->update_balance($dbh, "invoice", "allocated", qq|id = $inhref->{id}|, $qty * -1); last if (($ref->{qty} -= $qty) <= 0); } - $sth->finish; + $sth2->finish; } } $sth->finish; # delete acc_trans - $query = qq|DELETE FROM acc_trans - WHERE trans_id = $form->{id}|; - $dbh->do($query) || $form->dberror($query); - - # delete invoice entries - $query = qq|DELETE FROM invoice - WHERE trans_id = $form->{id}|; - $dbh->do($query) || $form->dberror($query); + @values = (conv_i($form->{id})); + do_query($form, $dbh, qq|DELETE FROM acc_trans WHERE trans_id = ?|, @values); + do_query($form, $dbh, qq|DELETE FROM invoice WHERE trans_id = ?|, @values); if ($form->{lizenzen}) { - $query = qq|DELETE FROM licenseinvoice - WHERE trans_id in (SELECT id FROM invoice WHERE trans_id = $form->{id})|; - $dbh->do($query) || $form->dberror($query); + $query = + qq|DELETE FROM licenseinvoice + WHERE trans_id in (SELECT id FROM invoice WHERE trans_id = ?)|; + do_query($form, $dbh, $query, @values); } - $query = qq|DELETE FROM shipto - WHERE trans_id = $form->{id} AND module = 'AR'|; - $dbh->do($query) || $form->dberror($query); + do_query($form, $dbh, qq|DELETE FROM shipto WHERE (trans_id = ?) AND (module = 'AR')|, @values); $main::lxdebug->leave_sub(); } @@ -1393,37 +1332,22 @@ sub delete_invoice { &reverse_invoice($dbh, $form); + my @values = (conv_i($form->{id})); + # delete AR record - my $query = qq|DELETE FROM ar - WHERE id = $form->{id}|; - $dbh->do($query) || $form->dberror($query); + do_query($form, $dbh, qq|DELETE FROM ar WHERE id = ?|, @values); # delete spool files - $query = qq|SELECT s.spoolfile FROM status s - WHERE s.trans_id = $form->{id}|; - my $sth = $dbh->prepare($query); - $sth->execute || $self->dberror($query); - - my $spoolfile; - my @spoolfiles = (); - - while (($spoolfile) = $sth->fetchrow_array) { - push @spoolfiles, $spoolfile; - } - $sth->finish; + my @spoolfiles = selectall_array_query($form, $dbh, qq|SELECT spoolfile FROM status WHERE trans_id = ?|, @values); # delete status entries - $query = qq|DELETE FROM status - WHERE trans_id = $form->{id}|; - $dbh->do($query) || $form->dberror($query); + do_query($form, $dbh, qq|DELETE FROM status WHERE trans_id = ?|, @values); my $rc = $dbh->commit; $dbh->disconnect; if ($rc) { - foreach $spoolfile (@spoolfiles) { - unlink "$spool/$spoolfile" if $spoolfile; - } + map { unlink "$spool/$_" if -f "$spool/$_"; } @{ $spoolfiles }; } $main::lxdebug->leave_sub(); @@ -1439,99 +1363,71 @@ sub retrieve_invoice { # connect to database my $dbh = $form->dbconnect_noauto($myconfig); - my $query; - - if ($form->{id}) { - - # get default accounts and last invoice number - $query = qq|SELECT (SELECT c.accno FROM chart c - WHERE d.inventory_accno_id = c.id) AS inventory_accno, - (SELECT c.accno FROM chart c - WHERE d.income_accno_id = c.id) AS income_accno, - (SELECT c.accno FROM chart c - WHERE d.expense_accno_id = c.id) AS expense_accno, - (SELECT c.accno FROM chart c - WHERE d.fxgain_accno_id = c.id) AS fxgain_accno, - (SELECT c.accno FROM chart c - WHERE d.fxloss_accno_id = c.id) AS fxloss_accno, - d.curr AS currencies - FROM defaults d|; - } else { - $query = qq|SELECT (SELECT c.accno FROM chart c - WHERE d.inventory_accno_id = c.id) AS inventory_accno, - (SELECT c.accno FROM chart c - WHERE d.income_accno_id = c.id) AS income_accno, - (SELECT c.accno FROM chart c - WHERE d.expense_accno_id = c.id) AS expense_accno, - (SELECT c.accno FROM chart c - WHERE d.fxgain_accno_id = c.id) AS fxgain_accno, - (SELECT c.accno FROM chart c - WHERE d.fxloss_accno_id = c.id) AS fxloss_accno, - d.curr AS currencies, current_date AS invdate - FROM defaults d|; - } - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - my $ref = $sth->fetchrow_hashref(NAME_lc); - map { $form->{$_} = $ref->{$_} } keys %$ref; - $sth->finish; + my ($sth, $ref, $query); + + my $query_transdate = ", current_date AS invdate" if !$form->{id}; + + $query = + qq|SELECT + (SELECT c.accno FROM chart c + WHERE d.inventory_accno_id = c.id) AS inventory_accno, + (SELECT c.accno FROM chart c + WHERE d.income_accno_id = c.id) AS income_accno, + (SELECT c.accno FROM chart c + WHERE d.expense_accno_id = c.id) AS expense_accno, + (SELECT c.accno FROM chart c + WHERE d.fxgain_accno_id = c.id) AS fxgain_accno, + (SELECT c.accno FROM chart c + WHERE d.fxloss_accno_id = c.id) AS fxloss_accno, + d.curr AS currencies + ${query_transdate} + FROM defaults d|; + + $ref = selectfirst_hashref_query($form, $dbh, $query); + map { $form->{$_} = $ref->{$_} } keys %{ $ref }; if ($form->{id}) { + my $id = conv_i($form->{id}); # retrieve invoice - $query = qq|SELECT a.invnumber, a.ordnumber, a.quonumber, a.cusordnumber, - a.orddate, a.quodate, a.globalproject_id, - a.transdate AS invdate, a.deliverydate, a.paid, a.storno, a.gldate, - a.shippingpoint, a.shipvia, a.terms, a.notes, a.intnotes, a.taxzone_id, - a.duedate, a.taxincluded, a.curr AS currency, a.shipto_id, a.cp_id, - a.employee_id, e.name AS employee, a.salesman_id, a.payment_id, a.language_id, a.delivery_customer_id, a.delivery_vendor_id, a.type - FROM ar a - LEFT JOIN employee e ON (e.id = a.employee_id) - WHERE a.id = $form->{id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = + qq|SELECT + a.invnumber, a.ordnumber, a.quonumber, a.cusordnumber, + a.orddate, a.quodate, a.globalproject_id, + a.transdate AS invdate, a.deliverydate, a.paid, a.storno, a.gldate, + a.shippingpoint, a.shipvia, a.terms, a.notes, a.intnotes, a.taxzone_id, + a.duedate, a.taxincluded, a.curr AS currency, a.shipto_id, a.cp_id, + a.employee_id, a.salesman_id, a.payment_id, + a.language_id, a.delivery_customer_id, a.delivery_vendor_id, a.type, + e.name AS employee + FROM ar a + LEFT JOIN employee e ON (e.id = a.employee_id) + WHERE a.id = ?|; + $ref = selectfirst_hashref_query($form, $dbh, $query, $id); + map { $form->{$_} = $ref->{$_} } keys %{ $ref }; - $ref = $sth->fetchrow_hashref(NAME_lc); - map { $form->{$_} = $ref->{$_} } keys %$ref; - $sth->finish; $form->{exchangerate} = - $form->get_exchangerate($dbh, $form->{currency}, $form->{invdate}, - "buy"); - # get shipto - $query = qq|SELECT s.* FROM shipto s - WHERE s.trans_id = $form->{id} AND s.module = 'AR'|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $form->get_exchangerate($dbh, $form->{currency}, $form->{invdate}, "buy"); - $ref = $sth->fetchrow_hashref(NAME_lc); - delete($ref->{id}); - map { $form->{$_} = $ref->{$_} } keys %$ref; - $sth->finish; - - if ($form->{delivery_customer_id}) { - $query = qq|SELECT name FROM customer WHERE id=$form->{delivery_customer_id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - ($form->{delivery_customer_string}) = $sth->fetchrow_array(); - $sth->finish; - } - - if ($form->{delivery_vendor_id}) { - $query = qq|SELECT name FROM customer WHERE id=$form->{delivery_vendor_id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - ($form->{delivery_vendor_string}) = $sth->fetchrow_array(); - $sth->finish; + # get shipto + $query = qq|SELECT * FROM shipto WHERE (trans_id = ?) AND (module = 'AR')|; + $ref = selectfirst_hashref_query($form, $dbh, $query, $id); + delete $ref->{id}; + map { $form->{$_} = $ref->{$_} } keys %{ $ref }; + + foreach my $vc (qw(customer vendor)) { + next if !$form->{"delivery_${vc}_id"}; + ($form->{"delivery_${vc}_string"}) + = selectrow_query($form, $dbh, qq|SELECT name FROM customer WHERE id = ?|, $id); } # get printed, emailed - $query = qq|SELECT s.printed, s.emailed, s.spoolfile, s.formname - FROM status s - WHERE s.trans_id = $form->{id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = + qq|SELECT printed, emailed, spoolfile, formname + FROM status + WHERE trans_id = ?|; + $sth = prepare_execute_query($form, $dbh, $query, $id); while ($ref = $sth->fetchrow_hashref(NAME_lc)) { $form->{printed} .= "$ref->{formname} " if $ref->{printed}; @@ -1544,75 +1440,99 @@ sub retrieve_invoice { my $transdate = $form->{deliverydate} ? $dbh->quote($form->{deliverydate}) : - $form->{invdate} ? $dbh->quote($form->{invdate}) : - "current_date"; + $form->{invdate} ? $dbh->quote($form->{invdate}) : + "current_date"; + + my $taxzone_id = $form->{taxzone_id} *= 1; + $taxzone_id = 0 if (0 > $taxzone_id) || (3 < $taxzone_id); - if (!$form->{taxzone_id}) { - $form->{taxzone_id} = 0; - } # retrieve individual items - $query = qq|SELECT - c1.accno AS inventory_accno, c1.new_chart_id AS inventory_new_chart, date($transdate) - c1.valid_from as inventory_valid, - c2.accno AS income_accno, c2.new_chart_id AS income_new_chart, date($transdate) - c2.valid_from as income_valid, - c3.accno AS expense_accno, c3.new_chart_id AS expense_new_chart, date($transdate) - c3.valid_from as expense_valid, - i.description, i.longdescription, i.qty, i.fxsellprice AS sellprice, - i.discount, i.parts_id AS id, i.unit, i.deliverydate, - i.project_id, pr.projectnumber, i.serialnumber, - p.partnumber, p.assembly, p.bin, p.notes AS partnotes, p.inventory_accno_id AS part_inventory_accno_id, i.id AS invoice_pos, - pg.partsgroup, i.pricegroup_id, (SELECT pricegroup FROM pricegroup WHERE id=i.pricegroup_id) as pricegroup, - i.ordnumber, i.transdate, i.cusordnumber, p.formel, i.subtotal - FROM invoice i - JOIN parts p ON (i.parts_id = p.id) - LEFT JOIN project pr ON (i.project_id = pr.id) - LEFT JOIN partsgroup pg ON (p.partsgroup_id = pg.id) - LEFT JOIN chart c1 ON ((select inventory_accno_id from buchungsgruppen where id=p.buchungsgruppen_id) = c1.id) - LEFT JOIN chart c2 ON ((select income_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c2.id) - LEFT JOIN chart c3 ON ((select expense_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c3.id) - WHERE i.trans_id = $form->{id} - AND NOT i.assemblyitem = '1' - ORDER BY i.id|; - $sth = $dbh->prepare($query); + $query = + qq|SELECT + c1.accno AS inventory_accno, + c1.new_chart_id AS inventory_new_chart, + date($transdate) - c1.valid_from AS inventory_valid, + + c2.accno AS income_accno, + c2.new_chart_id AS income_new_chart, + date($transdate) - c2.valid_from as income_valid, + + c3.accno AS expense_accno, + c3.new_chart_id AS expense_new_chart, + date($transdate) - c3.valid_from AS expense_valid, + + i.description, i.longdescription, i.qty, i.fxsellprice AS sellprice, + i.discount, i.parts_id AS id, i.unit, i.deliverydate, + i.project_id, i.serialnumber, i.id AS invoice_pos, i.pricegroup_id, + i.ordnumber, i.transdate, i.cusordnumber, i.subtotal, + + p.partnumber, p.assembly, p.bin, p.notes AS partnotes, + p.inventory_accno_id AS part_inventory_accno_id, p.formel, + + pr.projectnumber, + pg.partsgroup, + prg.pricegroup + + FROM invoice i + LEFT JOIN parts p ON (i.parts_id = p.id) + LEFT JOIN project pr ON (i.project_id = pr.id) + LEFT JOIN partsgroup pg ON (p.partsgroup_id = pg.id) + LEFT JOIN pricegroup prg ON (i.pricegroup_id = prg.id) + + LEFT JOIN chart c1 ON + ((SELECT inventory_accno_id + FROM buchungsgruppen + WHERE id = p.buchungsgruppen_id) = c1.id) + LEFT JOIN chart c2 ON + ((SELECT income_accno_id_${taxzone_id} + FROM buchungsgruppen + WHERE id=p.buchungsgruppen_id) = c2.id) + LEFT JOIN chart c3 ON + ((SELECT expense_accno_id_${taxzone_id} + FROM buchungsgruppen + WHERE id = p.buchungsgruppen_id) = c3.id) + + WHERE (i.trans_id = ?) + AND NOT (i.assemblyitem = '1') + ORDER BY i.id|; + + $sth = prepare_execute_query($form, $dbh, $query, $id); - $sth->execute || $form->dberror($query); while (my $ref = $sth->fetchrow_hashref(NAME_lc)) { if (!$ref->{"part_inventory_accno_id"}) { map({ delete($ref->{$_}); } qw(inventory_accno inventory_new_chart inventory_valid)); } delete($ref->{"part_inventory_accno_id"}); - while ($ref->{inventory_new_chart} && ($ref->{inventory_valid} >=0)) { - my $query = qq| SELECT accno AS inventory_accno, new_chart_id AS inventory_new_chart, date($transdate) - valid_from AS inventory_valid FROM chart WHERE id = $ref->{inventory_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{inventory_accno}, $ref->{inventory_new_chart}, $ref->{inventory_valid}) = $stw->fetchrow_array; - $stw->finish; - } - - while ($ref->{income_new_chart} && ($ref->{income_valid} >=0)) { - my $query = qq| SELECT accno AS income_accno, new_chart_id AS income_new_chart, date($transdate) - valid_from AS income_valid FROM chart WHERE id = $ref->{income_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{income_accno}, $ref->{income_new_chart}, $ref->{income_valid}) = $stw->fetchrow_array; - $stw->finish; - } - - while ($ref->{expense_new_chart} && ($ref->{expense_valid} >=0)) { - my $query = qq| SELECT accno AS expense_accno, new_chart_id AS expense_new_chart, date($transdate) - valid_from AS expense_valid FROM chart WHERE id = $ref->{expense_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{expense_accno}, $ref->{expense_new_chart}, $ref->{expense_valid}) = $stw->fetchrow_array; - $stw->finish; - } + foreach my $type (qw(inventory income expense)) { + while ($ref->{"${type}_new_chart"} && ($ref->{"${type}_valid"} >=0)) { + my $query = + qq|SELECT accno, new_chart_id, date($transdate) - valid_from + FROM chart + WHERE id = ?|; + ($ref->{"${type}_accno"}, + $ref->{"${type}_new_chart"}, + $ref->{"${type}_valid"}) + = selectrow_query($form, $dbh, $query, $ref->{"${type}_new_chart"}); + } + } # get tax rates and description - $accno_id = + my $accno_id = ($form->{vc} eq "customer") ? $ref->{income_accno} : $ref->{expense_accno}; - $query = qq|SELECT c.accno, t.taxdescription, t.rate, t.taxnumber - FROM tax t LEFT JOIN chart c on (c.id=t.chart_id) - WHERE t.id in (SELECT tk.tax_id from taxkeys tk where tk.chart_id = (SELECT id from chart WHERE accno='$accno_id') AND startdate<=$transdate ORDER BY startdate desc LIMIT 1) - ORDER BY c.accno|; - $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); + $query = + qq|SELECT c.accno, t.taxdescription, t.rate, t.taxnumber + FROM tax t + LEFT JOIN chart c ON (c.id = t.chart_id) + WHERE t.id IN + (SELECT tk.tax_id + FROM taxkeys tk + WHERE tk.chart_id = (SELECT id FROM chart WHERE accno = ?) + AND startdate <= $transdate + ORDER BY startdate DESC + LIMIT 1) + ORDER BY c.accno|; + my $stw = prepare_execute_query($form, $dbh, $query, $accno_id); $ref->{taxaccounts} = ""; my $i=0; while ($ptr = $stw->fetchrow_hashref(NAME_lc)) { @@ -1634,20 +1554,17 @@ sub retrieve_invoice { } if ($form->{lizenzen}) { - $query = qq|SELECT l.licensenumber, l.id AS licenseid - FROM license l, licenseinvoice li - WHERE l.id = li.license_id AND li.trans_id = $ref->{invoice_pos}|; - $stg = $dbh->prepare($query); - $stg->execute || $form->dberror($query); - ($licensenumber, $licenseid) = $stg->fetchrow_array(); - $ref->{lizenzen} = - ""; - $stg->finish(); - } - if ($form->{type} eq "credit_note") { - $ref->{qty} *= -1; + $query = + qq|SELECT l.licensenumber, l.id AS licenseid + FROM license l, licenseinvoice li + WHERE l.id = li.license_id AND li.trans_id = ?|; + my ($licensenumber, $licenseid) + = selectrow_query($form, $dbh, $query, conv_i($ref->{invoice_pos})); + $ref->{lizenzen} = ""; } + $ref->{qty} *= -1 if $form->{type} eq "credit_note"; + chop $ref->{taxaccounts}; push @{ $form->{invoice_details} }, $ref; $stw->finish; @@ -1676,74 +1593,85 @@ sub get_customer { my $dateformat = $myconfig->{dateformat}; $dateformat .= "yy" if $myconfig->{dateformat} !~ /^y/; - my $duedate = - ($form->{invdate}) - ? "to_date('$form->{invdate}', '$dateformat')" - : "current_date"; + my (@values, $duedate, $ref, $query); + + if ($form->{invdate}) { + $duedate = "to_date(?, '$dateformat')"; + push @values, $form->{invdate}; + } else { + $duedate = "current_date"; + } - $form->{customer_id} *= 1; + my $cid = conv_i($form->{customer_id}); # get customer - my $query = qq|SELECT c.name AS customer, c.discount, c.creditlimit, c.terms, - c.email, c.cc, c.bcc, c.language_id, c.payment_id AS customer_payment_id, - c.street, c.zipcode, c.city, c.country, - $duedate + COALESCE(pt.terms_netto, 0) AS duedate, c.notes AS intnotes, - b.discount AS tradediscount, b.description AS business, c.klass as customer_klass, c.taxzone_id, - c.salesman_id - FROM customer c - LEFT JOIN business b ON (b.id = c.business_id) - LEFT JOIN payment_terms pt ON c.payment_id = pt.id - WHERE c.id = ?|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $form->{customer_id}); + $query = + qq|SELECT + c.name AS customer, c.discount, c.creditlimit, c.terms, + c.email, c.cc, c.bcc, c.language_id, c.payment_id AS customer_payment_id, + c.street, c.zipcode, c.city, c.country, + c.notes AS intnotes, c.klass as customer_klass, c.taxzone_id, c.salesman_id, + $duedate + COALESCE(pt.terms_netto, 0) AS duedate, + b.discount AS tradediscount, b.description AS business + FROM customer c + LEFT JOIN business b ON (b.id = c.business_id) + LEFT JOIN payment_terms pt ON (c.payment_id = pt.id) + WHERE c.id = ?|; + push @values, $cid; + $ref = selectfirst_hashref_query($form, $dbh, $query, @values); map { $form->{$_} = $ref->{$_} } keys %$ref; - my $query = qq|SELECT sum(a.amount - a.paid) AS dunning_amount FROM ar a - WHERE a.paid < a.amount AND a.customer_id = ? AND a.dunning_config_id IS NOT NULL|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $form->{customer_id}); + $query = + qq|SELECT sum(amount - paid) AS dunning_amount + FROM ar + WHERE (paid < amount) + AND (customer_id = ?) + AND (dunning_config_id IS NOT NULL)|; + $ref = selectfirst_hashref_query($form, $dbh, $query, $cid); map { $form->{$_} = $ref->{$_} } keys %$ref; - my $query = qq|SELECT dnn.dunning_description AS max_dunning_level FROM dunning_config dnn - WHERE id in (SELECT dunning_config_id from ar WHERE paid < amount AND customer_id = ? AND dunning_config_id IS NOT NULL) - ORDER BY dunning_level DESC LIMIT 1|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $form->{customer_id}); + $query = + qq|SELECT dnn.dunning_description AS max_dunning_level + FROM dunning_config dnn + WHERE id IN (SELECT dunning_config_id + FROM ar + WHERE (paid < amount) AND (customer_id = ?) AND (dunning_config_id IS NOT NULL)) + ORDER BY dunning_level DESC LIMIT 1|; + $ref = selectfirst_hashref_query($form, $dbh, $query, $cid); map { $form->{$_} = $ref->{$_} } keys %$ref; #check whether payment_terms are better than old payment_terms if (($form->{payment_id} ne "") && ($form->{customer_payment_id} ne "")) { - my $query = qq|SELECT (SELECT ranking from payment_terms WHERE id = $form->{payment_id}), (SELECT ranking FROM payment_terms WHERE id = $form->{customer_payment_id})|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($old_ranking, $new_ranking) = $stw->fetchrow_array; - $stw->finish; + $query = + qq|SELECT + (SELECT ranking FROM payment_terms WHERE id = ?), + (SELECT ranking FROM payment_terms WHERE id = ?)|; + my ($old_ranking, $new_ranking) + = selectrow_query($form, $dbh, $query, conv_i($form->{payment_id}), conv_i($form->{customer_payment_id})); if ($new_ranking > $old_ranking) { - $form->{payment_id} =$form->{customer_payment_id}; + $form->{payment_id} = $form->{customer_payment_id}; } } + if ($form->{payment_id} eq "") { - $form->{payment_id} =$form->{customer_payment_id}; + $form->{payment_id} = $form->{customer_payment_id}; } $form->{creditremaining} = $form->{creditlimit}; - $query = qq|SELECT SUM(a.amount - a.paid) - FROM ar a - WHERE a.customer_id = $form->{customer_id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - ($form->{creditremaining}) -= $sth->fetchrow_array; - - $sth->finish; - - $query = qq|SELECT o.amount, - (SELECT e.buy FROM exchangerate e - WHERE e.curr = o.curr - AND e.transdate = o.transdate) - FROM oe o - WHERE o.customer_id = $form->{customer_id} - AND o.quotation = '0' - AND o.closed = '0'|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = qq|SELECT SUM(amount - paid) FROM ar WHERE customer_id = ?|; + my ($value) = selectrow_query($form, $dbh, $query, $cid); + $form->{creditremaining} -= $value; + + $query = + qq|SELECT o.amount, + (SELECT e.buy FROM exchangerate e + WHERE e.curr = o.curr + AND e.transdate = o.transdate) + FROM oe o + WHERE o.customer_id = ? + AND o.quotation = '0' + AND o.closed = '0'|; + $sth = prepare_execute_query($form, $dbh, $query, $cid); while (my ($amount, $exch) = $sth->fetchrow_array) { $exch = 1 unless $exch; @@ -1754,45 +1682,27 @@ sub get_customer { # get shipto if we did not converted an order or invoice if (!$form->{shipto}) { map { delete $form->{$_} } - qw(shiptoname shiptodepartment_1 shiptodepartment_2 shiptostreet shiptozipcode shiptocity shiptocountry shiptocontact shiptophone shiptofax shiptoemail); - - $query = qq|SELECT s.* FROM shipto s - WHERE s.trans_id = $form->{customer_id} AND s.module = 'CT'|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + qw(shiptoname shiptodepartment_1 shiptodepartment_2 + shiptostreet shiptozipcode shiptocity shiptocountry + shiptocontact shiptophone shiptofax shiptoemail); - $ref = $sth->fetchrow_hashref(NAME_lc); - undef($ref->{id}); + $query = qq|SELECT * FROM shipto WHERE trans_id = ? AND module = 'CT'|; + $ref = selectfirst_hashref_query($form, $dbh, $query, $cid); + delete $ref->{id}; map { $form->{$_} = $ref->{$_} } keys %$ref; - $sth->finish; } - # get taxes we charge for this customer - $query = qq|SELECT c.accno - FROM chart c - JOIN customertax ct ON (ct.chart_id = c.id) - WHERE ct.customer_id = $form->{customer_id}|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); - - my $customertax = (); - while ($ref = $sth->fetchrow_hashref(NAME_lc)) { - $customertax{ $ref->{accno} } = 1; - } - $sth->finish; - # setup last accounts used for this customer if (!$form->{id} && $form->{type} !~ /_(order|quotation)/) { - $query = qq|SELECT c.id, c.accno, c.description, c.link, c.category - FROM chart c - JOIN acc_trans ac ON (ac.chart_id = c.id) - JOIN ar a ON (a.id = ac.trans_id) - WHERE a.customer_id = $form->{customer_id} - AND NOT (c.link LIKE '%_tax%' OR c.link LIKE '%_paid%') - AND a.id IN (SELECT max(a2.id) FROM ar a2 - WHERE a2.customer_id = $form->{customer_id})|; - $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + $query = + qq|SELECT c.id, c.accno, c.description, c.link, c.category + FROM chart c + JOIN acc_trans ac ON (ac.chart_id = c.id) + JOIN ar a ON (a.id = ac.trans_id) + WHERE a.customer_id = ? + AND NOT (c.link LIKE '%_tax%' OR c.link LIKE '%_paid%') + AND a.id IN (SELECT max(a2.id) FROM ar a2 WHERE a2.customer_id = ?)|; + $sth = prepare_execute_query($form, $dbh, $query, $cid, $cid); my $i = 0; while ($ref = $sth->fetchrow_hashref(NAME_lc)) { @@ -1802,10 +1712,12 @@ sub get_customer { if ($form->{initial_transdate}) { my $tax_query = - qq|SELECT tk.tax_id, t.rate FROM taxkeys tk | . - qq|LEFT JOIN tax t ON tk.tax_id = t.id | . - qq|WHERE tk.chart_id = ? AND startdate <= ? | . - qq|ORDER BY tk.startdate DESC LIMIT 1|; + qq|SELECT tk.tax_id, t.rate + FROM taxkeys tk + LEFT JOIN tax t ON tk.tax_id = t.id + WHERE (tk.chart_id = ?) AND (startdate <= ?) + ORDER BY tk.startdate DESC + LIMIT 1|; my ($tax_id, $rate) = selectrow_query($form, $dbh, $tax_query, $ref->{id}, $form->{initial_transdate}); @@ -1835,55 +1747,76 @@ sub retrieve_item { my $i = $form->{rowcount}; - my $where = "NOT p.obsolete = '1'"; - - if ($form->{"partnumber_$i"}) { - my $partnumber = $form->like(lc $form->{"partnumber_$i"}); - $where .= " AND lower(p.partnumber) LIKE '$partnumber'"; - } - if ($form->{"description_$i"}) { - my $description = $form->like(lc $form->{"description_$i"}); - $where .= " AND lower(p.description) LIKE '$description'"; - } + my $where = qq|NOT p.obsolete = '1'|; + my @values; - if ($form->{"partsgroup_$i"}) { - my $partsgroup = $form->like(lc $form->{"partsgroup_$i"}); - $where .= " AND lower(pg.partsgroup) LIKE '$partsgroup'"; + foreach my $column (qw(p.partnumber p.description pgpartsgroup)) { + my ($table, $field) = split m/\./, $column; + next if !$form->{"${field}_${i}"}; + $where .= qq| AND lower(${column}) ILIKE ?|; + push @values, '%' . $form->{"${field}_${i}"} . '%'; } if ($form->{"description_$i"}) { - $where .= " ORDER BY p.description"; + $where .= qq| ORDER BY p.description|; } else { - $where .= " ORDER BY p.partnumber"; + $where .= qq| ORDER BY p.partnumber|; } my $transdate; if ($form->{type} eq "invoice") { $transdate = $form->{deliverydate} ? $dbh->quote($form->{deliverydate}) : - $form->{invdate} ? $dbh->quote($form->{invdate}) : - "current_date"; + $form->{invdate} ? $dbh->quote($form->{invdate}) : + "current_date"; } else { $transdate = - $form->{transdate} ? $dbh->quote($form->{transdate}) : - "current_date"; + $form->{transdate} ? $dbh->quote($form->{transdate}) : + "current_date"; } - my $query = qq|SELECT p.id, p.partnumber, p.description, p.sellprice, - p.listprice, p.inventory_accno_id, - c1.accno AS inventory_accno, c1.new_chart_id AS inventory_new_chart, date($transdate) - c1.valid_from as inventory_valid, - c2.accno AS income_accno, c2.new_chart_id AS income_new_chart, date($transdate) - c2.valid_from as income_valid, - c3.accno AS expense_accno, c3.new_chart_id AS expense_new_chart, date($transdate) - c3.valid_from as expense_valid, - p.unit, p.assembly, p.bin, p.onhand, p.notes AS partnotes, p.notes AS longdescription, p.not_discountable, - pg.partsgroup, p.formel, p.payment_id AS part_payment_id - FROM parts p - LEFT JOIN chart c1 ON ((select inventory_accno_id from buchungsgruppen where id=p.buchungsgruppen_id) = c1.id) - LEFT JOIN chart c2 ON ((select income_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c2.id) - LEFT JOIN chart c3 ON ((select expense_accno_id_$form->{taxzone_id} from buchungsgruppen where id=p.buchungsgruppen_id) = c3.id) - LEFT JOIN partsgroup pg ON (pg.id = p.partsgroup_id) - WHERE $where|; - my $sth = $dbh->prepare($query); - $sth->execute || $form->dberror($query); + my $taxzone_id = $form->{taxzone_id} * 1; + $taxzone_id = 0 if (0 > $taxzone_id) || (3 < $taxzone_id); + + my $query = + qq|SELECT + p.id, p.partnumber, p.description, p.sellprice, + p.listprice, p.inventory_accno_id, + + c1.accno AS inventory_accno, + c1.new_chart_id AS inventory_new_chart, + date($transdate) - c1.valid_from AS inventory_valid, + + c2.accno AS income_accno, + c2.new_chart_id AS income_new_chart, + date($transdate) - c2.valid_from AS income_valid, + + c3.accno AS expense_accno, + c3.new_chart_id AS expense_new_chart, + date($transdate) - c3.valid_from AS expense_valid, + + p.unit, p.assembly, p.bin, p.onhand, + p.notes AS partnotes, p.notes AS longdescription, + p.not_discountable, p.formel, p.payment_id AS part_payment_id, + + pg.partsgroup + + FROM parts p + LEFT JOIN chart c1 ON + ((SELECT inventory_accno_id + FROM buchungsgruppen + WHERE id = p.buchungsgruppen_id) = c1.id) + LEFT JOIN chart c2 ON + ((SELECT income_accno_id_${taxzone_id} + FROM buchungsgruppen + WHERE id = p.buchungsgruppen_id) = c2.id) + LEFT JOIN chart c3 ON + ((SELECT expense_accno_id_${taxzone_id} + FROM buchungsgruppen + WHERE id = p.buchungsgruppen_id) = c3.id) + LEFT JOIN partsgroup pg ON (pg.id = p.partsgroup_id) + WHERE $where|; + my $sth = prepare_execute_query($form, $dbh, $query, @values); while (my $ref = $sth->fetchrow_hashref(NAME_lc)) { @@ -1895,54 +1828,53 @@ sub retrieve_item { } delete($ref->{inventory_accno_id}); - #set expense_accno=inventory_accno if they are different => bilanz - - - while ($ref->{inventory_new_chart} && ($ref->{inventory_valid} >=0)) { - my $query = qq| SELECT accno AS inventory_accno, new_chart_id AS inventory_new_chart, date($transdate) - valid_from AS inventory_valid FROM chart WHERE id = $ref->{inventory_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{inventory_accno}, $ref->{inventory_new_chart}, $ref->{inventory_valid}) = $stw->fetchrow_array; - $stw->finish; - } - - while ($ref->{income_new_chart} && ($ref->{income_valid} >=0)) { - my $query = qq| SELECT accno AS income_accno, new_chart_id AS income_new_chart, date($transdate) - valid_from AS income_valid FROM chart WHERE id = $ref->{income_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{income_accno}, $ref->{income_new_chart}, $ref->{income_valid}) = $stw->fetchrow_array; - $stw->finish; - } - - while ($ref->{expense_new_chart} && ($ref->{expense_valid} >=0)) { - my $query = qq| SELECT accno AS expense_accno, new_chart_id AS expense_new_chart, date($transdate) - valid_from AS expense_valid FROM chart WHERE id = $ref->{expense_new_chart}|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($ref->{expense_accno}, $ref->{expense_new_chart}, $ref->{expense_valid}) = $stw->fetchrow_array; - $stw->finish; + foreach my $type (qw(inventory income expense)) { + while ($ref->{"${type}_new_chart"} && ($ref->{"${type}_valid"} >=0)) { + my $query = + qq|SELECT accno, new_chart_id, date($transdate) - valid_from + FROM chart + WHERE id = ?|; + ($ref->{"${type}_accno"}, + $ref->{"${type}_new_chart"}, + $ref->{"${type}_valid"}) + = selectrow_query($form, $dbh, $query, $ref->{"${type}_new_chart"}); + } } #check whether payment_terms are better than old payment_terms - if (($form->{payment_id} ne "") && ($ref->{part_payment_id} ne "")) { - my $query = qq|select (select ranking from payment_terms WHERE id = $form->{payment_id}), (select ranking from payment_terms WHERE id = $ref->{part_payment_id})|; - my $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - ($old_ranking, $new_ranking) = $stw->fetchrow_array; - $stw->finish; - if ($new_ranking <= $old_ranking) { - $ref->{part_payment_id} = ""; + if (($form->{payment_id} ne "") && ($form->{part_payment_id} ne "")) { + $query = + qq|SELECT + (SELECT ranking FROM payment_terms WHERE id = ?), + (SELECT ranking FROM payment_terms WHERE id = ?)|; + my ($old_ranking, $new_ranking) + = selectrow_query($form, $dbh, $query, conv_i($form->{payment_id}), conv_i($form->{part_payment_id})); + if ($new_ranking > $old_ranking) { + $form->{payment_id} = $form->{customer_payment_id}; } } + if ($form->{payment_id} eq "") { + $form->{payment_id} = $form->{part_payment_id}; + } + # get tax rates and description - $accno_id = - ($form->{vc} eq "customer") ? $ref->{income_accno} : $ref->{expense_accno}; - $query = qq|SELECT c.accno, t.taxdescription, t.rate, t.taxnumber - FROM tax t LEFT JOIN chart c on (c.id=t.chart_id) - WHERE t.id in (SELECT tk.tax_id from taxkeys tk where tk.chart_id = (SELECT id from chart WHERE accno='$accno_id') AND startdate<=$transdate ORDER BY startdate desc LIMIT 1) - ORDER BY c.accno|; + $accno_id = ($form->{vc} eq "customer") ? $ref->{income_accno} : $ref->{expense_accno}; + $query = + qq|SELECT c.accno, t.taxdescription, t.rate, t.taxnumber + FROM tax t + LEFT JOIN chart c ON (c.id = t.chart_id) + WHERE t.id in + (SELECT tk.tax_id + FROM taxkeys tk + WHERE tk.chart_id = (SELECT id from chart WHERE accno = ?) + AND startdate <= ? + ORDER BY startdate DESC + LIMIT 1) + ORDER BY c.accno|; + @values = ($accno_id, $transdate); $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); + $stw->execute(@values) || $form->dberror($query); $ref->{taxaccounts} = ""; my $i = 0; @@ -1967,30 +1899,33 @@ sub retrieve_item { $stw->finish; chop $ref->{taxaccounts}; if ($form->{language_id}) { - $query = qq|SELECT tr.translation, tr.longdescription - FROM translation tr - WHERE tr.language_id=$form->{language_id} AND tr.parts_id=$ref->{id}|; - $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - my ($translation, $longdescription) = $stw->fetchrow_array(); + $query = + qq|SELECT tr.translation, tr.longdescription + FROM translation tr + WHERE tr.language_id = ? AND tr.parts_id = ?|; + @values = (conv_i($form->{language_id}), conv_i($ref->{id})); + my ($translation, $longdescription) = selectrow_query($form, $dbh, $query, @values); if ($translation ne "") { $ref->{description} = $translation; $ref->{longdescription} = $longdescription; } else { - $query = qq|SELECT tr.translation, tr.longdescription - FROM translation tr - WHERE tr.language_id in (select id from language where article_code=(select article_code from language where id = $form->{language_id})) AND tr.parts_id=$ref->{id} LIMIT 1|; - $stg = $dbh->prepare($query); - $stg->execute || $form->dberror($query); - my ($translation) = $stg->fetchrow_array(); + $query = + qq|SELECT tr.translation, tr.longdescription + FROM translation tr + WHERE tr.language_id IN + (SELECT id + FROM language + WHERE article_code = (SELECT article_code FROM language WHERE id = ?)) + AND tr.parts_id = ? + LIMIT 1|; + @values = (conv_i($form->{language_id}), conv_i($ref->{id})); + my ($translation, $longdescription) = selectrow_query($form, $dbh, $query, @values); if ($translation ne "") { $ref->{description} = $translation; $ref->{longdescription} = $longdescription; } - $stg->finish; } - $stw->finish; } push @{ $form->{item_list} }, $ref; @@ -1998,10 +1933,11 @@ sub retrieve_item { if ($form->{lizenzen}) { if ($ref->{inventory_accno} > 0) { $query = - qq| SELECT l.* FROM license l WHERE l.parts_id = $ref->{id} AND NOT l.id IN (SELECT li.license_id FROM licenseinvoice li)|; - $stw = $dbh->prepare($query); - $stw->execute || $form->dberror($query); - while ($ptr = $stw->fetchrow_hashref(NAME_lc)) { + qq|SELECT l.* + FROM license l + WHERE l.parts_id = ? AND NOT l.id IN (SELECT li.license_id FROM licenseinvoice li)|; + my $stw = prepare_execute_query($form, $dbh, $query, conv_i($ref->{id})); + while (my $ptr = $stw->fetchrow_hashref(NAME_lc)) { push @{ $form->{LIZENZEN}{ $ref->{id} } }, $ptr; } $stw->finish; @@ -2045,27 +1981,23 @@ sub get_pricegroups_for_parts { $id = $form->{"new_id_$i"}; } - ($price, $selectedpricegroup_id) = split /--/, - $form->{"sellprice_pg_$i"}; + ($price, $selectedpricegroup_id) = split(/--/, + $form->{"sellprice_pg_$i"}); $pricegroup_old = $form->{"pricegroup_old_$i"}; $form->{"new_pricegroup_$i"} = $selectedpricegroup_id; $form->{"old_pricegroup_$i"} = $pricegroup_old; - $price_new = $form->{"price_new_$i"}; + $price_new = $form->{"price_new_$i"}; $price_old = $form->{"price_old_$i"}; - $query = - qq|SELECT pricegroup_id, (SELECT p.sellprice from parts p where p.id = $id) as default_sellprice,(SELECT pg.pricegroup FROM pricegroup pg WHERE id=pricegroup_id) AS pricegroup, price, '' AS selected FROM prices WHERE parts_id = $id UNION SELECT 0 as pricegroup_id,(SELECT sellprice FROM parts WHERE id=$id) as default_sellprice,'' as pricegroup, (SELECT DISTINCT sellprice from parts where id=$id) as price, 'selected' AS selected from prices ORDER BY pricegroup|; - $pkq = $dbh->prepare($query); - $pkq->execute || $form->dberror($query); if (!$form->{"unit_old_$i"}) { # Neue Ware aus der Datenbank. In diesem Fall ist unit_$i die # Einheit, wie sie in den Stammdaten hinterlegt wurde. # Es sollte also angenommen werden, dass diese ausgewaehlt war. $form->{"unit_old_$i"} = $form->{"unit_$i"}; } - + # Die zuletzt ausgewaehlte mit der aktuell ausgewaehlten Einheit # vergleichen und bei Unterschied den Preis entsprechend umrechnen. $form->{"selected_unit_$i"} = $form->{"unit_$i"} unless ($form->{"selected_unit_$i"}); @@ -2079,8 +2011,9 @@ sub get_pricegroups_for_parts { # um eine Dienstleistung). Dann keinerlei Umrechnung vornehmen. $form->{"unit_old_$i"} = $form->{"selected_unit_$i"} = $form->{"unit_$i"}; } + my $basefactor = 1; - + if ($form->{"unit_old_$i"} ne $form->{"selected_unit_$i"}) { if (defined($all_units->{$form->{"unit_old_$i"}}->{"factor"}) && $all_units->{$form->{"unit_old_$i"}}->{"factor"}) { @@ -2088,13 +2021,36 @@ sub get_pricegroups_for_parts { $all_units->{$form->{"unit_old_$i"}}->{"factor"}; } } + if (!$form->{"basefactor_$i"}) { $form->{"basefactor_$i"} = 1; } - while ($pkr = $pkq->fetchrow_hashref(NAME_lc)) { - # push @{ $form->{PRICES}{$id} }, $pkr; - #push @{ $form->{PRICES}{$i} }, $pkr; + $query = + qq|SELECT + pricegroup_id, + (SELECT p.sellprice FROM parts p WHERE p.id = ?) AS default_sellprice, + (SELECT pg.pricegroup FROM pricegroup pg WHERE id = pricegroup_id) AS pricegroup, + price, + '' AS selected + FROM prices + WHERE parts_id = ? + + UNION + + SELECT + 0 as pricegroup_id, + (SELECT sellprice FROM parts WHERE id = ?) AS default_sellprice, + '' AS pricegroup, + (SELECT DISTINCT sellprice FROM parts where id = ?) AS price, + 'selected' AS selected + FROM prices + + ORDER BY pricegroup|; + @values = (conv_i($id), conv_i($id), conv_i($id), conv_i($id)); + my $pkq = prepare_execute_query($form, $dbh, $query, @values); + + while ($pkr = $pkq->fetchrow_hashref(NAME_lc)) { $pkr->{id} = $id; $pkr->{selected} = ''; @@ -2105,9 +2061,9 @@ sub get_pricegroups_for_parts { } $pkr->{price} *= $form->{"basefactor_$i"}; - + $pkr->{price} *= $basefactor; - + $pkr->{price} = $form->format_amount($myconfig, $pkr->{price}, 5); if ($selectedpricegroup_id eq undef) { @@ -2125,12 +2081,9 @@ sub get_pricegroups_for_parts { $form->{"sellprice_$i"} = $pkr->{price}; } - } else { - if ($pkr->{price} == $pkr->{default_sellprice}) { - - $pkr->{price} = $form->{"sellprice_$i"}; - $pkr->{selected} = ' selected'; - } + } elsif ($pkr->{price} == $pkr->{default_sellprice}) { + $pkr->{price} = $form->{"sellprice_$i"}; + $pkr->{selected} = ' selected'; } } @@ -2139,23 +2092,18 @@ sub get_pricegroups_for_parts { if ($pkr->{pricegroup_id} eq $selectedpricegroup_id) { $pkr->{selected} = ' selected'; } - } else { - if (($price_new != $form->{"sellprice_$i"}) and ($price_new ne 0)) { - if ($pkr->{pricegroup_id} == 0) { - $pkr->{price} = $form->{"sellprice_$i"}; - $pkr->{selected} = ' selected'; - } + } elsif (($price_new != $form->{"sellprice_$i"}) and ($price_new ne 0)) { + if ($pkr->{pricegroup_id} == 0) { + $pkr->{price} = $form->{"sellprice_$i"}; + $pkr->{selected} = ' selected'; + } + } elsif ($pkr->{pricegroup_id} eq $selectedpricegroup_id) { + $pkr->{selected} = ' selected'; + if ( ($pkr->{pricegroup_id} == 0) + and ($pkr->{price} == $form->{"sellprice_$i"})) { + # $pkr->{price} = $form->{"sellprice_$i"}; } else { - if ($pkr->{pricegroup_id} eq $selectedpricegroup_id) { - $pkr->{selected} = ' selected'; - if ( ($pkr->{pricegroup_id} == 0) - and ($pkr->{price} == $form->{"sellprice_$i"})) { - - # $pkr->{price} = $form->{"sellprice_$i"}; - } else { - $pkr->{price} = $form->{"sellprice_$i"}; - } - } + $pkr->{price} = $form->{"sellprice_$i"}; } } } @@ -2198,4 +2146,3 @@ sub has_storno { } 1; - -- 2.20.1