CustomVariables: Zuweisen von zu parsenden Inhalten

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.

diff --git a/SL/DB/CustomVariable.pm b/SL/DB/CustomVariable.pm
index e48c1d82971d955f97dd2b8d4e7d50a166af19bc..08dd663c809d0450fb0c2a3c7b0517ea88ef4f48 100644 (file)
--- a/SL/DB/CustomVariable.pm
+++ b/SL/DB/CustomVariable.pm
@@ -35,10 +35,14 @@ sub parse_value {
 
   my $unparsed = delete $self->{__unparsed_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);
   }
 
     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);
   }
   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;
   if (scalar(@_) > 1) {
     $self->unparsed_value($_[1]);
     $self->parse_value;

+    @_ = ($self);

   }
 
   goto &bool_value      if $type eq 'bool';
   }
 
   goto &bool_value      if $type eq 'bool';