[Rt-commit] r8303 - rt/branches/3.7-EXPERIMENTAL-TUNIS/lib/RT/Interface

ruz at bestpractical.com ruz at bestpractical.com
Thu Jul 26 17:25:48 EDT 2007


Author: ruz
Date: Thu Jul 26 17:25:48 2007
New Revision: 8303

Modified:
   rt/branches/3.7-EXPERIMENTAL-TUNIS/lib/RT/Interface/Web.pm

Log:
* use defined && length check instead of just boolean check

Modified: rt/branches/3.7-EXPERIMENTAL-TUNIS/lib/RT/Interface/Web.pm
==============================================================================
--- rt/branches/3.7-EXPERIMENTAL-TUNIS/lib/RT/Interface/Web.pm	(original)
+++ rt/branches/3.7-EXPERIMENTAL-TUNIS/lib/RT/Interface/Web.pm	Thu Jul 26 17:25:48 2007
@@ -1260,7 +1260,8 @@
         # to fake it
         if ( $arg eq 'Values-Magic' ) {
             # We don't care about the magic, if there's really a values element;
-            next if $args{'ARGS'}->{'Value'} || $args{'ARGS'}->{'Values'};
+            next if defined $args{'ARGS'}->{'Value'} && length $args{'ARGS'}->{'Value'};
+            next if defined $args{'ARGS'}->{'Values'} && length $args{'ARGS'}->{'Values'};
 
             # "Empty" values does not mean anything for Image and Binary fields
             next if $cf_type =~ /^(?:Image|Binary)$/;


More information about the Rt-commit mailing list