projekt kivitendo

Anmerkungen jan:

1) send_email macht nicht das was es soll, es sendet nicht nur stumpf eine E-Mail, sondern prüft auch auf den Zustand des Background Jobs.

   Das ist aber eher in der Verantwortung der aufrufenden Routine, damit das besser abgegrenzt ist:

 ```@@ -56,9 +57,6 @@ sub _email_user {

 sub send_email {

   my ($self) = @_;

 -  my @ids = @{$self->{job_obj}->data_as_hash->{ids}};

 -  return unless (scalar @ids && $self->{config} && $self->{config}->{send_email_to});

 -

   my $user  = $self->_email_user;

   my $email = $self->{job_obj}->data_as_hash->{mail_to} ? $self->{job_obj}->data_as_hash->{mail_to}

             : $user                                     ? $user->get_config_value('email')

 @@ -140,13 +138,15 @@ sub run {

   $self->check_below_minimum_stock();

 -  $self->send_email();

 -

    my $data = $job_obj->data_as_hash;

 -  die $data->{errors} if $data->{errors};

 +  # errors indicate we have to inform the user

 +  if ($data->{errors}) {

 +    $self->send_email();

 +    die $data->{errors} if $data->{errors};

 +  }

 ```

 2) Die Routine send_email springt raus, falls sich in der kivitendo.conf kein mit einer gültigen E-Mail-Adresse befindet.

   Das ist für den kivi-Admin schwierig zu konfigurieren und für Admins die nur an der Oberfläche administrieren überhaupt einstellbar

   Ferner wird ja im weiteren Verlauf auf andere Mail-Adressen geprüft und _email_user ist ja auch schon ausgelagert.

   2.1) Prüfung auf valide Konfig in methode

 ```sub _email_user {

    my ($self) = @_;

 +  return unless ($self->{config} && $self->{config}->{send_email_to});

    $self->{email_user} ||= SL::DB::Manager::AuthUser->find_by(login => $self->{config}->{send_email_to});

  }

 ```

  2.2) Nicht zu früh bei send_email abbrechen:

   ```sub send_email {

   my ($self) = @_;

 -  my @ids = @{$self->{job_obj}->data_as_hash->{ids}};

 -  return unless (scalar @ids && $self->{config} && $self->{config}->{send_email_to});

```

 3.) Kosmetik

 ```-  return ;

 +  return;

 ```