From d7cdfbfd26e0581bd8eac1457a365eb86fe96ddf Mon Sep 17 00:00:00 2001 From: "Chris St. Pierre" Date: Fri, 1 Jun 2012 14:25:01 -0400 Subject: fixed return values from toggle_debug xcmds --- src/lib/Bcfg2/SSLServer.py | 2 +- src/lib/Bcfg2/Server/Plugin.py | 3 +++ src/lib/Bcfg2/Server/Plugins/Packages/__init__.py | 3 ++- 3 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/lib/Bcfg2/SSLServer.py b/src/lib/Bcfg2/SSLServer.py index 6aa46ea58..33707eda8 100644 --- a/src/lib/Bcfg2/SSLServer.py +++ b/src/lib/Bcfg2/SSLServer.py @@ -45,7 +45,7 @@ class XMLRPCDispatcher (SimpleXMLRPCServer.SimpleXMLRPCDispatcher): params = (address, ) + params response = self.instance._dispatch(method, params, self.funcs) # py3k compatibility - if type(response) not in [bool, str, list, dict] or response is None: + if type(response) not in [bool, str, list, dict]: response = (response.decode('utf-8'), ) else: response = (response, ) diff --git a/src/lib/Bcfg2/Server/Plugin.py b/src/lib/Bcfg2/Server/Plugin.py index 4e9199466..e71e51f4c 100644 --- a/src/lib/Bcfg2/Server/Plugin.py +++ b/src/lib/Bcfg2/Server/Plugin.py @@ -78,6 +78,9 @@ class Debuggable(object): def toggle_debug(self): self.debug_flag = not self.debug_flag + self.logger("%s: debug_flag = %s" % (self.__class__.__name__, + self.debug_flag)) + return self.debug_flag def debug_log(self, message, flag=None): if (flag is None and self.debug_flag) or flag: diff --git a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py index 72913a60a..5a804a4cd 100644 --- a/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py +++ b/src/lib/Bcfg2/Server/Plugins/Packages/__init__.py @@ -40,8 +40,9 @@ class Packages(Bcfg2.Server.Plugin.Plugin, self.core.setup) def toggle_debug(self): - Bcfg2.Server.Plugin.Plugin.toggle_debug(self) + rv = Bcfg2.Server.Plugin.Plugin.toggle_debug(self) self.sources.toggle_debug() + return rv @property def disableResolver(self): -- cgit v1.2.3-1-g7c22