Auftrags-Controller: item->id per Hand setzten (assign_attributes lässt Primärschlüss...

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

diff --git a/SL/Controller/Order.pm b/SL/Controller/Order.pm
index c4624072a731ac726f1bfc2dcf80451fdc537d45..b6cdef4753557ba1b45b0bc1964ab086cbde62e5 100644 (file)
--- a/SL/Controller/Order.pm
+++ b/SL/Controller/Order.pm
@@ -82,8 +82,14 @@ sub action_update {
 sub action_save {
   my ($self) = @_;
 
-  $self->_save();
+  my $errors = $self->_save();
 
+  if (scalar @{ $errors }) {
+    $self->js->flash('error', $_) foreach @{ $errors };
+    return $self->js->render($self);
+  }
+
+  flash_later('info', 'The order has been saved');
   my @redirect_params = (
     action => 'edit',
     type   => $self->type,
@@ -137,14 +143,15 @@ sub action_add_item {
   my $cv_discount = $self->order->customer? $self->order->customer->discount : 0.0;
 
   my %new_attr;
-  $new_attr{id}          = join('_', 'new', Time::HiRes::gettimeofday(), int rand 1000000000000);
   $new_attr{part}        = $part;
   $new_attr{description} = $part->description if ! $item->{description};
   $new_attr{qty}         = 1.0                if ! $item->{qty};
   $new_attr{unit}        = $part->unit;
   $new_attr{sellprice}   = $part->sellprice   if ! $item->{sellprice};
   $new_attr{discount}    = $cv_discount       if ! $item->{discount};
+
   $item->assign_attributes(%new_attr);
+  $item->id(join('_', 'new', Time::HiRes::gettimeofday(), int rand 1000000000000)) if !$item->id;
 
   $self->order->add_items($item);
 
@@ -308,12 +315,15 @@ sub _recalc {
 sub _save {
   my ($self) = @_;
 
+  my $errors = [];
   my $db = $self->order->db;
 
   $db->do_transaction(
     sub {
       $self->order->save();
-  }) || die($db->error);
+  }) || push(@{$errors}, $db->error);
+
+  return $errors;
 }