kivitendo-erp

Web-based ERP system for the German market

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:

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

   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:

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;