[node-patches] Change in ovirt-node[master]: installer: Fix password page

[fabiand at fedoraproject.org]

Fabian Deutsch has uploaded a new change for review.

Change subject: installer: Fix password page
......................................................................

installer: Fix password page

Due to the cmigration to the new ConfirmedEntry widget, it was missed to
store the password in the password page model. This is now fixed and the
password is stored in the password page model.

Change-Id: I290c69683f6a2a1126533befb4f35b816406ed42
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007811
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/installer/core/password_page.py
M src/ovirt/node/plugins.py
2 files changed, 4 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/20/19320/1

diff --git a/src/ovirt/node/installer/core/password_page.py b/src/ovirt/node/installer/core/password_page.py
index 1dc75ab..ca302b4 100644
--- a/src/ovirt/node/installer/core/password_page.py
+++ b/src/ovirt/node/installer/core/password_page.py
@@ -67,4 +67,5 @@
             self.application.ui.navigate.to_previous_plugin()
         elif changes.contains_any(["admin.password",
                                    "button.next"]):
+            self._model.update(effective_changes)
             self.application.ui.navigate.to_next_plugin()
diff --git a/src/ovirt/node/plugins.py b/src/ovirt/node/plugins.py
index c3936b0..de5cde6 100644
--- a/src/ovirt/node/plugins.py
+++ b/src/ovirt/node/plugins.py
@@ -275,8 +275,9 @@
         """Called when some widget was changed
         change is expected to be a dict.
         """
-        if type(change) is not dict:
-            self.logger.warning("Change is not a dict: %s" % str(change))
+        if type(change) not in [dict, Changeset]:
+            self.logger.warning("Change is not a dict: %s (%s)" %
+                                (repr(change), type(change)))
 
         change = Changeset(change)
 


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

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