api: fix user registration

Victor Costa
1 parent c2c835ac
Showing 3 changed files with 8 additions and 12 deletions Show diff stats
app/models/user.rb
lib/noosfero/api/session.rb
test/unit/api/session_test.rb
@@ -102,12 +102,6 @@ class User &lt; ActiveRecord::Base
     end
   end
-  def signup
-    User.transaction do
-      self.person.save if self.save
-    end
-  end
-
   has_one :person, :dependent => :destroy
   belongs_to :environment
@@ -45,14 +45,16 @@ module Noosfero
         # test_captcha will render_api_error! and exit in case of any problem
         # this return is just to improve the clarity of the execution path
         return unless test_captcha(remote_ip, params, environment)
+
         name = params[:name].present? ? params[:name] : attrs[:email]
         user = User.new(attrs.merge(:name => name))
-        if user.signup!
+
+        begin
+          user.signup!
           user.generate_private_token! if user.activated?
           present user, :with => Entities::UserLogin
-        else
-          user.destroy
-          message = user.errors.as_json.merge(user.person.errors.as_json).to_json
+        rescue ActiveRecord::RecordInvalid
+          message = user.errors.as_json.merge((user.person.present? ? user.person.errors : {}).as_json).to_json
           render_api_error!(message, 400)
         end
       end
@@ -27,8 +27,8 @@ class SessionTest &lt; ActiveSupport::TestCase
     assert_equal 201, last_response.status
     json = JSON.parse(last_response.body)
     assert User['newuserapi'].activated?
-    assert json['user']['activated']
-    assert json['user']['private_token'].present?
+    assert json['activated']
+    assert json['private_token'].present?
   end
   should 'register a user with name' do
	@@ -102,12 +102,6 @@ class User < ActiveRecord::Base		@@ -102,12 +102,6 @@ class User < ActiveRecord::Base
102	end	102	end
103	end	103	end
104		104
105	- def signup
106	- User.transaction do
107	- self.person.save if self.save
108	- end
109	- end
110	-
111	has_one :person, :dependent => :destroy	105	has_one :person, :dependent => :destroy
112	belongs_to :environment	106	belongs_to :environment
113		107