diff --git a/app/controllers/account_controller.rb b/app/controllers/account_controller.rb
index af848786..085ee0c6 100644
--- a/app/controllers/account_controller.rb
+++ b/app/controllers/account_controller.rb
@@ -143,13 +143,17 @@ class AccountController < ApplicationController
         end
        
        #added by bai
-        unless @user.id.nil?
-          ue = UserExtensions.create(:identity => params[:identity].to_i,:technical_title => params[:technical_title], :gender => params[:gender].to_i, :user_id => @user.id, :student_id => params[:no])
-          unless params[:province].nil? || params[:city].nil?
-            ue.location = params[:province]
-            ue.location_city = params[:city]
-            ue.save
-          end
+        if @user.id != nil
+          ue = @user.user_extensions ||= UserExtensions.new
+          #ue = UserExtensions.create(:identity => params[:identity].to_i,:technical_title => params[:technical_title], :gender => params[:gender].to_i, :user_id => @user.id, :student_id => )
+          ue.identity = params[:identity].to_i
+          ue.technical_title = params[:technical_title]
+          ue.gender = params[:gender].to_i
+          ue.user_id = @user.id
+          ue.student_id = params[:no]
+          ue.location = params[:province] if params[:province] != nil
+          ue.location_city = params[:city] if params[:city] != nil
+          ue.save
         end
        #end