From 333b70e6e1322c401db47adff848375e0d6fa734 Mon Sep 17 00:00:00 2001
From: "ian%hixie.ch" <ian%hixie.ch>
Date: Sun, 11 Nov 2001 11:47:02 +0000
Subject: [PATCH] Should have been checking for a defined value, not a true
value -- there is nothing to stop a failure from being '0', for instance.
(The important thing is to be consistent with the SelectingServiceList code,
really, and it uses defined().)
---
webtools/PLIF/PLIF/Service/Components/AdminCommands.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/webtools/PLIF/PLIF/Service/Components/AdminCommands.pm b/webtools/PLIF/PLIF/Service/Components/AdminCommands.pm
index 436e2d8939595..6fcc0055df374 100644
--- a/webtools/PLIF/PLIF/Service/Components/AdminCommands.pm
+++ b/webtools/PLIF/PLIF/Service/Components/AdminCommands.pm
@@ -81,7 +81,7 @@ sub cmdSetup {
$result = $app->getSelectingServiceList('setup.install')->setupInstall($app);
}
# report on the result
- if ($result) {
+ if (defined($result)) {
# if we failed, first report that then signal that
# configuration has ended
$self->dump(9, "Failed to setup because argument '$result' was missing.");
--
GitLab