[node-patches] Change in ovirt-node[master]: engine_page: Use checkbox

fabiand at fedoraproject.org fabiand at fedoraproject.org
Tue Dec 11 20:09:38 UTC 2012


Fabian Deutsch has uploaded a new change for review.

Change subject: engine_page: Use checkbox
......................................................................

engine_page: Use checkbox

Change-Id: Id389880701ccf9f882ab03961fbedcfca6d6c02d
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M scripts/tui/src/ovirt/node/plugins/engine_page.py
M scripts/tui/src/ovirt/node/ui/__init__.py
2 files changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/22/9922/1

diff --git a/scripts/tui/src/ovirt/node/plugins/engine_page.py b/scripts/tui/src/ovirt/node/plugins/engine_page.py
index b019fc3..8d97580 100644
--- a/scripts/tui/src/ovirt/node/plugins/engine_page.py
+++ b/scripts/tui/src/ovirt/node/plugins/engine_page.py
@@ -46,7 +46,7 @@
             self._model = {
                 "vdsm.address": "",
                 "vdsm.port": "7634",
-                "vdsm.connect_and_validate": True,
+                "vdsm.connect_and_validate": ovirt.node.utils.parse_bool(True),
                 "vdsm.password": "",
                 "vdsm.password_confirmation": "",
             }
@@ -66,10 +66,10 @@
 
             ("vdsm.address", ovirt.node.ui.Entry("Server Address:")),
             ("vdsm.port", ovirt.node.ui.Entry("Server Port:")),
-            ("vdsm.connect_and_validate", ovirt.node.ui.Options(
-                    "Connect to oVirt Engine and Validate Certificate",
-                    [("yes", "Yes"), ("no", "No")])),
+            ("vdsm.connect_and_validate", ovirt.node.ui.Checkbox(
+                    "Connect to oVirt Engine and Validate Certificate")),
 
+            ("vdsm.password._divider", ovirt.node.ui.Divider("-")),
             ("vdsm.password._label", ovirt.node.ui.Label(
                     "Optional password for adding Node through oVirt " +
                     "Engine UI")),
diff --git a/scripts/tui/src/ovirt/node/ui/__init__.py b/scripts/tui/src/ovirt/node/ui/__init__.py
index 3391ab0..aaa974b 100644
--- a/scripts/tui/src/ovirt/node/ui/__init__.py
+++ b/scripts/tui/src/ovirt/node/ui/__init__.py
@@ -262,7 +262,7 @@
         label: Caption of this checkbox
         state: The initial change
     """
-    def __init__(self, label, state):
+    def __init__(self, label, state=False):
         self.label = label
         self.state(state)
 


--
To view, visit http://gerrit.ovirt.org/9922
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id389880701ccf9f882ab03961fbedcfca6d6c02d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list