Adding exception handling to remove the problem of propogating error codes all the way up the stack. Still not quite super-consistant, but getting there. Util functions still return true/false since they just ask questions, but API functions will throw errors to ensure they are being dealt with. Main CLI class needs to take advantage of this fact and become simpler. Tests are already modified to detect new exceptions with one exception :)

1 files changed, 3 insertions, 3 deletions

diff --git a/cobbler/item_distro.py b/cobbler/item_distro.py
index 59250af..e61ea7b 100644
--- a/cobbler/item_distro.py
+++ b/cobbler/item_distro.py
@@ -10,7 +10,7 @@ import utils
 import item
 import weakref
 import os
-from cobbler_exception import CobblerException
+import cexceptions
 
 class Distro(item.Item):
 
@@ -51,7 +51,7 @@ class Distro(item.Item):
         if utils.find_kernel(kernel):
             self.kernel = kernel
             return True
-        raise CobblerException("no_kernel")
+        raise cexceptions.CobblerException("no_kernel")
 
     def set_initrd(self,initrd):
         """
@@ -61,7 +61,7 @@ class Distro(item.Item):
         if utils.find_initrd(initrd):
             self.initrd = initrd
             return True
-        raise CobblerException("no_initrd")
+        raise cexceptions.CobblerException("no_initrd")
 
     def is_valid(self):
         """