CVar-Helper: Beim Parsen darauf achten, ob unparsed_value ein Objekt enthält.
authorBernd Bleßmann <bernd@kivitendo-premium.de>
Mon, 14 Sep 2015 15:01:18 +0000 (17:01 +0200)
committerBernd Bleßmann <bernd@kivitendo-premium.de>
Mon, 14 Sep 2015 15:01:18 +0000 (17:01 +0200)
Um Objekte mit der value-Methode einer CVar vom Typ Customer, Vendor oder Part
zuweisen zu können, muss beim Parsen die Id des Objekts ermittelt werden.

Dieses wird z.B. im Helper (SL::DB::Helper::CustomVariables.pm) verwendet, um
bei neuen cvars eines sub-modules den Wert der Basis-CVar zu setzen, denn die
value-Methode der CVar (hier der Basis-CVar) liefert ein Objekt zurück.

Ohne diesen Patch gab es einen Fehler in der Art:
"No such SL::DB::Part where id = 111286144 at SL/DB/Helper/CustomVariables.pm
line 198"

SL/DB/CustomVariable.pm

index 46428fe..26ff748 100644 (file)
@@ -4,6 +4,9 @@
 package SL::DB::CustomVariable;
 
 use strict;
+
+use List::MoreUtils qw(any);
+
 use SL::DB::MetaSetup::CustomVariable;
 
 __PACKAGE__->meta->initialize;
@@ -36,7 +39,9 @@ sub parse_value {
   my $unparsed = delete $self->{__unparsed_value};
 
   if ($type =~ m{^(?:customer|vendor|part|number)}) {
-    return $self->number_value(defined($unparsed) ? $unparsed * 1 : undef);
+    return $self->number_value(!defined($unparsed) ? undef
+                               : (any { ref($unparsed) eq $_ } qw(SL::DB::Customer SL::DB::Vendor SL::DB::Part)) ? $unparsed->id * 1
+                               : $unparsed * 1);
   }
 
   if ($type =~ m{^(?:bool)}) {