fix: user update return info
This commit is contained in:
parent
6b436acfaf
commit
93169f8c96
|
@ -74,11 +74,10 @@ class UsersController < ApplicationController
|
||||||
|
|
||||||
def update
|
def update
|
||||||
return render_not_found unless @user = User.find_by_id(params[:id]) || User.find_by(login: params[:id])
|
return render_not_found unless @user = User.find_by_id(params[:id]) || User.find_by(login: params[:id])
|
||||||
|
return render_forbidden unless User.current.logged? && (current_user&.admin? || current_user.id == @user.id)
|
||||||
Util.write_file(@image, avatar_path(@user)) if user_params[:image].present?
|
Util.write_file(@image, avatar_path(@user)) if user_params[:image].present?
|
||||||
@user.attributes = user_params.except(:image)
|
@user.attributes = user_params.except(:image)
|
||||||
if @user.save
|
unless @user.save
|
||||||
render_ok
|
|
||||||
else
|
|
||||||
render_error(@user.errors.full_messages.join(", "))
|
render_error(@user.errors.full_messages.join(", "))
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -1,7 +1,20 @@
|
||||||
json.user_id user.id
|
json.user_id user.id
|
||||||
json.login user.login
|
|
||||||
json.name user.full_name
|
json.name user.full_name
|
||||||
|
json.username @user.full_name
|
||||||
|
json.real_name @user.real_name
|
||||||
json.grade user.grade
|
json.grade user.grade
|
||||||
json.identity user&.user_extension&.identity
|
json.gender @user.gender
|
||||||
# json.email user.mail # 邮箱原则上不暴露的,如果实在需要的话只能对某些具体的接口公开
|
json.login @user.login
|
||||||
json.image_url url_to_avatar(user)
|
json.user_id @user.id
|
||||||
|
json.image_url url_to_avatar(@user)
|
||||||
|
json.admin @user.admin?
|
||||||
|
json.user_identity @user.identity
|
||||||
|
json.is_watch current_user&.watched?(@user)
|
||||||
|
json.watched_count @user.fan_count #粉丝
|
||||||
|
json.watching_count @user.follow_count #关注数
|
||||||
|
json.created_time format_time(@user.created_on)
|
||||||
|
json.email @user.show_email ? @user.mail : nil
|
||||||
|
json.province @user.show_location ? @user.province : nil
|
||||||
|
json.city @user.show_location ? @user.city : nil
|
||||||
|
json.custom_department @user.show_department ? @user.custom_department : nil
|
||||||
|
json.description @user.description
|
|
@ -1,16 +1,4 @@
|
||||||
# json.partial! 'users/user', locals: { user: @user }
|
json.partial! 'users/user', locals: { user: @user }
|
||||||
|
|
||||||
json.username @user.full_name
|
|
||||||
json.real_name @user.real_name
|
|
||||||
json.gender @user.gender
|
|
||||||
json.login @user.login
|
|
||||||
json.user_id @user.id
|
|
||||||
json.image_url url_to_avatar(@user)
|
|
||||||
json.admin @user.admin?
|
|
||||||
json.user_identity @user.identity
|
|
||||||
json.is_watch current_user&.watched?(@user)
|
|
||||||
json.watched_count @user.fan_count #粉丝
|
|
||||||
json.watching_count @user.follow_count #关注数
|
|
||||||
json.undo_messages @waiting_applied_messages.size
|
json.undo_messages @waiting_applied_messages.size
|
||||||
json.undo_transfer_projects @common_applied_transfer_projects.size
|
json.undo_transfer_projects @common_applied_transfer_projects.size
|
||||||
json.undo_events @undo_events
|
json.undo_events @undo_events
|
||||||
|
@ -18,10 +6,4 @@ json.user_composes_count @user_composes_count
|
||||||
json.user_org_count @user_org_count
|
json.user_org_count @user_org_count
|
||||||
json.common_projects_count @projects_common_count
|
json.common_projects_count @projects_common_count
|
||||||
json.mirror_projects_count @projects_mirrior_count
|
json.mirror_projects_count @projects_mirrior_count
|
||||||
json.sync_mirror_projects_count @projects_sync_mirrior_count
|
json.sync_mirror_projects_count @projects_sync_mirrior_count
|
||||||
json.created_time format_time(@user.created_on)
|
|
||||||
json.email @user.show_email ? @user.mail : nil
|
|
||||||
json.province @user.show_location ? @user.province : nil
|
|
||||||
json.city @user.show_location ? @user.city : nil
|
|
||||||
json.custom_department @user.show_department ? @user.custom_department : nil
|
|
||||||
json.description @user.description
|
|
|
@ -0,0 +1 @@
|
||||||
|
json.partial! 'users/user', locals: { user: @user }
|
Loading…
Reference in New Issue