projekt kivitendo

# 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:

my ($self) = @_;

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

-

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

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

$self->check_below_minimum_stock();

-

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.

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});

}

```

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

```