X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FVK.pm;h=f5fb03f21e0d5c59a16108b1ed7a46a70b0eb452;hb=3f90c9cac7fea1568957718eba46dfe5f9a621ce;hp=2c0ce67259df4bdea355ef5182eca6445f7d9b9c;hpb=409d5d5fbf7020d69c7062cdde11040dc755ee38;p=kivitendo-erp.git diff --git a/SL/VK.pm b/SL/VK.pm index 2c0ce6725..f5fb03f21 100644 --- a/SL/VK.pm +++ b/SL/VK.pm @@ -55,7 +55,8 @@ sub invoice_transactions { # so we extract both versions in our query and later overwrite the description in article mode my $query = - qq|SELECT ct.id as customerid, ct.name as customername,ct.customernumber,ct.country,ar.invnumber,ar.id,ar.transdate,p.partnumber,p.description as description, pg.partsgroup,i.parts_id,i.qty,i.price_factor,i.discount,i.description as invoice_description,i.lastcost,i.sellprice,i.fxsellprice,i.marge_total,i.marge_percent,i.unit,b.description as business,e.name as employee,e2.name as salesman, to_char(ar.transdate,'Month') as month, to_char(ar.transdate, 'YYYYMM') as nummonth, p.unit as parts_unit, p.weight | . + qq|SELECT ct.id as customerid, ct.name as customername,ct.customernumber,ct.country,ar.invnumber,ar.id,ar.transdate,p.partnumber,p.description as description, pg.partsgroup,i.parts_id,i.qty,i.price_factor,i.discount,i.description as invoice_description,i.lastcost,i.sellprice,i.fxsellprice,i.marge_total,i.marge_percent,i.unit,b.description as business,e.name as employee,e2.name as salesman, to_char(ar.transdate,'Month') as month, to_char(ar.transdate, 'YYYYMM') as nummonth, p.unit as parts_unit, p.weight, ar.taxincluded | . + qq|, COALESCE((SELECT e.buy FROM exchangerate e WHERE e.transdate = ar.transdate and ar.currency_id = e.currency_id),1) as exchangerate | . qq|FROM invoice i | . qq|JOIN ar on (i.trans_id = ar.id) | . qq|JOIN parts p on (i.parts_id = p.id) | . @@ -71,7 +72,7 @@ sub invoice_transactions { # limits by employees (Bearbeiter), not salesmen! if (!$main::auth->assert('sales_all_edit', 1)) { $where .= " AND ar.employee_id = (select id from employee where login= ?)"; - push (@values, $form->{login}); + push (@values, $::myconfig{login}); } # Stornierte Rechnungen und Stornorechnungen in invoice rausfiltern @@ -109,14 +110,17 @@ sub invoice_transactions { if ($form->{customer_id}) { $where .= " AND ar.customer_id = ?"; push(@values, $form->{customer_id}); - }; + } elsif ($form->{customer}) { + $where .= " AND ct.name ILIKE ?"; + push(@values, like($form->{customer})); + } if ($form->{customernumber}) { $where .= qq| AND ct.customernumber = ? |; push(@values, $form->{customernumber}); } if ($form->{partnumber}) { $where .= qq| AND (p.partnumber ILIKE ?)|; - push(@values, '%' . $form->{partnumber} . '%'); + push(@values, like($form->{partnumber})); } if ($form->{partsgroup_id}) { $where .= qq| AND (pg.id = ?)|; @@ -124,7 +128,7 @@ sub invoice_transactions { } if ($form->{country}) { $where .= qq| AND (ct.country ILIKE ?)|; - push(@values, '%' . $form->{country} . '%'); + push(@values, like($form->{country})); } # when filtering for parts by description we probably want to filter by the description of the part as per the master data @@ -132,7 +136,7 @@ sub invoice_transactions { # at least in the translation case we probably want the report to also include translated articles, so we have to filter via parts.description if ($form->{description}) { $where .= qq| AND (p.description ILIKE ?)|; - push(@values, '%' . $form->{description} . '%'); + push(@values, like($form->{description})); } if ($form->{transdatefrom}) { $where .= " AND ar.transdate >= ?";