[node-patches] Change in ovirt-node[master]: Have the custom device dialog move to the next page all the ...

rbarry at redhat.com rbarry at redhat.com
Mon Jun 23 16:23:04 UTC 2014


Ryan Barry has uploaded a new change for review.

Change subject: Have the custom device dialog move to the next page all the time
......................................................................

Have the custom device dialog move to the next page all the time

Previously, the custom device dialog didn't move to the next plugin
on the installation device page, though it did on the boot device page.
Do so to maintain consistent behavior.

Change-Id: I075e8c9eed82ce0507eee7e878dbefdcb3ecdb23
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1065915
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/installer/core/installation_device_page.py
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/82/29082/1

diff --git a/src/ovirt/node/installer/core/installation_device_page.py b/src/ovirt/node/installer/core/installation_device_page.py
index 191ab0b..3e4af9c 100644
--- a/src/ovirt/node/installer/core/installation_device_page.py
+++ b/src/ovirt/node/installer/core/installation_device_page.py
@@ -164,7 +164,9 @@
         elif changes.contains_any(["installation.device.custom",
                                    "dialog.device.custom.save"]):
             self._dialog.close()
-            return self.ui_content()
+            cdev = self._model["installation.device.custom"]
+            self._model["installation.devices"].append(cdev)
+            self.application.ui.navigate.to_next_plugin()
 
         if changes.contains_any(["button.back"]):
             self.application.ui.navigate.to_previous_plugin()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I075e8c9eed82ce0507eee7e878dbefdcb3ecdb23
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list