Skip to content
Browse files

Fix primary email being set as most recently used if the user cancels…

… redirect to authenticate with primary.

issue #968
  • Loading branch information...
1 parent 57460fe commit 1738654eba21bf5388c604e54645f0ed3578ff8e @shane-tomlinson shane-tomlinson committed
View
3 resources/static/dialog/controllers/pick_email.js
@@ -48,9 +48,6 @@ BrowserID.Modules.PickEmail = (function() {
var valid = checkEmail.call(self, email);
if (valid) {
- var origin = user.getOrigin();
- storage.site.set(origin, "email", email);
-
self.close("email_chosen", { email: email });
}
}
View
3 resources/static/test/cases/controllers/pick_email.js
@@ -76,7 +76,7 @@
equal(label.hasClass("preselected"), false, "the label has no class");
});
- asyncTest("signIn - saves picked email to storage", function() {
+ asyncTest("signIn - trigger 'email_chosen message'", function() {
storage.addEmail("testuser@testuser.com", {});
storage.addEmail("testuser2@testuser.com", {});
@@ -87,7 +87,6 @@
var assertion;
register("email_chosen", function(msg, info) {
- equal(storage.site.get(testOrigin, "email"), "testuser2@testuser.com", "email saved correctly");
ok(info.email, "email_chosen message triggered with email");
start();
});

0 comments on commit 1738654

Please sign in to comment.
Something went wrong with that request. Please try again.