From 055e8decc147eda678fbec07b219f345171047c1 Mon Sep 17 00:00:00 2001
From: Marcel de Rooy <m.de.rooy@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@biblibre.com>
(cherry picked from commit 7bbc3580675a151bc08e884c206dc4e17bee6f04)

Signed-off-by: Chris Nighswonger <chris.nighswonger@gmail.com>
---
 C4/Installer.pm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/C4/Installer.pm b/C4/Installer.pm
index 1c177b9681..ae55296433 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;
 }
 
-- 
2.39.5