X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=bin%2Fmozilla%2Fdo.pl;h=54c650c313e5c78f4ab3ee0ee894113abc578859;hb=cb25314000db992f8b413d10a0d20f0e80b4b677;hp=c69052bd941dee713a8d9a880c1e38d01545ad38;hpb=381130c09175ee73d82737dd1a760fda5d47c7f3;p=kivitendo-erp.git diff --git a/bin/mozilla/do.pl b/bin/mozilla/do.pl index c69052bd9..54c650c31 100644 --- a/bin/mozilla/do.pl +++ b/bin/mozilla/do.pl @@ -34,6 +34,7 @@ use List::Util qw(max sum); use POSIX qw(strftime); use YAML; +use SL::DB::DeliveryOrder; use SL::DO; use SL::IR; use SL::IS; @@ -48,8 +49,6 @@ require "bin/mozilla/reportgenerator.pl"; use strict; -my $print_post; - 1; # end of main @@ -167,7 +166,7 @@ sub order_links { $form->all_vc(\%myconfig, $form->{vc}, ($form->{vc} eq 'customer') ? "AR" : "AP"); # retrieve order/quotation - $form->{webdav} = $main::webdav; + $form->{webdav} = $::lx_office_conf{features}->{webdav}; $form->{jsscript} = 1; my $editing = $form->{id}; @@ -181,8 +180,10 @@ sub order_links { # get customer / vendor if ($form->{vc} eq 'vendor') { IR->get_vendor(\%myconfig, \%$form); + $form->{discount} = $form->{vendor_discount}; } else { IS->get_customer(\%myconfig, \%$form); + $form->{discount} = $form->{customer_discount}; } $form->restore_vars(qw(payment_id language_id taxzone_id intnotes cp_id)); @@ -233,6 +234,7 @@ sub prepare_order { $form->{"reqdate_$i"} = $form->{"deliverydate_$i"} unless $form->{"reqdate_$i"}; $form->{"sellprice_$i"} = $form->format_amount(\%myconfig, $form->{"sellprice_$i"}, $decimalplaces); + $form->{"lastcost_$i"} = $form->format_amount(\%myconfig, $form->{"lastcost_$i"}, $decimalplaces); (my $dec_qty) = ($form->{"qty_$i"} =~ /\.(\d+)/); $dec_qty = length $dec_qty; @@ -336,7 +338,19 @@ sub update_delivery_order { $payment_id = $form->{payment_id} if $form->{payment_id}; check_name($form->{vc}); - + $form->{discount} = $form->{"$form->{vc}_discount"} if defined $form->{"$form->{vc}_discount"}; + # Problem: Wenn man ohne Erneuern einen Kunden/Lieferanten + # wechselt, wird der entsprechende Kunden/ Lieferantenrabatt + # nicht übernommen. Grundproblem: In Commit 82574e78 + # hab ich aus discount customer_discount und vendor_discount + # gemacht und entsprechend an den Oberflächen richtig hin- + # geschoben. Die damals bessere Lösung wäre gewesen: + # In den Templates nur die hidden für form-discount wieder ein- + # setzen dann wäre die Verrenkung jetzt nicht notwendig. + # TODO: Ggf. Bugfix 1284, 1575 und 817 wieder zusammenführen + # Testfälle: Kunden mit Rabatt 0 -> Rabatt 20 i.O. + # Kunde mit Rabatt 20 -> Rabatt 0 i.O. + # Kunde mit Rabatt 20 -> Rabatt 5,5 i.O. $form->{payment_id} = $payment_id if $form->{payment_id} eq ""; # for pricegroups @@ -350,21 +364,24 @@ sub update_delivery_order { } else { + my $mode; if ($form->{type} eq 'purchase_delivery_order') { IR->retrieve_item(\%myconfig, $form); + $mode = 'IR'; } else { IS->retrieve_item(\%myconfig, $form); + $mode = 'IS'; } my $rows = scalar @{ $form->{item_list} }; if ($rows) { - $form->{"qty_$i"} = 1 unless ($form->{"qty_$i"}); + $form->{"qty_$i"} = 1 unless $form->parse_amount(\%myconfig, $form->{"qty_$i"}); if ($rows > 1) { - select_item(); - exit; + select_item(mode => $mode); + ::end_of_request(); } else { @@ -372,6 +389,7 @@ sub update_delivery_order { $form->{"marge_price_factor_$i"} = $form->{item_list}->[0]->{price_factor}; $form->{"sellprice_$i"} = $form->format_amount(\%myconfig, $form->{"sellprice_$i"}); + $form->{"lastcost_$i"} = $form->format_amount(\%myconfig, $form->{"lastcost_$i"}); $form->{"qty_$i"} = $form->format_amount(\%myconfig, $form->{"qty_$i"}); } @@ -387,6 +405,7 @@ sub update_delivery_order { && ($form->{"description_$i"} eq "")) { $form->{rowcount}--; $form->{"discount_$i"} = ""; + $form->{"not_discountable_$i"} = ""; display_form(); } else { @@ -466,7 +485,7 @@ sub orders { my @hidden_variables = map { "l_${_}" } @columns; push @hidden_variables, $form->{vc}, qw(l_closed l_notdelivered open closed delivered notdelivered donumber ordnumber - transaction_description transdatefrom transdateto type vc employee_id salesman_id); + transaction_description transdatefrom transdateto type vc employee_id salesman_id project_id); my $href = build_std_url('action=orders', grep { $form->{$_} } @hidden_variables); @@ -477,7 +496,7 @@ sub orders { 'donumber' => { 'text' => $locale->text('Delivery Order'), }, 'ordnumber' => { 'text' => $locale->text('Order'), }, 'name' => { 'text' => $form->{vc} eq 'customer' ? $locale->text('Customer') : $locale->text('Vendor'), }, - 'employee' => { 'text' => $locale->text('Salesperson'), }, + 'employee' => { 'text' => $locale->text('Employee'), }, 'shipvia' => { 'text' => $locale->text('Ship via'), }, 'globalprojectnumber' => { 'text' => $locale->text('Project Number'), }, 'transaction_description' => { 'text' => $locale->text('Transaction description'), }, @@ -549,6 +568,7 @@ sub orders { 'attachment_basename' => $attachment_basename . strftime('_%Y%m%d', localtime time), ); $report->set_options_from_form(); + $locale->set_numberformat_wo_thousands_separator(\%myconfig) if lc($report->{options}->{output_format}) eq 'csv'; # add sort and escape callback, this one we use for the add sub $form->{callback} = $href .= "&sort=$form->{sort}"; @@ -590,6 +610,8 @@ sub orders { sub save { $main::lxdebug->enter_sub(); + my (%params) = @_; + check_do_access(); my $form = $main::form; @@ -609,31 +631,41 @@ sub save { # $locale->text('Customer missing!'); # $locale->text('Vendor missing!'); + remove_emptied_rows(); validate_items(); # if the name changed get new values if (check_name($form->{vc})) { update(); - exit; + ::end_of_request(); } $form->{id} = 0 if $form->{saveasnew}; - + # best case fix für bug 1079. Einkaufsrabatt wird nicht richtig + # aus Lieferantenauftrag -> Lieferschein -> Rechnung übernommen + # Tritt nur auf, wenn man direkt über Lieferschein -> speichern -> + # Workflow Rechnung geht (beim Aufruf über edit() i.O.) + # Gut. DO-save() speichert den Discount im DB-Format 0.12 für + # 12%, die Konvertierung wird leider in $form gemacht und daher + # wird die Maske mit dem falschen Rabatt wieder aufgebaut. + # Wie immer: backup_vars verwenden um nichts anderes kaputt zu + # machen. jan 03.03.2010 + # nicht mehr notwendig da für bug 1284 der backend aufruf entsprechend + # geändert wurde DO->save(); - # saving the history if(!exists $form->{addition}) { $form->{snumbers} = qq|donumber_| . $form->{donumber}; - $form->{addition} = "SAVED"; - $form->save_history($form->dbconnect(\%myconfig)); + $form->{addition} = "SAVED"; + $form->save_history; } # /saving the history $form->{simple_save} = 1; - if(!$form->{print_and_save}) { + if (!$params{no_redirect} && !$form->{print_and_save}) { set_headings("edit"); update(); - exit; + ::end_of_request(); } $main::lxdebug->leave_sub(); } @@ -670,13 +702,13 @@ sub delete_delivery_order { # saving the history if(!exists $form->{addition}) { $form->{snumbers} = qq|donumber_| . $form->{donumber}; - $form->{addition} = "DELETED"; - $form->save_history($form->dbconnect(\%myconfig)); + $form->{addition} = "DELETED"; + $form->save_history; } # /saving the history $form->info($locale->text('Delivery Order deleted!')); - exit(); + ::end_of_request(); } $form->error($locale->text('Cannot delete delivery order!')); @@ -719,19 +751,38 @@ sub invoice { } for my $i (1 .. $form->{rowcount}) { - map { $form->{"${_}_${i}"} = $form->parse_amount(\%myconfig, $form->{"${_}_${i}"}) if $form->{"${_}_${i}"} } qw(ship qty sellprice listprice basefactor); + # für bug 1284 + unless ($form->{"ordnumber"}) { + if ($form->{discount}) { # Falls wir einen Lieferanten-/Kundenrabatt haben + # und rabattfähig sind, dann + unless ($form->{"not_discountable_$i"}) { + $form->{"discount_$i"} = $form->{discount}*100; # ... nehmen wir diesen Rabatt + } + } + } + map { $form->{"${_}_${i}"} = $form->parse_amount(\%myconfig, $form->{"${_}_${i}"}) if $form->{"${_}_${i}"} } qw(ship qty sellprice listprice lastcost basefactor); } $form->{type} = "invoice"; # locale messages - $locale = new Locale "$myconfig{countrycode}", "$script"; + $main::locale = new Locale "$myconfig{countrycode}", "$script"; + $locale = $main::locale; require "bin/mozilla/$form->{script}"; my $currency = $form->{currency}; invoice_links(); + if ($form->{ordnumber}) { + require SL::DB::Order; + if (my $order = SL::DB::Manager::Order->find_by(ordnumber => $form->{ordnumber})) { + $order->load; + $form->{orddate} = $order->transdate_as_date; + $form->{$_} = $order->$_ for qw(payment_id salesman_id taxzone_id quonumber); + } + } + $form->{currency} = $currency; $form->{exchangerate} = ""; $form->{forex} = $form->check_exchangerate(\%myconfig, $form->{currency}, $form->{invdate}, $buysell); @@ -751,10 +802,15 @@ sub invoice { $form->{"deliverydate_$i"} = $form->{"reqdate_$i"} unless $form->{"deliverydate_$i"}; + $form->{"sellprice_$i"} = $form->format_amount(\%myconfig, $form->{"sellprice_$i"}, $decimalplaces); + $form->{"lastcost_$i"} = + $form->format_amount(\%myconfig, $form->{"lastcost_$i"}, + $decimalplaces); + (my $dec_qty) = ($form->{"qty_$i"} =~ /\.(\d+)/); $dec_qty = length $dec_qty; $form->{"qty_$i"} = @@ -798,7 +854,7 @@ sub invoice_multi { # zwischenspeichern (DO.pm) und als ' '-separierte Liste wieder zurueckschreiben # Hinweis: delete gibt den wert zurueck und loescht danach das element (nett und einfach) # $shell: perldoc perlunc; /delete EXPR - $form->{donumber} = delete $form->{donumber_array}; + $form->{donumber} = delete $form->{donumber_array}; $form->{deliverydate} = $form->{transdate}; $form->{transdate} = $form->current_date(\%myconfig); $form->{duedate} = $form->current_date(\%myconfig, $form->{invdate}, $form->{terms} * 1); @@ -822,15 +878,35 @@ sub invoice_multi { map { delete $form->{$_} } qw(id subject message cc bcc printed emailed queued); + # get vendor or customer discount + my $vc_discount; + my $saved_form = save_form(); + if ($form->{vc} eq 'vendor') { + IR->get_vendor(\%myconfig, \%$form); + $vc_discount = $form->{vendor_discount}; + } else { + IS->get_customer(\%myconfig, \%$form); + $vc_discount = $form->{customer_discount}; + } + restore_form($saved_form); + $form->{rowcount} = 0; foreach my $ref (@{ $form->{form_details} }) { $form->{rowcount}++; $ref->{reqdate} ||= $ref->{dord_transdate}; # copy transdates into each invoice row map { $form->{"${_}_$form->{rowcount}"} = $ref->{$_} } keys %{ $ref }; - map { $form->{"${_}_$form->{rowcount}"} = $form->format_amount(\%myconfig, $ref->{$_}) } qw(qty sellprice discount lastcost); - $form->{"discount_$form->{rowcount}"} = $form->{"discount_$form->{rowcount}"} * 100; #s.a. Bug 1151 - # Anm.: Eine Änderung des discounts in der SL/DO.pm->retrieve (select (doi.discount * 100) as discount) ergibt in psql einen - # Wert von 10.0000001490116. Ferner ist der Rabatt in der Rechnung dann bei 1.0 (?). Deswegen lasse ich das hier. jb 10.10.09 + map { $form->{"${_}_$form->{rowcount}"} = $form->format_amount(\%myconfig, $ref->{$_}) } qw(qty sellprice lastcost); + + if ($vc_discount){ # falls wir einen Lieferanten/Kundenrabatt haben + # und keinen anderen discount wert an $i ... + $form->{"discount_$form->{rowcount}"} ||= $vc_discount; # ... nehmen wir diesen Rabatt + } + + $form->{"discount_$form->{rowcount}"} = $form->{"discount_$form->{rowcount}"} * 100; #s.a. Bug 1151 + # Anm.: Eine Änderung des discounts in der SL/DO.pm->retrieve (select (doi.discount * 100) as discount) ergibt in psql einen + # Wert von 10.0000001490116. Ferner ist der Rabatt in der Rechnung dann bei 1.0 (?). Deswegen lasse ich das hier. jb 10.10.09 + + $form->{"discount_$form->{rowcount}"} = $form->format_amount(\%myconfig, $form->{"discount_$form->{rowcount}"}); } delete $form->{form_details}; @@ -856,6 +932,7 @@ sub save_as_new { $form->{closed} = 0; $form->{delivered} = 0; map { delete $form->{$_} } qw(printed emailed queued); + delete @{ $form }{ grep { m/^stock_(?:in|out)_\d+/ } keys %{ $form } }; # Let Lx-Office assign a new order number if the user hasn't changed the # previous one. If it has been changed manually then use it as-is. @@ -879,8 +956,6 @@ sub e_mail { $form->{print_and_save} = 1; - $print_post = 1; - my $saved_form = save_form(); save(); @@ -916,7 +991,7 @@ sub calculate_stock_in_out { 'amount_unit' => $all_units->{$form->{"partunit_$i"}}->{base_unit}, 'conv_units' => 'convertible_not_smaller', 'max_places' => 2); - $content .= qq| |; + $content = qq|${content} |; $main::lxdebug->leave_sub(); @@ -985,7 +1060,7 @@ sub update_stock_in { foreach my $i (1..$form->{rowcount}) { $form->{"qty_$i"} = $form->parse_amount(\%myconfig, $form->{"qty_$i"}); - push @{ $stock_info }, { map { $_ => $form->{"${_}_${i}"} } qw(warehouse_id bin_id chargenumber qty unit) }; + push @{ $stock_info }, { map { $_ => $form->{"${_}_${i}"} } qw(warehouse_id bin_id chargenumber bestbefore qty unit) }; } display_stock_in_form($stock_info); @@ -1014,12 +1089,15 @@ sub display_stock_in_form { my %myconfig = %main::myconfig; my $locale = $main::locale; + $form->{jsscript} = 1; + $form->{title} = $locale->text('Stock'); my $part_info = IC->get_basic_part_info('id' => $form->{parts_id}); my $units = AM->retrieve_units(\%myconfig, $form); - my $units_data = AM->unit_select_data($units, undef, undef, $part_info->{unit}); + # der zweite Parameter von unit_select_data gibt den default-Namen (selected) vor + my $units_data = AM->unit_select_data($units, $form->{do_unit}, undef, $part_info->{unit}); $form->get_lists('warehouses' => { 'key' => 'WAREHOUSES', 'bins' => 'BINS' }); @@ -1036,6 +1114,18 @@ sub display_stock_in_form { $main::lxdebug->leave_sub(); } +sub _stock_in_out_set_qty_display { + my $stock_info = shift; + my $form = $::form; + my $all_units = AM->retrieve_all_units(); + my $sum = AM->sum_with_unit(map { $_->{qty}, $_->{unit} } @{ $stock_info }); + $form->{qty_display} = $form->format_amount_units(amount => $sum * 1, + part_unit => $form->{partunit}, + amount_unit => $all_units->{ $form->{partunit} }->{base_unit}, + conv_units => 'convertible_not_smaller', + max_places => 2); +} + sub set_stock_in { $main::lxdebug->enter_sub(); @@ -1049,11 +1139,13 @@ sub set_stock_in { next if ($form->{"qty_$i"} <= 0); - push @{ $stock_info }, { map { $_ => $form->{"${_}_${i}"} } qw(warehouse_id bin_id chargenumber qty unit) }; + push @{ $stock_info }, { map { $_ => $form->{"${_}_${i}"} } qw(warehouse_id bin_id chargenumber bestbefore qty unit) }; } $form->{stock} = YAML::Dump($stock_info); + _stock_in_out_set_qty_display($stock_info); + $form->header(); print $form->parse_html_template('do/set_stock_in_out'); @@ -1088,7 +1180,8 @@ sub stock_out_form { foreach my $sinfo (@{ $stock_info }) { next if (($row->{bin_id} != $sinfo->{bin_id}) || ($row->{warehouse_id} != $sinfo->{warehouse_id}) || - ($row->{chargenumber} ne $sinfo->{chargenumber})); + ($row->{chargenumber} ne $sinfo->{chargenumber}) || + ($row->{bestbefore} ne $sinfo->{bestbefore})); map { $row->{"stock_$_"} = $sinfo->{$_} } qw(qty unit error); } @@ -1128,6 +1221,7 @@ sub set_stock_out { 'warehouse_id' => $form->{"warehouse_id_$i"}, 'bin_id' => $form->{"bin_id_$i"}, 'chargenumber' => $form->{"chargenumber_$i"}, + 'bestbefore' => $form->{"bestbefore_$i"}, 'qty' => $form->{"qty_$i"}, 'unit' => $form->{"unit_$i"}, 'row' => $i, @@ -1145,6 +1239,8 @@ sub set_stock_out { stock_in_out_form(); } else { + _stock_in_out_set_qty_display($stock_info); + $form->header(); print $form->parse_html_template('do/set_stock_in_out'); } @@ -1159,10 +1255,12 @@ sub transfer_in { my %myconfig = %main::myconfig; my $locale = $main::locale; - if (DO->is_marked_as_delivered('id' => $form->{id})) { + if ($form->{id} && DO->is_marked_as_delivered(id => $form->{id})) { $form->show_generic_error($locale->text('The parts for this delivery order have already been transferred in.'), 'back_button' => 1); } + save(no_redirect => 1); + my @part_ids = map { $form->{"id_${_}"} } grep { $form->{"id_${_}"} && $form->{"stock_in_${_}"} } (1 .. $form->{rowcount}); my @all_requests; @@ -1190,28 +1288,30 @@ sub transfer_in { my $do_base_qty = $form->parse_amount(\%myconfig, $form->{"qty_$i"}) * $units->{$form->{"unit_$i"}}->{factor} / $base_unit_factor; - if ($do_base_qty != $row_sum_base_qty) { - push @{ $form->{ERRORS} }, $locale->text('Error in position #1: You must either assign no stock at all or the full quantity of #2 #3.', - $i, $form->{"qty_$i"}, $form->{"unit_$i"}); - } +# if ($do_base_qty != $row_sum_base_qty) { +# push @{ $form->{ERRORS} }, $locale->text('Error in position #1: You must either assign no stock at all or the full quantity of #2 #3.', +# $i, $form->{"qty_$i"}, $form->{"unit_$i"}); +# } } if (@{ $form->{ERRORS} }) { push @{ $form->{ERRORS} }, $locale->text('The delivery order has not been marked as delivered. The warehouse contents have not changed.'); + set_headings('edit'); update(); $main::lxdebug->leave_sub(); - exit 0; + ::end_of_request(); } } DO->transfer_in_out('direction' => 'in', 'requests' => \@all_requests); - $form->{delivered} = 1; + SL::DB::DeliveryOrder->new(id => $form->{id})->load->update_attributes(delivered => 1); - save(); + $form->{callback} = 'do.pl?action=edit&type=purchase_delivery_order&id=' . $form->escape($form->{id}); + $form->redirect; $main::lxdebug->leave_sub(); } @@ -1223,10 +1323,12 @@ sub transfer_out { my %myconfig = %main::myconfig; my $locale = $main::locale; - if (DO->is_marked_as_delivered('id' => $form->{id})) { + if ($form->{id} && DO->is_marked_as_delivered(id => $form->{id})) { $form->show_generic_error($locale->text('The parts for this delivery order have already been transferred out.'), 'back_button' => 1); } + save(no_redirect => 1); + my @part_ids = map { $form->{"id_${_}"} } grep { $form->{"id_${_}"} && $form->{"stock_out_${_}"} } (1 .. $form->{rowcount}); my @all_requests; @@ -1246,8 +1348,9 @@ sub transfer_out { foreach my $request (@{ DO->unpack_stock_information('packed' => $form->{"stock_out_$i"}) }) { $request->{parts_id} = $form->{"id_$i"}; $request->{base_qty} = $request->{qty} * $units->{$request->{unit}}->{factor} / $base_unit_factor; + $request->{project_id} = $form->{"project_id_$i"} ? $form->{"project_id_$i"} : $form->{globalproject_id}; - my $map_key = join '--', ($form->{"id_$i"}, @{$request}{qw(warehouse_id bin_id chargenumber)}); + my $map_key = join '--', ($form->{"id_$i"}, @{$request}{qw(warehouse_id bin_id chargenumber bestbefore)}); $request_map{$map_key} ||= $request; $request_map{$map_key}->{sum_base_qty} ||= 0; @@ -1259,12 +1362,12 @@ sub transfer_out { next if (0 == $row_sum_base_qty); - my $do_base_qty = $form->parse_amount(\%myconfig, $form->{"qty_$i"}) * $units->{$form->{"unit_$i"}}->{factor} / $base_unit_factor; + my $do_base_qty = $form->{"qty_$i"} * $units->{$form->{"unit_$i"}}->{factor} / $base_unit_factor; - if ($do_base_qty != $row_sum_base_qty) { - push @{ $form->{ERRORS} }, $locale->text('Error in position #1: You must either assign no transfer at all or the full quantity of #2 #3.', - $i, $form->{"qty_$i"}, $form->{"unit_$i"}); - } +# if ($do_base_qty != $row_sum_base_qty) { +# push @{ $form->{ERRORS} }, $locale->text('Error in position #1: You must either assign no transfer at all or the full quantity of #2 #3.', +# $i, $form->{"qty_$i"}, $form->{"unit_$i"}); +# } } if (%request_map) { @@ -1273,7 +1376,7 @@ sub transfer_out { my @contents = DO->get_item_availability('parts_id' => \@part_ids); foreach my $inv (@contents) { - my $map_key = join '--', @{$inv}{qw(parts_id warehouse_id bin_id chargenumber)}; + my $map_key = join '--', @{$inv}{qw(parts_id warehouse_id bin_id chargenumber bestbefore)}; next unless ($request_map{$map_key}); @@ -1287,31 +1390,46 @@ sub transfer_out { my $pinfo = $part_info_map{$request->{parts_id}}; my $binfo = $bin_info_map{$request->{bin_id}}; - push @{ $form->{ERRORS} }, $locale->text("There is not enough available of '#1' at warehouse '#2', bin '#3', #4, for the transfer of #5.", - $pinfo->{description}, $binfo->{warehouse_description}, $binfo->{bin_description}, - $request->{chargenumber} ? $locale->text('chargenumber #1', $request->{chargenumber}) : $locale->text('no chargenumber'), - $form->format_amount_units('amount' => $request->{sum_base_qty}, - 'part_unit' => $pinfo->{unit}, - 'conv_units' => 'convertible_not_smaller')); + if ($::lx_office_conf{features}->{show_best_before}) { + push @{ $form->{ERRORS} }, $locale->text("There is not enough available of '#1' at warehouse '#2', bin '#3', #4, #5, for the transfer of #6.", + $pinfo->{description}, + $binfo->{warehouse_description}, + $binfo->{bin_description}, + $request->{chargenumber} ? $locale->text('chargenumber #1', $request->{chargenumber}) : $locale->text('no chargenumber'), + $request->{bestbefore} ? $locale->text('bestbefore #1', $request->{bestbefore}) : $locale->text('no bestbefore'), + $form->format_amount_units('amount' => $request->{sum_base_qty}, + 'part_unit' => $pinfo->{unit}, + 'conv_units' => 'convertible_not_smaller')); + } else { + push @{ $form->{ERRORS} }, $locale->text("There is not enough available of '#1' at warehouse '#2', bin '#3', #4, for the transfer of #5.", + $pinfo->{description}, + $binfo->{warehouse_description}, + $binfo->{bin_description}, + $request->{chargenumber} ? $locale->text('chargenumber #1', $request->{chargenumber}) : $locale->text('no chargenumber'), + $form->format_amount_units('amount' => $request->{sum_base_qty}, + 'part_unit' => $pinfo->{unit}, + 'conv_units' => 'convertible_not_smaller')); + } } } if (@{ $form->{ERRORS} }) { push @{ $form->{ERRORS} }, $locale->text('The delivery order has not been marked as delivered. The warehouse contents have not changed.'); + set_headings('edit'); update(); $main::lxdebug->leave_sub(); - exit 0; + ::end_of_request(); } } - DO->transfer_in_out('direction' => 'out', 'requests' => \@all_requests); - $form->{delivered} = 1; + SL::DB::DeliveryOrder->new(id => $form->{id})->load->update_attributes(delivered => 1); - save(); + $form->{callback} = 'do.pl?action=edit&type=sales_delivery_order&id=' . $form->escape($form->{id}); + $form->redirect; $main::lxdebug->leave_sub(); }