X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FDev%2FInventory.pm;h=5789b49462b9652130ef863c44d924ad4ffdea57;hb=ff159a4d47b9a2d10744dcfc23da2c63605c8a32;hp=d7f10829462e6bcfc0a82a3b5f22d984eddc63d2;hpb=e2db2d12ce284a195237c10d1cb8dc82b7cf26c0;p=kivitendo-erp.git diff --git a/SL/Dev/Inventory.pm b/SL/Dev/Inventory.pm index d7f108294..5789b4946 100644 --- a/SL/Dev/Inventory.pm +++ b/SL/Dev/Inventory.pm @@ -2,16 +2,23 @@ package SL::Dev::Inventory; use strict; use base qw(Exporter); -our @EXPORT = qw(create_warehouse_and_bins set_stock); +our @EXPORT_OK = qw( + create_warehouse_and_bins set_stock transfer_stock + transfer_sales_delivery_order transfer_purchase_delivery_order + transfer_delivery_order_item transfer_in transfer_out +); +our %EXPORT_TAGS = (ALL => \@EXPORT_OK); use SL::DB::Warehouse; use SL::DB::Bin; use SL::DB::Inventory; use SL::DB::TransferType; use SL::DB::Employee; +use SL::DB::DeliveryOrderItemsStock; use SL::WH; use DateTime; use Data::Dumper; +use Carp; sub create_warehouse_and_bins { my (%params) = @_; @@ -28,7 +35,9 @@ sub create_warehouse_and_bins { sub set_stock { my (%params) = @_; - die "param part is missing or not an SL::DB::Part object" unless ref($params{part}) eq 'SL::DB::Part'; + die "param part is missing or not an SL::DB::Part object" + unless ref($params{part}) eq 'SL::DB::Part'; + my $part = delete $params{part}; die "qty is missing" unless $params{qty} or $params{abs_qty}; die "need a bin or default bin" unless $part->warehouse_id or $part->bin_id or $params{bin} or $params{bin_id}; @@ -184,6 +193,154 @@ sub transfer_stock { # return 1; } +sub _transfer { + my (%params) = @_; + + my $transfer_type = delete $params{transfer_type}; + + die "param transfer_type is not a SL::DB::TransferType object: " . Dumper($transfer_type) + unless ref($transfer_type) eq 'SL::DB::TransferType'; + + my $shippingdate = delete $params{shippingdate} // DateTime->today; + + my $part = delete($params{part}) or croak 'part missing'; + my $qty = delete($params{qty}) or croak 'qty missing'; + + # distinguish absolute qty in inventory depending on transfer type direction + $qty *= -1 if $transfer_type->direction eq 'out'; + + # use defaults for unit/wh/bin is they exist and nothing else is specified + my $unit = delete($params{unit}) // $part->unit or croak 'unit missing'; + my $bin = delete($params{bin}) // $part->bin or croak 'bin missing'; + # if bin is given, we don't need a warehouse param + my $wh = $bin->warehouse or croak 'wh missing'; + + WH->transfer({ + parts_id => $part->id, + dst_bin => $bin, + dst_wh => $wh, + qty => $qty, + transfer_type => $transfer_type, + unit => $unit, + comment => delete $params{comment}, + shippingdate => $shippingdate, + }); +} + +sub transfer_in { + my (%params) = @_; + + my $transfer_type = delete $params{transfer_type} // 'stock'; + + my $transfer_type_obj = SL::DB::Manager::TransferType->find_by( + direction => 'in', + description => $transfer_type, + ) or die "Can't find transfer_type with direction in and description " . $params{transfer_type}; + + $params{transfer_type} = $transfer_type_obj; + + _transfer(%params); +} + +sub transfer_out { + my (%params) = @_; + + my $transfer_type = delete $params{transfer_type} // 'shipped'; + + my $transfer_type_obj = SL::DB::Manager::TransferType->find_by( + direction => 'out', + description => $transfer_type, + ) or die "Can't find transfer_type with direction in and description " . $params{transfer_type}; + + $params{transfer_type} = $transfer_type_obj; + + _transfer(%params); +} + +sub transfer_sales_delivery_order { + my ($sales_delivery_order) = @_; + die "first argument must be a sales delivery order Rose DB object" + unless ref($sales_delivery_order) eq 'SL::DB::DeliveryOrder' + and $sales_delivery_order->is_sales; + + die "the delivery order has already been delivered" if $sales_delivery_order->delivered; + + my ($wh, $bin, $trans_type); + + $sales_delivery_order->db->with_transaction(sub { + + foreach my $doi ( @{ $sales_delivery_order->items } ) { + next if $doi->part->is_service or $doi->part->is_assortment; + my $trans_type = SL::DB::Manager::TransferType->find_by(direction => 'out', description => 'shipped'); + transfer_delivery_order_item($doi, $wh, $bin, $trans_type); + }; + $sales_delivery_order->delivered(1); + $sales_delivery_order->save(changes_only=>1); + 1; + }) or die "error while transferring sales_delivery_order: " . $sales_delivery_order->db->error; +}; + +sub transfer_purchase_delivery_order { + my ($purchase_delivery_order) = @_; + die "first argument must be a purchase delivery order Rose DB object" + unless ref($purchase_delivery_order) eq 'SL::DB::DeliveryOrder' + and not $purchase_delivery_order->is_sales; + + my ($wh, $bin, $trans_type); + + $purchase_delivery_order->db->with_transaction(sub { + + foreach my $doi ( @{ $purchase_delivery_order->items } ) { + my $trans_type = SL::DB::Manager::TransferType->find_by(direction => 'in', description => 'stock'); + transfer_delivery_order_item($doi, $wh, $bin, $trans_type); + }; + 1; + }) or die "error while transferring purchase_Delivery_order: " . $purchase_delivery_order->db->error; +}; + +sub transfer_delivery_order_item { + my ($doi, $wh, $bin, $trans_type) = @_; + + unless ( defined $trans_type and ref($trans_type eq 'SL::DB::TransferType') ) { + if ( $doi->record->is_sales ) { + $trans_type //= SL::DB::Manager::TransferType->find_by(direction => 'out', description => 'shipped'); + } else { + $trans_type //= SL::DB::Manager::TransferType->find_by(direction => 'in', description => 'stock'); + } + } + + $bin //= $doi->part->bin; + $wh //= $doi->part->warehouse; + + die "no bin and wh specified and part has no default bin or wh" unless $bin and $wh; + + my $employee = SL::DB::Manager::Employee->current || die "No employee"; + + # dois are converted to base_qty, which is qty + # AM->convert_unit( 'g' => 'kg') * 1000; # 1 + # $doi->unit $doi->part->unit $doi->qty + my $dois = SL::DB::DeliveryOrderItemsStock->new( + delivery_order_item => $doi, + qty => AM->convert_unit($doi->unit => $doi->part->unit) * $doi->qty, + unit => $doi->part->unit, + warehouse_id => $wh->id, + bin_id => $bin->id, + )->save; + + my $inventory = SL::DB::Inventory->new( + parts => $dois->delivery_order_item->part, + qty => $dois->delivery_order_item->record->is_sales ? $dois->qty * -1 : $dois->qty, + oe => $doi->record, + warehouse_id => $dois->warehouse_id, + bin_id => $dois->bin_id, + trans_type_id => $trans_type->id, + delivery_order_items_stock => $dois, + trans_id => $dois->id, + employee_id => $employee->id, + shippingdate => $doi->record->transdate, + )->save; +}; + 1; __END__ @@ -199,14 +356,19 @@ defaults Creates a new warehouse and bins, and immediately saves them. Returns the warehouse and the first bin object. + my ($wh, $bin) = SL::Dev::Inventory::create_warehouse_and_bins(); Create named warehouse with 10 bins: - my ($wh, $bin) = SL::Dev::Inventory::create_warehouse_and_bins(warehouse_description => 'Testlager', - bin_description => 'Testlagerplatz', - number_of_bins => 10, - ); + + my ($wh, $bin) = SL::Dev::Inventory::create_warehouse_and_bins( + warehouse_description => 'Test warehouse', + bin_description => 'Test bin', + number_of_bins => 10, + ); + To access the second bin: + my $bin2 = $wh->bins->[1]; =head2 C @@ -214,55 +376,107 @@ To access the second bin: Change the stock level of a certain part by creating an inventory event. To access the updated onhand the part object needs to be loaded afterwards. -Mandatory params: - part - an SL::DB::Part object or a parts_id - qty | abs_qty - qty : the qty to increase of decrease the stock level by - abs_qty : sets stock level for a certain part to abs_qty by creating - a stock event with the current difference +Parameter: + +=over 4 -Optional params: - bin_id | bin - shippingdate : may be a DateTime object or a string that needs to be parsed by parse_date_to_object. - unit : SL::DB::Unit object, or the name of an SL::DB::Unit object +=item C + +Mandatory. An SL::DB::Part object or a parts_id. + +=item C + +The qty to increase of decrease the stock level by. + +Exactly one of C and C is mandatory. + +=item C + +Sets stock level for a certain part to abs_qty by creating a stock event with +the current difference. + +Exactly one of C and C is mandatory. + +=item C + +=item C + +Optional. The bin for inventory entry. If no bin is passed the default bin of the part is used, if that doesn't exist either there will be an error. +=item C + +Optional. May be a DateTime object or a string that needs to be parsed by +parse_date_to_object. + +=item C + +Optional. SL::DB::Unit object, or the name of an SL::DB::Unit object. + +=back + C creates the SL::DB::Inventory object from scratch, rather than passing params to WH->transfer_in or WH->transfer_out. Examples: + my $part = SL::DB::Manager::Part->find_by(partnumber => '1'); - SL::Dev::Inventory::set_stock(part => $part, qty => 5); + SL::Dev::Inventory::set_stock(part => $part, abs_qty => 5); SL::Dev::Inventory::set_stock(part => $part, qty => -2); $part->load; $part->onhand; # 3 Set stock level of a part in a certain bin_id to 10: + SL::Dev::Inventory::set_stock(part => $part, bin_id => 99, abs_qty => 10); Create 10 warehouses with 5 bins each, then create 100 parts and increase the stock qty in a random bin by a random positive qty for each of the parts: - SL::Dev::Inventory::create_warehouse_and_bins(warehouse_description => "Testlager $_") for ( 1 .. 10 ); - SL::Dev::Part::create_part(description => "Testpart $_")->save for ( 1 .. 100 ); + SL::Dev::Inventory::create_warehouse_and_bins( + warehouse_description => "Test Warehouse $_" + ) for 1 .. 10; + SL::Dev::Part::create_part( + description => "Test Part $_" + )->save for 1 .. 100; my $bins = SL::DB::Manager::Bin->get_all; - SL::Dev::Inventory::set_stock(part => $_, - qty => int(rand(99))+1, - bin => $bins->[ rand @{$bins} ], - ) foreach @{ SL::DB::Manager::Part->get_all() }; + SL::Dev::Inventory::set_stock( + part => $_, + qty => int(rand(99))+1, + bin => $bins->[ rand @{$bins} ], + ) for @{ SL::DB::Manager::Part->get_all }; =head2 C Transfers parts from one bin to another. -Mandatory params: - part | parts_id - an SL::DB::Part object or a parts_id - from_bin - an SL::DB::Bin object - to_bin qty - an SL::DB::Bin object +Parameters: + +=over 4 + +=item C + +=item C + +Mandatory. An SL::DB::Part object or a parts_id. + +=item C + +=item C + +Mandatory. SL::DB::Bin objects. + +=item C + +Mandatory. + +=item C -Optional params: shippingdate +Optional. + +=back The unit is always base_unit and there is no check for negative stock values. @@ -271,15 +485,129 @@ of the stock to a different bin inside the same warehouse: my ($wh, $bin) = SL::Dev::Inventory::create_warehouse_and_bins(); my $part = SL::Dev::Part::create_part->save; - SL::Dev::Inventory::set_stock(part => $part, bin_id => $wh->bins->[2]->id, qty => 5); - SL::Dev::Inventory::transfer_stock(part => $part, - from_bin => $wh->bins->[2], - to_bin => $wh->bins->[4], - qty => 3 - ); + SL::Dev::Inventory::set_stock( + part => $part, + bin_id => $wh->bins->[2]->id, + qty => 5, + ); + SL::Dev::Inventory::transfer_stock( + part => $part, + from_bin => $wh->bins->[2], + to_bin => $wh->bins->[4], + qty => 3, + ); $part->get_stock(bin_id => $wh->bins->[4]->id); # 3.00000 $part->get_stock(bin_id => $wh->bins->[2]->id); # 2.00000 +=head2 C + +Takes a SL::DB::DeliveryOrder object as its first argument and transfers out +all the items via their default bin, creating the delivery_order_stock and +inventory entries. + +Assumes a fresh delivery order where nothing has been transferred out yet. + +Should work like the functions in do.pl transfer_in/transfer_out and DO.pm +transfer_in_out, except that those work on the current form where as this just +works on database objects. + +As this is just Dev it doesn't check for negative stocks etc. + +Usage: + + my $sales_delivery_order = SL::DB::Manager::DeliveryOrder->find_by(donumber => 112); + SL::Dev::Inventory::transfer_sales_delivery_order($sales_delivery_order1); + +=head2 C + +Transfer in all the items in a purchase order. + +Behaves like C. + +=head2 C + +Transfers a delivery order item from a delivery order. The whole qty is transferred. +Doesn't check for available qty. + +Usage: + + SL::Dev::Inventory::transfer_delivery_order_item($doi, $wh, $bin, $trans_type); + +=head2 C + +Create stock in event for a part. Ideally the interface should mirror how data +is entered via the web interface. + +Does some param checking, sets some defaults, but otherwise uses WH->transfer. + +Parameters: + +=over 4 + +=item C + +Mandatory. An SL::DB::Part object. + +=item C + +Mandatory. + +=item C + +Optional. An SL::DB::Bin object, defaults to $part->bin. + +=item C + +Optional. An SL::DB::Bin object, defaults to $part->warehouse. + +=item C + +Optional. A string such as 't', 'Stck', defaults to $part->unit->name. + +=item C + +Optional. A DateTime object, defaults to today. + +=item C + +Optional. A string such as 'correction', defaults to 'stock'. + +=item C + +Optional. + +=back + +Example minimal usage using part default warehouse and bin: + + my ($wh, $bin) = SL::Dev::Inventory::create_warehouse_and_bins(); + my $part = SL::Dev::Part::create_part( + unit => 'kg', + warehouse => $wh, + bin => $bin, + )->save; + SL::Dev::Inventory::transfer_in( + part => $part, + qty => 0.9, + unit => 't', + comment => '900 kg in t', + ); + +Example with specific transfer_type and warehouse and bin and shipping_date: + + my $shipping_date = DateTime->today->subtract( days => 20 ); + SL::Dev::Inventory::transfer_in( + part => $part, + qty => 5, + transfer_type => 'correction', + bin => $bin, + shipping_date => $shipping_date, + ); + +=head2 C + +Create stock out event for a part. See C. + =head1 BUGS Nothing here yet.