[Patches] [PATCH] 6699 Add not-empty check for warnings from load_sql in

koha-patchbot at kohaaloha.com koha-patchbot at kohaaloha.com
Fri Nov 4 23:35:02 NZDT 2011


From: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date: Wed, 10 Aug 2011 15:23:07 +0200
Subject: [PATCH] 6699 Add not-empty check for warnings from load_sql in
 Installer

Eliminates fake warnings "Something is wrong on line 530".

Signed-off-by: Julian Maurice <julian.maurice at biblibre.com>
---
 C4/Installer.pm |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/C4/Installer.pm b/C4/Installer.pm
index 1c177b9..ae55296 100644
--- a/C4/Installer.pm
+++ b/C4/Installer.pm
@@ -529,8 +529,10 @@ sub load_sql {
         # report the import a failure although it really succeded -fbcit
     }
 #   errors thrown while loading installer data should be logged
-    warn "C4::Installer::load_sql returned the following errors while attempting to load $filename:\n";
-    warn $error;
+    if($error) {
+      warn "C4::Installer::load_sql returned the following errors while attempting to load $filename:\n";
+      warn $error;
+    }
     return $error;
 }
 
-- 
1.7.7.1




More information about the Patches mailing list