From 7664f50fac4a789932317d0e0d35e15bc2d7d1c4 Mon Sep 17 00:00:00 2001 From: Moritz Bunkus Date: Tue, 4 Feb 2014 12:49:57 +0100 Subject: [PATCH] SL::DB::Order, DeliveryOrder: Funktionen zum Umwandeln von Order in DeliveryOrder --- SL/DB/DeliveryOrder.pm | 133 +++++++++++++++++++++++++++++++++++++++++ SL/DB/Order.pm | 38 ++++++++++++ 2 files changed, 171 insertions(+) diff --git a/SL/DB/DeliveryOrder.pm b/SL/DB/DeliveryOrder.pm index 372a5cc3a..d84485476 100644 --- a/SL/DB/DeliveryOrder.pm +++ b/SL/DB/DeliveryOrder.pm @@ -74,4 +74,137 @@ sub date { goto &transdate; } +sub new_from { + my ($class, $source, %params) = @_; + + croak("Unsupported source object type '" . ref($source) . "'") unless ref($source) eq 'SL::DB::Order'; + + my $terms = $source->can('payment_id') && $source->payment_id ? $source->payment_terms->terms_netto : 0; + + my %args = ( map({ ( $_ => $source->$_ ) } qw(cp_id currency_id customer_id cusordnumber department_id employee_id globalproject_id intnotes language_id notes + ordnumber reqdate salesman_id shippingpoint shipvia taxincluded taxzone_id transaction_description vendor_id + )), + closed => 0, + is_sales => !!$source->customer_id, + delivered => 0, + terms => $terms, + transdate => DateTime->today_local, + ); + + # Custom shipto addresses (the ones specific to the sales/purchase + # record and not to the customer/vendor) are only linked from + # shipto -> delivery_orders. Meaning delivery_orders.shipto_id + # will not be filled in that case. Therefore we have to return the + # new shipto object as a separate object so that the caller can + # save it, too. + my $custom_shipto; + if (!$source->shipto_id && $source->id) { + require SL::DB::Shipto; + + my $old = SL::DB::Manager::Shipto->find_by(trans_id => $source->id); + if ($old) { + $custom_shipto = SL::DB::Shipto->new( + map { +($_ => $old->$_) } + grep { !m{^ (?: itime | mtime | shipto_id | trans_id ) $}x } + map { $_->name } + @{ $old->meta->columns } + ); + } + + } else { + $args{shipto_id} = $source->shipto_id; + } + + my $delivery_order = $class->new(%args, %params); + + my @items = map { + my $source_item = $_; + SL::DB::DeliveryOrderItem->new(map({ ( $_ => $source_item->$_ ) } + qw(base_qty cusordnumber description discount lastcost longdescription marge_price_factor ordnumber parts_id price_factor price_factor_id + project_id qty reqdate sellprice serialnumber transdate unit + ))); + } @{ $source->items_sorted }; + + $delivery_order->items(\@items); + + return ($delivery_order, $custom_shipto); +} + 1; +__END__ + +=pod + +=encoding utf8 + +=head1 NAME + +SL::DB::DeliveryOrder - Rose model for delivery orders (table +"delivery_orders") + +=head1 FUNCTIONS + +=over 4 + +=item C + +An alias for L for compatibility with other sales/purchase models. + +=item C + +Returns a human-readable description of the state regarding being +closed and delivered. + +=item C + +An alias for L for compatibility with other +sales/purchase models. + +=item C + +Returns the delivery order items sorted by their ID (same order they +appear in the frontend delivery order masks). + +=item C + +Creates a new C instance and copies as much +information from C<$source> as possible. At the moment only instances +of C (sales quotations, sales orders, requests for +quotations and purchase orders) are supported as sources. + +The conversion copies order items into delivery order items. Dates are copied +as appropriate, e.g. the C field will be set to the current date. + +Returns one or two objects depending on the context. In list context +the new delivery order instance and a shipto instance will be +returned. In scalar instance only the delivery order instance is +returned. + +Custom shipto addresses (the ones specific to the sales/purchase +record and not to the customer/vendor) are only linked from C +to C. Meaning C will not +be filled in that case. That's why a separate shipto object is created +and returned. + +The objects returned are not saved. + +=item C + +TODO: Describe sales_order + +=item C + +Returns a stringdescribing this record's type: either +C or C. + +=back + +=head1 BUGS + +Nothing here yet. + +=head1 AUTHOR + +Moritz Bunkus Em.bunkus@linet-services.deE + +=cut diff --git a/SL/DB/Order.pm b/SL/DB/Order.pm index 194431dcd..a3c9de0f5 100644 --- a/SL/DB/Order.pm +++ b/SL/DB/Order.pm @@ -142,6 +142,24 @@ sub convert_to_invoice { return $invoice; } +sub convert_to_delivery_order { + my ($self, %params) = @_; + + my ($delivery_order, $custom_shipto); + if (!$self->db->do_transaction(sub { + require SL::DB::DeliveryOrder; + ($delivery_order, $custom_shipto) = SL::DB::DeliveryOrder->new_from($self); + $delivery_order->save; + $custom_shipto->save if $custom_shipto; + $self->link_to_record($delivery_order); + # die; + })) { + return undef; + } + + return wantarray ? ($delivery_order, $custom_shipto) : $delivery_order; +} + sub number { my $self = shift; @@ -189,6 +207,26 @@ Returns one of the following string types: Returns true if the order is of the given type. +=head2 C + +Creates a new delivery order with C<$self> as the basis by calling +L. That delivery order is saved, and +C<$self> is linked to the new invoice via L. + +The arguments in C<%params> are passed to +L. + +Returns C on failure. Otherwise the return value depends on the +context. In list context the new delivery order and a shipto instance +will be returned. In scalar instance only the delivery order instance +is returned. + +Custom shipto addresses (the ones specific to the sales/purchase +record and not to the customer/vendor) are only linked from C +to C. Meaning C will not +be filled in that case. That's why a separate shipto object is created +and returned. + =head2 C Creates a new invoice with C<$self> as the basis by calling -- 2.20.1