From: Sven Schöling <s.schoeling@linet-services.de>
Date: Thu, 5 Mar 2015 14:43:25 +0000 (+0100)
Subject: CustomVariables: Zuweisen von zu parsenden Inhalten
X-Git-Tag: release-3.2.1~49
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=2b9e50ba726ae1cebd2d51fea7635dedeeae5e1a;p=kivitendo-erp.git

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 e48c1d829..08dd663c8 100644
--- a/SL/DB/CustomVariable.pm
+++ b/SL/DB/CustomVariable.pm
@@ -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';