Merge lp:~nataliabidart/ubuntuone-windows-installer/stable-3-0-update-2.99.1 into lp:ubuntuone-windows-installer/stable-3-0

Proposed by Natalia Bidart on 2012-01-03
Status: Merged
Approved by: Natalia Bidart on 2012-01-03
Approved revision: 93
Merged at revision: 93
Proposed branch: lp:~nataliabidart/ubuntuone-windows-installer/stable-3-0-update-2.99.1
Merge into: lp:ubuntuone-windows-installer/stable-3-0
Diff against target: 74 lines (+7/-9)
4 files modified
ubuntuone_installer/gui/qt/local_folders.py (+1/-1)
ubuntuone_installer/gui/qt/tests/test_local_folders.py (+2/-4)
ubuntuone_installer/gui/qt/utils/tests/test_windows.py (+2/-2)
ubuntuone_installer/gui/qt/utils/windows.py (+2/-2)
To merge this branch: bzr merge lp:~nataliabidart/ubuntuone-windows-installer/stable-3-0-update-2.99.1
Reviewer Review Type Date Requested Status
Roberto Alsina (community) 2012-01-03 Approve on 2012-01-03
Review via email: mp+87385@code.launchpad.net

Commit Message

[ Roberto Alsina <email address hidden> ]
  - Make UDF creation from the wizard work (LP: #904799).
  - Quote the paths for autostart (LP: #887715).

To post a comment you must log in.
Roberto Alsina (ralsina) wrote :

+1 code and IRL

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/local_folders.py'
2--- ubuntuone_installer/gui/qt/local_folders.py 2011-10-06 20:28:12 +0000
3+++ ubuntuone_installer/gui/qt/local_folders.py 2012-01-03 16:50:18 +0000
4@@ -176,7 +176,7 @@
5 self.wizard().currentIdChanged.disconnect(self.changed_page)
6 except KeyError:
7 pass
8- if page_id == self.wizard().SYNC_NOW_OR_LATER_PAGE:
9+ if page_id == self.wizard().CONGRATULATIONS_PAGE:
10 # The page following this one
11 self.wizard().overlay.show()
12 for path, item in self.items.items():
13
14=== modified file 'ubuntuone_installer/gui/qt/tests/test_local_folders.py'
15--- ubuntuone_installer/gui/qt/tests/test_local_folders.py 2011-11-07 20:18:58 +0000
16+++ ubuntuone_installer/gui/qt/tests/test_local_folders.py 2012-01-03 16:50:18 +0000
17@@ -1,6 +1,4 @@
18 # -*- coding: utf-8 -*-
19-
20-# Authors: Roberto Alsina <roberto.alsina@canonical.com>
21 #
22 # Copyright 2011 Canonical Ltd.
23 #
24@@ -482,7 +480,7 @@
25 yield self.ui.get_info()
26 show_counter = self.ui.wizard().overlay.show_counter
27 hide_counter = self.ui.wizard().overlay.hide_counter
28- self.ui.changed_page(self.ui.wizard().SYNC_NOW_OR_LATER_PAGE)
29+ self.ui.changed_page(self.ui.wizard().CONGRATULATIONS_PAGE)
30 changes = sorted(self.ui.cp_backend.volume_setings_changes)
31 self.assertEqual(changes,
32 [('asdfgh', {'subscribed': True})])
33@@ -503,7 +501,7 @@
34 item.setCheckState(0, QtCore.Qt.Checked)
35 item = yield self.ui.add_folder("whatever2")
36 item.setCheckState(0, QtCore.Qt.Unchecked)
37- self.ui.changed_page(self.ui.wizard().SYNC_NOW_OR_LATER_PAGE)
38+ self.ui.changed_page(self.ui.wizard().CONGRATULATIONS_PAGE)
39 self.assertEqual(self.ui.cp_backend.folders_created,
40 [('whatever',)])
41 self.assertEqual(self.ui.wizard().overlay.show_counter,
42
43=== modified file 'ubuntuone_installer/gui/qt/utils/tests/test_windows.py'
44--- ubuntuone_installer/gui/qt/utils/tests/test_windows.py 2011-11-30 17:09:12 +0000
45+++ ubuntuone_installer/gui/qt/utils/tests/test_windows.py 2012-01-03 16:50:18 +0000
46@@ -204,9 +204,9 @@
47 self.assertEqual(self.registry.query_args, None)
48 self.assertEqual(self.registry.set_args,
49 [(('KEY', 'Ubuntu One', 0, 1,
50- 'C:\\Python27\\ubuntuone-syncdaemon.exe'), {}),
51+ '"C:\\Python27\\ubuntuone-syncdaemon.exe"'), {}),
52 (('KEY', 'Ubuntu One Icon', 0, 1,
53- 'C:\\Python27\\ubuntuone-control-panel-qt.exe --minimized'),
54+ '"C:\\Python27\\ubuntuone-control-panel-qt.exe" --minimized'),
55 {})])
56
57 def test_not_added_if_not_frozen(self):
58
59=== modified file 'ubuntuone_installer/gui/qt/utils/windows.py'
60--- ubuntuone_installer/gui/qt/utils/windows.py 2011-11-30 17:17:27 +0000
61+++ ubuntuone_installer/gui/qt/utils/windows.py 2012-01-03 16:50:18 +0000
62@@ -172,10 +172,10 @@
63 def add_syncdaemon_to_autostart():
64 """Add syncdaemon to the session's autostart."""
65 if hasattr(sys, "frozen"):
66- sd_path = os.path.join(os.path.dirname(
67+ sd_path = '"%s"' % os.path.join(os.path.dirname(
68 os.path.abspath(sys.executable)),
69 "ubuntuone-syncdaemon.exe")
70- u1cp_path = os.path.join(os.path.dirname(
71+ u1cp_path = '"%s"' % os.path.join(os.path.dirname(
72 os.path.abspath(sys.executable)),
73 "ubuntuone-control-panel-qt.exe")
74 else:

Subscribers

People subscribed via source and target branches