X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FCsvImport%2FBase.pm;h=6a66e8ca1dd964a8f2692ebd4db7c3205f6d9f76;hb=e74dac176e01090f7e3b6bd9e99596a36e614284;hp=0381b4b0ba8f3bd74bcd9be8dcea284f0fc093e8;hpb=35948584de818b00974b9dab65682c4d5ceba67f;p=kivitendo-erp.git diff --git a/SL/Controller/CsvImport/Base.pm b/SL/Controller/CsvImport/Base.pm index 0381b4b0b..6a66e8ca1 100644 --- a/SL/Controller/CsvImport/Base.pm +++ b/SL/Controller/CsvImport/Base.pm @@ -5,13 +5,17 @@ use strict; use List::MoreUtils qw(pairwise); use SL::Helper::Csv; +use SL::DB::Customer; +use SL::DB::Language; +use SL::DB::PaymentTerm; +use SL::DB::Vendor; use parent qw(Rose::Object); use Rose::Object::MakeMethods::Generic ( scalar => [ qw(controller file csv) ], - 'scalar --get_set_init' => [ qw(profile existing_objects class manager_class) ], + 'scalar --get_set_init' => [ qw(profile displayable_columns existing_objects class manager_class cvar_columns all_cvar_configs all_languages payment_terms_by all_vc vc_by) ], ); sub run { @@ -26,6 +30,10 @@ sub run { strict_profile => 1, map { ( $_ => $self->controller->profile->get($_) ) } qw(sep_char escape_char quote_char), )); + + my $old_numberformat = $::myconfig{numberformat}; + $::myconfig{numberformat} = $self->controller->profile->get('numberformat'); + $self->csv->parse; $self->controller->errors([ $self->csv->errors ]) if $self->csv->errors; @@ -37,16 +45,17 @@ sub run { $headers->{used} = { map { ($_ => 1) } @{ $headers->{headers} } }; $self->controller->headers($headers); $self->controller->raw_data_headers({ used => { }, headers => [ ] }); + $self->controller->info_headers({ used => { }, headers => [ ] }); - # my @data; - # foreach my $object ($self->csv->get_objects) my @objects = $self->csv->get_objects; my @raw_data = @{ $self->csv->get_data }; - $self->controller->data([ pairwise { { object => $a, raw_data => $b, errors => [] } } @objects, @raw_data ]); + $self->controller->data([ pairwise { { object => $a, raw_data => $b, errors => [], information => [], info_data => {} } } @objects, @raw_data ]); $self->check_objects; $self->check_duplicates if $self->controller->profile->get('duplicates', 'no_check') ne 'no_check'; - $self->fix_field_lenghts; + $self->fix_field_lengths; + + $::myconfig{numberformat} = $old_numberformat; } sub add_columns { @@ -61,6 +70,18 @@ sub add_columns { } } +sub add_info_columns { + my ($self, @columns) = @_; + + my $h = $self->controller->info_headers; + + foreach my $column (grep { !$h->{used}->{ $_->{method} } } map { ref $_ eq 'HASH' ? $_ : { method => $_, header => $_ } } @columns) { + $h->{used}->{ $column->{method} } = 1; + push @{ $h->{methods} }, $column->{method}; + push @{ $h->{headers} }, $column->{header}; + } +} + sub add_raw_data_columns { my ($self, @columns) = @_; @@ -72,6 +93,103 @@ sub add_raw_data_columns { } } +sub add_cvar_raw_data_columns { + my ($self) = @_; + + map { $self->add_raw_data_columns($_) if exists $self->controller->data->[0]->{raw_data}->{$_} } @{ $self->cvar_columns }; +} + +sub init_cvar_columns { + my ($self) = @_; + + return [ map { "cvar_" . $_->name } (@{ $self->all_cvar_configs }) ]; +} + +sub init_all_languages { + my ($self) = @_; + + return SL::DB::Manager::Language->get_all; +} + +sub init_payment_terms_by { + my ($self) = @_; + + my $all_payment_terms = SL::DB::Manager::PaymentTerm->get_all; + return { map { my $col = $_; ( $col => { map { ( $_->$col => $_ ) } @{ $all_payment_terms } } ) } qw(id description) }; +} + +sub init_all_vc { + my ($self) = @_; + + return { customers => SL::DB::Manager::Customer->get_all, + vendors => SL::DB::Manager::Vendor->get_all }; +} + +sub init_vc_by { + my ($self) = @_; + + my %by_id = map { ( $_->id => $_ ) } @{ $self->all_vc->{customers} }, @{ $self->all_vc->{vendors} }; + my %by_number = ( customers => { map { ( $_->customernumber => $_ ) } @{ $self->all_vc->{customers} } }, + vendors => { map { ( $_->vendornumber => $_ ) } @{ $self->all_vc->{vendors} } } ); + my %by_name = ( customers => { map { ( $_->name => $_ ) } @{ $self->all_vc->{customers} } }, + vendors => { map { ( $_->name => $_ ) } @{ $self->all_vc->{vendors} } } ); + + return { id => \%by_id, + number => \%by_number, + name => \%by_name, }; +} + +sub check_vc { + my ($self, $entry, $id_column) = @_; + + if ($entry->{object}->$id_column) { + $entry->{object}->$id_column(undef) if !$self->vc_by->{id}->{ $entry->{object}->$id_column }; + } + + if (!$entry->{object}->$id_column) { + my $vc = $self->vc_by->{number}->{customers}->{ $entry->{raw_data}->{customernumber} } + || $self->vc_by->{number}->{vendors}->{ $entry->{raw_data}->{vendornumber} }; + $entry->{object}->$id_column($vc->id) if $vc; + } + + if (!$entry->{object}->$id_column) { + my $vc = $self->vc_by->{name}->{customers}->{ $entry->{raw_data}->{customer} } + || $self->vc_by->{name}->{vendors}->{ $entry->{raw_data}->{vendor} }; + $entry->{object}->$id_column($vc->id) if $vc; + } + + if ($entry->{object}->$id_column) { + $entry->{info_data}->{vc_name} = $self->vc_by->{id}->{ $entry->{object}->$id_column }->name; + } else { + push @{ $entry->{errors} }, $::locale->text('Error: Customer/vendor not found'); + } +} + +sub handle_cvars { + my ($self, $entry) = @_; + + return unless $self->can('all_cvar_configs'); + + my %type_to_column = ( text => 'text_value', + textfield => 'text_value', + select => 'text_value', + date => 'timestamp_value_as_date', + timestamp => 'timestamp_value_as_date', + number => 'number_value_as_number', + bool => 'bool_value' ); + + my @cvars; + foreach my $config (@{ $self->all_cvar_configs }) { + next unless exists $entry->{raw_data}->{ "cvar_" . $config->name }; + my $value = $entry->{raw_data}->{ "cvar_" . $config->name }; + my $column = $type_to_column{ $config->type } || die "Program logic error: unknown custom variable storage type"; + + push @cvars, SL::DB::CustomVariable->new(config_id => $config->id, $column => $value); + } + + $entry->{object}->custom_variables(\@cvars); +} + sub init_profile { my ($self) = @_; @@ -93,6 +211,39 @@ sub init_profile { $self->profile(\%profile); } +sub add_displayable_columns { + my ($self, @columns) = @_; + + my @cols = @{ $self->controller->displayable_columns || [] }; + my %ex_col_map = map { $_->{name} => $_ } @cols; + + foreach my $column (@columns) { + if ($ex_col_map{ $column->{name} }) { + @{ $ex_col_map{ $column->{name} } }{ keys %{ $column } } = @{ $column }{ keys %{ $column } }; + } else { + push @cols, $column; + } + } + + $self->controller->displayable_columns([ sort { $a->{name} cmp $b->{name} } @cols ]); +} + +sub setup_displayable_columns { + my ($self) = @_; + + $self->add_displayable_columns(map { { name => $_ } } keys %{ $self->profile }); +} + +sub add_cvar_columns_to_displayable_columns { + my ($self) = @_; + + return unless $self->can('all_cvar_configs'); + + $self->add_displayable_columns(map { { name => 'cvar_' . $_->name, + description => $::locale->text('#1 (custom variable)', $_->description) } } + @{ $self->all_cvar_configs }); +} + sub init_existing_objects { my ($self) = @_; @@ -117,6 +268,32 @@ sub check_objects { sub check_duplicates { } +sub check_payment { + my ($self, $entry) = @_; + + my $object = $entry->{object}; + + # Check whether or not payment ID is valid. + if ($object->payment_id && !$self->payment_terms_by->{id}->{ $object->payment_id }) { + push @{ $entry->{errors} }, $::locale->text('Error: Invalid payment terms'); + return 0; + } + + # Map name to ID if given. + if (!$object->payment_id && $entry->{raw_data}->{payment}) { + my $terms = $self->payment_terms_by->{description}->{ $entry->{raw_data}->{payment} }; + + if (!$terms) { + push @{ $entry->{errors} }, $::locale->text('Error: Invalid payment terms'); + return 0; + } + + $object->payment_id($terms->id); + } + + return 1; +} + sub save_objects { my ($self, %params) = @_; @@ -125,7 +302,9 @@ sub save_objects { foreach my $entry (@{ $data }) { next if @{ $entry->{errors} }; - if (!$entry->{object}->save) { + my $object = $entry->{object_to_save} || $entry->{object}; + + if (!$object->save) { push @{ $entry->{errors} }, $::locale->text('Error when saving: #1', $entry->{object}->db->error); } else { $self->controller->num_imported($self->controller->num_imported + 1); @@ -137,7 +316,7 @@ sub field_lengths { return (); } -sub fix_field_lenghts { +sub fix_field_lengths { my ($self) = @_; my %field_lengths = $self->field_lengths;