summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorelijah <elijah@riseup.net>2014-06-23 13:52:00 -0700
committerelijah <elijah@riseup.net>2014-06-23 13:52:00 -0700
commit03b56bc0d94558ad7ff8b82e17314511a40ed16a (patch)
treed4311c3e18609ed9f0a39f815833937657a25d85 /app
parent110bc16297d7736f1025bfa85f3551313e913d84 (diff)
Account.create - do a User.new instead of User.create, so that we can report the errors on the object if not saved.
Diffstat (limited to 'app')
-rw-r--r--app/models/account.rb5
1 files changed, 3 insertions, 2 deletions
diff --git a/app/models/account.rb b/app/models/account.rb
index bee540e..67fec58 100644
--- a/app/models/account.rb
+++ b/app/models/account.rb
@@ -16,7 +16,8 @@ class Account
# Returns the user record so it can be used in views.
def self.create(attrs)
- @user = User.create(attrs)
+ @user = User.new(attrs)
+ @user.save
if @user.persisted?
@identity = @user.identity
@identity.user_id = @user.id
@@ -28,7 +29,7 @@ class Account
rescue StandardError => ex
@user.errors.add(:base, ex.to_s)
ensure
- if @user.persisted? && (@identity.nil? || !@identity.persisted?)
+ if @user && @user.persisted? && (@identity.nil? || !@identity.persisted?)
@user.destroy
end
return @user