Merge lp:~diegosarmentero/ubuntuone-windows-installer/forgotten845737 into lp:ubuntuone-windows-installer

Proposed by Diego Sarmentero
Status: Merged
Approved by: Alejandro J. Cura
Approved revision: 71
Merged at revision: 72
Proposed branch: lp:~diegosarmentero/ubuntuone-windows-installer/forgotten845737
Merge into: lp:ubuntuone-windows-installer
Diff against target: 24 lines (+2/-2)
2 files modified
ubuntuone_installer/gui/qt/forgotten.py (+0/-1)
ubuntuone_installer/gui/qt/tests/test_gui.py (+2/-1)
To merge this branch: bzr merge lp:~diegosarmentero/ubuntuone-windows-installer/forgotten845737
Reviewer Review Type Date Requested Status
Alejandro J. Cura (community) Approve
Roberto Alsina (community) Approve
Review via email: mp+76462@code.launchpad.net

Commit message

- Fixed: When entering email to reset password, errors are not highlighted (LP: #845737).

Description of the change

- Fixed: When entering email to reset password, errors are not highlighted (LP: #845737).

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
Revision history for this message
Alejandro J. Cura (alecu) wrote :

Code looks great, and all tests pass.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntuone_installer/gui/qt/forgotten.py'
2--- ubuntuone_installer/gui/qt/forgotten.py 2011-08-02 02:36:16 +0000
3+++ ubuntuone_installer/gui/qt/forgotten.py 2011-09-21 18:31:16 +0000
4@@ -29,6 +29,5 @@
5
6 def on_password_reset_error(self, app_name, error):
7 """Hide the overlay, then do as the default controller does."""
8- self.view.wizard().overlay.hide()
9 super(ForgottenPasswordController,
10 self).on_password_reset_error(app_name, error)
11
12=== modified file 'ubuntuone_installer/gui/qt/tests/test_gui.py'
13--- ubuntuone_installer/gui/qt/tests/test_gui.py 2011-09-16 12:55:16 +0000
14+++ ubuntuone_installer/gui/qt/tests/test_gui.py 2011-09-21 18:31:16 +0000
15@@ -362,7 +362,8 @@
16 def test_forgotten_password_controller_error(self):
17 """When there is an error, it should hide the overlay."""
18 self.assertEqual(self.ui.overlay.hide_counter, 1)
19- controller = forgotten.ForgottenPasswordController()
20+ controller = forgotten.ForgottenPasswordController(
21+ message_box=self.ui)
22 controller.view = self.ui.forgotten
23 controller.on_password_reset_error("app_name",
24 {'errtype': 'ResetPasswordTokenError'})

Subscribers

People subscribed via source and target branches