X-Git-Url: http://wagnertech.de/gitweb/gitweb.cgi/mfinanz.git/blobdiff_plain/c4d248c6fc53195f1c566d8084a80fb5d936d8df..1d78de409b3536e1dba613700eb8ec5ffa531567:/SL/Controller/CsvImport/Order.pm?ds=inline diff --git a/SL/Controller/CsvImport/Order.pm b/SL/Controller/CsvImport/Order.pm index c2c78c4d4..2cd1b42af 100644 --- a/SL/Controller/CsvImport/Order.pm +++ b/SL/Controller/CsvImport/Order.pm @@ -14,6 +14,7 @@ use SL::DB::Contact; use SL::DB::Department; use SL::DB::Project; use SL::DB::Shipto; +use SL::DB::TaxZone; use SL::TransNumber; use parent qw(SL::Controller::CsvImport::BaseMulti); @@ -21,7 +22,7 @@ use parent qw(SL::Controller::CsvImport::BaseMulti); use Rose::Object::MakeMethods::Generic ( - 'scalar --get_set_init' => [ qw(settings languages_by all_parts parts_by all_contacts contacts_by all_departments departments_by all_projects projects_by all_ct_shiptos ct_shiptos_by) ], + 'scalar --get_set_init' => [ qw(settings languages_by all_parts parts_by all_contacts contacts_by all_departments departments_by all_projects projects_by all_ct_shiptos ct_shiptos_by all_taxzones taxzones_by) ], ); @@ -79,7 +80,8 @@ sub setup_displayable_columns { { name => 'language', description => $::locale->text('Language (name)') }, { name => 'payment_id', description => $::locale->text('Payment terms (database ID)') }, { name => 'payment', description => $::locale->text('Payment terms (name)') }, - { name => 'taxzone_id', description => $::locale->text('Steuersatz') }, + { name => 'taxzone_id', description => $::locale->text('Steuersatz (database ID') }, + { name => 'taxzone', description => $::locale->text('Steuersatz (description)') }, { name => 'cp_id', description => $::locale->text('Contact Person (database ID)') }, { name => 'contact', description => $::locale->text('Contact Person (name)') }, { name => 'department_id', description => $::locale->text('Department (database ID)') }, @@ -91,8 +93,11 @@ sub setup_displayable_columns { ); $self->add_displayable_columns('OrderItem', - { name => 'parts_id', description => $::locale->text('Part (database ID)') }, - { name => 'partnumber', description => $::locale->text('Part Number') }, + { name => 'parts_id', description => $::locale->text('Part (database ID)') }, + { name => 'partnumber', description => $::locale->text('Part Number') }, + { name => 'project_id', description => $::locale->text('Project (database ID)') }, + { name => 'projectnumber', description => $::locale->text('Project (number)') }, + { name => 'project', description => $::locale->text('Project (description)') }, ); } @@ -173,6 +178,18 @@ sub init_ct_shiptos_by { return $sby; } +sub init_all_taxzones { + my ($self) = @_; + + return SL::DB::Manager::TaxZone->get_all; +} + +sub init_taxzones_by { + my ($self) = @_; + + return { map { my $col = $_; ( $col => { map { ( $_->$col => $_ ) } @{ $self->all_taxzones } } ) } qw(id description) }; +} + sub check_objects { my ($self) = @_; @@ -202,6 +219,7 @@ sub check_objects { $self->check_department($entry); $self->check_project($entry, global => 1); $self->check_ct_shipto($entry); + $self->check_taxzone($entry); if ($vc_obj) { # copy from customer if not given @@ -230,8 +248,9 @@ sub check_objects { $self->add_info_columns($self->settings->{'order_column'}, { header => $::locale->text('Customer/Vendor'), method => 'vc_name' }); + # Todo: access via ->[0] ok? Better: search first order column and use this $self->add_columns($self->settings->{'order_column'}, - map { "${_}_id" } grep { exists $self->controller->data->[0]->{raw_data}->{$_} } qw(payment language department globalproject)); + map { "${_}_id" } grep { exists $self->controller->data->[0]->{raw_data}->{$_} } qw(payment language department globalproject taxzone)); $self->add_columns($self->settings->{'order_column'}, 'globalproject_id') if exists $self->controller->data->[0]->{raw_data}->{globalprojectnumber}; foreach my $entry (@{ $self->controller->data }) { @@ -249,11 +268,17 @@ sub check_objects { # set to 0 if not given $entry->{object}->discount(0) unless $entry->{object}->discount; $entry->{object}->ship(0) unless $entry->{object}->ship; + + $self->check_project($entry, global => 0); } } $self->add_info_columns($self->settings->{'item_column'}, { header => $::locale->text('Part Number'), method => 'partnumber' }); + # Todo: access via ->[1] ok? Better: search first item column and use this + $self->add_columns($self->settings->{'item_column'}, + map { "${_}_id" } grep { exists $self->controller->data->[1]->{raw_data}->{$_} } qw(project)); + $self->add_columns($self->settings->{'item_column'}, 'project_id') if exists $self->controller->data->[1]->{raw_data}->{projectnumber}; # add orderitems to order my $order_entry; @@ -315,6 +340,7 @@ sub check_objects { } ); foreach my $tv (@to_verify) { + # Todo: access via ->[0] ok? Better: search first order column and use this if (exists $self->controller->data->[0]->{raw_data}->{ $tv->{raw_column} }) { $self->add_raw_data_columns($self->settings->{'order_column'}, $tv->{raw_column}); $self->add_info_columns($self->settings->{'order_column'}, @@ -538,6 +564,30 @@ sub check_ct_shipto { return 1; } +sub check_taxzone { + my ($self, $entry) = @_; + + my $object = $entry->{object}; + + # Check wether or not taxzone ID is valid. + if ($object->taxzone_id && !$self->taxzones_by->{id}->{ $object->taxzone_id }) { + push @{ $entry->{errors} }, $::locale->text('Error: Invalid taxzone'); + return 0; + } + + # Map description to ID if given. + if (!$object->taxzone_id && $entry->{raw_data}->{taxzone}) { + my $taxzone = $self->taxzones_by->{description}->{ $entry->{raw_data}->{taxzone} }; + if (!$taxzone) { + push @{ $entry->{errors} }, $::locale->text('Error: Invalid taxzone'); + return 0; + } + + $object->taxzone_id($taxzone->id); + } + + return 1; +} sub save_objects {