CustomVariables: Zuweisen von zu parsenden Inhalten
authorSven Schöling <s.schoeling@linet-services.de>
Thu, 5 Mar 2015 14:43:25 +0000 (15:43 +0100)
committerSven Schöling <s.schoeling@linet-services.de>
Thu, 5 Mar 2015 14:43:25 +0000 (15:43 +0100)
2 Probleme.

1. Booleans wurden beim Parsen in den numeric Slot gesteckt
2. Die zwei-wertige Version von value() hat das zweite Argument nicht
   aus dem Stack entfernt, und damit beim goto weitergegeben an die
   unterliegenden Rose-Accessoren. Das hat dann das parsen wieder
   zunichte gemacht.

SL/DB/CustomVariable.pm

index e48c1d8..08dd663 100644 (file)
@@ -35,10 +35,14 @@ sub parse_value {
 
   my $unparsed = delete $self->{__unparsed_value};
 
-  if ($type =~ m{^(?:customer|vendor|part|bool|number)}) {
+  if ($type =~ m{^(?:customer|vendor|part|number)}) {
     return $self->number_value(defined($unparsed) ? $unparsed * 1 : undef);
   }
 
+  if ($type =~ m{^(?:bool)}) {
+    return $self->bool_value(defined($unparsed) ? !!$unparsed : undef);
+  }
+
   if ($type =~ m{^(?:date|timestamp)}) {
     return $self->timestamp_value(defined($unparsed) ? DateTime->from_kivitendo($unparsed) : undef);
   }
@@ -54,6 +58,7 @@ sub value {
   if (scalar(@_) > 1) {
     $self->unparsed_value($_[1]);
     $self->parse_value;
+    @_ = ($self);
   }
 
   goto &bool_value      if $type eq 'bool';