X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FCsvImport%2FContact.pm;h=5ca2ae848dcef9ae2ff801ca3f5671fdd80a4843;hb=b16b33c220a567bf8a8b998ff69d44d380b0a181;hp=ace6654018d8f65dca33ea74572cb74493be7197;hpb=7b9d1666e389bfedba43b2d0fd5d1b0e3f129e88;p=kivitendo-erp.git

diff --git a/SL/Controller/CsvImport/Contact.pm b/SL/Controller/CsvImport/Contact.pm
index ace665401..5ca2ae848 100644
--- a/SL/Controller/CsvImport/Contact.pm
+++ b/SL/Controller/CsvImport/Contact.pm
@@ -3,13 +3,14 @@ package SL::Controller::CsvImport::Contact;
 use strict;
 
 use SL::Helper::Csv;
+use SL::DB::CustomVariable;
+use SL::DB::CustomVariableConfig;
 
 use parent qw(SL::Controller::CsvImport::Base);
 
 use Rose::Object::MakeMethods::Generic
 (
- scalar                  => [ qw(table) ],
- 'scalar --get_set_init' => [ qw(all_vc) ],
+ scalar => [ qw(table) ],
 );
 
 sub init_class {
@@ -17,91 +18,112 @@ sub init_class {
   $self->class('SL::DB::Contact');
 }
 
-sub init_all_vc {
+sub init_all_cvar_configs {
   my ($self) = @_;
 
-  $self->all_vc({ customers => SL::DB::Manager::Customer->get_all(with_objects => [ 'contacts' ]),
-                  vendors   => SL::DB::Manager::Vendor->get_all(  with_objects => [ 'contacts' ]) });
+  return SL::DB::Manager::CustomVariableConfig->get_all(where => [ module => 'Contacts' ]);
+}
+
+sub force_allow_columns {
+  return qw(cp_id);
 }
 
 sub check_objects {
   my ($self) = @_;
 
-  my %by_id     = map { ( $_->id => $_ ) } @{ $self->all_vc->{customers} }, @{ $self->all_vc->{vendors} };
-  my %by_number = ( customers => { map { ( $_->customernumber => $_->id ) } @{ $self->all_vc->{customers} } },
-                    vendors   => { map { ( $_->vendornumber   => $_->id ) } @{ $self->all_vc->{vendors}   } } );
-  my %by_name   = ( customers => { map { ( $_->name           => $_->id ) } @{ $self->all_vc->{customers} } },
-                    vendors   => { map { ( $_->name           => $_->id ) } @{ $self->all_vc->{vendors}   } } );
+  $self->controller->track_progress(phase => 'building data', progress => 0);
+
+  my $i              = 0;
+  my $num_data       = scalar @{ $self->controller->data };
+  my $update_policy  = $self->controller->profile->get('update_policy') || 'update_existing';
+  my %contacts_by_id = map { ( $_->cp_id => $_ ) } @{ $self->existing_objects };
+  my $methods        = $self->controller->headers->{methods};
+  my %used_methods   = map { ( $_ => 1 ) } @{ $methods };
 
   foreach my $entry (@{ $self->controller->data }) {
-    my $object   = $entry->{object};
-    my $raw_data = $entry->{raw_data};
+    $self->controller->track_progress(progress => $i/$num_data * 100) if $i % 100 == 0;
 
-    my $name     =  $object->cp_name;
-    $name        =~ s/^\s+//;
-    $name        =~ s/\s+$//;
+    my $object = $entry->{object};
+    if ($object->cp_id) {
+      my $existing_contact = $contacts_by_id{ $object->cp_id };
+      if (!$existing_contact) {
+        $contacts_by_id{ $object->cp_id } = $object;
 
-    if (!$name) {
-      push @{ $entry->{errors} }, $::locale->text('Error: Name missing');
-      next;
-    }
+      } elsif ($update_policy eq 'skip') {
+        push(@{ $entry->{errors} }, $::locale->text('Skipping due to existing entry in database'));
+        next;
 
-    if ($object->cp_cv_id) {
-      $object->cp_cv_id(undef) if !$by_id{ $object->cp_cv_id };
-    }
+      } elsif ($update_policy eq 'update_existing') {
+        # Update existing customer/vendor records.
+        $entry->{object_to_save} = $existing_contact;
 
-    if (!$object->cp_cv_id) {
-      $::lxdebug->message(0, "cnum" . $raw_data->{customernumber});
-      my $vc_id = $by_number{customers}->{ $raw_data->{customernumber} } || $by_number{vendors}->{ $raw_data->{vendornumber} };
-      $object->cp_cv_id($vc_id) if $vc_id;
-    }
+        $object->cp_cv_id($existing_contact->cp_cv_id);
 
-    if (!$object->cp_cv_id) {
-      my $vc_id = $by_name{customers}->{ $raw_data->{customer} } || $by_name{vendors}->{ $raw_data->{vendor} };
-      $object->cp_cv_id($vc_id) if $vc_id;
-    }
+        foreach (qw(cp_name cp_gender)) {
+          $object->$_($existing_contact->$_) if !$object->$_;
+        }
 
-    if (!$object->cp_cv_id) {
-      push @{ $entry->{errors} }, $::locale->text('Error: Customer/vendor not found');
-      next;
-    }
+        $existing_contact->$_( $entry->{object}->$_ ) for @{ $methods };
 
-    $entry->{vc} = $by_id{ $object->cp_cv_id };
+        push @{ $entry->{information} }, $::locale->text('Updating existing entry in database');
 
-    if (($object->cp_gender ne 'm') && ($object->cp_gender ne 'f')) {
-      push @{ $entry->{errors} }, $::locale->text('Error: Gender (cp_gender) missing or invalid');
-      next;
+      } else {
+        $object->cp_id(undef);
+      }
     }
-  }
-}
 
-sub check_duplicates {
-  my ($self, %params) = @_;
+    $self->check_name($entry);
+    $self->check_vc($entry, 'cp_cv_id');
+    $self->check_gender($entry);
+    $self->handle_cvars($entry);
 
-  my $normalizer = sub { my $name = $_[0]; $name =~ s/[\s,\.\-]//g; return $name; };
+    my @cleaned_fields = $self->clean_fields(qr{[\r\n]}, $object, qw(cp_title cp_givenname cp_name cp_email cp_phone1 cp_phone2 cp_fax cp_mobile1 cp_mobile2 cp_satphone cp_satfax
+                                                                     cp_privatphone cp_privatemail cp_abteilung cp_street cp_zipcode cp_city cp_position));
 
-  my %by_id_and_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 { ( $normalizer->($_->cp_name) => 'db' ) } @{ $vc->contacts } };
-      }
-    }
+    push @{ $entry->{information} }, $::locale->text('Illegal characters have been removed from the following fields: #1', join(', ', @cleaned_fields))
+      if @cleaned_fields;
+  } continue {
+    $i++;
   }
 
-  foreach my $entry (@{ $self->controller->data }) {
-    next if @{ $entry->{errors} };
+  $self->add_info_columns({ header => $::locale->text('Customer/Vendor'), method => 'vc_name' });
+  $self->add_cvar_raw_data_columns;
+}
 
-    my $name = $normalizer->($entry->{object}->cp_name);
+sub check_name {
+  my ($self, $entry) = @_;
 
-    $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';
+  my $name     =  $entry->{object}->cp_name;
+  $name        =~ s/^\s+//;
+  $name        =~ s/\s+$//;
 
-    } 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} }, $::locale->text('Error: Name missing') unless $name;
+}
+
+sub check_gender {
+  my ($self, $entry) = @_;
+
+  push @{ $entry->{errors} }, $::locale->text('Error: Gender (cp_gender) missing or invalid') if ($entry->{object}->cp_gender ne 'm') && ($entry->{object}->cp_gender ne 'f');
+}
+
+sub get_duplicate_check_fields {
+  return {
+    cp_name => {
+      label     => $::locale->text('Name'),
+      default   => 1,
+      maker     => sub {
+        my $o = shift;
+        return join(
+                 '--',
+                 $o->cp_cv_id,
+                 map(
+                   { s/[\s,\.\-]//g; $_ }
+                   $o->cp_name
+                 )
+        );
+      }
+    },
+  };
 }
 
 sub field_lengths {
@@ -114,4 +136,38 @@ sub field_lengths {
          );
 }
 
+sub setup_displayable_columns {
+  my ($self) = @_;
+
+  $self->SUPER::setup_displayable_columns;
+  $self->add_cvar_columns_to_displayable_columns;
+
+  $self->add_displayable_columns({ name => 'cp_abteilung',   description => $::locale->text('Department')                    },
+                                 { name => 'cp_birthday',    description => $::locale->text('Birthday')                      },
+                                 { name => 'cp_cv_id',       description => $::locale->text('Customer/Vendor (database ID)') },
+                                 { name => 'cp_email',       description => $::locale->text('E-mail')                        },
+                                 { name => 'cp_fax',         description => $::locale->text('Fax')                           },
+                                 { name => 'cp_gender',      description => $::locale->text('Gender')                        },
+                                 { name => 'cp_givenname',   description => $::locale->text('Given Name')                    },
+                                 { name => 'cp_id',          description => $::locale->text('Database ID')                   },
+                                 { name => 'cp_mobile1',     description => $::locale->text('Mobile1')                       },
+                                 { name => 'cp_mobile2',     description => $::locale->text('Mobile2')                       },
+                                 { name => 'cp_name',        description => $::locale->text('Name')                          },
+                                 { name => 'cp_phone1',      description => $::locale->text('Phone1')                        },
+                                 { name => 'cp_phone2',      description => $::locale->text('Phone2')                        },
+                                 { name => 'cp_privatemail', description => $::locale->text('Private E-mail')                },
+                                 { name => 'cp_privatphone', description => $::locale->text('Private Phone')                 },
+                                 { name => 'cp_project',     description => $::locale->text('Project')                       },
+                                 { name => 'cp_satfax',      description => $::locale->text('Sat. Fax')                      },
+                                 { name => 'cp_satphone',    description => $::locale->text('Sat. Phone')                    },
+                                 { name => 'cp_title',       description => $::locale->text('Title')                         },
+                                 { name => 'cp_position',    description => $::locale->text('Function/position')             },
+
+                                 { name => 'customer',       description => $::locale->text('Customer (name)')               },
+                                 { name => 'customernumber', description => $::locale->text('Customer Number')               },
+                                 { name => 'vendor',         description => $::locale->text('Vendor (name)')                 },
+                                 { name => 'vendornumber',   description => $::locale->text('Vendor Number')                 },
+                                );
+}
+
 1;