Merge lp:~ssweeny/unity-greeter/remote-login-no-email into lp:unity-greeter

Proposed by Scott Sweeny
Status: Needs review
Proposed branch: lp:~ssweeny/unity-greeter/remote-login-no-email
Merge into: lp:unity-greeter
Diff against target: 81 lines (+26/-6)
2 files modified
debian/changelog (+6/-0)
src/user-list.vala (+20/-6)
To merge this branch: bzr merge lp:~ssweeny/unity-greeter/remote-login-no-email
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Unity Greeter Development Team Pending
Review via email: mp+215053@code.launchpad.net

Commit message

Allow either username or email for remote login (LP: #1172877)

Description of the change

This change removes the strict requirement of an email address from remote login and allows the process to go forward with a user name as well.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:1150
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~ssweeny/unity-greeter/remote-login-no-email/+merge/215053/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-greeter-ci/46/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-greeter-trusty-amd64-ci/29
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-greeter-trusty-armhf-ci/29

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-greeter-ci/46/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Unmerged revisions

1150. By Scott Sweeny

Merge from commercial repo (LP: #1172877)

1149. By Scott Sweeny

Merge from upstream

1148. By Scott Sweeny

Merge from trunk

1147. By Scott Sweeny

Merge from commercial repo

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-04-08 03:58:58 +0000
3+++ debian/changelog 2014-04-09 22:30:33 +0000
4@@ -1,3 +1,9 @@
5+unity-greeter (14.04.9-0ubuntu2) trusty; urgency=low
6+
7+ * Merge from commercial repo (LP: #1172877)
8+
9+ -- Scott Sweeny <scott.sweeny@canonical.com> Fri, 28 Mar 2014 00:46:45 -0400
10+
11 unity-greeter (14.04.9-0ubuntu1) trusty; urgency=medium
12
13 * New upstream release:
14
15=== modified file 'src/user-list.vala'
16--- src/user-list.vala 2014-03-26 21:25:01 +0000
17+++ src/user-list.vala 2014-04-09 22:30:33 +0000
18@@ -402,20 +402,24 @@
19
20 var password_field = current_remote_fields.get ("password") as DashEntry;
21 var email_field = current_remote_fields.get ("email") as Gtk.Entry;
22+ var username_field = current_remote_fields.get ("username") as Gtk.Entry;
23 if (password_field == null)
24 {
25 debug ("Something wrong happened in remote_directory_respond_cb. There was no password field");
26 return;
27 }
28- if (email_field == null)
29+ if (email_field == null && username_field == null)
30 {
31- debug ("Something wrong happened in remote_directory_respond_cb. There was no email field");
32+ debug ("Something wrong happened in remote_directory_respond_cb. There was no email or username field");
33 return;
34 }
35
36 RemoteServer[] server_list = {};
37- var email = email_field.text;
38- var email_valid = false;
39+ var email = (email_field == null) ? username_field.text : email_field.text;
40+ var email_valid = (email_field == null) ? true : false;
41+
42+ if (! email_valid) {
43+
44 try
45 {
46 /* Check email address is valid
47@@ -431,6 +435,8 @@
48 debug ("Calling email regex match failed. Error: %s", e.message);
49 }
50
51+ } /* (! email_valid) */
52+
53 selected_entry.reset_messages ();
54 if (!email_valid)
55 {
56@@ -497,8 +503,7 @@
57 password_field.did_respond = false;
58 if (server_list.length == 0)
59 show_remote_account_dialog ();
60- else
61- {
62+ else {
63 var last_used_server_list_name = "";
64 foreach (var remote_server in server_list)
65 {
66@@ -510,6 +515,15 @@
67 set_active_entry (last_used_server_list_name);
68 else
69 set_active_first_entry_with_prefix ("*remote_login");
70+
71+ }
72+ if (server_list.length == 1) {
73+ var remote_server = server_list[0];
74+ if (is_supported_remote_session (remote_server.type)) {
75+ debug ("pass-thru shortcut");
76+ set_active_first_entry_with_prefix ("*remote_login");
77+ selected_entry.respond ({});
78+ }
79 }
80 }
81 else

Subscribers

People subscribed via source and target branches