From adecbacded3eff18d51dbf7e2e537811190b3888 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sven=20Sch=C3=B6ling?= Date: Thu, 21 Oct 2010 16:17:36 +0200 Subject: [PATCH] SL::DB::Part.pm - types, methoden, doku type modell santizied. orphaned funktion erstellt. Dokumentation erwitert. --- SL/DB/Part.pm | 122 +++++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 106 insertions(+), 16 deletions(-) diff --git a/SL/DB/Part.pm b/SL/DB/Part.pm index 52b7520fa..1acd8ef20 100644 --- a/SL/DB/Part.pm +++ b/SL/DB/Part.pm @@ -25,19 +25,59 @@ __PACKAGE__->meta->initialize; sub is_type { my $self = shift; my $type = lc(shift || ''); + die 'invalid type' unless $type =~ /^(?:part|service|assembly)$/; - if ($type =~ m/^part/) { - return !$self->assembly && $self->inventory_accno_id ? 1 : 0; - - } elsif ($type =~ m/^service/) { - return !$self->inventory_accno_id && !$self->assembly ? 1 : 0; + return $self->type eq $type ? 1 : 0; +} - } elsif ($type =~ m/^assembl/) { - return $self->assembly ? 1 : 0; +sub is_part { $_[0]->is_type('part') } +sub is_assembly { $_[0]->is_type('assembly') } +sub is_service { $_[0]->is_type('service') } +sub type { + my ($self, $type) = @_; + if (@_ > 1) { + die 'invalid type' unless $type =~ /^(?:part|service|assembly)$/; + $self->assembly( $type eq 'assembly' ? 1 : 0); + $self->inventory_accno_id($type ne 'service' ? 1 : undef); } - confess "Unknown type parameter '$type'"; + return 'assembly' if $self->assembly; + return 'part' if $self->inventory_accno_id; + return 'service'; +} + +sub new_part { + my ($class, %params) = @_; + $class->new(%params, type => 'part'); +} + +sub new_assembly { + my ($class, %params) = @_; + $class->new(%params, type => 'assembly'); +} + +sub new_service { + my ($class, %params) = @_; + $class->new(%params, type => 'service'); +} + +sub orphaned { + my ($self) = @_; + die 'not an accessor' if @_ > 1; + + my @relations = qw( + SL::DB::InvoiceItem + SL::DB::OrderItem + SL::DB::Inventory + SL::DB::RMAItem + ); + + for my $class (@relations) { + eval "require $class"; + return 0 if $class->_get_manager_class->get_all_count(query => [ parts_id => $self->id ]); + } + return 1; } sub get_sellprice_info { @@ -77,11 +117,47 @@ SL::DB::Part: Model for the 'parts' table This is a standard Rose::DB::Object based model and can be used as one. -=head1 FUNCTIONS +=head1 TYPES + +Although the base class is called C we usually talk about C if +we mean instances of this class. This is because articles come in three +flavours called: =over 4 -=item is_type $type +=item Part - a single part + +=item Service - a part without onhand, and without inventory accounting + +=item Assembly - a collection of both parts and services + +=back + +These types are sadly represented by data inside the class and cannot be +migrated into a flag. To work around this, each C object knows what type +it currently is. Since the type ist data driven, there ist no explicit setting +method for it, but you can construct them explicitly with C, +C, and C. A Buchungsgruppe should be supplied in this +case, but it will use the default Buchungsgruppe if you don't. + +Matching these there are assorted helper methods dealing with type: + +=head2 new_part PARAMS + +=head2 new_service PARAMS + +=head2 new_assembly PARAMS + +Will set the appropriate data fields so that the resulting instance will be of +tthe requested type. Since part of the distinction are accounting targets, +providing a C is recommended. If none is given the constructor +will load a default one and set the accounting targets from it. + +=head2 type + +Returns the type as a string. Can be one of C, C, C. + +=head2 is_type TYPE Tests if the current object is a part, a service or an assembly. C<$type> must be one of the words 'part', 'service' or @@ -90,7 +166,17 @@ assembly. C<$type> must be one of the words 'part', 'service' or Returns 1 if the requested type matches, 0 if it doesn't and Ces if an unknown C<$type> parameter is encountered. -=item get_sellprice_info %params +=head2 is_part + +=head2 is_service + +=head2 is_assembly + +Shorthand for is_type('part') etc. + +=head1 FUNCTIONS + +=head2 get_sellprice_info %params Retrieves the C and C for a part under different conditions and returns a hash reference with those two keys. @@ -104,17 +190,21 @@ entry without a country set will be used. If none of the above conditions is met then the information from C<$self> is used. -=item get_ordered_qty %params +=head2 get_ordered_qty %params Retrieves the quantity that has been ordered from a vendor but that has not been delivered yet. Only open purchase orders are considered. -=item get_uncommissioned_qty %params +=head2 orphaned -Retrieves the quantity that has been ordered by a customer but that -has not been commissioned yet. Only open sales orders are considered. +Checks if this articke is used in orders, invoices, delivery orders or +assemblies. -=back +=head2 buchungsgruppe BUCHUNGSGRUPPE + +Used to set the accounting informations from a L object. +Please note, that this is a write only accessor, the original Buchungsgruppe can +not be retrieved from an article once set. =head1 AUTHOR -- 2.20.1