Auftrags-Controller: item-Methoden richtig nutzen, nicht als Hash-Elemente.

diff --git a/SL/Controller/Order.pm b/SL/Controller/Order.pm
index b6cdef4..eff55ca 100644 (file)
--- a/SL/Controller/Order.pm
+++ b/SL/Controller/Order.pm
@@ -144,11 +144,11 @@ sub action_add_item {
 
   my %new_attr;
   $new_attr{part}        = $part;
-  $new_attr{description} = $part->description if ! $item->{description};
-  $new_attr{qty}         = 1.0                if ! $item->{qty};
+  $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};
+  $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;