diff options
| author | francis <francis> | 2008-01-03 12:54:40 +0000 |
|---|---|---|
| committer | francis <francis> | 2008-01-03 12:54:40 +0000 |
| commit | d8118bafcbb1f7951c8fa5ab7a48b0861b333e68 (patch) | |
| tree | a0cf8d8b473f01623eee044d6d64ffab85200177 /app/controllers/user_controller.rb | |
| parent | 859f9b5db0634f4ae2485348b3d10bcd1e8f1942 (diff) | |
Always call save!, which raises an exception on error, rather than ignoring any error.
Diffstat (limited to 'app/controllers/user_controller.rb')
| -rw-r--r-- | app/controllers/user_controller.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb index b83d85d18..222f60791 100644 --- a/app/controllers/user_controller.rb +++ b/app/controllers/user_controller.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: user_controller.rb,v 1.21 2007-12-14 18:46:08 francis Exp $ +# $Id: user_controller.rb,v 1.22 2008-01-03 12:54:40 francis Exp $ class UserController < ApplicationController # XXX See controllers/application.rb simplify_url_part for reverse of expression in SQL below @@ -55,7 +55,7 @@ class UserController < ApplicationController else # New unconfirmed user @user_signup.email_confirmed = false - @user_signup.save + @user_signup.save! send_confirmation_mail @user_signup return @@ -73,7 +73,7 @@ class UserController < ApplicationController @user = post_redirect.user @user.email_confirmed = true - @user.save + @user.save! session[:user_id] = @user.id |