Browse Source

FIX pr merge bug

tags/v3.0.3
jasder 5 years ago
parent
commit
0133ec24ca
2 changed files with 5 additions and 1 deletions
  1. +1
    -1
      app/controllers/pull_requests_controller.rb
  2. +4
    -0
      app/models/concerns/project_operable.rb

+ 1
- 1
app/controllers/pull_requests_controller.rb View File

@@ -140,7 +140,7 @@ class PullRequestsController < ApplicationController
end

def pr_merge
return render_forbidden("你没有权限操作.") unless @project.develper?(current_user)
return render_forbidden("你没有权限操作.") unless @project.operator?(current_user)

if params[:do].blank?
normal_status(-1, "请选择合并方式")


+ 4
- 0
app/models/concerns/project_operable.rb View File

@@ -84,6 +84,10 @@ module ProjectOperable
end
end

def operator?(user)
user.admin? || !reporter?(user)
end

def set_developer_role(member, role_name)
role = Role.find_by(name: role_name)
member.member_roles.create!(role: role)


Loading…
Cancel
Save