Needs to be compatible with older versions

Evandro Junior
1 parent cebdccf8
Showing 3 changed files with 14 additions and 4 deletions Show diff stats
app/models/user.rb
lib/noosfero/api/entities.rb
lib/noosfero/api/session.rb
@@ -94,6 +94,12 @@ 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
@@ -112,6 +112,7 @@ module Noosfero
         root 'users', 'user'
         expose :id
         expose :login
+        expose :email
         expose :person, :using => Profile
         expose :activated?, as: :activated
         expose :permissions do |user, options|
@@ -43,11 +43,14 @@ module Noosfero
         # this return is just to improve the clarity of the execution path
         return unless test_captcha(remote_ip, params, environment)
         user = User.new(attrs)
-        if user.save
-          user.generate_private_token! if user.activated?
-          present user, :with => Entities::UserLogin
+        person = Person.new(name: params[:name], identifier: user.login)
+        user.person = person
+        if user.signup
+            user.generate_private_token! if user.activated?
+            present user, :with => Entities::UserLogin
         else
-          message = user.errors.to_json
+          user.destroy
+          message = user.errors.as_json.merge(person.errors.as_json).to_json
           render_api_error!(message, 400)
         end
       end
	@@ -94,6 +94,12 @@ class User < ActiveRecord::Base		@@ -94,6 +94,12 @@ class User < ActiveRecord::Base
94	end	94	end
95	end	95	end
96		96
		97	+ def signup
		98	+ User.transaction do
		99	+ self.person.save if self.save
		100	+ end
		101	+ end
		102	+
97	has_one :person, :dependent => :destroy	103	has_one :person, :dependent => :destroy
98	belongs_to :environment	104	belongs_to :environment
99		105