commit
07d11737e9
|
@ -47,7 +47,7 @@ class MembersController < ApplicationController
|
||||||
end
|
end
|
||||||
|
|
||||||
def member_exists?
|
def member_exists?
|
||||||
@project.member?(params[:user_id])
|
@project.members.exists?(params[:user_id])
|
||||||
end
|
end
|
||||||
|
|
||||||
def operate!
|
def operate!
|
||||||
|
@ -59,6 +59,6 @@ class MembersController < ApplicationController
|
||||||
end
|
end
|
||||||
|
|
||||||
def check_member_not_exists!
|
def check_member_not_exists!
|
||||||
return render_result(1, "user_id为#{params[:user_id]}的用户还不是项目成员") unless member_exists?
|
return render_result(1, "user_id为#{params[:user_id]}的用户还不是项目成员") unless @project.member?(params[:user_id])
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -140,7 +140,7 @@ class PullRequestsController < ApplicationController
|
||||||
end
|
end
|
||||||
|
|
||||||
def pr_merge
|
def pr_merge
|
||||||
return render_forbidden("你没有权限操作.") if @project.reporter?(current_user)
|
return render_forbidden("你没有权限操作.") unless current_user.project_manager?(@project)
|
||||||
|
|
||||||
if params[:do].blank?
|
if params[:do].blank?
|
||||||
normal_status(-1, "请选择合并方式")
|
normal_status(-1, "请选择合并方式")
|
||||||
|
|
|
@ -203,7 +203,7 @@ class User < Owner
|
||||||
end
|
end
|
||||||
|
|
||||||
def project_manager?(project)
|
def project_manager?(project)
|
||||||
project.managers.exists?(user: self) || self.admin?
|
project.manager?(self) || self.admin?
|
||||||
end
|
end
|
||||||
|
|
||||||
# 学号
|
# 学号
|
||||||
|
|
Loading…
Reference in New Issue