Kundenstammdaten: Lieferadresse speichern, wenn beliebiges Feld gesetzt

[mfinanz.git] / SL / Controller / CustomerVendor.pm

diff --git a/SL/Controller/CustomerVendor.pm b/SL/Controller/CustomerVendor.pm
index b8672f14d28804b4f27d69b0c04c1f303987b360..fdccdc9d77b58a799fbfe87136c7f90c0b784be2 100644 (file)
--- a/SL/Controller/CustomerVendor.pm
+++ b/SL/Controller/CustomerVendor.pm
@@ -3,9 +3,13 @@ package SL::Controller::CustomerVendor;
 use strict;
 use parent qw(SL::Controller::Base);
 
 use strict;
 use parent qw(SL::Controller::Base);
 

+use List::MoreUtils qw(any);
+

 use SL::JSON;
 use SL::DBUtils;
 use SL::Helper::Flash;
 use SL::JSON;
 use SL::DBUtils;
 use SL::Helper::Flash;

+use SL::Locale::String;
+use SL::Controller::Helper::GetModels;

 
 use SL::DB::Customer;
 use SL::DB::Vendor;
 
 use SL::DB::Customer;
 use SL::DB::Vendor;


@@ -18,16 +22,20 @@ use SL::DB::PaymentTerm;
 use SL::DB::Pricegroup;
 use SL::DB::Contact;
 use SL::DB::FollowUp;
 use SL::DB::Pricegroup;
 use SL::DB::Contact;
 use SL::DB::FollowUp;

+use SL::DB::FollowUpLink;

 use SL::DB::History;
 use SL::DB::Currency;
 
 use SL::DB::History;
 use SL::DB::Currency;
 

+use Rose::Object::MakeMethods::Generic (
+  'scalar --get_set_init' => [ qw(customer_models vendor_models) ],
+);
+

 # safety
 __PACKAGE__->run_before(
   sub {
     $::auth->assert('customer_vendor_edit');
   }
 );
 # safety
 __PACKAGE__->run_before(
   sub {
     $::auth->assert('customer_vendor_edit');
   }
 );

-

 __PACKAGE__->run_before(
   '_instantiate_args',
   only => [
 __PACKAGE__->run_before(
   '_instantiate_args',
   only => [


@@ -49,11 +57,35 @@ __PACKAGE__->run_before(
   '_load_customer_vendor',
   only => [
     'edit',
   '_load_customer_vendor',
   only => [
     'edit',

+    'show',

     'update',
     'ajaj_get_shipto',
     'ajaj_get_contact',
   ]
 );
     'update',
     'ajaj_get_shipto',
     'ajaj_get_contact',
   ]
 );

+
+# make sure this comes after _load_customer_vendor
+__PACKAGE__->run_before(
+  '_check_customer_vendor_all_edit',
+  only => [
+    'edit',
+    'show',
+    'update',
+    'delete',
+    'save',
+    'save_and_ap_transaction',
+    'save_and_ar_transaction',
+    'save_and_close',
+    'save_and_invoice',
+    'save_and_order',
+    'save_and_quotation',
+    'save_and_rfq',
+    'delete',
+    'delete_contact',
+    'delete_shipto',
+  ]
+);
+

 __PACKAGE__->run_before(
   '_create_customer_vendor',
   only => [
 __PACKAGE__->run_before(
   '_create_customer_vendor',
   only => [


@@ -61,10 +93,14 @@ __PACKAGE__->run_before(
   ]
 );
 
   ]
 );
 

+__PACKAGE__->run_before('normalize_name');
+
+

 sub action_add {
   my ($self) = @_;
 
   $self->_pre_render();
 sub action_add {
   my ($self) = @_;
 
   $self->_pre_render();

+  $self->{cv}->assign_attributes(hourly_rate => $::instance_conf->get_customer_hourly_rate) if $self->{cv}->is_customer;

   $self->render(
     'customer_vendor/form',
     title => ($self->is_vendor() ? $::locale->text('Add Vendor') : $::locale->text('Add Customer')),
   $self->render(
     'customer_vendor/form',
     title => ($self->is_vendor() ? $::locale->text('Add Vendor') : $::locale->text('Add Customer')),


@@ -83,9 +119,37 @@ sub action_edit {
   );
 }
 
   );
 }
 

+sub action_show {
+  my ($self) = @_;
+
+  if ($::request->type eq 'json') {
+    my $cv_hash;
+    if (!$self->{cv}) {
+      # TODO error
+    } else {
+      $cv_hash          = $self->{cv}->as_tree;
+      $cv_hash->{cvars} = $self->{cv}->cvar_as_hashref;
+    }
+
+    $self->render(\ SL::JSON::to_json($cv_hash), { layout => 0, type => 'json', process => 0 });
+  }
+}
+

 sub _save {
   my ($self) = @_;
 
 sub _save {
   my ($self) = @_;
 

+  my @errors = $self->{cv}->validate;
+  if (@errors) {
+    flash('error', @errors);
+    $self->_pre_render();
+    $self->render(
+      'customer_vendor/form',
+      title => ($self->is_vendor() ? t8('Edit Vendor') : t8('Edit Customer')),
+      %{$self->{template_args}}
+    );
+    $::dispatcher->end_request;
+  }
+

   my $db = $self->{cv}->db;
 
   $db->do_transaction(sub {
   my $db = $self->{cv}->db;
 
   $db->do_transaction(sub {


@@ -113,21 +177,30 @@ sub _save {
 
     $self->{contact}->cp_cv_id($self->{cv}->id);
     if( $self->{contact}->cp_name ne '' || $self->{contact}->cp_givenname ne '' ) {
 
     $self->{contact}->cp_cv_id($self->{cv}->id);
     if( $self->{contact}->cp_name ne '' || $self->{contact}->cp_givenname ne '' ) {

-      $self->{contact}->save();
+      $self->{contact}->save(cascade => 1);

     }
 
     if( $self->{note}->subject ne '' && $self->{note}->body ne '' ) {
     }
 
     if( $self->{note}->subject ne '' && $self->{note}->body ne '' ) {

+
+      if ( !$self->{note_followup}->follow_up_date ) {
+        $::form->error($::locale->text('Date missing!'));
+      }
+

       $self->{note}->trans_id($self->{cv}->id);
       $self->{note}->save();
       $self->{note}->trans_id($self->{cv}->id);
       $self->{note}->save();

+

       $self->{note_followup}->save();
 
       $self->{note_followup}->save();
 

-      $self->{note} = SL::DB::Note->new();
-      $self->{note_followup} = SL::DB::FollowUp->new();
+      $self->{note_followup_link}->follow_up_id($self->{note_followup}->id);
+      $self->{note_followup_link}->trans_id($self->{cv}->id);
+      $self->{note_followup_link}->save();
+
+      SL::Helper::Flash::flash_later('info', $::locale->text('Follow-Up saved.'));

     }
 
     $self->{shipto}->trans_id($self->{cv}->id);
     }
 
     $self->{shipto}->trans_id($self->{cv}->id);

-    if( $self->{shipto}->shiptoname ne '' ) {
-      $self->{shipto}->save();
+    if(any { $self->{shipto}->$_ ne '' } qw(shiptoname shiptodepartment_1 shiptodepartment_2 shiptostreet shiptozipcode shiptocity shiptocountry shiptogln shiptocontact shiptophone shiptofax shiptoemail)) {
+      $self->{shipto}->save(cascade => 1);

     }
 
     my $snumbers = $self->is_vendor() ? 'vendornumber_'. $self->{cv}->vendornumber : 'customernumber_'. $self->{cv}->customernumber;
     }
 
     my $snumbers = $self->is_vendor() ? 'vendornumber_'. $self->{cv}->vendornumber : 'customernumber_'. $self->{cv}->customernumber;


@@ -137,6 +210,19 @@ sub _save {
       employee_id => SL::DB::Manager::Employee->current->id,
       addition => 'SAVED',
     )->save();
       employee_id => SL::DB::Manager::Employee->current->id,
       addition => 'SAVED',
     )->save();

+
+    if ( $::form->{delete_notes} ) {
+      foreach my $note_id (@{ $::form->{delete_notes} }) {
+        my $note = SL::DB::Note->new(id => $note_id)->load();
+        if ( $note->follow_up ) {
+          if ( $note->follow_up->follow_up_link ) {
+            $note->follow_up->follow_up_link->delete(cascade => 'delete');
+          }
+          $note->follow_up->delete(cascade => 'delete');
+        }
+        $note->delete(cascade => 'delete');
+      }
+    }

   }) || die($db->error);
 
 }
   }) || die($db->error);
 
 }


@@ -146,7 +232,21 @@ sub action_save {
 
   $self->_save();
 
 
   $self->_save();
 

-  $self->redirect_to(action => 'edit', id => $self->{cv}->id);
+  my @redirect_params = (
+    action => 'edit',
+    id     => $self->{cv}->id,
+    db     => ($self->is_vendor() ? 'vendor' : 'customer'),
+  );
+
+  if ( $self->{contact}->cp_id ) {
+    push(@redirect_params, contact_id => $self->{contact}->cp_id);
+  }
+
+  if ( $self->{shipto}->shipto_id ) {
+    push(@redirect_params, shipto_id => $self->{shipto}->shipto_id);
+  }
+
+  $self->redirect_to(@redirect_params);

 }
 
 sub action_save_and_close {
 }
 
 sub action_save_and_close {


@@ -166,18 +266,18 @@ sub _transaction {
 
   $self->_save();
 
 
   $self->_save();
 

-  my $callback = $::form->escape($::form->{callback}, 1);

   my $name = $::form->escape($self->{cv}->name, 1);
   my $db = $self->is_vendor() ? 'vendor' : 'customer';
 
   my $name = $::form->escape($self->{cv}->name, 1);
   my $db = $self->is_vendor() ? 'vendor' : 'customer';
 

-  my $url =
-    $script .'?'.
-    'action=add&'.
-    'vc='. $db .'&'.
-    $db .'_id=' . $self->{cv}->id .'&'.
-    $db .'='. $name .'&'.
-    'type='. $::form->{type} .'&'.
-    'callback='. $callback;
+  my $url = $self->url_for(
+    controller => $script,
+    action     => 'add',
+    vc         => $db,
+    $db .'_id' => $self->{cv}->id,
+    $db        => $name,
+    type       => $::form->{type},
+    callback   => $::form->{callback},
+  );

 
   print $::form->redirect_header($url);
 }
 
   print $::form->redirect_header($url);
 }


@@ -290,7 +390,7 @@ sub action_delete_contact {
       }
     }) || die($db->error);
 
       }
     }) || die($db->error);
 

-    $self->{contact} = SL::DB::Contact->new();
+    $self->{contact} = $self->_new_contact_object;

   }
 
   $self->action_edit();
   }
 
   $self->action_edit();


@@ -326,13 +426,17 @@ sub action_delete_shipto {
 sub action_search {
   my ($self) = @_;
 
 sub action_search {
   my ($self) = @_;
 

-  my $url = 'ct.pl?action=search&db='. ($self->is_vendor() ? 'vendor' : 'customer');
+  my @url_params = (
+    controller => 'ct.pl',
+    action => 'search',
+    db => $self->is_vendor() ? 'vendor' : 'customer',
+  );

 
   if ( $::form->{callback} ) {
 
   if ( $::form->{callback} ) {

-    $url .= '&callback='. $::from->escape($::form->{callback});
+    push(@url_params, callback => $::form->{callback});

   }
 
   }
 

-  print $::form->redirect_header($url);
+  $self->redirect_to(@url_params);

 }
 
 
 }
 
 


@@ -342,7 +446,7 @@ sub action_search_contact {
   my $url = 'ct.pl?action=search_contact&db=customer';
 
   if ( $::form->{callback} ) {
   my $url = 'ct.pl?action=search_contact&db=customer';
 
   if ( $::form->{callback} ) {

-    $url .= '&callback='. $::from->escape($::form->{callback});
+    $url .= '&callback='. $::form->escape($::form->{callback});

   }
 
   print $::form->redirect_header($url);
   }
 
   print $::form->redirect_header($url);


@@ -352,6 +456,8 @@ sub action_search_contact {
 sub action_get_delivery {
   my ($self) = @_;
 
 sub action_get_delivery {
   my ($self) = @_;
 

+  $::auth->assert('sales_all_edit');
+

   my $dbh = $::form->get_standard_dbh();
 
   my ($arap, $db, $qty_sign);
   my $dbh = $::form->get_standard_dbh();
 
   my ($arap, $db, $qty_sign);


@@ -365,21 +471,24 @@ sub action_get_delivery {
     $qty_sign = '';
   }
 
     $qty_sign = '';
   }
 

-  my $where = ' WHERE 1=1 ';
+  my $where = ' WHERE 1=1';

   my @values;
 
   if ( !$self->is_vendor() && $::form->{shipto_id} && $::form->{shipto_id} ne 'all' ) {
   my @values;
 
   if ( !$self->is_vendor() && $::form->{shipto_id} && $::form->{shipto_id} ne 'all' ) {

-    $where .= "AND ${arap}.shipto_id = ?";
+    $where .= " AND ${arap}.shipto_id = ?";

     push(@values, $::form->{shipto_id});
     push(@values, $::form->{shipto_id});

+  } else {
+    $where .= " AND ${arap}.${db}_id = ?";
+    push(@values, $::form->{id});

   }
 
   if ( $::form->{delivery_from} ) {
   }
 
   if ( $::form->{delivery_from} ) {

-    $where .= "AND ${arap}.transdate >= ?";
+    $where .= " AND ${arap}.transdate >= ?";

     push(@values, conv_date($::form->{delivery_from}));
   }
 
   if ( $::form->{delivery_to} ) {
     push(@values, conv_date($::form->{delivery_from}));
   }
 
   if ( $::form->{delivery_to} ) {

-    $where .= "AND ${arap}.transdate <= ?";
+    $where .= " AND ${arap}.transdate <= ?";

     push(@values, conv_date($::form->{delivery_to}));
   }
 
     push(@values, conv_date($::form->{delivery_to}));
   }
 


@@ -408,7 +517,8 @@ sub action_get_delivery {
        ON p.id = i.parts_id
 
      LEFT JOIN oe
        ON p.id = i.parts_id
 
      LEFT JOIN oe

-       ON (oe.ordnumber = ${arap}.ordnumber AND NOT ${arap}.ordnumber = '')
+       ON (oe.ordnumber = ${arap}.ordnumber AND NOT ${arap}.ordnumber = ''
+           AND ". ($arap eq 'ar' ? 'oe.customer_id IS NOT NULL' : 'oe.vendor_id IS NOT NULL') ." )

 
      ${where}
      ORDER BY ${arap}.transdate DESC LIMIT 15";
 
      ${where}
      ORDER BY ${arap}.transdate DESC LIMIT 15";


@@ -421,16 +531,19 @@ sub action_get_delivery {
 sub action_ajaj_get_shipto {
   my ($self) = @_;
 
 sub action_ajaj_get_shipto {
   my ($self) = @_;
 

-  my $data = {
+  my $data = {};
+  $data->{shipto} = {

     map(
       {
         my $name = 'shipto'. $_;
         $name => $self->{shipto}->$name;
       }
     map(
       {
         my $name = 'shipto'. $_;
         $name => $self->{shipto}->$name;
       }

-      qw(_id name department_1 department_2 street zipcode city country contact phone fax email)
+      qw(_id name department_1 department_2 street zipcode city gln country contact phone fax email)

     )
   };
 
     )
   };
 

+  $data->{shipto_cvars} = $self->_prepare_cvar_configs_for_ajaj($self->{shipto}->cvars_by_config);
+

   $self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
 }
 
   $self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
 }
 


@@ -457,57 +570,70 @@ sub action_ajaj_get_contact {
     )
   };
 
     )
   };
 

-  $data->{contact_cvars} = {
-    map(
-      {
-        if ( $_->config->type eq 'number' ) {
-          $_->config->name => $::form->format_amount(\%::myconfig, $_->value, -2);
-        } else {
-          $_->config->name => $_->value;
-        }
-      }
-      grep(
-        { $_->is_valid; }
-        @{$self->{contact}->cvars_by_config}
-      )
-    )
-  };
+  $data->{contact_cvars} = $self->_prepare_cvar_configs_for_ajaj($self->{contact}->cvars_by_config);

 
   $self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
 }
 
 
   $self->render(\SL::JSON::to_json($data), { type => 'json', process => 0 });
 }
 

-sub action_ajaj_customer_autocomplete {
+sub action_ajaj_autocomplete {

   my ($self, %params) = @_;
 
   my ($self, %params) = @_;
 

-  my $limit = $::form->{limit} || 20;
-  my $type  = $::form->{type}  || {};
-  my $query = { ilike => '%'. $::form->{term} .'%' };
+  my ($model, $manager, $number, $matches);
+
+  # first see if this is customer or vendor picking
+  if ($::form->{type} eq 'customer') {
+     $model   = $self->customer_models;
+     $manager = 'SL::DB::Manager::Customer';
+     $number  = 'customernumber';
+  } elsif ($::form->{type} eq 'vendor')  {
+     $model   = $self->vendor_models;
+     $manager = 'SL::DB::Manager::Vendor';
+     $number  = 'vendornumber';
+  } else {
+     die "unknown type $::form->{type}";
+  }

 
 

-  my @filter;
-  push(
-    @filter,
-    $::form->{column} ? ($::form->{column} => $query) : (or => [ customernumber => $query, name => $query ])
-  );
+  # if someone types something, and hits enter, assume he entered the full name.
+  # if something matches, treat that as sole match
+  # unfortunately get_models can't do more than one per package atm, so we d it
+  # the oldfashioned way.
+  if ($::form->{prefer_exact}) {
+    my $exact_matches;
+    if (1 == scalar @{ $exact_matches = $manager->get_all(
+      query => [
+        obsolete => 0,
+        (salesman_id => SL::DB::Manager::Employee->current->id) x !$::auth->assert('customer_vendor_all_edit', 1),
+        or => [
+          name    => { ilike => $::form->{filter}{'all:substr:multi::ilike'} },
+          $number => { ilike => $::form->{filter}{'all:substr:multi::ilike'} },
+        ]
+      ],
+      limit => 2,
+    ) }) {
+      $matches = $exact_matches;
+    }
+  }

 
 

-  my $customers = SL::DB::Manager::Customer->get_all(query => [ @filter ], limit => $limit);
-  my $value_col = $::form->{column} || 'name';
+  $matches //= $model->get;
+
+  my @hashes = map {
+   +{
+     value       => $_->displayable_name,
+     label       => $_->displayable_name,
+     id          => $_->id,
+     $number     => $_->$number,
+     name        => $_->name,
+     type        => $::form->{type},
+     cvars       => { map { ($_->config->name => { value => $_->value_as_text, is_valid => $_->is_valid }) } @{ $_->cvars_by_config } },
+    }
+  } @{ $matches };

 
 

-  my $data = [
-    map(
-      {
-        {
-          value => $_->can($value_col)->($_),
-          label => $_->displayable_name,
-          id    => $_->id,
-          customernumber => $_->customernumber,
-          name  => $_->name,
-        }
-      }
-      @{$customers}
-    )
-  ];
+  $self->render(\ SL::JSON::to_json(\@hashes), { layout => 0, type => 'json', process => 0 });
+}

 
 

-  $self->render(\SL::JSON::to_json($data), { layout => 0, type => 'json' });
+sub action_test_page {
+  $::request->{layout}->add_javascripts('autocomplete_customer.js');
+  $_[0]->render('customer_vendor/test_page');

 }
 
 sub is_vendor {
 }
 
 sub is_vendor {


@@ -526,7 +652,7 @@ sub is_orphaned {
   }
 
   my $arap      = $self->is_vendor ? 'ap' : 'ar';
   }
 
   my $arap      = $self->is_vendor ? 'ap' : 'ar';

-  my $num_args  = 2;
+  my $num_args  = 3;

 
   my $cv = $self->is_vendor ? 'vendor' : 'customer';
 
 
   my $cv = $self->is_vendor ? 'vendor' : 'customer';
 


@@ -541,6 +667,13 @@ sub is_orphaned {
     SELECT a.id
     FROM oe a
     JOIN '. $cv .' ct ON (a.'. $cv .'_id = ct.id)
     SELECT a.id
     FROM oe a
     JOIN '. $cv .' ct ON (a.'. $cv .'_id = ct.id)

+    WHERE ct.id = ?
+
+    UNION
+
+    SELECT a.id
+    FROM delivery_orders a
+    JOIN '. $cv .' ct ON (a.'. $cv .'_id = ct.id)

     WHERE ct.id = ?';
 
 
     WHERE ct.id = ?';
 
 


@@ -556,6 +689,17 @@ sub is_orphaned {
   return $self->{_is_orphaned} = !$dummy;
 }
 
   return $self->{_is_orphaned} = !$dummy;
 }
 

+sub _copy_form_to_cvars {
+  my ($self, %params) = @_;
+
+  foreach my $cvar (@{ $params{target}->cvars_by_config }) {
+    my $value = $params{source}->{$cvar->config->name};
+    $value    = $::form->parse_amount(\%::myconfig, $value) if $cvar->config->type eq 'number';
+
+    $cvar->value($value);
+  }
+}
+

 sub _instantiate_args {
   my ($self) = @_;
 
 sub _instantiate_args {
   my ($self) = @_;
 


@@ -568,38 +712,37 @@ sub _instantiate_args {
       $self->{cv} = SL::DB::Customer->new(id => $::form->{cv}->{id})->load();
     }
   } else {
       $self->{cv} = SL::DB::Customer->new(id => $::form->{cv}->{id})->load();
     }
   } else {

-    if ( $self->is_vendor() ) {
-      $self->{cv} = SL::DB::Vendor->new();
-    } else {
-      $self->{cv} = SL::DB::Customer->new();
-    }
+    $self->{cv} = $self->_new_customer_vendor_object;

   }
   $self->{cv}->assign_attributes(%{$::form->{cv}});
 
   }
   $self->{cv}->assign_attributes(%{$::form->{cv}});
 

-  foreach my $cvar (@{$self->{cv}->cvars_by_config()}) {
-    my $value = $::form->{cv_cvars}->{$cvar->config->name};
-
-    if ( $cvar->config->type eq 'number' ) {
-      $value = $::form->parse_amount(\%::myconfig, $value);
-    }
-
-    $cvar->value($value);
+  if ( $self->is_customer() && $::form->{cv}->{taxincluded_checked} eq '' ) {
+    $self->{cv}->taxincluded_checked(undef);

   }
 
   }
 

+  $self->{cv}->hourly_rate($::instance_conf->get_customer_hourly_rate) if $self->is_customer && !$self->{cv}->hourly_rate;
+

   if ( $::form->{note}->{id} ) {
     $self->{note} = SL::DB::Note->new(id => $::form->{note}->{id})->load();
   if ( $::form->{note}->{id} ) {
     $self->{note} = SL::DB::Note->new(id => $::form->{note}->{id})->load();

+    $self->{note_followup} = $self->{note}->follow_up;
+    $self->{note_followup_link} = $self->{note_followup}->follow_up_link;

   } else {
     $self->{note} = SL::DB::Note->new();
   } else {
     $self->{note} = SL::DB::Note->new();

+    $self->{note_followup} = SL::DB::FollowUp->new();
+    $self->{note_followup_link} = SL::DB::FollowUpLink->new();

   }
   }

+

   $self->{note}->assign_attributes(%{$::form->{note}});
   $self->{note}->created_by($curr_employee->id);
   $self->{note}->trans_module('ct');
 
   $self->{note}->assign_attributes(%{$::form->{note}});
   $self->{note}->created_by($curr_employee->id);
   $self->{note}->trans_module('ct');
 

-  $self->{note_followup} = SL::DB::FollowUp->new();

   $self->{note_followup}->assign_attributes(%{$::form->{note_followup}});
   $self->{note_followup}->note($self->{note});
   $self->{note_followup}->created_by($curr_employee->id);
 
   $self->{note_followup}->assign_attributes(%{$::form->{note_followup}});
   $self->{note_followup}->note($self->{note});
   $self->{note_followup}->created_by($curr_employee->id);
 

+  $self->{note_followup_link}->trans_type($self->is_vendor() ? 'vendor' : 'customer');
+  $self->{note_followup_link}->trans_info($self->{cv}->name);
+

   if ( $::form->{shipto}->{shipto_id} ) {
     $self->{shipto} = SL::DB::Shipto->new(shipto_id => $::form->{shipto}->{shipto_id})->load();
   } else {
   if ( $::form->{shipto}->{shipto_id} ) {
     $self->{shipto} = SL::DB::Shipto->new(shipto_id => $::form->{shipto}->{shipto_id})->load();
   } else {


@@ -611,19 +754,13 @@ sub _instantiate_args {
   if ( $::form->{contact}->{cp_id} ) {
     $self->{contact} = SL::DB::Contact->new(cp_id => $::form->{contact}->{cp_id})->load();
   } else {
   if ( $::form->{contact}->{cp_id} ) {
     $self->{contact} = SL::DB::Contact->new(cp_id => $::form->{contact}->{cp_id})->load();
   } else {

-    $self->{contact} = SL::DB::Contact->new();
+    $self->{contact} = $self->_new_contact_object;

   }
   $self->{contact}->assign_attributes(%{$::form->{contact}});
 
   }
   $self->{contact}->assign_attributes(%{$::form->{contact}});
 

-  foreach my $cvar (@{$self->{contact}->cvars_by_config()}) {
-    my $value = $::form->{contact_cvars}->{$cvar->config->name};
-
-    if ( $cvar->config->type eq 'number' ) {
-      $value = $::form->parse_amount(\%::myconfig, $value);
-    }
-
-    $cvar->value($value);
-  }
+  $self->_copy_form_to_cvars(target => $self->{cv},      source => $::form->{cv_cvars});
+  $self->_copy_form_to_cvars(target => $self->{contact}, source => $::form->{contact_cvars});
+  $self->_copy_form_to_cvars(target => $self->{shipto},  source => $::form->{shipto_cvars});

 }
 
 sub _load_customer_vendor {
 }
 
 sub _load_customer_vendor {


@@ -635,8 +772,15 @@ sub _load_customer_vendor {
     $self->{cv} = SL::DB::Customer->new(id => $::form->{id})->load();
   }
 
     $self->{cv} = SL::DB::Customer->new(id => $::form->{id})->load();
   }
 

-  $self->{note} = SL::DB::Note->new();
-  $self->{note_followup} = SL::DB::FollowUp->new();
+  if ( $::form->{note_id} ) {
+    $self->{note} = SL::DB::Note->new(id => $::form->{note_id})->load();
+    $self->{note_followup} = $self->{note}->follow_up;
+    $self->{note_followup_link} = $self->{note_followup}->follow_up_link;
+  } else {
+    $self->{note} = SL::DB::Note->new();
+    $self->{note_followup} = SL::DB::FollowUp->new();
+    $self->{note_followup_link} = SL::DB::FollowUpLink->new();
+  }

 
   if ( $::form->{shipto_id} ) {
     $self->{shipto} = SL::DB::Shipto->new(shipto_id => $::form->{shipto_id})->load();
 
   if ( $::form->{shipto_id} ) {
     $self->{shipto} = SL::DB::Shipto->new(shipto_id => $::form->{shipto_id})->load();


@@ -655,18 +799,25 @@ sub _load_customer_vendor {
       die($::locale->text('Error'));
     }
   } else {
       die($::locale->text('Error'));
     }
   } else {

-    $self->{contact} = SL::DB::Contact->new();
+    $self->{contact} = $self->_new_contact_object;

   }
 }
 
   }
 }
 

+sub _check_customer_vendor_all_edit {
+  my ($self) = @_;
+
+  unless ($::auth->assert('customer_vendor_all_edit', 1)) {
+    die($::locale->text("You don't have the rights to edit this customer.") . "\n")
+      if $self->{cv}->is_customer and
+         SL::DB::Manager::Employee->current->id != $self->{cv}->salesman_id;
+  };
+};
+

 sub _create_customer_vendor {
   my ($self) = @_;
 
 sub _create_customer_vendor {
   my ($self) = @_;
 

-  if ( $self->is_vendor() ) {
-    $self->{cv} = SL::DB::Vendor->new();
-  } else {
-    $self->{cv} = SL::DB::Customer->new();
-  }
+  $self->{cv} = $self->_new_customer_vendor_object;
+  $self->{cv}->currency_id($::instance_conf->get_currency_id());

 
   $self->{note} = SL::DB::Note->new();
 
 
   $self->{note} = SL::DB::Note->new();
 


@@ -674,7 +825,7 @@ sub _create_customer_vendor {
 
   $self->{shipto} = SL::DB::Shipto->new();
 
 
   $self->{shipto} = SL::DB::Shipto->new();
 

-  $self->{contact} = SL::DB::Contact->new();
+  $self->{contact} = $self->_new_contact_object;

 }
 
 sub _pre_render {
 }
 
 sub _pre_render {


@@ -686,7 +837,7 @@ sub _pre_render {
 
   $self->{all_business} = SL::DB::Manager::Business->get_all();
 
 
   $self->{all_business} = SL::DB::Manager::Business->get_all();
 

-  $self->{all_employees} = SL::DB::Manager::Employee->get_all();
+  $self->{all_employees} = SL::DB::Manager::Employee->get_all(query => [ deleted => 0 ]);

 
   $query =
     'SELECT DISTINCT(greeting)
 
   $query =
     'SELECT DISTINCT(greeting)


@@ -720,14 +871,33 @@ sub _pre_render {
 
   $self->{all_languages} = SL::DB::Manager::Language->get_all();
 
 
   $self->{all_languages} = SL::DB::Manager::Language->get_all();
 

-  $self->{all_taxzones} = SL::DB::Manager::TaxZone->get_all();
+  $self->{all_taxzones} = SL::DB::Manager::TaxZone->get_all_sorted();
+
+  if ( $::instance_conf->get_vertreter() ) {
+    $query =
+      'SELECT id
+       FROM business
+       WHERE salesman';
+    my $business_ids = [
+      map(
+        { $_->{id}; }
+        selectall_hashref_query($::form, $dbh, $query)
+      )
+    ];

 
 

-  #Employee:
-  #TODO: ALL_SALESMAN
-  #TODO: ALL_SALESMAN_CUSTOMERS
+    if ( $business_ids->[0] ) {
+      $self->{all_salesman_customers} = SL::DB::Manager::Customer->get_all(query => [business_id => $business_ids]);
+    } else {
+      $self->{all_salesman_customers} = [];
+    }
+  } else {
+    $self->{all_salesmen} = SL::DB::Manager::Employee->get_all(query => [ or => [ id => $self->{cv}->salesman_id,  deleted => 0 ] ]);
+  }

 
   $self->{all_payment_terms} = SL::DB::Manager::PaymentTerm->get_all();
 
 
   $self->{all_payment_terms} = SL::DB::Manager::PaymentTerm->get_all();
 

+  $self->{all_delivery_terms} = SL::DB::Manager::DeliveryTerm->get_all();
+

   $self->{all_pricegroups} = SL::DB::Manager::Pricegroup->get_all();
 
   $query =
   $self->{all_pricegroups} = SL::DB::Manager::Pricegroup->get_all();
 
   $query =


@@ -748,9 +918,126 @@ sub _pre_render {
   $self->{shiptos} = $self->{cv}->shipto;
   $self->{shiptos} ||= [];
 
   $self->{shiptos} = $self->{cv}->shipto;
   $self->{shiptos} ||= [];
 

-  $self->{template_args} = {};
+  $self->{notes} = SL::DB::Manager::Note->get_all(
+    query => [
+      trans_id => $self->{cv}->id,
+      trans_module => 'ct',
+    ],
+    with_objects => ['follow_up'],
+  );
+
+  $self->{template_args} ||= {};

 
   $::request->{layout}->add_javascripts('autocomplete_customer.js');
 
   $::request->{layout}->add_javascripts('autocomplete_customer.js');

+  $::request->{layout}->add_javascripts('kivi.CustomerVendor.js');
+}
+
+sub _prepare_cvar_configs_for_ajaj {
+  my ($self, $cvars) = @_;
+
+  return {
+    map {
+      my $cvar   = $_;
+      my $result = { type => $cvar->config->type };
+
+      if ($cvar->config->type eq 'number') {
+        $result->{value} = $::form->format_amount(\%::myconfig, $cvar->value, -2);
+
+      } elsif ($result->{type} eq 'date') {
+        $result->{value} = $cvar->value ? $cvar->value->to_kivitendo : undef;
+
+      } elsif ($result->{type} =~ m{customer|vendor|part}) {
+        my $object       = $cvar->value;
+        my $method       = $result->{type} eq 'part' ? 'description' : 'name';
+
+        $result->{id}    = int($cvar->number_value) || undef;
+        $result->{value} = $object ? $object->$method // '' : '';
+
+      } else {
+        $result->{value} = $cvar->value;
+      }
+
+      ( $cvar->config->name => $result )
+
+    } grep { $_->is_valid } @{ $cvars }
+  };
+}
+
+sub normalize_name {
+  my ($self) = @_;
+
+  # check if feature is enabled (select normalize_vc_names from defaults)
+  return unless ($::instance_conf->get_normalize_vc_names);
+
+  return unless $self->{cv};
+  my $name = $self->{cv}->name;
+  $name =~ s/\s+$//;
+  $name =~ s/^\s+//;
+  $name =~ s/\s+/ /g;
+  $self->{cv}->name($name);
+}
+
+sub home_address_for_google_maps {
+  my ($self)  = @_;
+
+  my $address = $::instance_conf->get_address // '';
+  $address    =~ s{^\s+|\s+$|\r+}{}g;
+  $address    =~ s{\n+}{,}g;
+  $address    =~ s{\s+}{ }g;
+
+  return $address;
+}
+
+sub init_customer_models {
+  my ($self) = @_;
+
+  SL::Controller::Helper::GetModels->new(
+    controller   => $self,
+    model        => 'Customer',
+    sorted => {
+      _default  => {
+        by => 'customernumber',
+        dir  => 1,
+      },
+      customernumber => t8('Customer Number'),
+    },
+    query => [
+     ( salesman_id => SL::DB::Manager::Employee->current->id) x !$::auth->assert('customer_vendor_all_edit', 1),
+    ],
+  );
+}
+
+sub init_vendor_models {
+  my ($self) = @_;
+
+  SL::Controller::Helper::GetModels->new(
+    controller => $self,
+    model      => 'Vendor',
+    sorted => {
+      _default  => {
+        by => 'vendornumber',
+        dir  => 1,
+      },
+      vendornumber => t8('Vendor Number'),
+    },
+  );
+}
+
+sub _new_customer_vendor_object {
+  my ($self) = @_;
+
+  my $class  = 'SL::DB::' . ($self->is_vendor ? 'Vendor' : 'Customer');
+  return $class->new(
+    contacts         => [],
+    shipto           => [],
+    custom_variables => [],
+  );
+}
+
+sub _new_contact_object {
+  my ($self) = @_;
+
+  return SL::DB::Contact->new(custom_variables => []);

 }
 
 1;
 }
 
 1;