X-Git-Url: http://wagnertech.de/gitweb/gitweb.cgi/mfinanz.git/blobdiff_plain/81379539e7f75fc6f886057176f8a51ebb8f9cf4..1c84c7f2353749b92ab39bb6f3fec67e5e645756:/SL/Controller/CsvImport/Part.pm diff --git a/SL/Controller/CsvImport/Part.pm b/SL/Controller/CsvImport/Part.pm index 596f038e0..af3b10755 100644 --- a/SL/Controller/CsvImport/Part.pm +++ b/SL/Controller/CsvImport/Part.pm @@ -5,9 +5,12 @@ use strict; use SL::Helper::Csv; use SL::DB::Buchungsgruppe; +use SL::DB::CustomVariable; +use SL::DB::CustomVariableConfig; use SL::DB::PartsGroup; use SL::DB::PaymentTerm; use SL::DB::PriceFactor; +use SL::DB::Translation; use SL::DB::Unit; use parent qw(SL::Controller::CsvImport::Base); @@ -15,7 +18,8 @@ use parent qw(SL::Controller::CsvImport::Base); use Rose::Object::MakeMethods::Generic ( scalar => [ qw(table) ], - 'scalar --get_set_init' => [ qw(bg_by settings parts_by price_factors_by units_by payment_terms_by packing_types_by partsgroups_by) ], + 'scalar --get_set_init' => [ qw(bg_by settings parts_by price_factors_by units_by packing_types_by partsgroups_by + translation_columns) ], ); sub init_class { @@ -37,13 +41,6 @@ sub init_price_factors_by { return { map { my $col = $_; ( $col => { map { ( $_->$col => $_ ) } @{ $all_price_factors } } ) } qw(id description) }; } -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_packing_types_by { my ($self) = @_; @@ -88,15 +85,23 @@ sub init_settings { shoparticle_if_missing parts_type) }; } +sub init_all_cvar_configs { + my ($self) = @_; + + return SL::DB::Manager::CustomVariableConfig->get_all(where => [ module => 'IC' ]); +} + +sub init_translation_columns { + my ($self) = @_; + + return [ map { ("description_" . $_->article_code, "notes_" . $_->article_code) } (@{ $self->all_languages }) ]; +} + sub check_objects { my ($self) = @_; return unless @{ $self->controller->data }; - $self->add_columns(qw(type)) if $self->settings->{parts_type} eq 'mixed'; - $self->add_columns(qw(buchungsgruppen_id unit)); - $self->add_columns(map { "${_}_id" } grep { exists $self->controller->data->[0]->{$_} } qw (price_factor payment packing_type partsgroup)); - foreach my $entry (@{ $self->controller->data }) { my $object = $entry->{object}; my $raw_data = $entry->{raw_data}; @@ -111,25 +116,28 @@ sub check_objects { $self->check_existing($entry); $self->handle_prices($entry) if $self->settings->{sellprice_adjustment}; $self->handle_shoparticle($entry); + $self->handle_translations($entry); + $self->handle_cvars($entry); $self->set_various_fields($entry); } + + $self->add_columns(qw(type)) if $self->settings->{parts_type} eq 'mixed'; + $self->add_columns(qw(buchungsgruppen_id unit)); + $self->add_columns(map { "${_}_id" } grep { exists $self->controller->data->[0]->{raw_data}->{$_} } qw (price_factor payment packing_type partsgroup)); + $self->add_columns(qw(shop)) if $self->settings->{shoparticle_if_missing}; + $self->add_cvar_raw_data_columns; + map { $self->add_raw_data_columns($_) if exists $self->controller->data->[0]->{raw_data}->{$_} } @{ $self->translation_columns }; } sub check_duplicates { my ($self, %params) = @_; - die "IMPLEMENT ME"; - my $normalizer = sub { my $name = $_[0]; $name =~ s/[\s,\.\-]//g; return $name; }; - my $name_maker = sub { return $normalizer->($_[0]->shiptoname) . '--' . $normalizer->($_[0]->shiptostreet) }; + my $name_maker = sub { return $normalizer->($_[0]->description) }; - my %by_id_and_name; + my %by_name; if ('check_db' eq $self->controller->profile->get('duplicates')) { - foreach my $type (qw(customers vendors)) { - foreach my $vc (@{ $self->all_vc->{$type} }) { - $by_id_and_name{ $vc->id } = { map { ( $name_maker->($_) => 'db' ) } @{ $vc->shipto } }; - } - } + %by_name = map { ( $name_maker->($_) => 'db' ) } @{ $self->existing_objects }; } foreach my $entry (@{ $self->controller->data }) { @@ -137,12 +145,11 @@ sub check_duplicates { my $name = $name_maker->($entry->{object}); - $by_id_and_name{ $entry->{vc}->id } ||= { }; - if (!$by_id_and_name{ $entry->{vc}->id }->{ $name }) { - $by_id_and_name{ $entry->{vc}->id }->{ $name } = 'csv'; + if (!$by_name{ $name }) { + $by_name{ $name } = 'csv'; } else { - push @{ $entry->{errors} }, $by_id_and_name{ $entry->{vc}->id }->{ $name } eq 'db' ? $::locale->text('Duplicate in database') : $::locale->text('Duplicate in CSV file'); + push @{ $entry->{errors} }, $by_name{ $name } eq 'db' ? $::locale->text('Duplicate in database') : $::locale->text('Duplicate in CSV file'); } } } @@ -207,7 +214,7 @@ sub handle_prices { my $value = $entry->{object}->$column; $value = $self->settings->{sellprice_adjustment_type} eq 'percent' ? $value * (100 + $adjustment) / 100 : $value + $adjustment; - $entry->{object}->$column($::form->round_amount($self->settings->{sellprice_places}, $value)); + $entry->{object}->$column($::form->round_amount($value, $self->settings->{sellprice_places})); } } @@ -257,40 +264,14 @@ sub check_price_factor { # Map name to ID if given. if (!$object->price_factor_id && $entry->{raw_data}->{price_factor}) { - my $id = $self->price_factors_by->{description}->{ $entry->{raw_data}->{price_factor} }; + my $pf = $self->price_factors_by->{description}->{ $entry->{raw_data}->{price_factor} }; - if (!$id) { + if (!$pf) { push @{ $entry->{errors} }, $::locale->text('Error: Invalid price factor'); return 0; } - $object->price_factor_id($id); - } - - return 1; -} - -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 $id = $self->payment_terms_by->{description}->{ $entry->{raw_data}->{payment} }; - - if (!$id) { - push @{ $entry->{errors} }, $::locale->text('Error: Invalid payment terms'); - return 0; - } - - $object->payment_id($id); + $object->price_factor_id($pf->id); } return 1; @@ -309,14 +290,14 @@ sub check_packing_type { # Map name to ID if given. if (!$object->packing_type_id && $entry->{raw_data}->{packing_type}) { - my $id = $self->packing_type_by->{description}->{ $entry->{raw_data}->{packing_type} }; + my $type = $self->packing_types_by->{description}->{ $entry->{raw_data}->{packing_type} }; - if (!$id) { + if (!$type) { push @{ $entry->{errors} }, $::locale->text('Error: Invalid packing type'); return 0; } - $object->packing_type_id($id); + $object->packing_type_id($type->id); } return 1; @@ -335,14 +316,14 @@ sub check_partsgroup { # Map name to ID if given. if (!$object->partsgroup_id && $entry->{raw_data}->{partsgroup}) { - my $id = $self->partsgroups_by->{partsgroup}->{ $entry->{raw_data}->{partsgroup} }; + my $pg = $self->partsgroups_by->{partsgroup}->{ $entry->{raw_data}->{partsgroup} }; - if (!$id) { + if (!$pg) { push @{ $entry->{errors} }, $::locale->text('Error: Invalid parts group'); return 0; } - $object->partsgroup_id($id); + $object->partsgroup_id($pg->id); } return 1; @@ -362,6 +343,22 @@ sub check_unit { return 1; } +sub handle_translations { + my ($self, $entry) = @_; + + my @translations; + foreach my $language (@{ $self->all_languages }) { + my ($desc, $notes) = @{ $entry->{raw_data} }{ "description_" . $language->article_code, "notes_" . $language->article_code }; + next unless $desc || $notes; + + push @translations, SL::DB::Translation->new(language_id => $language->id, + translation => $desc, + longdescription => $notes); + } + + $entry->{object}->translations(\@translations); +} + sub set_various_fields { my ($self, $entry) = @_; @@ -372,16 +369,21 @@ sub init_profile { my ($self) = @_; my $profile = $self->SUPER::init_profile; - delete $profile->{type}; - - $::lxdebug->dump(0, "prof", $profile); + delete @{$profile}{qw(type priceupdate)}; return $profile; } +sub save_objects { + my ($self, %params) = @_; + + my $with_number = [ grep { $_->{object}->partnumber ne '####' } @{ $self->controller->data } ]; + my $without_number = [ grep { $_->{object}->partnumber eq '####' } @{ $self->controller->data } ]; + + map { $_->{object}->partnumber('') } @{ $without_number }; -# TODO: -# priceupdate aus Profil raus -# CVARs ins Profil rein + $self->SUPER::save_objects(data => $with_number); + $self->SUPER::save_objects(data => $without_number); +} 1;