Merge pull request 'merge develop to standalone_develop' (#267) from develop into standalone_develop

app/controllers/compare_controller.rb

@@ -43,7 +43,7 @@ class CompareController < ApplicationController
 
   def load_compare_params
     @base = Addressable::URI.unescape(params[:base])
-    @head = params[:head].include?('json') ? params[:head]&.split('.json')[0] : params[:head]
+    @head = params[:head].include?('.json') ? params[:head][0..-6] : params[:head]
 
   end
 

app/controllers/forks_controller.rb

@@ -13,7 +13,7 @@ class ForksController < ApplicationController
     if current_user&.id == @project.user_id
       render_result(-1, "自己不能fork自己的项目")
     elsif Project.exists?(user_id: current_user.id, identifier: @project.identifier)
-      render_result(-1, "fork失败，你已拥有了这个项目")
+      render_result(0, "fork失败，你已拥有了这个项目")
     end
     # return if current_user != @project.owner
     # render_result(-1, "自己不能fork自己的项目")

app/controllers/issue_tags_controller.rb

@@ -2,7 +2,7 @@ class IssueTagsController < ApplicationController
   before_action :require_login, except: [:index]
   before_action :load_repository
   before_action :set_user
-  before_action :check_issue_permission, except: :index
+  before_action :check_issue_tags_permission
   before_action :set_issue_tag, only: [:edit, :update, :destroy]
 
 
@@ -122,9 +122,9 @@ class IssueTagsController < ApplicationController
     @user = @project.owner
   end
 
-  def check_issue_permission
+  def check_issue_tags_permission
-    unless @project.member?(current_user) || current_user.admin?
+    unless @project.manager?(current_user) || current_user.admin?
-      normal_status(-1, "您没有权限")
+      return render_forbidden('你不是管理员，没有权限操作')
     end
   end
 

app/controllers/members_controller.rb

@@ -3,7 +3,7 @@ class MembersController < ApplicationController
   before_action :load_project
   before_action :find_user_with_id, only: %i[create remove change_role]
   before_action :check_user_profile_completed, only: [:create]
-  before_action :operate!, except: %i[index]
+  before_action :operate!
   before_action :check_member_exists!, only: %i[create]
   before_action :check_member_not_exists!, only: %i[remove change_role]
 

app/controllers/organizations/organizations_controller.rb

@@ -46,7 +46,7 @@ class Organizations::OrganizationsController < Organizations::BaseController
       @organization.save!
       sync_organization_extension!
       
-      Gitea::Organization::UpdateService.call(@organization.gitea_token, login, @organization.reload)
+      Gitea::Organization::UpdateService.call(current_user.gitea_token, login, @organization.reload)
       Util.write_file(@image, avatar_path(@organization)) if params[:image].present?
     end
   rescue Exception => e
@@ -57,10 +57,14 @@ class Organizations::OrganizationsController < Organizations::BaseController
   def destroy
     tip_exception("密码不正确") unless current_user.check_password?(password)
     ActiveRecord::Base.transaction do
-      Gitea::Organization::DeleteService.call(@organization.gitea_token, @organization.login)
+      gitea_status, gitea_message = Gitea::Organization::DeleteService.call(current_user.gitea_token, @organization.login)
+      if gitea_status == 204 
         @organization.destroy!
-    end
         render_ok
+      else 
+        tip_exception("当组织内含有仓库时，无法删除此组织")
+      end
+    end
   rescue Exception => e
     uid_logger_error(e.message)
     tip_exception(e.message)

app/controllers/pull_requests_controller.rb

@@ -56,6 +56,7 @@ class PullRequestsController < ApplicationController
   end
 
   def create
+    return render_forbidden("你没有权限操作.") unless @project.operator?(current_user)
     ActiveRecord::Base.transaction do
       @pull_request, @gitea_pull_request = PullRequests::CreateService.call(current_user, @owner, @project, params)
       if @gitea_pull_request[:status] == :success
@@ -78,6 +79,7 @@ class PullRequestsController < ApplicationController
   end
 
   def update
+    return render_forbidden("你没有权限操作.") unless @project.operator?(current_user)
     if params[:title].nil?
       normal_status(-1, "名称不能为空")
     elsif params[:issue_tag_ids].nil?
@@ -197,7 +199,7 @@ class PullRequestsController < ApplicationController
   def check_can_merge
     target_head = params[:head]  #源分支
     target_base = params[:base]  #目标分支
-    is_original = params[:is_original]
+    is_original = params[:is_original] || false
     if target_head.blank? || target_base.blank?
       normal_status(-2, "请选择分支")
     elsif target_head === target_base && !is_original
@@ -228,11 +230,11 @@ class PullRequestsController < ApplicationController
 
   private
   def load_pull_request
-    @pull_request = PullRequest.find params[:id]
+    @pull_request = @project.pull_requests.where(gitea_number: params[:id]).where.not(id: params[:id]).take || PullRequest.find_by_id(params[:id])
   end
 
   def find_pull_request
-    @pull_request = PullRequest.find_by_id(params[:id])
+    @pull_request = @project.pull_requests.where(gitea_number: params[:id]).where.not(id: params[:id]).take || PullRequest.find_by_id(params[:id])
     @issue = @pull_request&.issue
     if @pull_request.blank?
       normal_status(-1, "合并请求不存在")

app/controllers/users_controller.rb

@@ -32,7 +32,8 @@ class UsersController < ApplicationController
         @waiting_applied_messages = @user.applied_messages.waiting
         @common_applied_transfer_projects = AppliedTransferProject.where(owner_id: @user.id).common + AppliedTransferProject.where(owner_id: Organization.joins(team_users: :team).where(team_users: {user_id: @user.id}, teams: {authorize: %w(admin owner)} )).common
         @common_applied_projects = AppliedProject.where(project_id: @user.full_admin_projects).common
-        @undo_events = @waiting_applied_messages.size + @common_applied_transfer_projects.size + @common_applied_projects.size
+        #@undo_events = @waiting_applied_messages.size + @common_applied_transfer_projects.size + @common_applied_projects.size
+        @undo_events = @common_applied_transfer_projects.size + @common_applied_projects.size
       else 
         @waiting_applied_messages = AppliedMessage.none
         @common_applied_transfer_projects = AppliedTransferProject.none

app/forms/organizations/create_form.rb

@@ -8,4 +8,11 @@ class Organizations::CreateForm < BaseForm
   validates :description, length: { maximum: 200 }
   validates :name, format: { with: NAME_REGEX, multiline: true, message: "只能含有数字、字母、下划线且不能以下划线开头和结尾" }
 
+  validate do 
+    check_name(name) unless name.blank?
+  end
+
+  def check_name(name)
+    raise "组织账号已被使用." if Owner.where(login: name.strip).exists?
+  end
 end

app/models/message_template/issue_assigned.rb

@@ -36,7 +36,6 @@ class MessageTemplate::IssueAssigned < MessageTemplate
   def self.get_email_message_content(receiver, operator, issue)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::IssueAssigned"]
-    end
       project = issue&.project
       owner = project&.owner 
       title = email_title
@@ -56,6 +55,9 @@ class MessageTemplate::IssueAssigned < MessageTemplate
       content.gsub!('{id}', issue&.id.to_s)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::IssueAssigned.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/issue_changed.rb

@@ -191,7 +191,6 @@ class MessageTemplate::IssueChanged < MessageTemplate
     return '', '', '' if change_params.blank?
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["CreateOrAssign::IssueChanged"]
-    end
       project = issue&.project
       owner = project&.owner 
       title = email_title
@@ -362,6 +361,10 @@ class MessageTemplate::IssueChanged < MessageTemplate
       end
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+    
   rescue => e
     Rails.logger.info("MessageTemplate::IssueChanged.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/issue_deleted.rb

@@ -33,7 +33,6 @@ class MessageTemplate::IssueDeleted < MessageTemplate
   def self.get_email_message_content(receiver, operator, issue_title)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["CreateOrAssign::IssueChanged"]
-    end
       title = email_title
       title.gsub!('{title}', issue_title)
       content = email
@@ -44,6 +43,9 @@ class MessageTemplate::IssueDeleted < MessageTemplate
       content.gsub!('{title}', issue_title)
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::IssueDeleted.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/organization_joined.rb

@@ -34,7 +34,6 @@ class MessageTemplate::OrganizationJoined < MessageTemplate
   def self.get_email_message_content(receiver, organization) 
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Organization"]
-    end
       title = email_title
       title.gsub!('{organization}', organization&.real_name)
       content = email 
@@ -44,6 +43,10 @@ class MessageTemplate::OrganizationJoined < MessageTemplate
       content.gsub!('{organization}', organization&.real_name)
     
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::OrganizationJoined.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/organization_left.rb

@@ -34,7 +34,6 @@ class MessageTemplate::OrganizationLeft < MessageTemplate
   def self.get_email_message_content(receiver, organization)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Organization"]
-    end
       title = email_title
       title.gsub!('{organization}', organization&.real_name)
       content = email 
@@ -44,6 +43,10 @@ class MessageTemplate::OrganizationLeft < MessageTemplate
       content.gsub!('{organization}', organization&.real_name)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::OrganizationLeft.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/organization_role.rb

@@ -34,7 +34,6 @@ class MessageTemplate::OrganizationRole < MessageTemplate
   def self.get_email_message_content(receiver, organization, role) 
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Permission"]
-    end
       title = email_title
       title.gsub!('{organization}', organization&.real_name)
       title.gsub!('{role}', role)
@@ -46,6 +45,10 @@ class MessageTemplate::OrganizationRole < MessageTemplate
       content.gsub!('{role}', role)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::OrganizationRole.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_issue.rb

@@ -38,7 +38,6 @@ class MessageTemplate::ProjectIssue < MessageTemplate
   def self.get_email_message_content(receiver, is_manager, operator, issue)
     if receiver.user_template_message_setting.present? && is_manager
       return '', '', '' unless receiver.user_template_message_setting.email_body["ManageProject::Issue"]
-    end
       project = issue&.project 
       owner = project&.owner 
       title = email_title
@@ -59,6 +58,10 @@ class MessageTemplate::ProjectIssue < MessageTemplate
       content.gsub!('{title}', issue&.subject)
       
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+   
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectIssue.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_joined.rb

@@ -34,7 +34,6 @@ class MessageTemplate::ProjectJoined < MessageTemplate
   def self.get_email_message_content(receiver, project) 
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Project"]
-    end
       title = email_title
       title.gsub!('{repository}', project&.name)
       
@@ -47,6 +46,10 @@ class MessageTemplate::ProjectJoined < MessageTemplate
       content.gsub!('{repository}', project&.name)
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+    
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectJoined.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_left.rb

@@ -33,7 +33,6 @@ class MessageTemplate::ProjectLeft < MessageTemplate
   def self.get_email_message_content(receiver, project)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Project"]
-    end
       title = email_title
       title.gsub!('{repository}', project&.name)
       
@@ -46,6 +45,10 @@ class MessageTemplate::ProjectLeft < MessageTemplate
       content.gsub!('{repository}', project&.name)
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+    
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectLeft.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_member_joined.rb

@@ -34,7 +34,6 @@ class MessageTemplate::ProjectMemberJoined < MessageTemplate
   def self.get_email_message_content(receiver, user, project)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["ManageProject::Member"]
-    end
       title = email_title
       title.gsub!('{nickname1}', user&.real_name)
       title.gsub!('{nickname2}', project&.owner&.real_name)
@@ -51,6 +50,9 @@ class MessageTemplate::ProjectMemberJoined < MessageTemplate
       content.gsub!('{repository}', project&.name)
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectMemberJoined.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_member_left.rb

@@ -34,7 +34,6 @@ class MessageTemplate::ProjectMemberLeft < MessageTemplate
   def self.get_email_message_content(receiver, user, project)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["ManageProject::Member"]
-    end
       title = email_title
       title.gsub!('{nickname1}', user&.real_name)
       title.gsub!('{nickname2}', project&.owner&.real_name)
@@ -51,6 +50,10 @@ class MessageTemplate::ProjectMemberLeft < MessageTemplate
       content.gsub!('{repository}', project&.name)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectMemberLeft.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_pull_request.rb

@@ -38,7 +38,6 @@ class MessageTemplate::ProjectPullRequest < MessageTemplate
   def self.get_email_message_content(receiver, is_manager, operator, pull_request)
     if receiver.user_template_message_setting.present? && is_manager
       return '', '', '' unless receiver.user_template_message_setting.email_body["ManageProject::PullRequest"]
-    end
       project = pull_request&.project 
       owner = project&.owner 
       title = email_title
@@ -59,6 +58,9 @@ class MessageTemplate::ProjectPullRequest < MessageTemplate
       content.gsub!('{title}', pull_request&.title)
       
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectPullRequest.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_role.rb

@@ -34,7 +34,6 @@ class MessageTemplate::ProjectRole < MessageTemplate
   def self.get_email_message_content(receiver, project, role)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::Permission"]
-    end
       title = email_title
       title.gsub!('{repository}', project&.name)
       title.gsub!('{role}', role)
@@ -49,6 +48,10 @@ class MessageTemplate::ProjectRole < MessageTemplate
       content.gsub!('{role}', role)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectRole.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/project_setting_changed.rb

@@ -159,10 +159,9 @@ class MessageTemplate::ProjectSettingChanged < MessageTemplate
   end
 
   def self.get_email_message_content(receiver, operator, project, change_params)
+    return '', '', '' if change_params.blank?
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["ManageProject::SettingChanged"]
-    end
-    return '', '', '' if change_params.blank?
       owner = project&.owner
       title = email_title
       title.gsub!('{nickname2}', owner&.real_name)
@@ -303,6 +302,9 @@ class MessageTemplate::ProjectSettingChanged < MessageTemplate
       end
       
       return receiver&.mail, title, content
+    else
+      return '', '', '' 
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::ProjectSettingChanged.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/pull_request_assigned.rb

@@ -36,7 +36,6 @@ class MessageTemplate::PullRequestAssigned < MessageTemplate
   def self.get_email_message_content(receiver, operator, pull_request)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["Normal::PullRequestAssigned"]
-    end
       project = pull_request&.project
       owner = project&.owner 
       title = email_title
@@ -56,6 +55,9 @@ class MessageTemplate::PullRequestAssigned < MessageTemplate
       content.gsub!('{id}', pull_request&.id.to_s)
 
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::PullRequestAssigned.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/pull_request_changed.rb

@@ -103,7 +103,6 @@ class MessageTemplate::PullRequestChanged < MessageTemplate
     return '', '', '' if change_params.blank?
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["CreateOrAssign::PullRequestChanged"]
-    end
       project = pull_request&.project
       owner = project&.owner 
       title = email_title
@@ -186,6 +185,10 @@ class MessageTemplate::PullRequestChanged < MessageTemplate
       end
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::PullRequestChanged.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/pull_request_closed.rb

@@ -36,7 +36,6 @@ class MessageTemplate::PullRequestClosed < MessageTemplate
   def self.get_email_message_content(receiver, operator, pull_request)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["CreateOrAssign::PullRequestChanged"]
-    end
       project = pull_request&.project
       owner = project&.owner 
       title = email_title
@@ -54,6 +53,10 @@ class MessageTemplate::PullRequestClosed < MessageTemplate
       content.gsub!('{id}', pull_request&.id.to_s)
   
       return receiver&.mail, title, content
+    else
+      return '', '', '' 
+    end
+
   rescue => e
     Rails.logger.info("MessageTemplate::PullRequestClosed.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/message_template/pull_request_merged.rb

@@ -36,7 +36,6 @@ class MessageTemplate::PullRequestMerged < MessageTemplate
   def self.get_email_message_content(receiver, operator, pull_request)
     if receiver.user_template_message_setting.present? 
       return '', '', '' unless receiver.user_template_message_setting.email_body["CreateOrAssign::PullRequestChanged"]
-    end
       project = pull_request&.project
       owner = project&.owner 
       title = email_title
@@ -54,6 +53,9 @@ class MessageTemplate::PullRequestMerged < MessageTemplate
       content.gsub!('{id}', pull_request&.id.to_s)
   
       return receiver&.mail, title, content
+    else
+      return '', '', ''
+    end
   rescue => e
     Rails.logger.info("MessageTemplate::PullRequestMerged.get_email_message_content [ERROR] #{e}")
     return '', '', ''

app/models/project.rb

@@ -130,7 +130,7 @@ class Project < ApplicationRecord
   after_create :incre_user_statistic, :incre_platform_statistic
   after_save :check_project_members, :reset_cache_data
   before_save :set_invite_code, :reset_unmember_followed, :set_recommend_and_is_pinned
-  before_destroy :decre_project_common
+  before_destroy :decre_project_common, :decre_forked_from_project_count
   after_destroy :decre_user_statistic, :decre_platform_statistic
   scope :project_statics_select, -> {select(:id,:name, :is_public, :identifier, :status, :project_type, :user_id, :forked_count, :description, :visits, :project_category_id, :project_language_id, :license_id, :ignore_id, :watchers_count, :created_on)}
   scope :no_anomory_projects, -> {where("projects.user_id is not null and projects.user_id != ?", 2)}
@@ -172,6 +172,14 @@ class Project < ApplicationRecord
     CacheAsyncClearJob.perform_later('project_common_service', self.id)
   end
 
+  def decre_forked_from_project_count
+    forked_project = self.forked_from_project
+    if forked_project.present?
+      forked_project.decrement(:forked_count, 1)
+      forked_project.save
+    end
+  end
+
   def incre_user_statistic 
     CacheAsyncSetJob.perform_later("user_statistic_service", {project_count: 1, project_language_count_key: self.project_language&.name, project_language_count: 1}, self.user_id)
   end

app/models/user_template_message_setting.rb

@@ -44,17 +44,17 @@ class UserTemplateMessageSetting < ApplicationRecord
 
   def self.init_email_body 
     {
-      "Normal::Permission": true,
+      "Normal::Permission": false,
-      "Normal::Project": true,
+      "Normal::Project": false,
-      "Normal::Organization": true,
+      "Normal::Organization": false,
-      "Normal::IssueAssigned": true,
+      "Normal::IssueAssigned": false,
-      "Normal::PullRequestAssigned": true,
+      "Normal::PullRequestAssigned": false,
-      "CreateOrAssign::IssueChanged": true,
+      "CreateOrAssign::IssueChanged": false,
-      "CreateOrAssign::PullRequestChanged": true,
+      "CreateOrAssign::PullRequestChanged": false,
-      "ManageProject::Issue": true,
+      "ManageProject::Issue": false,
-      "ManageProject::PullRequest": true,
+      "ManageProject::PullRequest": false,
-      "ManageProject::Member": true,
+      "ManageProject::Member": false,
-      "ManageProject::SettingChanged": true,
+      "ManageProject::SettingChanged": false,
     }.stringify_keys!
   end
 

app/services/cache/v2/project_common_service.rb

@@ -196,6 +196,7 @@ class Cache::V2::ProjectCommonService < ApplicationService
 
   def reset_project_common
     load_project
+    return unless @project.present?
     return unless @project.is_full_public
     $redis_cache.del(project_common_key)
     reset_project_owner_id

app/services/gitea/client_service.rb

@@ -144,6 +144,8 @@ class Gitea::ClientService < ApplicationService
       {status: 403, message: '你没有权限操作!'}
     when 404
       {status: 404, message: '你访问的链接不存在!'}
+    when 500
+      {status: 500, message: ''}
     else
       if response&.body.blank?
         message = "请求失败"

app/services/pull_requests/create_service.rb

@@ -94,7 +94,7 @@ class PullRequests::CreateService < ApplicationService
       user: @current_user,
       issue: pull_issue,
       fork_project_id: @params[:fork_project_id],
-      is_original: @params[:is_original],
+      is_original: is_original,
       files_count: @params[:files_count] || 0,
       commits_count: @params[:commits_count] || 0
     })
@@ -145,7 +145,13 @@ class PullRequests::CreateService < ApplicationService
     raise "title参数不能为空" if @params[:title].blank?
     raise "head参数不能为空" if @params[:head].blank?
     raise "base参数不能为空" if @params[:base].blank?
+    raise "分支内容相同，无需创建合并请求" if @params[:head] === @params[:base] && !is_original
+    raise "合并请求已存在" if @project&.pull_requests.where(head: @params[:head], base: @params[:base], status: 0, is_original: is_original, fork_project_id: @params[:fork_project_id]).present?
     raise @pull_issue.errors.full_messages.join(", ") unless pull_issue.valid?
     raise @pull_request.errors.full_messages.join(", ") unless pull_request.valid?
   end
+
+  def is_original
+    @params[:is_original] || false
+  end
 end

app/views/projects/applied_transfer_projects/_detail.json.jbuilder

@@ -1,5 +1,6 @@
 project = object.project
-json.project do 
+if project.present?
+  json.project do 
     json.id project.id
     json.identifier project.identifier
     json.name project.name
@@ -8,6 +9,9 @@ json.project do
     json.owner do 
       json.partial! "/users/user_simple", locals: {user: project.owner}
     end
+  end
+else
+  json.project nil 
 end
 json.user do 
   json.partial! "/users/user_simple", locals: {user: object.user}

app/views/pull_requests/index.json.jbuilder

@@ -14,6 +14,7 @@ json.issues do
   json.array! @issues.to_a do |issue|
     pr = issue.pull_request
     json.pull_request_id pr.id
+    json.pull_request_number pr.gitea_number
     json.pull_request_status pr.status
     json.pull_request_head pr.head 
     json.pull_request_base pr.base

config/locales/zh-CN.yml

@@ -16,7 +16,7 @@ zh-CN:
       'agreed': '已同意'
   trend:
     Issue: 易修(Issue)
-    PullRequest: 合并请求
+    PullRequest: 合并请求(PR)
     VersionRelease: 版本发布
     create: 创建了
     journal: 回复了

public/500.html

@@ -12,12 +12,27 @@
     body {
       font-family: "微软雅黑","宋体";
       background: #fff;
+      margin: 0px;
+      padding: 0px;
     }
     h1 {
       font-size: 1.5em;
     }
-    p {
+    img{
-      font-size: 0.8em;
+      vertical-align: middle;
+      border-style: none;
+    }
+    a{
+      text-decoration: none;
+    }
+    .font-16{
+      font-size: 16px !important;
+    }
+    .mt56{
+      margin-top: 56px;
+    }
+    .color-blue{
+      color:#466AFF !important;
     }
     .h_content{
       text-align: center;
@@ -33,6 +48,32 @@
       align-items: center;
       display: -webkit-flex;
     }
+    .edu-txt-center {
+      text-align: center !important;
+    }
+    .newTable{
+      background-color: #F0F5FF;
+      position: relative;
+      height: 100vh;
+      text-align: center;
+    }
+    .newMargin{
+      position: absolute;
+      left: 0;
+      bottom: 0;
+      width: 100%;
+    }
+
+    .mainImg {
+        position: absolute;
+        width: 100%;
+        height: 100%;
+        display: flex;
+        flex-direction: column;
+        align-items: center;
+        justify-content: center;
+        z-index: 10;
+    }
   </style>
   <script type="text/javascript">
     $(function(){
@@ -49,13 +90,20 @@
 <p>An error occurred on the page you were trying to access.<br />
 If you continue to experience problems please contact your Trustie administrator for assistance.</p>
 <p>If you are the Trustie administrator, check your log files for details about the error.</p> -->
-<div class="verticalCenter">
+<div class="newTable clearfix">
-  <div class="edu-txt-center">
+    <div class="mainImg">
-    <img src="/images/warn/pic_500.jpg" >
+        <img src="images/warn/pic_500.jpg" width="639px"/>
-    <p class="font-18 mt40">您可以稍后尝试&nbsp;
+        <div class="font-16 mt56" style="text-align: center;">
-      您可以稍后尝试&nbsp;<a href="javascript:history.back()" class="color-blue">返回上页</a>，或者&nbsp;
+            <p>服务器异常，请稍后重试</p>
-      <a target="_blank" href="//shang.qq.com/wpa/qunwpa?idkey=2f2043d88c1bd61d182b98bf1e061c6185e23055bec832c07d8148fe11c5a6cd" class="color-blue">QQ反馈>></a>
+            您可尝试<a href="javascript:location.reload();" class="color-blue">刷新页面</a>或<a href="/explore"
-    </p>
+                                                                                       class="color-blue">返回首页</a>，也可以通过
+            <a target="_blank"
+               href="https://qm.qq.com/cgi-bin/qm/qr?k=YVGUhY7uK8ovpyd7tG_lHe2qGZ63LOij&jump_from=webapi"
+               class="color-blue">QQ</a>向我们反馈
+        </div>
+    </div>
+    <div class="newMargin">
+      <img src="images/warn/pic_bg.png" width="100%" height="292px"/>
     </div>
 </div>
 </body>