X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FDB%2FPart.pm;h=7fd35d80ef2e0120c4da93560b6c0b4a7219e907;hb=1663057f67e6dc72fd492d52e5a97b659ec90f63;hp=ecdaeaabe4f24139766c2e77f030ea634fff216a;hpb=666d4cad684028ef5ebb5b62adca17127b9bd864;p=kivitendo-erp.git diff --git a/SL/DB/Part.pm b/SL/DB/Part.pm index ecdaeaabe..7fd35d80e 100644 --- a/SL/DB/Part.pm +++ b/SL/DB/Part.pm @@ -4,11 +4,13 @@ use strict; use Carp; use List::MoreUtils qw(any); +use Rose::DB::Object::Helpers qw(as_tree); use SL::DBUtils; use SL::DB::MetaSetup::Part; use SL::DB::Manager::Part; use SL::DB::Chart; +use SL::DB::Helper::AttrHTML; use SL::DB::Helper::TransNumberGenerator; use SL::DB::Helper::CustomVariables ( module => 'IC', @@ -19,6 +21,7 @@ __PACKAGE__->meta->add_relationships( assemblies => { type => 'one to many', class => 'SL::DB::Assembly', + manager_args => { sort_by => 'position, oid' }, column_map => { id => 'id' }, }, prices => { @@ -29,6 +32,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 => { @@ -36,10 +40,17 @@ __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' }, + }, ); __PACKAGE__->meta->initialize; +__PACKAGE__->attr_html('notes'); + __PACKAGE__->before_save('_before_save_set_partnumber'); sub _before_save_set_partnumber { @@ -49,44 +60,74 @@ sub _before_save_set_partnumber { return 1; } +sub validate { + my ($self) = @_; + + my @errors; + push @errors, $::locale->text('The partnumber is missing.') unless $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 orphaned { @@ -97,6 +138,8 @@ sub orphaned { SL::DB::InvoiceItem SL::DB::OrderItem SL::DB::Inventory + SL::DB::Assembly + SL::DB::AssortmentItem ); for my $class (@relations) { @@ -140,21 +183,19 @@ sub get_taxkey { my $date = $params{date} || DateTime->today_local; my $is_sales = !!$params{is_sales}; my $taxzone = $params{ defined($params{taxzone}) ? 'taxzone' : 'taxzone_id' } * 1; + my $tk_info = $::request->cache('get_taxkey'); - $self->{__partpriv_taxkey_information} ||= { }; - my $tk_info = $self->{__partpriv_taxkey_information}; + $tk_info->{$self->id} //= {}; + $tk_info->{$self->id}->{$taxzone} //= { }; + my $cache = $tk_info->{$self->id}->{$taxzone}->{$is_sales} //= { }; - $tk_info->{$taxzone} ||= { }; - $tk_info->{$taxzone}->{$is_sales} ||= { }; - - if (!exists $tk_info->{$taxzone}->{$is_sales}->{$date}) { - $tk_info->{$taxzone}->{$is_sales}->{$date} = + if (!exists $cache->{$date}) { + $cache->{$date} = $self->get_chart(type => $is_sales ? 'income' : 'expense', taxzone => $taxzone) - ->load ->get_active_taxkey($date); } - return $tk_info->{$taxzone}->{$is_sales}->{$date}; + return $cache->{$date}; } sub get_chart { @@ -163,17 +204,22 @@ sub get_chart { my $type = (any { $_ eq $params{type} } qw(income expense inventory)) ? $params{type} : croak("Invalid 'type' parameter '$params{type}'"); my $taxzone = $params{ defined($params{taxzone}) ? 'taxzone' : 'taxzone_id' } * 1; - $self->{__partpriv_get_chart_id} ||= { }; - my $charts = $self->{__partpriv_get_chart_id}; + my $charts = $::request->cache('get_chart_id/by_part_id_and_taxzone')->{$self->id} //= {}; + my $all_charts = $::request->cache('get_chart_id/by_id'); $charts->{$taxzone} ||= { }; if (!exists $charts->{$taxzone}->{$type}) { - my $bugru = $self->buchungsgruppe; + 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) - : $bugru->call_sub("${type}_accno_id_${taxzone}"); + : $bugru->call_sub("${type}_accno_id", $taxzone); - $charts->{$taxzone}->{$type} = $chart_id ? SL::DB::Chart->new(id => $chart_id)->load : undef; + if ($chart_id) { + my $chart = $all_charts->{$chart_id} // SL::DB::Chart->load_cached($chart_id)->load; + $all_charts->{$chart_id} = $chart; + $charts->{$taxzone}->{$type} = $chart; + } } return $charts->{$taxzone}->{$type}; @@ -198,10 +244,41 @@ 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; +} + 1; __END__ @@ -232,13 +309,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, @@ -256,7 +335,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. @@ -311,7 +390,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. @@ -323,19 +402,18 @@ 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.