SL::DB::CustomVariable->value für Typ Nummer auch wirklich Nummer zurückgeben
authorMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 16 Jul 2015 15:04:54 +0000 (17:04 +0200)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 16 Jul 2015 15:07:39 +0000 (17:07 +0200)
Da die Spalte number_value in der DB vom Typ numeric() ist, wird das von
Rose als String eingelesen. Das bedeutet, dass ->number_value z.B. der
Wert '0.00000' liefert, was im Booleschen Kontext trueish ist – nicht
das, was der Programmierer erwarten würde.

Statt dessen erwartet der Programmierer, dass !$zahl für den Wert 0 auch
wirklich zutrifft.

Daher sollte ->value für CVars vom Typ Nummer auch wirklich eine Zahl
zurückgeben, was durch ein * 1 erzwungen wird. Ausnahme: undef, was
weiterhin undef bleibt.

SL/DB/CustomVariable.pm

index 08dd663..0971e64 100644 (file)
@@ -63,7 +63,10 @@ sub value {
 
   goto &bool_value      if $type eq 'bool';
   goto &timestamp_value if $type eq 'timestamp';
-  goto &number_value    if $type eq 'number';
+
+  if ($type eq 'number') {
+    return defined($self->number_value) ? $self->number_value * 1 : undef;
+  }
 
   if ( $type eq 'customer' ) {
     require SL::DB::Customer;