Fix syntax errors
[snf-image-creator] / image_creator / dialog_wizard.py
index ee62963..60360d3 100644 (file)
@@ -58,6 +58,7 @@ class Wizard:
         self.session = session
         self.pages = []
         self.session['wizard'] = {}
+        self.d = session['dialog']
 
     def add_page(self, page):
         self.pages.append(page)
@@ -73,29 +74,52 @@ class Wizard:
                 continue
 
             if idx >= len(self.pages):
-                break
+                msg = "All necessary information has been gathered:\n\n"
+                for page in self.pages:
+                    msg += " * %s\n" % page.info
+                msg += "\nContinue with the image creation process?"
+
+                ret = self.d.yesno(
+                    msg, width=PAGE_WIDTH, height=8 + len(self.pages),
+                    ok_label="Yes", cancel="Back", extra_button=1,
+                    extra_label="Quit", title="Confirmation")
+
+                if ret == self.d.DIALOG_CANCEL:
+                    idx -= 1
+                elif ret == self.d.DIALOG_EXTRA:
+                    return False
+                elif ret == self.d.DIALOG_OK:
+                    return True
 
             if idx < 0:
                 return False
-        return True
 
 
-class WizardPage:
+class WizardPage(object):
     NEXT = 1
     PREV = -1
 
+    def __init__(self, **kargs):
+        validate = kargs['validate'] if 'validate' in kargs else lambda x: x
+        setattr(self, "validate", validate)
+
+        display = kargs['display'] if 'display' in kargs else lambda x: x
+        setattr(self, "display", display)
+
     def run(self, session, index, total):
         raise NotImplementedError
 
 
 class WizardRadioListPage(WizardPage):
 
-    def __init__(self, name, message, choices, **kargs):
+    def __init__(self, name, printable, message, choices, **kargs):
+        super(WizardRadioListPage, self).__init__(**kargs)
         self.name = name
+        self.printable = printable
         self.message = message
         self.choices = choices
         self.title = kargs['title'] if 'title' in kargs else ''
-        self.default = kargs['default'] if 'default' in kargs else 0
+        self.default = kargs['default'] if 'default' in kargs else ""
 
     def run(self, session, index, total):
         d = session['dialog']
@@ -103,98 +127,75 @@ class WizardRadioListPage(WizardPage):
 
         choices = []
         for i in range(len(self.choices)):
-            default = 1 if i == self.default else 0
+            default = 1 if self.choices[i][0] == self.default else 0
             choices.append((self.choices[i][0], self.choices[i][1], default))
 
-        while True:
-            (code, answer) = \
-                d.radiolist(self.message, width=PAGE_WIDTH,
-                            ok_label="Next", cancel="Back", choices=choices,
-                            title="(%d/%d) %s" % (index + 1, total, self.title)
-                            )
+        (code, answer) = d.radiolist(
+            self.message, height=10, width=PAGE_WIDTH, ok_label="Next",
+            cancel="Back", choices=choices,
+            title="(%d/%d) %s" % (index + 1, total, self.title))
 
-            if code in (d.DIALOG_CANCEL, d.DIALOG_ESC):
-                return self.PREV
+        if code in (d.DIALOG_CANCEL, d.DIALOG_ESC):
+            return self.PREV
 
-            for i in range(len(choices)):
-                if self.choices[i] == answer:
-                    self.default = i
-                    w[name] = i
-                    break
+        w[self.name] = self.validate(answer)
+        self.default = answer
+        self.info = "%s: %s" % (self.printable, self.display(w[self.name]))
 
-            return self.NEXT
+        return self.NEXT
 
 
 class WizardInputPage(WizardPage):
 
-    def __init__(self, name, message, **kargs):
+    def __init__(self, name, printable, message, **kargs):
+        super(WizardInputPage, self).__init__(**kargs)
         self.name = name
+        self.printable = printable
         self.message = message
         self.title = kargs['title'] if 'title' in kargs else ''
         self.init = kargs['init'] if 'init' in kargs else ''
-        if 'validate' in kargs:
-            validate = kargs['validate']
-        else:
-            validate = lambda x: x
-
-        setattr(self, "validate", validate)
 
     def run(self, session, index, total):
         d = session['dialog']
         w = session['wizard']
 
-        while True:
-            (code, answer) = \
-                d.inputbox(self.message, init=self.init,
-                           width=PAGE_WIDTH, ok_label="Next", cancel="Back",
-                           title="(%d/%d) %s" % (index + 1, total, self.title))
+        (code, answer) = d.inputbox(
+            self.message, init=self.init, width=PAGE_WIDTH, ok_label="Next",
+            cancel="Back", title="(%d/%d) %s" % (index + 1, total, self.title))
 
-            if code in (d.DIALOG_CANCEL, d.DIALOG_ESC):
-                return self.PREV
+        if code in (d.DIALOG_CANCEL, d.DIALOG_ESC):
+            return self.PREV
 
-            value = answer.strip()
-            self.init = value
-            w[self.name] = self.validate(value)
-            break
+        value = answer.strip()
+        self.init = value
+        w[self.name] = self.validate(value)
+        self.info = "%s: %s" % (self.printable, self.display(w[self.name]))
 
         return self.NEXT
 
 
-class WizardYesNoPage(WizardPage):
-
-    def __init__(self, message, **kargs):
-        self.message = message
-        self.title = kargs['title'] if 'title' in kargs else ''
-
-    def run(self, session, index, total):
-        d = session['dialog']
-
-        while True:
-            ret = d.yesno(self.message, width=PAGE_WIDTH, ok_label="Yes",
-                          cancel="Back", extra_button=1, extra_label="Quit",
-                          title="(%d/%d) %s" % (index + 1, total, self.title))
-
-            if ret == d.DIALOG_CANCEL:
-                return self.PREV
-            elif ret == d.DIALOG_EXTRA:
-                raise WizardExit
-            elif ret == d.DIALOG_OK:
-                return self.NEXT
-
-
 def wizard(session):
 
     init_token = Kamaki.get_token()
     if init_token is None:
         init_token = ""
 
-    name = WizardInputPage("ImageName", "Please provide a name for the image:",
-                           title="Image Name", init=session['device'].distro)
-    descr = WizardInputPage("ImageDescription",
-                            "Please provide a description for the image:",
-                            title="Image Description",
-                            init=session['metadata']['DESCRIPTION'] if
-                            'DESCRIPTION' in session['metadata'] else '')
+    name = WizardInputPage(
+        "ImageName", "Image Name", "Please provide a name for the image:",
+        title="Image Name", init=session['device'].distro)
+
+    descr = WizardInputPage(
+        "ImageDescription", "Image Description",
+        "Please provide a description for the image:",
+        title="Image Description", init=session['metadata']['DESCRIPTION'] if
+        'DESCRIPTION' in session['metadata'] else '')
+
+    registration = WizardRadioListPage(
+        "ImageRegistration", "Registration Type",
+        "Please provide a registration type:",
+        [("Private", "Image is accessible only by this user"),
+         ("Public", "Everyone can create VMs from this image")],
+        title="Registration Type", default="Private")
 
     def validate_account(token):
         if len(token) == 0:
@@ -204,25 +205,23 @@ def wizard(session):
         account = Kamaki.get_account(token)
         if account is None:
             session['dialog'].msgbox("The token you provided in not valid!",
-                width=PAGE_WIDTH)
+                                     width=PAGE_WIDTH)
             raise WizardInvalidData
 
         return account
 
-    account = WizardInputPage("account",
+    account = WizardInputPage(
+        "Account", "Account",
         "Please provide your ~okeanos authentication token:",
-        title="~okeanos account token", init=init_token,
-        validate=validate_account)
-
-    msg = "All necessary information has been gathered. Confirm and Proceed."
-    proceed = WizardYesNoPage(msg, title="Confirmation")
+        title="~okeanos account", init=init_token, validate=validate_account,
+        display=lambda account: account['username'])
 
     w = Wizard(session)
 
     w.add_page(name)
     w.add_page(descr)
+    w.add_page(registration)
     w.add_page(account)
-    w.add_page(proceed)
 
     if w.run():
         create_image(session)
@@ -241,7 +240,7 @@ def create_image(session):
     wizard = session['wizard']
 
     # Save Kamaki credentials
-    Kamaki.save_token(wizard['account']['auth_token'])
+    Kamaki.save_token(wizard['Account']['auth_token'])
 
     with_progress = OutputWthProgress(True)
     out = disk.out
@@ -275,7 +274,7 @@ def create_image(session):
         out.output()
         try:
             out.output("Uploading image to pithos:")
-            kamaki = Kamaki(wizard['account'], out)
+            kamaki = Kamaki(wizard['Account'], out)
 
             name = "%s-%s.diskdump" % (wizard['ImageName'],
                                        time.strftime("%Y%m%d%H%M"))
@@ -296,8 +295,12 @@ def create_image(session):
             out.success('done')
             out.output()
 
-            out.output('Registering image with ~okeanos ...', False)
-            kamaki.register(wizard['ImageName'], pithos_file, metadata)
+            is_public = True if wizard['ImageRegistration'] == "Public" else \
+                False
+            out.output('Registering %s image with ~okeanos ...' %
+                       wizard['ImageRegistration'].lower(), False)
+            kamaki.register(wizard['ImageName'], pithos_file, metadata,
+                            is_public)
             out.success('done')
             out.output()
 
@@ -306,8 +309,9 @@ def create_image(session):
     finally:
         out.remove(with_progress)
 
-    msg = "The image was successfully uploaded and registered with " \
-          "~okeanos. Would you like to keep a local copy of the image?"
+    msg = "The %s image was successfully uploaded and registered with " \
+          "~okeanos. Would you like to keep a local copy of the image?" \
+          % wizard['ImageRegistration'].lower()
     if not d.yesno(msg, width=PAGE_WIDTH):
         extract_image(session)