weitere Verbesserungen für die helper-funktion

Die Funktion sollte unabhängig vom Variablen-Namen sein, damit dies
langfristig einfacher wartbar ist.

diff --git a/SL/DB/OrderItem.pm b/SL/DB/OrderItem.pm
index 0c28a52435faa54aa49b2abf54f5ffab76591824..5e8aad0cc759806fc05def344eccf58dfbe900cd 100644 (file)
--- a/SL/DB/OrderItem.pm
+++ b/SL/DB/OrderItem.pm
@@ -37,14 +37,13 @@ sub shipped_qty {
 sub delivered_qty {
   my ($self) = @_;
 
 sub delivered_qty {
   my ($self) = @_;
 

-  $self->_delivered_qty;
-  return $self->{delivered_qty};
+  return $self->_delivered_qty;

 }
 
 sub _delivered_qty {
   my ($self) = @_;
 
 }
 
 sub _delivered_qty {
   my ($self) = @_;
 

-  return if $self->{delivered_qty};
+  return $self->{delivered_qty} if $self->{delivered_qty};

 
   my $d_orders = $self->order->linked_records(direction => 'to', to => 'SL::DB::DeliveryOrder');
 
 
   my $d_orders = $self->order->linked_records(direction => 'to', to => 'SL::DB::DeliveryOrder');
 


@@ -54,6 +53,8 @@ sub _delivered_qty {
 
   require SL::AM;
   $self->{delivered_qty} =  sum(map { AM->convert_unit($_->unit => $self->unit) * $_->qty } @doi_delivered);
 
   require SL::AM;
   $self->{delivered_qty} =  sum(map { AM->convert_unit($_->unit => $self->unit) * $_->qty } @doi_delivered);

+
+  return $self->{delivered_qty};

 }
 
 sub value_of_goods {
 }
 
 sub value_of_goods {