X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FDB%2FPart.pm;h=5ea3b26989c223d071411179b8398b947bef01a3;hb=06e79bf0f4ad9d5499f4435d2158d8a064034a8f;hp=2535efca9a2c427a448ed57eb04d1609f58ecb0b;hpb=fb5b97ff8fd166a4a3a2cf29663c1dda2a9d98a7;p=kivitendo-erp.git diff --git a/SL/DB/Part.pm b/SL/DB/Part.pm index 2535efca9..5ea3b2698 100644 --- a/SL/DB/Part.pm +++ b/SL/DB/Part.pm @@ -16,6 +16,7 @@ use SL::DB::Helper::CustomVariables ( module => 'IC', cvars_alias => 1, ); +use List::Util qw(sum); __PACKAGE__->meta->add_relationships( assemblies => { @@ -45,6 +46,13 @@ __PACKAGE__->meta->add_relationships( 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; @@ -60,11 +68,32 @@ sub _before_save_set_partnumber { return 1; } +sub items { + my ($self) = @_; + + if ( $self->part_type eq 'assembly' ) { + return $self->assemblies; + } elsif ( $self->part_type eq 'assortment' ) { + return $self->assortment_items; + } else { + return undef; + } +} + +sub items_checksum { + my ($self) = @_; + + # for detecting if the items of an (orphaned) assembly or assortment have + # changed when saving + + return join(' ', sort map { $_->part->id } @{$self->items}); +}; + sub validate { my ($self) = @_; my @errors; - push @errors, $::locale->text('The partnumber is missing.') unless $self->partnumber; + 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; @@ -72,7 +101,8 @@ sub validate { 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) { + if ($self->is_assortment && $self->orphaned && scalar @{$self->assortment_items} == 0) { + # when assortment isn't orphaned form doesn't contain any items push @errors, $::locale->text('The assortment doesn\'t have any items.'); } @@ -139,9 +169,12 @@ sub orphaned { my ($self) = @_; die 'not an accessor' if @_ > 1; + return 1 unless $self->id; + my @relations = qw( SL::DB::InvoiceItem SL::DB::OrderItem + SL::DB::DeliveryOrderItem SL::DB::Inventory SL::DB::Assembly SL::DB::AssortmentItem @@ -257,14 +290,13 @@ 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}); + $clone->makemodels( map { $_->clone_and_reset } @{$self->makemodels} ) if @{$self->makemodels}; + $clone->translations( map { $_->clone_and_reset } @{$self->translations} ) if @{$self->translations}; if ( $self->is_assortment ) { + # use clone rather than reset_and_clone because the unique constraint would also remove parts_id $clone->assortment_items( map { $_->clone } @{$self->assortment_items} ); - foreach my $ai ( @{ $clone->assortment_items } ) { - $ai->assortment_id(undef); - }; + $_->assortment_id(undef) foreach @{ $clone->assortment_items } }; if ( $self->is_assembly ) { @@ -284,6 +316,56 @@ sub clone_and_reset_deep { return $clone; } +sub item_diffs { + my ($self, $comparison_part) = @_; + + die "item_diffs needs a part object" unless ref($comparison_part) eq 'SL::DB::Part'; + die "part and comparison_part need to be of the same part_type" unless + ( $self->part_type eq 'assembly' or $self->part_type eq 'assortment' ) + and ( $comparison_part->part_type eq 'assembly' or $comparison_part->part_type eq 'assortment' ) + and $self->part_type eq $comparison_part->part_type; + + # return [], [] if $self->items_checksum eq $comparison_part->items_checksum; + my @self_part_ids = map { $_->parts_id } $self->items; + my @comparison_part_ids = map { $_->parts_id } $comparison_part->items; + + my %orig = map{ $_ => 1 } @self_part_ids; + my %comparison = map{ $_ => 1 } @comparison_part_ids; + my (@additions, @removals); + @additions = grep { !exists( $orig{$_} ) } @comparison_part_ids if @comparison_part_ids; + @removals = grep { !exists( $comparison{$_} ) } @self_part_ids if @self_part_ids; + + return \@additions, \@removals; +}; + +sub items_sellprice_sum { + my ($self, %params) = @_; + + return unless $self->is_assortment or $self->is_assembly; + return unless $self->items; + + if ($self->is_assembly) { + return sum map { $_->linetotal_sellprice } @{$self->items}; + } else { + return sum map { $_->linetotal_sellprice(%params) } grep { $_->charge } @{$self->items}; + } +} + +sub items_lastcost_sum { + my ($self) = @_; + + return unless $self->is_assortment or $self->is_assembly; + return unless $self->items; + sum map { $_->linetotal_lastcost } @{$self->items}; +}; + +sub assortment_lastcost_sum { + my ($self) = @_; + + return unless $self->is_assortment; + sum map { $_->linetotal_lastcost } @{$self->assortment_items}; +}; + 1; __END__ @@ -314,7 +396,7 @@ flavours called: =item Assembly - a collection of both parts and services -=item Assortment - a collection of parts +=item Assortment - a collection of items (parts or assemblies) =back @@ -422,6 +504,22 @@ 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