LP#1418772: (follow-up) tweak undef-edness check
authorGalen Charlton <gmc@esilibrary.com>
Thu, 19 Feb 2015 15:59:33 +0000 (15:59 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 19 Feb 2015 16:08:08 +0000 (16:08 +0000)
Use the Perl defined() function rather than "eq undef"
to avoid logging warnings about undefined values.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm

index 4ff0a1a..1e86db6 100644 (file)
@@ -196,7 +196,7 @@ sub init_ro_object_cache {
         my $date = shift;
 
         # Calling parse_datetime() with empty $date will lead to Internal Server Error
-        return '' if ($date eq '' or $date eq undef);
+        return '' if (!defined($date) or $date eq '');
 
         # Probably an accidental entry like '0212' instead of '2012',
         # but 1) the leading 0 may get stripped in cstore and