X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FDB%2FPart.pm;h=46a06513c06336c7c6775ec38c318e6b5008d566;hb=f27794056c50783ab7af7fb4af5c48097001cc3b;hp=adb650e0646e3bd7db0bd59f507995f1c2b1ee70;hpb=b989d7cfb4bd8de5c3e290b8df0519ad34d5dd0c;p=kivitendo-erp.git diff --git a/SL/DB/Part.pm b/SL/DB/Part.pm index adb650e06..46a06513c 100644 --- a/SL/DB/Part.pm +++ b/SL/DB/Part.pm @@ -16,11 +16,13 @@ use SL::DB::Helper::CustomVariables ( module => 'IC', cvars_alias => 1, ); +use List::Util qw(sum); __PACKAGE__->meta->add_relationships( assemblies => { type => 'one to many', class => 'SL::DB::Assembly', + manager_args => { sort_by => 'position, oid' }, column_map => { id => 'id' }, }, prices => { @@ -31,6 +33,7 @@ __PACKAGE__->meta->add_relationships( makemodels => { type => 'one to many', class => 'SL::DB::MakeModel', + manager_args => { sort_by => 'sortorder' }, column_map => { id => 'parts_id' }, }, translations => { @@ -38,6 +41,18 @@ __PACKAGE__->meta->add_relationships( class => 'SL::DB::Translation', column_map => { id => 'parts_id' }, }, + assortment_items => { + type => 'one to many', + class => 'SL::DB::AssortmentItem', + column_map => { id => 'assortment_id' }, + }, + history_entries => { + type => 'one to many', + class => 'SL::DB::History', + column_map => { id => 'trans_id' }, + query_args => [ what_done => 'part' ], + manager_args => { sort_by => 'itime' }, + }, ); __PACKAGE__->meta->initialize; @@ -53,46 +68,81 @@ sub _before_save_set_partnumber { return 1; } +sub validate { + my ($self) = @_; + + my @errors; + push @errors, $::locale->text('The partnumber is missing.') if $self->id and !$self->partnumber; + push @errors, $::locale->text('The unit is missing.') unless $self->unit; + push @errors, $::locale->text('The buchungsgruppe is missing.') unless $self->buchungsgruppen_id or $self->buchungsgruppe; + + unless ( $self->id ) { + push @errors, $::locale->text('The partnumber already exists.') if SL::DB::Manager::Part->get_all_count(where => [ partnumber => $self->partnumber ]); + }; + + if ($self->is_assortment && scalar @{$self->assortment_items} == 0) { + push @errors, $::locale->text('The assortment doesn\'t have any items.'); + } + + if ($self->is_assembly && scalar @{$self->assemblies} == 0) { + push @errors, $::locale->text('The assembly doesn\'t have any items.'); + } + + return @errors; +} + sub is_type { my $self = shift; my $type = lc(shift || ''); - die 'invalid type' unless $type =~ /^(?:part|service|assembly)$/; + die 'invalid type' unless $type =~ /^(?:part|service|assembly|assortment)$/; return $self->type eq $type ? 1 : 0; } -sub is_part { $_[0]->is_type('part') } -sub is_assembly { $_[0]->is_type('assembly') } -sub is_service { $_[0]->is_type('service') } +sub is_part { $_[0]->part_type eq 'part' } +sub is_assembly { $_[0]->part_type eq 'assembly' } +sub is_service { $_[0]->part_type eq 'service' } +sub is_assortment { $_[0]->part_type eq 'assortment' } 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); - } - - return 'assembly' if $self->assembly; - return 'part' if $self->inventory_accno_id; - return 'service'; + return $_[0]->part_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); + # } + + # 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'); + $class->new(%params, part_type => 'part'); } sub new_assembly { my ($class, %params) = @_; - $class->new(%params, type => 'assembly'); + $class->new(%params, part_type => 'assembly'); } sub new_service { my ($class, %params) = @_; - $class->new(%params, type => 'service'); + $class->new(%params, part_type => 'service'); +} + +sub new_assortment { + my ($class, %params) = @_; + $class->new(%params, part_type => 'assortment'); } +sub last_modification { + my ($self) = @_; + return $self->mtime or $self->itime; +}; + sub orphaned { my ($self) = @_; die 'not an accessor' if @_ > 1; @@ -101,6 +151,8 @@ sub orphaned { SL::DB::InvoiceItem SL::DB::OrderItem SL::DB::Inventory + SL::DB::Assembly + SL::DB::AssortmentItem ); for my $class (@relations) { @@ -173,7 +225,7 @@ sub get_chart { if (!exists $charts->{$taxzone}->{$type}) { require SL::DB::Buchungsgruppe; my $bugru = SL::DB::Buchungsgruppe->load_cached($self->buchungsgruppen_id); - my $chart_id = ($type eq 'inventory') ? ($self->inventory_accno_id ? $bugru->inventory_accno_id : undef) + my $chart_id = ($type eq 'inventory') ? ($self->is_part ? $bugru->inventory_accno_id : undef) : $bugru->call_sub("${type}_accno_id", $taxzone); if ($chart_id) { @@ -205,10 +257,69 @@ sub get_simple_stock { sub bin { require SL::DB::Bin; SL::DB::Manager::Bin ->find_by_or_create(id => $_[0]->{bin_id}) } } -sub long_description { +sub displayable_name { join ' ', grep $_, map $_[0]->$_, qw(partnumber description); } +sub clone_and_reset_deep { + my ($self) = @_; + + my $clone = $self->clone_and_reset; # resets id and partnumber (primary key and unique constraint) + $clone->makemodels( map { $_->clone_and_reset } @{$self->makemodels}); + $clone->translations( map { $_->clone_and_reset } @{$self->translations}); + + if ( $self->is_assortment ) { + $clone->assortment_items( map { $_->clone } @{$self->assortment_items} ); + foreach my $ai ( @{ $clone->assortment_items } ) { + $ai->assortment_id(undef); + }; + }; + + if ( $self->is_assembly ) { + $clone->assemblies( map { $_->clone_and_reset } @{$self->assemblies}); + }; + + if ( $self->prices ) { + $clone->prices( map { $_->clone } @{$self->prices}); # pricegroup_id gets reset here because it is part of a unique contraint + if ( $clone->prices ) { + foreach my $price ( @{$clone->prices} ) { + $price->id(undef); + $price->parts_id(undef); + }; + }; + }; + + return $clone; +} + +sub assembly_sellprice_sum { + my ($self) = @_; + + return unless $self->is_assembly; + sum map { $_->linetotal_sellprice } @{$self->assemblies}; +}; + +sub assembly_lastcost_sum { + my ($self) = @_; + + return unless $self->is_assembly; + sum map { $_->linetotal_lastcost } @{$self->assemblies}; +}; + +sub assortment_sellprice_sum { + my ($self) = @_; + + return unless $self->is_assortment; + sum map { $_->linetotal_sellprice } @{$self->assortment_items}; +}; + +sub assortment_lastcost_sum { + my ($self) = @_; + + return unless $self->is_assortment; + sum map { $_->linetotal_lastcost } @{$self->assortment_items}; +}; + 1; __END__ @@ -239,13 +350,15 @@ flavours called: =item Assembly - a collection of both parts and services +=item Assortment - a collection of parts + =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 +it currently is. Since the type is 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 +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 types, @@ -263,7 +376,7 @@ L and others. =item C 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, +the requested type. Since accounting targets are part of the distinction, providing a C is recommended. If none is given the constructor will load a default one and set the accounting targets from it. @@ -318,7 +431,7 @@ This function looks up the income (for trueish values of C<$params{is_sales}>) or expense (for falsish values of C<$params{is_sales}>) account for the current part. It uses the part's associated buchungsgruppe and uses the fields belonging to the tax -zone given by C<$params{taxzone}> (range 0..3). +zone given by C<$params{taxzone}>. The information retrieved by the function is cached. @@ -330,22 +443,37 @@ C<$params{type}> and tax zone C<$params{taxzone}> the three key words C, C and C. This function uses the part's associated buchungsgruppe and uses the -fields belonging to the tax zone given by C<$params{taxzone}> (range -0..3). +fields belonging to the tax zone given by C<$params{taxzone}>. The information retrieved by the function is cached. =item C -Checks if this articke is used in orders, invoices, delivery orders or +Checks if this article is used in orders, invoices, delivery orders or assemblies. =item C -Used to set the accounting informations from a L object. +Used to set the accounting information 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. +=item C + +Non-recursive sellprice sum of all the assembly item sellprices. + +=item C + +Non-recursive sellprice sum of all the assortment item sellprices. + +=item C + +Non-recursive lastcost sum of all the assembly item lastcosts. + +=item C + +Non-recursive lastcost sum of all the assortment item lastcosts. + =back =head1 AUTHORS