public
Description: Gitorious aims to provide a great way of doing distributed opensource code collaboration.
Homepage: http://gitorious.org/projects/gitorious
Clone URL: git://github.com/dysinger/gitorious.git
Changing password with a openid enabled account works fine after all
js (author)
Tue Jun 10 09:00:28 -0700 2008
commit  d4bf0ea1e7138ca4f6cf18ee7d0bc0f97416bc8f
tree    d91c2ee90473d9333c1c107f9a564830f7e7c9af
parent  1313f28d3a69a50a0806bc1a46ad0060d39e5252
...
1
2
3
4
5
...
 
 
1
2
3
0
@@ -1,5 +1,3 @@
0
-* fix issue with accounts who added openid not being able to change their password
0
-
0
 events:
0
 * need to create user-less events even we don't find matching users, otherwise it gets weird whenever there's commits from non-registered gitorious users (or users commit from another email).
0
 - maybe save name+email in events table in that case, so we can hook them up to a user later if needed.
...
56
57
58
 
 
 
 
 
 
 
 
 
 
 
59
60
...
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
0
@@ -56,5 +56,16 @@ describe AccountsController do
0
     User.authenticate(users(:johan).email, "test").should == users(:johan)
0
     User.authenticate(users(:johan).email, "fubar").should == nil
0
   end
0
+
0
+ it "should be able to update password, even if user is openid enabled" do
0
+ user = users(:johan)
0
+ user.update_attribute(:identity_url, "http://johan.someprovider.com/")
0
+ put :update_password, :user => {
0
+ :current_password => "test",
0
+ :password => "fubar",
0
+ :password_confirmation => "fubar" }
0
+ flash[:notice].should match(/Your password has been changed/i)
0
+ User.authenticate(users(:johan).email, "fubar").should == users(:johan)
0
+ end
0
 
0
 end

Comments

    No one has commented yet.