From 97f40f6d7661145915f67ab1015a2ada7a51742a Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Thu, 30 Jul 2020 09:53:10 +0800
Subject: [PATCH 01/22] Change Issue series Update

---
 app/controllers/issues_controller.rb | 14 +++++++++++++-
 app/jobs/sync_repository_job.rb      |  1 +
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb
index 8fdb06371..418ae4f81 100644
--- a/app/controllers/issues_controller.rb
+++ b/app/controllers/issues_controller.rb
@@ -248,7 +248,19 @@ class IssuesController < ApplicationController
     update_hash = {}
     update_hash.merge!(assigned_to_id: params[:assigned_to_id]) if params[:assigned_to_id].present?
     update_hash.merge!(fixed_version_id: params[:fixed_version_id]) if params[:fixed_version_id].present?
-    update_hash.merge!(status_id: params[:status_id]) if params[:status_id].present?
+    # update_hash.merge!(status_id: params[:status_id]) if params[:status_id].present?
+    if params[:status_id].present?
+      status_id = params[:status_id].to_i
+      update_hash.merge!(status_id: status_id)
+      done_ratio = nil
+      case status_id
+      when 1
+        done_ratio = 0
+      when 3
+        done_ratio = 100
+      end
+      update_hash.merge!(done_ratio: done_ratio) if done_ratio
+    end
     # update_hash = params[:issue]
     issue_ids = params[:ids]
     if issue_ids.present?
diff --git a/app/jobs/sync_repository_job.rb b/app/jobs/sync_repository_job.rb
index c59154d99..9edbae983 100644
--- a/app/jobs/sync_repository_job.rb
+++ b/app/jobs/sync_repository_job.rb
@@ -28,6 +28,7 @@ class SyncRepositoryJob < ApplicationJob
           SyncLog.sync_log("=============force_push_erros==#{path}/#{image_repo_name}++branch:#{branch}")
         else
           SyncLog.sync_log("=============force_push_success==#{path}/#{image_repo_name}++branch+++#{branch}")
+          system("rm -rf #{path}/#{image_repo_name}")
         end
       end
     else

From c63c916d9ded65cfff24e0e4a75a52d319dfe85c Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Thu, 6 Aug 2020 14:29:34 +0800
Subject: [PATCH 02/22] Update sync gitea user info

---
 app/controllers/accounts_controller.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/controllers/accounts_controller.rb b/app/controllers/accounts_controller.rb
index 46ea83a2e..23290d91c 100644
--- a/app/controllers/accounts_controller.rb
+++ b/app/controllers/accounts_controller.rb
@@ -66,8 +66,8 @@ class AccountsController < ApplicationController
 
         sync_params = {}
 
-        if (user_params["mail"] && user_params["mail"] != user_mail) || (user_params["login"] && user_params["login"] != params[:old_user_login])
-          sync_params = sync_params.merge(email: user_params["mail"], login_name: user_params["login"], full_name: user_params["login"])
+        if (user_params["mail"] && user_params["mail"] != user_mail)
+          sync_params = sync_params.merge(email: user_params["mail"])
         end
 
         if sync_params.present?

From 6bd809525815e7d4c4bf6e62b637742d56626c75 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Tue, 11 Aug 2020 23:20:04 +0800
Subject: [PATCH 03/22] =?UTF-8?q?FIX=20=E5=8D=87=E7=BA=A7=E5=AE=8C?=
 =?UTF-8?q?=E5=96=84=E9=A1=B9=E7=9B=AE=E7=9B=B8=E5=85=B3=E8=B7=AF=E7=94=B1?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 README.md                                     |  66 +++-
 app/controllers/application_controller.rb     |  33 +-
 app/controllers/forks_controller.rb           |   3 +-
 app/controllers/issue_tags_controller.rb      |  13 +-
 app/controllers/issues_controller.rb          |  11 +-
 app/controllers/members_controller.rb         |   2 +-
 app/controllers/project_trends_controller.rb  |   4 +-
 app/controllers/projects/base_controller.rb   |   4 +-
 app/controllers/projects_controller.rb        |   7 +-
 app/controllers/pull_requests_controller.rb   |  11 +-
 app/controllers/repositories_controller.rb    |   8 +-
 .../version_releases_controller.rb            |  22 +-
 app/controllers/versions_controller.rb        |   4 +-
 app/models/concerns/project_ability.rb        |  14 +
 app/models/project.rb                         |  13 +-
 app/models/user.rb                            |   5 +-
 config/routes.rb                              | 310 ++++++++----------
 17 files changed, 284 insertions(+), 246 deletions(-)
 create mode 100644 app/models/concerns/project_ability.rb

diff --git a/README.md b/README.md
index 61f68b32a..8450de925 100644
--- a/README.md
+++ b/README.md
@@ -532,17 +532,18 @@ curl -X POST http://localhost:3000/api/repositories/1244/sync_mirror  | jq
 
 #### 项目详情
 ```
-GET api/projects/:id
+GET /api/:namespace_id/:id
 ```
 *示例*
 ```
-curl -X GET http://localhost:3000/api/projects/3263  | jq
+curl -X GET http://localhost:3000/api/jasder/jasder_test  | jq
 ```
 *请求参数说明:*
 
 |参数名|必选|类型|说明|
 |-|-|-|-|
-|id            |是|int    |项目id  |
+|namespace_id             |是|string |用户登录名  |
+|id             |是|string |项目标识identifier  |
 
 
 *返回参数说明:*
@@ -571,6 +572,50 @@ curl -X GET http://localhost:3000/api/projects/3263  | jq
 ```
 ---
 
+#### 项目详情(简版)
+```
+GET /api/:namespace_id/:id/simple
+```
+*示例*
+```
+curl -X GET http://localhost:3000/api/jasder/jasder_test/simple  | jq
+```
+*请求参数说明:*
+
+|参数名|必选|类型|说明|
+|-|-|-|-|
+|id            |是|int    |项目id  |
+
+
+*返回参数说明:*
+
+|参数名|类型|说明|
+|-|-|-|
+|id             |int   |id |
+|name           |string|项目名称|
+|identifier     |string|项目标识|
+|is_public      |boolean|项目是否公开， true:公开，false:私有|
+|description    |string|项目简介|
+|repo_id        |int|仓库id|
+|repo_identifier|string|仓库标识|
+
+
+返回值
+```
+{
+  "identifier": "jasder_test",
+  "name": "jasder的测试项目",
+  "id": 4967,
+  "type": 0,
+  "author": {
+    "login": "jasder",
+    "name": "姓名",
+    "image_url": "avatars/User/b"
+  }
+}
+```
+---
+
 #### 编辑仓库信息
 ```
 GET  /api/repositories/:id/edit.json
@@ -898,13 +943,13 @@ curl -X POST http://localhost:3000/api/projects/3297/forks  | jq
 
 #### 获取代码目录列表
 ```
-POST /api/repositories/:id/entries.json
+POST /api/:namespace_id/:project_id/repository/entries
 ```
 *示例*
 ```
 curl -X GET \
 -d "ref=develop" \
-http://localhost:3000//api/repositories/3687/entries.json  | jq
+http://localhost:3000//api/jasder/jasder_test/repository/entries  | jq
 ```
 *请求参数说明:*
 
@@ -1321,11 +1366,11 @@ http://localhost:3000/api/projects  | jq
 
 ### 获取分支列表
 ```
-GET /api/projects/:id/branches
+GET /api/:namespace_id/:id/branches
 ```
 *示例*
 ```
-curl -X GET http://localhost:3000/api/projects/4797/branches | jq
+curl -X GET http://localhost:3000/api/jasder/jasder_test/branches | jq
 ```
 *请求参数说明:*
 
@@ -1488,18 +1533,19 @@ http://localhost:3000/api/repositories/5836/tags.json | jq
 
 ## 仓库详情
 ```
-GET /api/repositories/:id
+GET /api/:namespace_id/:project_id/repository
 ```
 *示例*
 ```
 curl -X GET \
-http://localhost:3000/api/repositories/23.json | jq
+http://192.168.2.230:3000/api/jasder/forgeplus/repository | jq
 ```
 *请求参数说明:*
 
 |参数名|必选|类型|说明|
 |-|-|-|-|
-|id             |是|string |项目id  |
+|namespace_id             |是|string |用户登录名  |
+|project_id             |是|string |项目标识identifier  |
 
 
 *返回参数说明:*
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 68f0b2a48..a4ae06504 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -384,7 +384,7 @@ class ApplicationController < ActionController::Base
 
 	def current_user
 		if Rails.env.development?
-			User.current = User.find 1
+			User.current = User.find 36480
 		else
 			User.current
 		end
@@ -727,11 +727,6 @@ class ApplicationController < ActionController::Base
 		render_not_found("未找到’#{project}’相关的项目") unless @project
 	end
 
-	def find_project_with_identifier
-		@project = Project.find_by_identifier! params[:id]
-		render_not_found("未找到’#{params[:id]}’相关的项目") unless @project
-	end
-
 	def find_project_with_id
 		@project = Project.find(params[:project_id] || params[:id])
 	rescue Exception => e
@@ -743,6 +738,32 @@ class ApplicationController < ActionController::Base
 		interactor.success? ? render_ok : render_error(interactor.error)
 	end
 
+	# projects
+	def load_project
+    namespace = params[:namespace_id]
+    id = params[:project_id] || params[:id]
+
+    @project = Project.find_with_namespace(namespace, id)
+
+    if @project and current_user.can_read_project?(@project)
+			logger.info "###########： has project and can read project"
+			@project
+    elsif current_user.is_a?(AnonymousUser)
+			logger.info "###########：This is AnonymousUser"
+			@project = nil if !@project.is_public?
+			render_forbidden and return
+    else
+			logger.info "###########：project not found"
+      @project = nil
+      render_not_found and return
+    end
+    @project
+	end
+
+	def load_repository
+		@repository ||= load_project.repository
+	end
+
   private
 	def object_not_found
 		uid_logger("Missing template or cant't find record, responding with 404")
diff --git a/app/controllers/forks_controller.rb b/app/controllers/forks_controller.rb
index 93772cd19..cb18091c1 100644
--- a/app/controllers/forks_controller.rb
+++ b/app/controllers/forks_controller.rb
@@ -1,5 +1,6 @@
 class ForksController < ApplicationController
-  before_action :require_login, :find_project_with_id
+  before_action :require_login
+  before_action :load_project
   before_action :authenticate_project!, :authenticate_user!
 
   def create
diff --git a/app/controllers/issue_tags_controller.rb b/app/controllers/issue_tags_controller.rb
index dd4f2476c..fa12aefae 100644
--- a/app/controllers/issue_tags_controller.rb
+++ b/app/controllers/issue_tags_controller.rb
@@ -1,7 +1,7 @@
 class IssueTagsController < ApplicationController
   before_action :require_login, except: [:index]
-  before_action :find_project_with_id
-  before_action :set_project
+  before_action :load_repository
+  before_action :set_user
   before_action :check_issue_permission, except: :index
   before_action :set_issue_tag, only: [:edit, :update, :destroy]
 
@@ -121,13 +121,8 @@ class IssueTagsController < ApplicationController
 
   private
 
-  def set_project
-    # @project = Project.find_by_identifier! params[:project_id]
-    @repository = @project.repository
+  def set_user
     @user = @project.owner
-    normal_status(-1, "项目不存在") unless @project.present?
-    normal_status(-1, "仓库不存在") unless @repository.present?
-    normal_status(-1, "用户不存在") unless @user.present?
   end
 
   def check_issue_permission
@@ -143,4 +138,4 @@ class IssueTagsController < ApplicationController
     end
   end
 
-end
\ No newline at end of file
+end
diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb
index 418ae4f81..f0578d10e 100644
--- a/app/controllers/issues_controller.rb
+++ b/app/controllers/issues_controller.rb
@@ -1,7 +1,7 @@
 class IssuesController < ApplicationController
   before_action :require_login, except: [:index, :show, :index_chosen]
-  before_action :find_project_with_id
-  before_action :set_project_and_user
+  before_action :load_project
+  before_action :set_user
   before_action :check_issue_permission
   before_action :check_project_public, only: [:index ,:show, :copy, :index_chosen, :close_issue]
 
@@ -15,7 +15,7 @@ class IssuesController < ApplicationController
     @user_admin_or_member = current_user.present? && current_user.logged? && (current_user.admin || @project.member?(current_user))
     issues = @project.issues.issue_issue.issue_index_includes
     issues = issues.where(is_private: false) unless @user_admin_or_member
-    
+
     @all_issues_size = issues.size
     @open_issues_size = issues.where.not(status_id: 5).size
     @close_issues_size = issues.where(status_id: 5).size
@@ -347,11 +347,8 @@ class IssuesController < ApplicationController
   end
 
   private
-  def set_project_and_user
-    # @project = Project.find_by_identifier(params[:project_id]) || (Project.find params[:project_id]) || (Project.find params[:id])
+  def set_user
     @user = @project&.owner
-    # normal_status(-1, "项目不存在") unless @project.present?
-    normal_status(-1, "用户不存在") unless @user.present?
   end
 
   def check_project_public
diff --git a/app/controllers/members_controller.rb b/app/controllers/members_controller.rb
index 0a47f6a0d..2964eb1af 100644
--- a/app/controllers/members_controller.rb
+++ b/app/controllers/members_controller.rb
@@ -1,6 +1,6 @@
 class MembersController < ApplicationController
   before_action :require_login
-  before_action :find_project_with_id
+  before_action :load_project
   before_action :find_user_with_id, only: %i[create remove change_role]
   before_action :operate!, except: %i[index]
   before_action :check_member_exists!, only: %i[create]
diff --git a/app/controllers/project_trends_controller.rb b/app/controllers/project_trends_controller.rb
index 476a571dc..dc1ffbdb4 100644
--- a/app/controllers/project_trends_controller.rb
+++ b/app/controllers/project_trends_controller.rb
@@ -1,5 +1,5 @@
 class ProjectTrendsController < ApplicationController
-  before_action :find_project_with_id
+  before_action :load_repository
   before_action :check_project_public
 
   def index
@@ -44,4 +44,4 @@ class ProjectTrendsController < ApplicationController
       normal_status(-1, "您没有权限")
     end
   end
-end
\ No newline at end of file
+end
diff --git a/app/controllers/projects/base_controller.rb b/app/controllers/projects/base_controller.rb
index d874b4759..9811a2136 100644
--- a/app/controllers/projects/base_controller.rb
+++ b/app/controllers/projects/base_controller.rb
@@ -1,5 +1,7 @@
 class Projects::BaseController < ApplicationController
   include PaginateHelper
 
-  before_action :require_login, :check_auth
+  before_action :load_project
+  before_action :load_repository
+
 end
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index eb6c79777..55e4bed97 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -2,8 +2,8 @@ class ProjectsController < ApplicationController
   include ApplicationHelper
   include OperateProjectAbilityAble
   include ProjectsHelper
-  before_action :require_login, except: %i[index branches group_type_list simple]
-  before_action :find_project_with_id, only: %i[show branches update destroy fork_users praise_users watch_users]
+  before_action :require_login, except: %i[index branches group_type_list simple, show]
+  before_action :load_project
   before_action :authorizate_user_can_edit_project!, only: %i[update]
   before_action :project_public?, only: %i[fork_users praise_users watch_users]
 
@@ -100,8 +100,7 @@ class ProjectsController < ApplicationController
   end
 
   def simple
-    project = Project.includes(:owner, :repository).select(:id, :name, :identifier, :user_id, :project_type).find params[:id]
-    json_response(project)
+    json_response(@project)
   end
 
   private
diff --git a/app/controllers/pull_requests_controller.rb b/app/controllers/pull_requests_controller.rb
index bd4e0d8f8..15b03520c 100644
--- a/app/controllers/pull_requests_controller.rb
+++ b/app/controllers/pull_requests_controller.rb
@@ -1,7 +1,6 @@
 class PullRequestsController < ApplicationController
   before_action :require_login, except: [:index, :show]
-  before_action :find_project_with_id
-  before_action :set_repository
+  before_action :load_repository
   before_action :find_pull_request, except: [:index, :new, :create, :check_can_merge,:get_branches,:create_merge_infos]
   # before_action :get_relatived, only: [:edit]
   include TagChosenHelper
@@ -234,14 +233,6 @@ class PullRequestsController < ApplicationController
 
 
   private
-
-  def set_repository
-    @repository = @project.repository
-    @user = @project.owner
-    normal_status(-1, "仓库不存在") unless @repository.present?
-    normal_status(-1, "用户不存在") unless @user.present?
-  end
-
   def find_pull_request
     @pull_request = PullRequest.find_by_id(params[:id])
     @issue = @pull_request&.issue
diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb
index c8633eb92..a6c81a8de 100644
--- a/app/controllers/repositories_controller.rb
+++ b/app/controllers/repositories_controller.rb
@@ -1,11 +1,11 @@
 class RepositoriesController < ApplicationController
   include ApplicationHelper
   include OperateProjectAbilityAble
+
   before_action :require_login, only: %i[edit update create_file update_file delete_file sync_mirror]
-  before_action :find_project_with_includes, only: :show
-  before_action :find_project, except: [:tags, :commit, :sync_mirror, :show]
+  before_action :load_project
   before_action :authorizate!, except: [:sync_mirror, :tags, :commit]
-  before_action :find_repository_by_id, only: %i[commit sync_mirror tags]
+  before_action :find_repository_by_id, only: %i[commit sync_mirror]
   before_action :authorizate_user_can_edit_repo!, only: %i[sync_mirror]
   before_action :get_ref, only: %i[entries sub_entries top_counts]
   before_action :get_latest_commit, only: %i[entries sub_entries top_counts]
@@ -60,7 +60,7 @@ class RepositoriesController < ApplicationController
   end
 
   def tags
-    @tags = Gitea::Repository::Tags::ListService.new(current_user&.gitea_token, @repo.user.login, @repo.identifier, {page: params[:page], limit: params[:limit]}).call
+    @tags = Gitea::Repository::Tags::ListService.new(current_user&.gitea_token, @project.owner.login, @project.identifier, {page: params[:page], limit: params[:limit]}).call
   end
 
   def edit
diff --git a/app/controllers/version_releases_controller.rb b/app/controllers/version_releases_controller.rb
index 6717bf064..bc6f03037 100644
--- a/app/controllers/version_releases_controller.rb
+++ b/app/controllers/version_releases_controller.rb
@@ -1,6 +1,6 @@
 class VersionReleasesController < ApplicationController
-  before_action :find_project_with_id
-  before_action :set_user_and_project
+  before_action :load_repository
+  before_action :set_user
   before_action :require_login, except: [:index]
   before_action :find_version , only: [:edit, :update, :destroy]
 
@@ -78,14 +78,14 @@ class VersionReleasesController < ApplicationController
       ActiveRecord::Base.transaction do
         begin
           version_params = releases_params
-         
+
           if @version.update_attributes!(version_params)
             create_attachments(params[:attachment_ids], @version) if params[:attachment_ids].present?
             git_version_release = Gitea::Versions::UpdateService.new(@user.gitea_token, @user.try(:login), @repository.try(:identifier), version_params, @version.try(:version_gid)).call
             unless git_version_release
               raise Error, "更新失败"
             end
-            
+
             normal_status(0, "更新成功")
           else
             normal_status(-1, "更新失败")
@@ -123,14 +123,8 @@ class VersionReleasesController < ApplicationController
 
 
   private
-
-  def set_user_and_project
-    # @project = Project.find_by_id(params[:project_id])
-    @repository = @project.repository  #项目的仓库
-    @user = @project.owner
-    unless @user.present? && @project.present? && @repository.present?
-      normal_status(-1, "仓库不存在")
-    end
+  def set_user
+    @user = @repository.user
   end
 
   def find_version
@@ -140,7 +134,7 @@ class VersionReleasesController < ApplicationController
     end
   end
 
-  def releases_params 
+  def releases_params
    {
       body:	params[:body],
       draft: params[:draft] || false,
@@ -151,7 +145,7 @@ class VersionReleasesController < ApplicationController
     }
   end
 
-  def create_attachments(attachment_ids, target) 
+  def create_attachments(attachment_ids, target)
     attachment_ids.each do |id|
       attachment = Attachment.select(:id, :container_id, :container_type)&.find_by_id(id)
       unless attachment.blank?
diff --git a/app/controllers/versions_controller.rb b/app/controllers/versions_controller.rb
index 742d48313..5ec769e94 100644
--- a/app/controllers/versions_controller.rb
+++ b/app/controllers/versions_controller.rb
@@ -1,6 +1,6 @@
 class VersionsController < ApplicationController
   before_action :require_login, except: [:index, :show]
-  before_action :find_project_with_id
+  before_action :load_repository
   before_action :check_issue_permission, except: [:show, :index]
   before_action :set_version, only: [:edit, :update, :destroy, :show,:update_status]
 
@@ -166,4 +166,4 @@ class VersionsController < ApplicationController
     end
   end
 
-end
\ No newline at end of file
+end
diff --git a/app/models/concerns/project_ability.rb b/app/models/concerns/project_ability.rb
new file mode 100644
index 000000000..682f6fdc0
--- /dev/null
+++ b/app/models/concerns/project_ability.rb
@@ -0,0 +1,14 @@
+module ProjectAbility
+  extend ActiveSupport::Concern
+
+  included do
+
+  end
+
+  def can_read_project?(project)
+    return true if self.admin?
+    return false if !project.is_public? && !project.member?(self.id)
+    true
+  end
+
+end
diff --git a/app/models/project.rb b/app/models/project.rb
index f4fb1122f..bd2008af6 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -160,7 +160,7 @@ class Project < ApplicationRecord
     member&.roles&.last&.name || permission
   end
 
-  def fork_project 
+  def fork_project
     Project.find_by(id: self.forked_from_project_id)
   end
 
@@ -168,4 +168,15 @@ class Project < ApplicationRecord
     joins(:members).where(members: { user_id: member_user_id})
   end
 
+  def self.find_with_namespace(namespace_path, identifier)
+    logger.info "########namespace_path: #{namespace_path} ########identifier: #{identifier} "
+
+    user = User.find_by_login namespace_path
+    return nil if user.blank?
+
+    project = user.projects.find_by(identifier: identifier)
+
+    return nil if project.blank?
+    project
+  end
 end
diff --git a/app/models/user.rb b/app/models/user.rb
index f919ed1bf..9518f59c6 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -5,6 +5,7 @@ class User < ApplicationRecord
   include Likeable
   include BaseModel
   include ProjectOperable
+  include ProjectAbility
   # include Searchable::Dependents::User
 
   # Account statuses
@@ -70,8 +71,8 @@ class User < ApplicationRecord
   # 关注
   has_many :be_watchers, foreign_key: :user_id, dependent: :destroy # 我的关注
   has_many :be_watcher_users, through: :be_watchers, dependent: :destroy # 我关注的用户
-  
-  has_many :watchers, as: :watchable, dependent: :destroy 
+
+  has_many :watchers, as: :watchable, dependent: :destroy
 
   # 认证
   has_many :apply_user_authentication
diff --git a/config/routes.rb b/config/routes.rb
index 45ba48550..8f61f0477 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -16,8 +16,8 @@ Rails.application.routes.draw do
   resources :edu_settings
 
   scope '/api' do
-    resources :sync_forge, only: [:create] do 
-      collection do 
+    resources :sync_forge, only: [:create] do
+      collection do
         post :sync_users
       end
     end
@@ -67,41 +67,6 @@ Rails.application.routes.draw do
       end
     end
     resources :projects do
-      resources :hooks
-      resources :pull_requests, except: [:destroy] do
-        member do
-          post :pr_merge
-          # post :check_merge
-          post :refuse_merge
-        end
-        collection do
-          post :check_can_merge
-          get :create_merge_infos
-          get :get_branches
-        end
-      end
-      resources :version_releases, only: [:index,:new, :create, :edit, :update, :destroy]
-      resources :project_trends, only: [:index, :create]
-      resources :issues do
-        collection do
-          get :commit_issues
-          get :index_chosen
-          post :clean
-          post :series_update
-        end
-        member do
-         post :copy
-         post :close_issue
-         post :lock_issue
-        end
-      end
-      resources :issue_tags, only: [:create, :edit, :update, :destroy, :index]
-      resources :versions do
-        member do
-          post :update_status
-        end
-      end
-
       resources :praise_tread, only: [:index] do
         collection do
           post :like
@@ -109,28 +74,147 @@ Rails.application.routes.draw do
           get :check_like
         end
       end
-      resources :members, only: [:index, :create] do
-        collection do
-          delete :remove
-          put :change_role
-        end
-      end
-      resources :forks, only: [:create]
+
       collection do
         post :migrate
         get :group_type_list
-        post :watch
-      end
-      member do
-        get :branches
-        post :watch
-        get :watch_users
-        get :praise_users
-        get :fork_users
-        get :simple
       end
     end
 
+    # Project Area START
+    resources :namespaces, path: '/', constraints: { id: /[a-zA-Z.0-9_\-]+/ }, only: [] do
+      resources(:projects, constraints: { id: /[a-zA-Z.0-9_\-]+(?<!\.atom)/ }, except:
+              [:new, :create, :index], path: "/") do
+        member do
+          get :activity
+          get :branches
+          get :simple
+          get :branches
+          post :watch
+          get :watch_users, :path => :watchers
+          get :praise_users, :path => :stargazers
+          get :fork_users, :path => :members
+        end
+
+        resource :repository, only: [:show, :create, :edit] do
+          member do
+            get 'archive'
+            get 'top_counts'
+            get 'entries'
+            get 'sub_entries'
+            get 'commits'
+            get 'tags'
+          end
+        end
+
+        resources :issues do
+          collection do
+            get :commit_issues
+            get :index_chosen
+            post :clean
+            post :series_update
+          end
+          member do
+           post :copy
+           post :close_issue
+           post :lock_issue
+          end
+        end
+
+        resources :pull_requests, :path => :pulls, except: [:destroy] do
+          member do
+            post :pr_merge
+            # post :check_merge
+            post :refuse_merge
+          end
+          collection do
+            post :check_can_merge
+            get :create_merge_infos
+            get :get_branches
+          end
+        end
+
+        resources :versions, :path => :milestones do
+          member do
+            post :update_status
+          end
+        end
+
+        resources :members, :path => :collaborators, only: [:index, :create] do
+          collection do
+            delete :remove
+            put :change_role
+          end
+        end
+
+        resources :hooks
+        resources :forks, only: [:create]
+        resources :project_trends, :path => :activity, only: [:index, :create]
+        resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
+        resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
+
+        scope module: :projects do
+          scope do
+            get(
+              '/blob/*id/diff',
+              to: 'blob#diff',
+              constraints: { id: /.+/, format: false },
+              as: :blob_diff
+            )
+            get(
+              '/blob/*id',
+              to: 'blob#show',
+              constraints: { id: /.+/, format: false },
+              as: :blob
+            )
+            delete(
+              '/blob/*id',
+              to: 'blob#destroy',
+              constraints: { id: /.+/, format: false }
+            )
+            put(
+              '/blob/*id',
+              to: 'blob#update',
+              constraints: { id: /.+/, format: false }
+            )
+            post(
+              '/blob/*id',
+              to: 'blob#create',
+              constraints: { id: /.+/, format: false }
+            )
+          end
+
+          scope do
+            get(
+              '/raw/*id',
+              to: 'raw#show',
+              constraints: { id: /.+/, format: /(html|js)/ },
+              as: :raw
+            )
+          end
+
+          scope do
+            get(
+              '/blame/*id',
+              to: 'blame#show',
+              constraints: { id: /.+/, format: /(html|js)/ },
+              as: :blame
+            )
+          end
+
+          scope do
+            get(
+              '/tree/*id',
+              to: 'tree#show',
+              constraints: { id: /.+/, format: /(html|js)/ },
+              as: :tree
+            )
+          end
+        end
+      end
+    end
+    # Project Area END
+
     resources :accounts do
       collection do
         post :login
@@ -176,7 +260,7 @@ Rails.application.routes.draw do
 
       scope module: :users do
         # resources :courses, only: [:index]
-        resources :projects, only: [:index]
+        # resources :projects, only: [:index]
         # resources :subjects, only: [:index]
         resources :project_packages, only: [:index]
         # 私信
@@ -186,11 +270,6 @@ Rails.application.routes.draw do
         # resource :unread_message_info, only: [:show]
       end
 
-
-      resources :projects, module: :users, only: [] do
-        get :search, on: :collection
-      end
-
       resources :tidings, only: [:index]
 
       scope module: :users do
@@ -245,119 +324,6 @@ Rails.application.routes.draw do
       end
     end
 
-    resources :courses do
-      member do
-        get 'settings', :action => 'settings', :as => 'settings'
-        post 'set_invite_code_halt'
-        post 'set_public_or_private'
-        post 'search_teacher_candidate'
-        post 'add_teacher'
-        post 'create_graduation_group'
-        post 'join_graduation_group'
-        post 'set_course_group'
-        post 'change_course_admin'
-        post 'change_member_role'
-        post 'change_course_teacher'
-        post 'delete_course_teacher'
-        post 'teacher_application_review'
-        post 'transfer_to_course_group'
-        post 'delete_from_course'
-        post 'add_students_by_search'
-        post 'create_group_by_importing_file'
-        post 'duplicate_course'
-        post 'visits_plus_one'
-        get 'get_historical_courses'
-        get 'get_historical_course_students'
-        get 'course_group_list'
-        get 'add_teacher_popup'
-        get 'teachers'
-        get 'apply_teachers'
-        get 'graduation_group_list'
-        get 'top_banner'
-        get 'left_banner'
-        get 'students'
-        get 'all_course_groups'
-        get 'search_users'
-        get 'base_info'
-        get 'attahcment_category_list'
-        get 'export_member_scores_excel'  #导出课堂信息
-        get 'export_couser_info'
-        get 'export_member_act_score'
-        post 'switch_to_teacher'
-        post 'switch_to_assistant'
-        post 'switch_to_student'
-        post 'exit_course'
-        get 'informs'
-        post 'update_informs'
-        post 'new_informs'
-        delete 'delete_informs'
-        get 'online_learning'
-        post 'join_excellent_course'
-        get 'tasks_list'
-        post 'update_task_position'
-        get 'course_groups'
-        post 'join_course_group'
-        get 'work_score'
-        get 'act_score'
-        get 'statistics'
-        get 'course_videos'
-        delete 'delete_course_video'
-        post :inform_up
-        post :inform_down
-      end
-
-      collection do
-        post 'apply_to_join_course'
-        post 'search_course_list'
-        get 'board_list'
-        get 'mine'
-        get 'search_slim'
-      end
-
-      resources :course_stages, shallow: true do
-        member do
-          post :up_position
-          post :down_position
-        end
-      end
-
-      resources :course_groups, shallow: true do
-        member do
-          post 'rename_group'
-          post 'move_category'
-          post 'set_invite_code_halt'
-        end
-      end
-    end
-
-
-    resources :course_modules, shallow: true do
-      member do
-        get 'sticky_module'
-        get 'hidden_module'
-        post 'rename_module'
-        post 'add_second_category'
-      end
-      collection do
-        post 'unhidden_modules'
-      end
-    end
-
-    resources :course_second_categories, shallow: true do
-      member do
-        post 'rename_category'
-        post 'move_category'
-      end
-    end
-
-
-    resources :repertoires, only: [:index]
-
-    scope module: :projects do
-      resources :project_applies, only: [:create]
-    end
-
-
     namespace :wechats do
       resource :js_sdk_signature, only: [:create]
     end

From 5f36b517b4ebb0e705acdc14e3901cd14a228fd8 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 10:32:51 +0800
Subject: [PATCH 04/22] =?UTF-8?q?FIX=20=E5=AE=8C=E5=96=84=E8=B7=AF?=
 =?UTF-8?q?=E7=94=B1?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 config/routes.rb | 266 ++++++++++++++++++++++++++---------------------
 1 file changed, 145 insertions(+), 121 deletions(-)

diff --git a/config/routes.rb b/config/routes.rb
index 8f61f0477..6a1f91174 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -82,134 +82,158 @@ Rails.application.routes.draw do
     end
 
     # Project Area START
-    resources :namespaces, path: '/', constraints: { id: /[a-zA-Z.0-9_\-]+/ }, only: [] do
-      resources(:projects, constraints: { id: /[a-zA-Z.0-9_\-]+(?<!\.atom)/ }, except:
-              [:new, :create, :index], path: "/") do
+    scope "/:owner/:repo" do
+      scope do
+        get(
+          '/activity',
+          to: 'project_trends#index',
+          as: :project_activity
+        )
+
+        get(
+          '/branches',
+          to: 'projects#branches',
+          as: :project_branches
+        )
+
+        get(
+          '/simple',
+          to: 'projects#simple',
+          as: :project_simple
+        )
+
+        get(
+          '/watchers',
+          to: 'projects#watch_users',
+          as: :project_watchers
+        )
+
+        get(
+          '/stargazers',
+          to: 'projects#praise_users',
+          as: :project_stargazers
+        )
+
+        get(
+          '/members',
+          to: 'projects#fork_users',
+          as: :project_members
+        )
+      end
+
+      resource :repositories, path: '/', only: [:show, :create, :edit] do
         member do
-          get :activity
-          get :branches
-          get :simple
-          get :branches
-          post :watch
-          get :watch_users, :path => :watchers
-          get :praise_users, :path => :stargazers
-          get :fork_users, :path => :members
+          get 'archive'
+          get 'top_counts'
+          get 'entries'
+          get 'sub_entries'
+          get 'commits'
+          get 'tags'
+        end
+      end
+
+      resources :issues do
+        collection do
+          get :commit_issues
+          get :index_chosen
+          post :clean
+          post :series_update
+        end
+        member do
+         post :copy
+         post :close_issue
+         post :lock_issue
+        end
+      end
+
+      resources :pull_requests, :path => :pulls, except: [:destroy] do
+        member do
+          post :pr_merge
+          # post :check_merge
+          post :refuse_merge
+        end
+        collection do
+          post :check_can_merge
+          get :create_merge_infos
+          get :get_branches
+        end
+      end
+
+      resources :versions, :path => :milestones do
+        member do
+          post :update_status
+        end
+      end
+
+      resources :members, :path => :collaborators, only: [:index, :create] do
+        collection do
+          delete :remove
+          put :change_role
+        end
+      end
+
+      resources :hooks
+      resources :forks, only: [:create]
+      resources :project_trends, :path => :activity, only: [:index, :create]
+      resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
+      resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
+
+      scope module: :projects do
+        scope do
+          get(
+            '/blob/*id/diff',
+            to: 'blob#diff',
+            constraints: { id: /.+/, format: false },
+            as: :blob_diff
+          )
+          get(
+            '/blob/*id',
+            to: 'blob#show',
+            constraints: { id: /.+/, format: false },
+            as: :blob
+          )
+          delete(
+            '/blob/*id',
+            to: 'blob#destroy',
+            constraints: { id: /.+/, format: false }
+          )
+          put(
+            '/blob/*id',
+            to: 'blob#update',
+            constraints: { id: /.+/, format: false }
+          )
+          post(
+            '/blob/*id',
+            to: 'blob#create',
+            constraints: { id: /.+/, format: false }
+          )
         end
 
-        resource :repository, only: [:show, :create, :edit] do
-          member do
-            get 'archive'
-            get 'top_counts'
-            get 'entries'
-            get 'sub_entries'
-            get 'commits'
-            get 'tags'
-          end
+        scope do
+          get(
+            '/raw/*id',
+            to: 'raw#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :raw
+          )
         end
 
-        resources :issues do
-          collection do
-            get :commit_issues
-            get :index_chosen
-            post :clean
-            post :series_update
-          end
-          member do
-           post :copy
-           post :close_issue
-           post :lock_issue
-          end
+        scope do
+          get(
+            '/blame/*id',
+            to: 'blame#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :blame
+          )
         end
 
-        resources :pull_requests, :path => :pulls, except: [:destroy] do
-          member do
-            post :pr_merge
-            # post :check_merge
-            post :refuse_merge
-          end
-          collection do
-            post :check_can_merge
-            get :create_merge_infos
-            get :get_branches
-          end
-        end
-
-        resources :versions, :path => :milestones do
-          member do
-            post :update_status
-          end
-        end
-
-        resources :members, :path => :collaborators, only: [:index, :create] do
-          collection do
-            delete :remove
-            put :change_role
-          end
-        end
-
-        resources :hooks
-        resources :forks, only: [:create]
-        resources :project_trends, :path => :activity, only: [:index, :create]
-        resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
-        resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
-
-        scope module: :projects do
-          scope do
-            get(
-              '/blob/*id/diff',
-              to: 'blob#diff',
-              constraints: { id: /.+/, format: false },
-              as: :blob_diff
-            )
-            get(
-              '/blob/*id',
-              to: 'blob#show',
-              constraints: { id: /.+/, format: false },
-              as: :blob
-            )
-            delete(
-              '/blob/*id',
-              to: 'blob#destroy',
-              constraints: { id: /.+/, format: false }
-            )
-            put(
-              '/blob/*id',
-              to: 'blob#update',
-              constraints: { id: /.+/, format: false }
-            )
-            post(
-              '/blob/*id',
-              to: 'blob#create',
-              constraints: { id: /.+/, format: false }
-            )
-          end
-
-          scope do
-            get(
-              '/raw/*id',
-              to: 'raw#show',
-              constraints: { id: /.+/, format: /(html|js)/ },
-              as: :raw
-            )
-          end
-
-          scope do
-            get(
-              '/blame/*id',
-              to: 'blame#show',
-              constraints: { id: /.+/, format: /(html|js)/ },
-              as: :blame
-            )
-          end
-
-          scope do
-            get(
-              '/tree/*id',
-              to: 'tree#show',
-              constraints: { id: /.+/, format: /(html|js)/ },
-              as: :tree
-            )
-          end
+        scope do
+          get(
+            '/tree/*id',
+            to: 'tree#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :tree
+          )
         end
       end
     end

From 009b1976a79685996c915904abd972bef437d857 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 10:35:19 +0800
Subject: [PATCH 05/22] FIX bug

---
 app/controllers/application_controller.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index a4ae06504..159832177 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -740,8 +740,8 @@ class ApplicationController < ActionController::Base
 
 	# projects
 	def load_project
-    namespace = params[:namespace_id]
-    id = params[:project_id] || params[:id]
+    namespace = params[:owner]
+    id = params[:repo] || params[:id]
 
     @project = Project.find_with_namespace(namespace, id)
 

From eb0c69e802b9672eb359f759ca402c86c8c077ed Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 10:38:45 +0800
Subject: [PATCH 06/22] Add Sync Project Military Job

---
 app/controllers/projects_controller.rb   |   1 +
 app/controllers/sync_forge_controller.rb | 575 ++++++++++++-----------
 app/jobs/sync_project_military_job.rb    |  22 +
 config/routes.rb                         |   1 +
 4 files changed, 317 insertions(+), 282 deletions(-)
 create mode 100644 app/jobs/sync_project_military_job.rb

diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index eb6c79777..f36d9e8f0 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -104,6 +104,7 @@ class ProjectsController < ApplicationController
     json_response(project)
   end
 
+
   private
   def project_params
     params.permit(:user_id, :name, :description, :repository_name,
diff --git a/app/controllers/sync_forge_controller.rb b/app/controllers/sync_forge_controller.rb
index e5dbc211d..a6d266970 100644
--- a/app/controllers/sync_forge_controller.rb
+++ b/app/controllers/sync_forge_controller.rb
@@ -1,302 +1,313 @@
 class SyncForgeController < ApplicationController
   # before_action :check_token
 
-  def create 
-    ActiveRecord::Base.transaction do
-      params.permit!
-      sync_params = params[:sync_params]
-      project_user = User.where(login: sync_params[:owner_login])&.first 
-      #以前已同步的项目,那么肯定存在仓库
-      SyncLog.sync_log("=================begin_to_sync_forge: project_identifier: #{sync_params[:identifier]}========")
-      user_projects = Project.where(user_id: project_user.id)
-      if  user_projects.where(id: sync_params[:id], identifier: sync_params[:identifier]).present?
-        has_project = true
-        project = user_projects.where(id: sync_params[:id], identifier: sync_params[:identifier])&.first
-      elsif user_projects.where(id: sync_params[:id]).present?
-        has_project = true
-        project = user_projects.where(id: sync_params[:id])&.first
-      elsif user_projects.where(identifier: sync_params[:identifier]).present?
-        has_project = true
-        project = user_projects.where(identifier: sync_params[:identifier])&.first
-      else
-        has_project = false
-      end
+  def 
+    sync_counts = params[:sync_count] || 10
+    projects = Project.includes(:project_score, :repository)
+                      .where(is_public: true)
+                      .where.not(identifier: ["educoder","trustieforge", "gitlab"])
+                      .joins(:project_score).order("project_scores.changeset_num desc").limit(sync_counts.to_i)
+    projects.each do | project |
+      SyncProjectMilitaryJob.perform_later(project, project.repository, project.project_score)
+    end
+  end
 
-      if has_project
-        SyncLog.sync_log("=================begin_to_update_project========")
-        check_sync_project(project, sync_params)
-      else #新建项目
-        SyncLog.sync_log("=================begin_to_create_new_project========")
+  # def create 
+  #   ActiveRecord::Base.transaction do
+  #     params.permit!
+  #     sync_params = params[:sync_params]
+  #     project_user = User.where(login: sync_params[:owner_login])&.first 
+  #     #以前已同步的项目,那么肯定存在仓库
+  #     SyncLog.sync_log("=================begin_to_sync_forge: project_identifier: #{sync_params[:identifier]}========")
+  #     user_projects = Project.where(user_id: project_user.id)
+  #     if  user_projects.where(id: sync_params[:id], identifier: sync_params[:identifier]).present?
+  #       has_project = true
+  #       project = user_projects.where(id: sync_params[:id], identifier: sync_params[:identifier])&.first
+  #     elsif user_projects.where(id: sync_params[:id]).present?
+  #       has_project = true
+  #       project = user_projects.where(id: sync_params[:id])&.first
+  #     elsif user_projects.where(identifier: sync_params[:identifier]).present?
+  #       has_project = true
+  #       project = user_projects.where(identifier: sync_params[:identifier])&.first
+  #     else
+  #       has_project = false
+  #     end
+
+  #     if has_project
+  #       SyncLog.sync_log("=================begin_to_update_project========")
+  #       check_sync_project(project, sync_params)
+  #     else #新建项目
+  #       SyncLog.sync_log("=================begin_to_create_new_project========")
         
-        project_params = {
-          repository_name: sync_params[:identifier],
-          user_id: project_user.id,
-          private: !sync_params[:is_public],
-          name: sync_params[:name]
-        }
-        project = Projects::CreateService.new(project_user, project_params).call
-        if project.present?
-          if sync_params[:project_score].present?
-            sync_params.permit!
-            score_params = sync_params[:project_score].merge(project_id: project.id)
-            new_project_score = ProjectScore.create(score_params)
-            SyncLog.sync_log("=================new_project_score:#{new_project_score.try(:id)}========")
-          end
+  #       project_params = {
+  #         repository_name: sync_params[:identifier],
+  #         user_id: project_user.id,
+  #         private: !sync_params[:is_public],
+  #         name: sync_params[:name]
+  #       }
+  #       project = Projects::CreateService.new(project_user, project_params).call
+  #       if project.present?
+  #         if sync_params[:project_score].present?
+  #           sync_params.permit!
+  #           score_params = sync_params[:project_score].merge(project_id: project.id)
+  #           new_project_score = ProjectScore.create(score_params)
+  #           SyncLog.sync_log("=================new_project_score:#{new_project_score.try(:id)}========")
+  #         end
               
-          SyncRepositoryJob.perform_later(sync_params[:owner_login], sync_params[:identifier], sync_params[:repository], get_sudomain) if sync_params[:repository].present?
-          check_new_project(project, sync_params)
-        else
-          SyncLog.sync_project_log("=============new_project_create_failed, trustie_project_id==:#{params[:sync_params][:id]}")
-        end
-      end
-    end
-  rescue Exception => e
-    SyncLog.sync_project_log("=============sync_has_errors:==#{e.message}, project_id==:#{params[:sync_params][:id]}")
-  end
+  #         SyncRepositoryJob.perform_later(sync_params[:owner_login], sync_params[:identifier], sync_params[:repository], get_sudomain) if sync_params[:repository].present?
+  #         check_new_project(project, sync_params)
+  #       else
+  #         SyncLog.sync_project_log("=============new_project_create_failed, trustie_project_id==:#{params[:sync_params][:id]}")
+  #       end
+  #     end
+  #   end
+  # rescue Exception => e
+  #   SyncLog.sync_project_log("=============sync_has_errors:==#{e.message}, project_id==:#{params[:sync_params][:id]}")
+  # end
 
-  def sync_users
-    params.permit!
-    sync_params = params[:sync_params]
-    users_params = sync_params[:users]
+  # def sync_users
+  #   params.permit!
+  #   sync_params = params[:sync_params]
+  #   users_params = sync_params[:users]
 
-    users_params.each do |u|
-      if User.exists?(login: u[:user_params][:login])
-        SyncLog.sync_log("=================sync_to_user_been_exists====#{u[:user_params][:login]}")
-      else
-        # new_user = User.new(u[:user_params])
+  #   users_params.each do |u|
+  #     if User.exists?(login: u[:user_params][:login])
+  #       SyncLog.sync_log("=================sync_to_user_been_exists====#{u[:user_params][:login]}")
+  #     else
+  #       # new_user = User.new(u[:user_params])
 
-        if u[:user_params][:mail].blank?
-          u_mail = "#{u[:user_params][:login]}@example.com"
-        else
-          u_mail = u[:user_params][:mail]
-        end
+  #       if u[:user_params][:mail].blank?
+  #         u_mail = "#{u[:user_params][:login]}@example.com"
+  #       else
+  #         u_mail = u[:user_params][:mail]
+  #       end
 
-        new_user = User.new(u[:user_params].merge(mail: u_mail))
+  #       new_user = User.new(u[:user_params].merge(mail: u_mail))
 
-        username = new_user.login
-        password = "12345678"
-        # if new_user.save!
-        #   SyncLog.sync_log("=================sync_to_user_success==#{new_user.login}")
-        # else
-        #   SyncLog.sync_log("=================sync_to_user_failed,user_login==#{new_user.login}")
-        # end
-        ActiveRecord::Base.transaction do
-          interactor = Gitea::RegisterInteractor.call({username: username, email: new_user.mail, password: password})
-          if interactor.success?
-            gitea_user = interactor.result
-            result = Gitea::User::GenerateTokenService.new(username, password).call
-            new_user.gitea_token = result['sha1']
-            new_user.gitea_uid = gitea_user['id']
-            if new_user.save!
-              UserExtension.create!(u[:user_extensions][:user_extensions].merge(user_id: new_user.id)) if u[:user_extensions].present? && u[:user_extensions][:user_extensions].present?
-            else
-              SyncLog.sync_log("=================sync_to_user_failed,user_login==#{new_user.login}")
-            end
-          else
-            SyncLog.sync_log("=============sync_to_user_failed,user_login====#{new_user.login}")
-            SyncLog.sync_log("=================sync_to_user_failed,user_login====#{new_user.login}")
-          end
-        end
-      end
-    end
-    # normal_status(1, "completed_sync")
-  rescue Exception => e
-     SyncLog.sync_log("=================sync_user_failed====#{e}")
-  end
+  #       username = new_user.login
+  #       password = "12345678"
+  #       # if new_user.save!
+  #       #   SyncLog.sync_log("=================sync_to_user_success==#{new_user.login}")
+  #       # else
+  #       #   SyncLog.sync_log("=================sync_to_user_failed,user_login==#{new_user.login}")
+  #       # end
+  #       ActiveRecord::Base.transaction do
+  #         interactor = Gitea::RegisterInteractor.call({username: username, email: new_user.mail, password: password})
+  #         if interactor.success?
+  #           gitea_user = interactor.result
+  #           result = Gitea::User::GenerateTokenService.new(username, password).call
+  #           new_user.gitea_token = result['sha1']
+  #           new_user.gitea_uid = gitea_user['id']
+  #           if new_user.save!
+  #             UserExtension.create!(u[:user_extensions][:user_extensions].merge(user_id: new_user.id)) if u[:user_extensions].present? && u[:user_extensions][:user_extensions].present?
+  #           else
+  #             SyncLog.sync_log("=================sync_to_user_failed,user_login==#{new_user.login}")
+  #           end
+  #         else
+  #           SyncLog.sync_log("=============sync_to_user_failed,user_login====#{new_user.login}")
+  #           SyncLog.sync_log("=================sync_to_user_failed,user_login====#{new_user.login}")
+  #         end
+  #       end
+  #     end
+  #   end
+  #   # normal_status(1, "completed_sync")
+  # rescue Exception => e
+  #    SyncLog.sync_log("=================sync_user_failed====#{e}")
+  # end
 
-  private 
+  # private 
 
-  def check_sync_project(project,sync_params)
-    begin
-      gitea_main = "https://www.trustie.net/"
-      # if request.subdomain === 'testforgeplus'
-      #   gitea_main = "https://ucloudtest.trustie.net/"
-      # end
+  # def check_sync_project(project,sync_params)
+  #   begin
+  #     gitea_main = "https://www.trustie.net/"
+  #     # if request.subdomain === 'testforgeplus'
+  #     #   gitea_main = "https://ucloudtest.trustie.net/"
+  #     # end
 
-      SyncLog.sync_log("----begin_to_check_sync_project----project_id:#{project.id}---------------")
-      change_project_score(project, sync_params[:project_score], sync_params[:repository]) if sync_params[:repository].present?  #更新project_score 
-      change_project_issues(project, sync_params[:issues],project.id, gitea_main) 
-      change_project_members(project, sync_params[:members],gitea_main)
-      change_project_versions(project, sync_params[:project_versions],gitea_main)
-      change_project_watchers(project, sync_params[:project_watchers],gitea_main)
-      change_project_praises(project, sync_params[:praise_trends],gitea_main)
-    rescue => e
-      SyncLog.sync_log("=========check_sync_project_errors:#{e}===================")
-    end
+  #     SyncLog.sync_log("----begin_to_check_sync_project----project_id:#{project.id}---------------")
+  #     change_project_score(project, sync_params[:project_score], sync_params[:repository]) if sync_params[:repository].present?  #更新project_score 
+  #     change_project_issues(project, sync_params[:issues],project.id, gitea_main) 
+  #     change_project_members(project, sync_params[:members],gitea_main)
+  #     change_project_versions(project, sync_params[:project_versions],gitea_main)
+  #     change_project_watchers(project, sync_params[:project_watchers],gitea_main)
+  #     change_project_praises(project, sync_params[:praise_trends],gitea_main)
+  #   rescue => e
+  #     SyncLog.sync_log("=========check_sync_project_errors:#{e}===================")
+  #   end
     
-  end
+  # end
 
-  def check_new_project(project,sync_params)
-    SyncLog.sync_log("***8. begin_to_sync_new_project---------------")
-    sync_projects_params = {
-        type: "Project",
-        ids: sync_params[:id],
-        token: get_token,
-        sync_params: sync_params,
-        new_project_id: project.id
-      }
+  # def check_new_project(project,sync_params)
+  #   SyncLog.sync_log("***8. begin_to_sync_new_project---------------")
+  #   sync_projects_params = {
+  #       type: "Project",
+  #       ids: sync_params[:id],
+  #       token: get_token,
+  #       sync_params: sync_params,
+  #       new_project_id: project.id
+  #     }
 
-      gitea_main = "https://www.trustie.net/"
-      # if request.subdomain === 'testforgeplus'
-      #   gitea_main = "https://ucloudtest.trustie.net/"
-      # end
-      SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
-      SyncLog.sync_log("***8. end_to_sync_new_project---------------")
-  end
+  #     gitea_main = "https://www.trustie.net/"
+  #     # if request.subdomain === 'testforgeplus'
+  #     #   gitea_main = "https://ucloudtest.trustie.net/"
+  #     # end
+  #     SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
+  #     SyncLog.sync_log("***8. end_to_sync_new_project---------------")
+  # end
 
-  def change_project_praises(project, praises,gitea_main)
-    SyncLog.sync_log("***6. begin_to_sync_parises---------------")
-    forge_praises_ids = project&.praise_treads&.select(:id)&.pluck(:id)
-    diff_target_ids = praises[:ids] - forge_praises_ids
-    if diff_target_ids.size > 0
-      sync_projects_params = {
-        type: "PraiseTread",
-        ids: diff_target_ids,
-        token: get_token,
-        parent_id: project.id
-      }
-      SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
+  # def change_project_praises(project, praises,gitea_main)
+  #   SyncLog.sync_log("***6. begin_to_sync_parises---------------")
+  #   forge_praises_ids = project&.praise_treads&.select(:id)&.pluck(:id)
+  #   diff_target_ids = praises[:ids] - forge_praises_ids
+  #   if diff_target_ids.size > 0
+  #     sync_projects_params = {
+  #       type: "PraiseTread",
+  #       ids: diff_target_ids,
+  #       token: get_token,
+  #       parent_id: project.id
+  #     }
+  #     SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
       
-      SyncLog.sync_log("***6. end_to_sync_parises---------------")
-    end
-  end
+  #     SyncLog.sync_log("***6. end_to_sync_parises---------------")
+  #   end
+  # end
 
-  #检查repository和project_score
-  def change_project_score(project, project_scores, repository_params)
-    SyncLog.sync_log("***1. begin_to_sync_project_score---------------")
-    begin
-      pre_project_score = project.project_score
-      if pre_project_score.present?
-        change_num = 0
-        project_scores.each do |k,v|
-          unless pre_project_score.send("#{k}") == v
-            change_num += 1
-            pre_project_score[:"#{k}"] = v
-          end
-          if k == "changeset_num" && v.to_i > pre_project_score.changeset_num.to_i && repository_params[:url].present?
-            SyncRepositoryJob.perform_later(project.owner.try(:login), project.identifier, repository_params, get_sudomain)
-          end
-        end
-        pre_project_score.save! if change_num > 0   #如果 project_score有变化则更新
-      else 
-        ProjectScore.create!(project_scores.merge(project_id: project.id))
-      end
-      SyncLog.sync_log("***1. end_to_sync_project_score---------------")
-    rescue Exception => e
-      SyncLog.sync_log("=========change_project_score_errors:#{e}===================")
-    end
-  end
+  # #检查repository和project_score
+  # def change_project_score(project, project_scores, repository_params)
+  #   SyncLog.sync_log("***1. begin_to_sync_project_score---------------")
+  #   begin
+  #     pre_project_score = project.project_score
+  #     if pre_project_score.present?
+  #       change_num = 0
+  #       project_scores.each do |k,v|
+  #         unless pre_project_score.send("#{k}") == v
+  #           change_num += 1
+  #           pre_project_score[:"#{k}"] = v
+  #         end
+  #         if k == "changeset_num" && v.to_i > pre_project_score.changeset_num.to_i && repository_params[:url].present?
+  #           SyncRepositoryJob.perform_later(project.owner.try(:login), project.identifier, repository_params, get_sudomain)
+  #         end
+  #       end
+  #       pre_project_score.save! if change_num > 0   #如果 project_score有变化则更新
+  #     else 
+  #       ProjectScore.create!(project_scores.merge(project_id: project.id))
+  #     end
+  #     SyncLog.sync_log("***1. end_to_sync_project_score---------------")
+  #   rescue Exception => e
+  #     SyncLog.sync_log("=========change_project_score_errors:#{e}===================")
+  #   end
+  # end
 
-  def change_project_issues(project, old_issues_params,project_id, gitea_main)
-    SyncLog.sync_log("***2. begin_to_syncissues---------------")
-    begin
-      forge_issue_ids = project&.issues&.select(:id)&.pluck(:id)
-      sync_projects_params = {}
-      SyncLog.sync_log("***2--01. forge_issue_ids-#{forge_issue_ids.size.to_i}--------------")
-      if forge_issue_ids.size.to_i <= old_issues_params[:count].to_i
-        diff_issue_ids = old_issues_params[:ids] - forge_issue_ids
+  # def change_project_issues(project, old_issues_params,project_id, gitea_main)
+  #   SyncLog.sync_log("***2. begin_to_syncissues---------------")
+  #   begin
+  #     forge_issue_ids = project&.issues&.select(:id)&.pluck(:id)
+  #     sync_projects_params = {}
+  #     SyncLog.sync_log("***2--01. forge_issue_ids-#{forge_issue_ids.size.to_i}--------------")
+  #     if forge_issue_ids.size.to_i <= old_issues_params[:count].to_i
+  #       diff_issue_ids = old_issues_params[:ids] - forge_issue_ids
         
-        if diff_issue_ids.size == 0  #issue数量一样，判断评论是否有增减
-          forge_journal_ids = Journal.select([:id, :journalized_id, :journalized_type]).where(journalized_id: forge_issue_ids).pluck(:id)
-          diff_journal_ids = old_issues_params[:journals][:ids] - forge_journal_ids
-          unless diff_journal_ids.size == 0
-            sync_projects_params = {
-              type: "Journal",
-              ids: diff_journal_ids,
-              token: get_token,
-              parent_id: project_id
-            }
-            SyncLog.sync_log("***2--02. sync_projects_params-#{sync_projects_params}--------------")
-            SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
-          end
-        else
-          new_diff_ids = diff_issue_ids.in_groups_of(200).map{|k| k.reject(&:blank?)}
-          diff_len = new_diff_ids.length
-          (1..diff_len).each do |len|
-            sync_projects_params = {
-              type: "Issue",
-              ids: new_diff_ids[len-1],
-              token: get_token,
-              parent_id: project_id
-            }
-            SyncLog.sync_log("***2--030#{len}. sync_projects_params_groups-#{sync_projects_params}--------------")
-            SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
-          end
-          # sync_projects_params = {
-          #   type: "Issue",
-          #   ids: diff_issue_ids,
-          #   token: get_token,
-          #   parent_id: project_id
-          # }
-          # SyncLog.sync_log("***2--03. sync_projects_params_groups-#{sync_projects_params}--------------")
-          # SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
+  #       if diff_issue_ids.size == 0  #issue数量一样，判断评论是否有增减
+  #         forge_journal_ids = Journal.select([:id, :journalized_id, :journalized_type]).where(journalized_id: forge_issue_ids).pluck(:id)
+  #         diff_journal_ids = old_issues_params[:journals][:ids] - forge_journal_ids
+  #         unless diff_journal_ids.size == 0
+  #           sync_projects_params = {
+  #             type: "Journal",
+  #             ids: diff_journal_ids,
+  #             token: get_token,
+  #             parent_id: project_id
+  #           }
+  #           SyncLog.sync_log("***2--02. sync_projects_params-#{sync_projects_params}--------------")
+  #           SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
+  #         end
+  #       else
+  #         new_diff_ids = diff_issue_ids.in_groups_of(200).map{|k| k.reject(&:blank?)}
+  #         diff_len = new_diff_ids.length
+  #         (1..diff_len).each do |len|
+  #           sync_projects_params = {
+  #             type: "Issue",
+  #             ids: new_diff_ids[len-1],
+  #             token: get_token,
+  #             parent_id: project_id
+  #           }
+  #           SyncLog.sync_log("***2--030#{len}. sync_projects_params_groups-#{sync_projects_params}--------------")
+  #           SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
+  #         end
+  #         # sync_projects_params = {
+  #         #   type: "Issue",
+  #         #   ids: diff_issue_ids,
+  #         #   token: get_token,
+  #         #   parent_id: project_id
+  #         # }
+  #         # SyncLog.sync_log("***2--03. sync_projects_params_groups-#{sync_projects_params}--------------")
+  #         # SyncProjectsJob.perform_later(sync_projects_params, gitea_main)
          
-        end
-      end
+  #       end
+  #     end
       
-      # SyncProjectsJob.perform_later(sync_projects_params, gitea_main) if sync_projects_params.present?
-      SyncLog.sync_log("***2. end_to_syncissues---------------")
-    rescue Exception => e
-      SyncLog.sync_log("=========change_project_issues_errors:#{e}===================")
-    end
-  end
+  #     # SyncProjectsJob.perform_later(sync_projects_params, gitea_main) if sync_projects_params.present?
+  #     SyncLog.sync_log("***2. end_to_syncissues---------------")
+  #   rescue Exception => e
+  #     SyncLog.sync_log("=========change_project_issues_errors:#{e}===================")
+  #   end
+  # end
 
-  def change_project_watchers(project, watchers,gitea_main)
-    SyncLog.sync_log("***5. begin_to_sync_watchers---------------")
-    forge_watchers_ids = project&.watchers&.select(:id)&.pluck(:id)
-    if forge_watchers_ids.size.to_i <= watchers[:count].to_i
-      diff_target_ids = watchers[:ids] - forge_watchers_ids
-      if diff_target_ids.size > 0
-        sync_projects_params = {
-          type: "Watcher",
-          ids: diff_target_ids,
-          token: get_token,
-          parent_id: project.id
-        }
-        SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
+  # def change_project_watchers(project, watchers,gitea_main)
+  #   SyncLog.sync_log("***5. begin_to_sync_watchers---------------")
+  #   forge_watchers_ids = project&.watchers&.select(:id)&.pluck(:id)
+  #   if forge_watchers_ids.size.to_i <= watchers[:count].to_i
+  #     diff_target_ids = watchers[:ids] - forge_watchers_ids
+  #     if diff_target_ids.size > 0
+  #       sync_projects_params = {
+  #         type: "Watcher",
+  #         ids: diff_target_ids,
+  #         token: get_token,
+  #         parent_id: project.id
+  #       }
+  #       SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
         
-      end
-    end
-    SyncLog.sync_log("***5. begin_to_sync_watchers---------------")
-  end
+  #     end
+  #   end
+  #   SyncLog.sync_log("***5. begin_to_sync_watchers---------------")
+  # end
 
-  def change_project_versions(project, versions,gitea_main)
-    SyncLog.sync_log("***4. begin_to_sync_versions---------------")
-    forge_version_ids = project&.versions&.select(:id)&.pluck(:id)
-    if forge_version_ids.size <= versions[:count].to_i
-      diff_version_ids = versions[:ids] - forge_version_ids
-      if diff_version_ids.size > 0
-        sync_projects_params = {
-          type: "Version",
-          ids: diff_version_ids,
-          token: get_token,
-          parent_id: project.id
-        }
-        SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
-    end
+  # def change_project_versions(project, versions,gitea_main)
+  #   SyncLog.sync_log("***4. begin_to_sync_versions---------------")
+  #   forge_version_ids = project&.versions&.select(:id)&.pluck(:id)
+  #   if forge_version_ids.size <= versions[:count].to_i
+  #     diff_version_ids = versions[:ids] - forge_version_ids
+  #     if diff_version_ids.size > 0
+  #       sync_projects_params = {
+  #         type: "Version",
+  #         ids: diff_version_ids,
+  #         token: get_token,
+  #         parent_id: project.id
+  #       }
+  #       SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
+  #   end
     
-      SyncLog.sync_log("***4. end_to_sync_versions---------------")
-    end
-  end
+  #     SyncLog.sync_log("***4. end_to_sync_versions---------------")
+  #   end
+  # end
 
-  def change_project_members(project, members,gitea_main)
-    SyncLog.sync_log("***3. begin_to_sync_members---------------")
-    forge_member_ids = project&.members&.select(:id)&.pluck(:id)
-    if forge_member_ids.size <= members[:count]
-      diff_member_ids = members[:ids] - forge_member_ids
-      if diff_member_ids.size > 0
-        sync_projects_params = {
-          type: "Member",
-          ids: diff_member_ids,
-          token: get_token,
-          parent_id: project.id
-        }
-        SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
-    end
+  # def change_project_members(project, members,gitea_main)
+  #   SyncLog.sync_log("***3. begin_to_sync_members---------------")
+  #   forge_member_ids = project&.members&.select(:id)&.pluck(:id)
+  #   if forge_member_ids.size <= members[:count]
+  #     diff_member_ids = members[:ids] - forge_member_ids
+  #     if diff_member_ids.size > 0
+  #       sync_projects_params = {
+  #         type: "Member",
+  #         ids: diff_member_ids,
+  #         token: get_token,
+  #         parent_id: project.id
+  #       }
+  #       SyncProjectsJob.perform_later(sync_projects_params,gitea_main)
+  #   end
     
-      SyncLog.sync_log("***3. end_to_sync_members---------------")
-    end
-  end
+  #     SyncLog.sync_log("***3. end_to_sync_members---------------")
+  #   end
+  # end
 
   # def check_token 
   #   sync_params = params[:sync_params]
@@ -305,19 +316,19 @@ class SyncForgeController < ApplicationController
   #   end
   # end
 
-  def get_token
-    "34c82f51e0b699d9d16d70fd6497c9b1e4821d6ea3e872558a6537a091076b8e"
-  end
+  # def get_token
+  #   "34c82f51e0b699d9d16d70fd6497c9b1e4821d6ea3e872558a6537a091076b8e"
+  # end
 
-  def get_sudomain
-    SyncLog.sync_log("=================request.subdomain:#{request.subdomain}========")
-    gitea_main = "gitea.trustie.net"
-    if request.subdomain === 'testforgeplus'
-      gitea_main = "testgitea2.trustie.net"
-    # elsif request.subdomain === 'forgeplus'
-    #   gitea_main = "gitea.trustie.net"
-    end
-    return gitea_main
-  end
+  # def get_sudomain
+  #   SyncLog.sync_log("=================request.subdomain:#{request.subdomain}========")
+  #   gitea_main = "gitea.trustie.net"
+  #   if request.subdomain === 'testforgeplus'
+  #     gitea_main = "testgitea2.trustie.net"
+  #   # elsif request.subdomain === 'forgeplus'
+  #   #   gitea_main = "gitea.trustie.net"
+  #   end
+  #   return gitea_main
+  # end
 
 end
\ No newline at end of file
diff --git a/app/jobs/sync_project_military_job.rb b/app/jobs/sync_project_military_job.rb
new file mode 100644
index 000000000..0cf76f7a9
--- /dev/null
+++ b/app/jobs/sync_project_military_job.rb
@@ -0,0 +1,22 @@
+require 'uri'
+require 'net/http'
+
+class SyncProjectMilitaryJob < ApplicationJob
+  queue_as :default
+
+  def perform(project, repository, project_socre)
+    Rails.logger.info("============begin to sync project ===========")
+    project_except_params = %w(praises_count watchers_count issues_count pull_requests_count versions_count issue_tags_count closed_issues_count)
+    project_params = {
+      project: project.as_json(except: project_except_params),
+      repository: repository.as_json, 
+      project_socre: project_socre.as_json
+    }
+    url = "http://47.93.212.82:49999/sync_forges"  #trustie上的相关路由
+    uri = URI.parse(url)
+    http = Net::HTTP.new(uri.hostname, uri.port)
+    http.use_ssl = false
+    response = http.send_request('POST', uri.path, project_params.to_json, {'Content-Type' => 'application/json'})
+    Rails.logger.info("============end to sync project, status: #{response.code} ===========")
+  end
+end
diff --git a/config/routes.rb b/config/routes.rb
index 45ba48550..80ca95018 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -19,6 +19,7 @@ Rails.application.routes.draw do
     resources :sync_forge, only: [:create] do 
       collection do 
         post :sync_users
+        post :sync_range_projects
       end
     end
     resources :composes do

From 5ecd4532f2eb7a44247e30b4a31dfeaba35c6b2e Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 10:42:19 +0800
Subject: [PATCH 07/22] Repair

---
 app/controllers/sync_forge_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/sync_forge_controller.rb b/app/controllers/sync_forge_controller.rb
index a6d266970..db44a3b8c 100644
--- a/app/controllers/sync_forge_controller.rb
+++ b/app/controllers/sync_forge_controller.rb
@@ -1,7 +1,7 @@
 class SyncForgeController < ApplicationController
   # before_action :check_token
 
-  def 
+  def sync_range_projects
     sync_counts = params[:sync_count] || 10
     projects = Project.includes(:project_score, :repository)
                       .where(is_public: true)

From 0a321fd73edc5b6ec59cafac3d6ed2bcab478d59 Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 10:52:20 +0800
Subject: [PATCH 08/22] Change

---
 app/views/users/projects/shared/_project.json.jbuilder | 4 ++--
 app/views/users/subjects/shared/_subject.json.jbuilder | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/views/users/projects/shared/_project.json.jbuilder b/app/views/users/projects/shared/_project.json.jbuilder
index 5cc985909..9d1bad441 100644
--- a/app/views/users/projects/shared/_project.json.jbuilder
+++ b/app/views/users/projects/shared/_project.json.jbuilder
@@ -5,8 +5,8 @@ json.members_count project.members.count
 json.issues_count project.issues.count
 json.changesets_count project.project_score&.changeset_num.to_i
 
-json.is_public project.is_public?
-json.can_visited project.can_visited?
+json.is_public project&.is_public
+json.can_visited project&.can_visited?
 
 json.owner do
   json.partial! 'users/shared/real_user', user: project.owner
diff --git a/app/views/users/subjects/shared/_subject.json.jbuilder b/app/views/users/subjects/shared/_subject.json.jbuilder
index 2059bc461..ad83b6ece 100644
--- a/app/views/users/subjects/shared/_subject.json.jbuilder
+++ b/app/views/users/subjects/shared/_subject.json.jbuilder
@@ -5,4 +5,4 @@ json.image_url url_to_avatar(subject)
 json.owner_id subject.user.id
 json.owner_name subject.user.full_name
 json.visits_count subject.visits
-json.can_visited subject.can_visited?
+json.can_visited subject&.can_visited?

From 54b1af8d06c2d0693617284fce04cb72e3e173b6 Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 10:55:13 +0800
Subject: [PATCH 09/22] Change

---
 app/controllers/application_controller.rb | 2 +-
 app/controllers/projects_controller.rb    | 2 +-
 app/models/project.rb                     | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 159832177..0ea9db177 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -750,7 +750,7 @@ class ApplicationController < ActionController::Base
 			@project
     elsif current_user.is_a?(AnonymousUser)
 			logger.info "###########：This is AnonymousUser"
-			@project = nil if !@project.is_public?
+			@project = nil if !@project.is_public
 			render_forbidden and return
     else
 			logger.info "###########：project not found"
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 93c5f6941..30fa96601 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -116,7 +116,7 @@ class ProjectsController < ApplicationController
   end
 
   def project_public?
-    return if @project.is_public?
+    return if @project.is_public
 
     if current_user
       return if current_user.admin? || @project.member?(current_user.id)
diff --git a/app/models/project.rb b/app/models/project.rb
index bd2008af6..2519700ba 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -122,7 +122,7 @@ class Project < ApplicationRecord
 
 
   def can_visited?
-    is_public? || User.current.admin? || member?(User.current)
+    is_public || User.current.admin? || member?(User.current)
   end
 
   def releases_size(current_user_id, type)

From b46f9667ca15632375aa0ad4111eaace2251e5fe Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 10:56:28 +0800
Subject: [PATCH 10/22] =?UTF-8?q?FIX=20=E5=A4=84=E7=90=86=E7=89=88?=
 =?UTF-8?q?=E6=9C=AC=E5=BA=93=E7=9B=B8=E5=85=B3routes=E5=AF=BC=E8=87=B4?=
 =?UTF-8?q?=E5=85=B6=E4=BB=96=E8=B7=AF=E7=94=B1=E6=97=A0=E6=95=88=E7=9A=84?=
 =?UTF-8?q?=E9=97=AE=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 config/routes.rb | 317 ++++++++++++++++++++++++-----------------------
 1 file changed, 159 insertions(+), 158 deletions(-)

diff --git a/config/routes.rb b/config/routes.rb
index 6a1f91174..b1910c93a 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -81,164 +81,6 @@ Rails.application.routes.draw do
       end
     end
 
-    # Project Area START
-    scope "/:owner/:repo" do
-      scope do
-        get(
-          '/activity',
-          to: 'project_trends#index',
-          as: :project_activity
-        )
-
-        get(
-          '/branches',
-          to: 'projects#branches',
-          as: :project_branches
-        )
-
-        get(
-          '/simple',
-          to: 'projects#simple',
-          as: :project_simple
-        )
-
-        get(
-          '/watchers',
-          to: 'projects#watch_users',
-          as: :project_watchers
-        )
-
-        get(
-          '/stargazers',
-          to: 'projects#praise_users',
-          as: :project_stargazers
-        )
-
-        get(
-          '/members',
-          to: 'projects#fork_users',
-          as: :project_members
-        )
-      end
-
-      resource :repositories, path: '/', only: [:show, :create, :edit] do
-        member do
-          get 'archive'
-          get 'top_counts'
-          get 'entries'
-          get 'sub_entries'
-          get 'commits'
-          get 'tags'
-        end
-      end
-
-      resources :issues do
-        collection do
-          get :commit_issues
-          get :index_chosen
-          post :clean
-          post :series_update
-        end
-        member do
-         post :copy
-         post :close_issue
-         post :lock_issue
-        end
-      end
-
-      resources :pull_requests, :path => :pulls, except: [:destroy] do
-        member do
-          post :pr_merge
-          # post :check_merge
-          post :refuse_merge
-        end
-        collection do
-          post :check_can_merge
-          get :create_merge_infos
-          get :get_branches
-        end
-      end
-
-      resources :versions, :path => :milestones do
-        member do
-          post :update_status
-        end
-      end
-
-      resources :members, :path => :collaborators, only: [:index, :create] do
-        collection do
-          delete :remove
-          put :change_role
-        end
-      end
-
-      resources :hooks
-      resources :forks, only: [:create]
-      resources :project_trends, :path => :activity, only: [:index, :create]
-      resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
-      resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
-
-      scope module: :projects do
-        scope do
-          get(
-            '/blob/*id/diff',
-            to: 'blob#diff',
-            constraints: { id: /.+/, format: false },
-            as: :blob_diff
-          )
-          get(
-            '/blob/*id',
-            to: 'blob#show',
-            constraints: { id: /.+/, format: false },
-            as: :blob
-          )
-          delete(
-            '/blob/*id',
-            to: 'blob#destroy',
-            constraints: { id: /.+/, format: false }
-          )
-          put(
-            '/blob/*id',
-            to: 'blob#update',
-            constraints: { id: /.+/, format: false }
-          )
-          post(
-            '/blob/*id',
-            to: 'blob#create',
-            constraints: { id: /.+/, format: false }
-          )
-        end
-
-        scope do
-          get(
-            '/raw/*id',
-            to: 'raw#show',
-            constraints: { id: /.+/, format: /(html|js)/ },
-            as: :raw
-          )
-        end
-
-        scope do
-          get(
-            '/blame/*id',
-            to: 'blame#show',
-            constraints: { id: /.+/, format: /(html|js)/ },
-            as: :blame
-          )
-        end
-
-        scope do
-          get(
-            '/tree/*id',
-            to: 'tree#show',
-            constraints: { id: /.+/, format: /(html|js)/ },
-            as: :tree
-          )
-        end
-      end
-    end
-    # Project Area END
-
     resources :accounts do
       collection do
         post :login
@@ -675,4 +517,163 @@ Rails.application.routes.draw do
 
   ## react用
   get '*path', to: 'main#index', constraints: ReactConstraint.new
+
+
+  # Project Area START
+  scope "/:owner/:repo" do
+    scope do
+      get(
+        '/activity',
+        to: 'project_trends#index',
+        as: :project_activity
+      )
+
+      get(
+        '/branches',
+        to: 'projects#branches',
+        as: :project_branches
+      )
+
+      get(
+        '/simple',
+        to: 'projects#simple',
+        as: :project_simple
+      )
+
+      get(
+        '/watchers',
+        to: 'projects#watch_users',
+        as: :project_watchers
+      )
+
+      get(
+        '/stargazers',
+        to: 'projects#praise_users',
+        as: :project_stargazers
+      )
+
+      get(
+        '/members',
+        to: 'projects#fork_users',
+        as: :project_members
+      )
+    end
+
+    resource :repositories, path: '/', only: [:show, :create, :edit] do
+      member do
+        get 'archive'
+        get 'top_counts'
+        get 'entries'
+        get 'sub_entries'
+        get 'commits'
+        get 'tags'
+      end
+    end
+
+    resources :issues do
+      collection do
+        get :commit_issues
+        get :index_chosen
+        post :clean
+        post :series_update
+      end
+      member do
+       post :copy
+       post :close_issue
+       post :lock_issue
+      end
+    end
+
+    resources :pull_requests, :path => :pulls, except: [:destroy] do
+      member do
+        post :pr_merge
+        # post :check_merge
+        post :refuse_merge
+      end
+      collection do
+        post :check_can_merge
+        get :create_merge_infos
+        get :get_branches
+      end
+    end
+
+    resources :versions, :path => :milestones do
+      member do
+        post :update_status
+      end
+    end
+
+    resources :members, :path => :collaborators, only: [:index, :create] do
+      collection do
+        delete :remove
+        put :change_role
+      end
+    end
+
+    resources :hooks
+    resources :forks, only: [:create]
+    resources :project_trends, :path => :activity, only: [:index, :create]
+    resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
+    resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
+
+    scope module: :projects do
+      scope do
+        get(
+          '/blob/*id/diff',
+          to: 'blob#diff',
+          constraints: { id: /.+/, format: false },
+          as: :blob_diff
+        )
+        get(
+          '/blob/*id',
+          to: 'blob#show',
+          constraints: { id: /.+/, format: false },
+          as: :blob
+        )
+        delete(
+          '/blob/*id',
+          to: 'blob#destroy',
+          constraints: { id: /.+/, format: false }
+        )
+        put(
+          '/blob/*id',
+          to: 'blob#update',
+          constraints: { id: /.+/, format: false }
+        )
+        post(
+          '/blob/*id',
+          to: 'blob#create',
+          constraints: { id: /.+/, format: false }
+        )
+      end
+
+      scope do
+        get(
+          '/raw/*id',
+          to: 'raw#show',
+          constraints: { id: /.+/, format: /(html|js)/ },
+          as: :raw
+        )
+      end
+
+      scope do
+        get(
+          '/blame/*id',
+          to: 'blame#show',
+          constraints: { id: /.+/, format: /(html|js)/ },
+          as: :blame
+        )
+      end
+
+      scope do
+        get(
+          '/tree/*id',
+          to: 'tree#show',
+          constraints: { id: /.+/, format: /(html|js)/ },
+          as: :tree
+        )
+      end
+    end
+  end
+  # Project Area END
 end

From d80600a58ce553c714d3342beaf4609dbc2574f8 Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 10:58:16 +0800
Subject: [PATCH 11/22] Change

---
 app/controllers/application_controller.rb              | 2 +-
 app/controllers/projects_controller.rb                 | 2 +-
 app/models/project.rb                                  | 2 +-
 app/views/users/projects/shared/_project.json.jbuilder | 4 ++--
 app/views/users/subjects/shared/_subject.json.jbuilder | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 0ea9db177..d5f879696 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -750,7 +750,7 @@ class ApplicationController < ActionController::Base
 			@project
     elsif current_user.is_a?(AnonymousUser)
 			logger.info "###########：This is AnonymousUser"
-			@project = nil if !@project.is_public
+			@project = nil if @project && !@project.is_public?
 			render_forbidden and return
     else
 			logger.info "###########：project not found"
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 30fa96601..93c5f6941 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -116,7 +116,7 @@ class ProjectsController < ApplicationController
   end
 
   def project_public?
-    return if @project.is_public
+    return if @project.is_public?
 
     if current_user
       return if current_user.admin? || @project.member?(current_user.id)
diff --git a/app/models/project.rb b/app/models/project.rb
index 2519700ba..bd2008af6 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -122,7 +122,7 @@ class Project < ApplicationRecord
 
 
   def can_visited?
-    is_public || User.current.admin? || member?(User.current)
+    is_public? || User.current.admin? || member?(User.current)
   end
 
   def releases_size(current_user_id, type)
diff --git a/app/views/users/projects/shared/_project.json.jbuilder b/app/views/users/projects/shared/_project.json.jbuilder
index 9d1bad441..5cc985909 100644
--- a/app/views/users/projects/shared/_project.json.jbuilder
+++ b/app/views/users/projects/shared/_project.json.jbuilder
@@ -5,8 +5,8 @@ json.members_count project.members.count
 json.issues_count project.issues.count
 json.changesets_count project.project_score&.changeset_num.to_i
 
-json.is_public project&.is_public
-json.can_visited project&.can_visited?
+json.is_public project.is_public?
+json.can_visited project.can_visited?
 
 json.owner do
   json.partial! 'users/shared/real_user', user: project.owner
diff --git a/app/views/users/subjects/shared/_subject.json.jbuilder b/app/views/users/subjects/shared/_subject.json.jbuilder
index ad83b6ece..2059bc461 100644
--- a/app/views/users/subjects/shared/_subject.json.jbuilder
+++ b/app/views/users/subjects/shared/_subject.json.jbuilder
@@ -5,4 +5,4 @@ json.image_url url_to_avatar(subject)
 json.owner_id subject.user.id
 json.owner_name subject.user.full_name
 json.visits_count subject.visits
-json.can_visited subject&.can_visited?
+json.can_visited subject.can_visited?

From facf7a0040296810fdc8d107df27e7f266392958 Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 11:00:39 +0800
Subject: [PATCH 12/22] Chnage

---
 app/controllers/application_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index d5f879696..f78089558 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -750,7 +750,7 @@ class ApplicationController < ActionController::Base
 			@project
     elsif current_user.is_a?(AnonymousUser)
 			logger.info "###########：This is AnonymousUser"
-			@project = nil if @project && !@project.is_public?
+			@project = nil if !@project&.is_public?
 			render_forbidden and return
     else
 			logger.info "###########：project not found"

From ff847184ccd2e7271598329feedcb419c8641d47 Mon Sep 17 00:00:00 2001
From: "sylor_huang@126.com" <sylor_huang@126.com>
Date: Wed, 12 Aug 2020 11:01:34 +0800
Subject: [PATCH 13/22] Change

---
 app/controllers/application_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index f78089558..159832177 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -750,7 +750,7 @@ class ApplicationController < ActionController::Base
 			@project
     elsif current_user.is_a?(AnonymousUser)
 			logger.info "###########：This is AnonymousUser"
-			@project = nil if !@project&.is_public?
+			@project = nil if !@project.is_public?
 			render_forbidden and return
     else
 			logger.info "###########：project not found"

From 6525608f0b1ec74098c35fec6065cfe00c54de93 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 11:04:25 +0800
Subject: [PATCH 14/22] FIX routes namepace

---
 config/routes.rb | 315 ++++++++++++++++++++++++-----------------------
 1 file changed, 158 insertions(+), 157 deletions(-)

diff --git a/config/routes.rb b/config/routes.rb
index d09036bb0..42b053497 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -243,6 +243,164 @@ Rails.application.routes.draw do
         end
       end
     end
+
+    # Project Area START
+    scope "/:owner/:repo" do
+      scope do
+        get(
+          '/activity',
+          to: 'project_trends#index',
+          as: :project_activity
+        )
+
+        get(
+          '/branches',
+          to: 'projects#branches',
+          as: :project_branches
+        )
+
+        get(
+          '/simple',
+          to: 'projects#simple',
+          as: :project_simple
+        )
+
+        get(
+          '/watchers',
+          to: 'projects#watch_users',
+          as: :project_watchers
+        )
+
+        get(
+          '/stargazers',
+          to: 'projects#praise_users',
+          as: :project_stargazers
+        )
+
+        get(
+          '/members',
+          to: 'projects#fork_users',
+          as: :project_members
+        )
+      end
+
+      resource :repositories, path: '/', only: [:show, :create, :edit] do
+        member do
+          get 'archive'
+          get 'top_counts'
+          get 'entries'
+          get 'sub_entries'
+          get 'commits'
+          get 'tags'
+        end
+      end
+
+      resources :issues do
+        collection do
+          get :commit_issues
+          get :index_chosen
+          post :clean
+          post :series_update
+        end
+        member do
+         post :copy
+         post :close_issue
+         post :lock_issue
+        end
+      end
+
+      resources :pull_requests, :path => :pulls, except: [:destroy] do
+        member do
+          post :pr_merge
+          # post :check_merge
+          post :refuse_merge
+        end
+        collection do
+          post :check_can_merge
+          get :create_merge_infos
+          get :get_branches
+        end
+      end
+
+      resources :versions, :path => :milestones do
+        member do
+          post :update_status
+        end
+      end
+
+      resources :members, :path => :collaborators, only: [:index, :create] do
+        collection do
+          delete :remove
+          put :change_role
+        end
+      end
+
+      resources :hooks
+      resources :forks, only: [:create]
+      resources :project_trends, :path => :activity, only: [:index, :create]
+      resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
+      resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
+
+      scope module: :projects do
+        scope do
+          get(
+            '/blob/*id/diff',
+            to: 'blob#diff',
+            constraints: { id: /.+/, format: false },
+            as: :blob_diff
+          )
+          get(
+            '/blob/*id',
+            to: 'blob#show',
+            constraints: { id: /.+/, format: false },
+            as: :blob
+          )
+          delete(
+            '/blob/*id',
+            to: 'blob#destroy',
+            constraints: { id: /.+/, format: false }
+          )
+          put(
+            '/blob/*id',
+            to: 'blob#update',
+            constraints: { id: /.+/, format: false }
+          )
+          post(
+            '/blob/*id',
+            to: 'blob#create',
+            constraints: { id: /.+/, format: false }
+          )
+        end
+
+        scope do
+          get(
+            '/raw/*id',
+            to: 'raw#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :raw
+          )
+        end
+
+        scope do
+          get(
+            '/blame/*id',
+            to: 'blame#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :blame
+          )
+        end
+
+        scope do
+          get(
+            '/tree/*id',
+            to: 'tree#show',
+            constraints: { id: /.+/, format: /(html|js)/ },
+            as: :tree
+          )
+        end
+      end
+    end
+    # Project Area END
   end
 
   namespace :admins do
@@ -520,161 +678,4 @@ Rails.application.routes.draw do
   get '*path', to: 'main#index', constraints: ReactConstraint.new
 
 
-  # Project Area START
-  scope "/:owner/:repo" do
-    scope do
-      get(
-        '/activity',
-        to: 'project_trends#index',
-        as: :project_activity
-      )
-
-      get(
-        '/branches',
-        to: 'projects#branches',
-        as: :project_branches
-      )
-
-      get(
-        '/simple',
-        to: 'projects#simple',
-        as: :project_simple
-      )
-
-      get(
-        '/watchers',
-        to: 'projects#watch_users',
-        as: :project_watchers
-      )
-
-      get(
-        '/stargazers',
-        to: 'projects#praise_users',
-        as: :project_stargazers
-      )
-
-      get(
-        '/members',
-        to: 'projects#fork_users',
-        as: :project_members
-      )
-    end
-
-    resource :repositories, path: '/', only: [:show, :create, :edit] do
-      member do
-        get 'archive'
-        get 'top_counts'
-        get 'entries'
-        get 'sub_entries'
-        get 'commits'
-        get 'tags'
-      end
-    end
-
-    resources :issues do
-      collection do
-        get :commit_issues
-        get :index_chosen
-        post :clean
-        post :series_update
-      end
-      member do
-       post :copy
-       post :close_issue
-       post :lock_issue
-      end
-    end
-
-    resources :pull_requests, :path => :pulls, except: [:destroy] do
-      member do
-        post :pr_merge
-        # post :check_merge
-        post :refuse_merge
-      end
-      collection do
-        post :check_can_merge
-        get :create_merge_infos
-        get :get_branches
-      end
-    end
-
-    resources :versions, :path => :milestones do
-      member do
-        post :update_status
-      end
-    end
-
-    resources :members, :path => :collaborators, only: [:index, :create] do
-      collection do
-        delete :remove
-        put :change_role
-      end
-    end
-
-    resources :hooks
-    resources :forks, only: [:create]
-    resources :project_trends, :path => :activity, only: [:index, :create]
-    resources :issue_tags, :path => :labels, only: [:create, :edit, :update, :destroy, :index]
-    resources :version_releases, :path => :releases, only: [:index,:new, :create, :edit, :update, :destroy]
-
-    scope module: :projects do
-      scope do
-        get(
-          '/blob/*id/diff',
-          to: 'blob#diff',
-          constraints: { id: /.+/, format: false },
-          as: :blob_diff
-        )
-        get(
-          '/blob/*id',
-          to: 'blob#show',
-          constraints: { id: /.+/, format: false },
-          as: :blob
-        )
-        delete(
-          '/blob/*id',
-          to: 'blob#destroy',
-          constraints: { id: /.+/, format: false }
-        )
-        put(
-          '/blob/*id',
-          to: 'blob#update',
-          constraints: { id: /.+/, format: false }
-        )
-        post(
-          '/blob/*id',
-          to: 'blob#create',
-          constraints: { id: /.+/, format: false }
-        )
-      end
-
-      scope do
-        get(
-          '/raw/*id',
-          to: 'raw#show',
-          constraints: { id: /.+/, format: /(html|js)/ },
-          as: :raw
-        )
-      end
-
-      scope do
-        get(
-          '/blame/*id',
-          to: 'blame#show',
-          constraints: { id: /.+/, format: /(html|js)/ },
-          as: :blame
-        )
-      end
-
-      scope do
-        get(
-          '/tree/*id',
-          to: 'tree#show',
-          constraints: { id: /.+/, format: /(html|js)/ },
-          as: :tree
-        )
-      end
-    end
-  end
-  # Project Area END
 end

From e2926acd2f495aa34b96b430f7aa989c81353363 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 11:14:57 +0800
Subject: [PATCH 15/22] FIX some routes bug

---
 app/controllers/projects_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 93c5f6941..40f3f910e 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -3,7 +3,7 @@ class ProjectsController < ApplicationController
   include OperateProjectAbilityAble
   include ProjectsHelper
   before_action :require_login, except: %i[index branches group_type_list simple, show]
-  before_action :load_project
+  before_action :load_project, except: %i[group_type_list migrate]
   before_action :authorizate_user_can_edit_project!, only: %i[update]
   before_action :project_public?, only: %i[fork_users praise_users watch_users]
 

From 066dede6e3b11a12f0c1e5bc688724536409a2d6 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 11:18:14 +0800
Subject: [PATCH 16/22] FIX bug

---
 app/controllers/projects_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 40f3f910e..739e9ec68 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -3,7 +3,7 @@ class ProjectsController < ApplicationController
   include OperateProjectAbilityAble
   include ProjectsHelper
   before_action :require_login, except: %i[index branches group_type_list simple, show]
-  before_action :load_project, except: %i[group_type_list migrate]
+  before_action :load_project, except: %i[index group_type_list migrate]
   before_action :authorizate_user_can_edit_project!, only: %i[update]
   before_action :project_public?, only: %i[fork_users praise_users watch_users]
 

From 082c3b05b37d4283e6f2b797ab84d2b006b703a7 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 14:51:12 +0800
Subject: [PATCH 17/22] FIX repositories controller bug

---
 .../concerns/operate_project_ability_able.rb  |  2 +-
 app/controllers/repositories_controller.rb    | 11 +++---
 config/routes.rb                              | 36 +++++++------------
 3 files changed, 19 insertions(+), 30 deletions(-)

diff --git a/app/controllers/concerns/operate_project_ability_able.rb b/app/controllers/concerns/operate_project_ability_able.rb
index 13f48e6cb..4d18ae1e4 100644
--- a/app/controllers/concerns/operate_project_ability_able.rb
+++ b/app/controllers/concerns/operate_project_ability_able.rb
@@ -10,7 +10,7 @@ module OperateProjectAbilityAble
   end
 
   def authorizate_user_can_edit_repo!
-    return if @repo.project.manager?(current_user) || current_user.admin?
+    return if @repository.project.manager?(current_user) || current_user.admin?
     render_forbidden('你没有权限操作.')
   end
 
diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb
index a6c81a8de..c8ab51cba 100644
--- a/app/controllers/repositories_controller.rb
+++ b/app/controllers/repositories_controller.rb
@@ -3,9 +3,8 @@ class RepositoriesController < ApplicationController
   include OperateProjectAbilityAble
 
   before_action :require_login, only: %i[edit update create_file update_file delete_file sync_mirror]
-  before_action :load_project
+  before_action :load_repository
   before_action :authorizate!, except: [:sync_mirror, :tags, :commit]
-  before_action :find_repository_by_id, only: %i[commit sync_mirror]
   before_action :authorizate_user_can_edit_repo!, only: %i[sync_mirror]
   before_action :get_ref, only: %i[entries sub_entries top_counts]
   before_action :get_latest_commit, only: %i[entries sub_entries top_counts]
@@ -56,7 +55,7 @@ class RepositoriesController < ApplicationController
   end
 
   def commit
-    @commit = Gitea::Repository::Commits::GetService.new(@repo.user.login, @repo.identifier, params[:sha], current_user.gitea_token).call
+    @commit = Gitea::Repository::Commits::GetService.new(@repository.user.login, @repository.identifier, params[:sha], current_user.gitea_token).call
   end
 
   def tags
@@ -102,10 +101,10 @@ class RepositoriesController < ApplicationController
   end
 
   def sync_mirror
-    return render_error("正在镜像中..") if  @repo.mirror.waiting?
+    return render_error("正在镜像中..") if  @repository.mirror.waiting?
 
-    @repo.sync_mirror!
-    SyncMirroredRepositoryJob.perform_later(@repo.id, current_user.id)
+    @repository.sync_mirror!
+    SyncMirroredRepositoryJob.perform_later(@repository.id, current_user.id)
     render_ok
   end
 
diff --git a/config/routes.rb b/config/routes.rb
index 42b053497..fb7d8be63 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -155,23 +155,6 @@ Rails.application.routes.draw do
       end
     end
 
-    resources :repositories, only: [:index, :show, :edit] do
-      member do
-        get :entries
-        match :sub_entries, :via => [:get, :put]
-        get :commits
-        post :files
-        get :tags
-        post :create_file
-        put :update_file
-        delete :delete_file
-        post :repo_hook
-        post :sync_mirror
-        get :top_counts
-        get 'commits/:sha', to: 'repositories#commit', as: 'commit'
-      end
-    end
-
     resources :users_for_private_messages, only: [:index]
 
     resources :files, only: [:index, :show, :update] do
@@ -286,12 +269,19 @@ Rails.application.routes.draw do
 
       resource :repositories, path: '/', only: [:show, :create, :edit] do
         member do
-          get 'archive'
-          get 'top_counts'
-          get 'entries'
-          get 'sub_entries'
-          get 'commits'
-          get 'tags'
+          get :archive
+          get :top_counts
+          get :entries
+          match :sub_entries, :via => [:get, :put]
+          get :commits
+          get :tags
+          post :create_file
+          put :update_file
+          delete :delete_file
+          post :repo_hook
+          post :sync_mirror
+          get :top_counts
+          get 'commits/:sha', to: 'repositories#commit', as: 'commit'
         end
       end
 

From 0e951df5d0104f18773578973c7a688d269e9f6d Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 15:03:23 +0800
Subject: [PATCH 18/22] FIX pull_requests new action bug

---
 app/controllers/pull_requests_controller.rb | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/controllers/pull_requests_controller.rb b/app/controllers/pull_requests_controller.rb
index 15b03520c..ad145f492 100644
--- a/app/controllers/pull_requests_controller.rb
+++ b/app/controllers/pull_requests_controller.rb
@@ -1,6 +1,7 @@
 class PullRequestsController < ApplicationController
   before_action :require_login, except: [:index, :show]
   before_action :load_repository
+  before_action :set_user, only: [:new, :get_branches]
   before_action :find_pull_request, except: [:index, :new, :create, :check_can_merge,:get_branches,:create_merge_infos]
   # before_action :get_relatived, only: [:edit]
   include TagChosenHelper
@@ -233,6 +234,10 @@ class PullRequestsController < ApplicationController
 
 
   private
+  def set_user
+    @user = @project.owner
+  end
+
   def find_pull_request
     @pull_request = PullRequest.find_by_id(params[:id])
     @issue = @pull_request&.issue

From a0789cbd2253d285b74ac1fdce429e8e0af4fbfa Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 18:22:03 +0800
Subject: [PATCH 19/22] FIX project routes bug

---
 config/routes.rb | 38 +++++++++-----------------------------
 1 file changed, 9 insertions(+), 29 deletions(-)

diff --git a/config/routes.rb b/config/routes.rb
index fb7d8be63..a46d8d880 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -235,36 +235,16 @@ Rails.application.routes.draw do
           to: 'project_trends#index',
           as: :project_activity
         )
+      end
 
-        get(
-          '/branches',
-          to: 'projects#branches',
-          as: :project_branches
-        )
-
-        get(
-          '/simple',
-          to: 'projects#simple',
-          as: :project_simple
-        )
-
-        get(
-          '/watchers',
-          to: 'projects#watch_users',
-          as: :project_watchers
-        )
-
-        get(
-          '/stargazers',
-          to: 'projects#praise_users',
-          as: :project_stargazers
-        )
-
-        get(
-          '/members',
-          to: 'projects#fork_users',
-          as: :project_members
-        )
+      resource :projects, path: '/' do
+        member do
+          get :branches
+          get :simple
+          get :watchers, to: 'projects#watch_users'
+          get :stargazers, to: 'projects#praise_users'
+          get :members, to: 'projects#fork_users'
+        end
       end
 
       resource :repositories, path: '/', only: [:show, :create, :edit] do

From 0a9fbfd101800e2dd0b6ef50214ab644c17fdc8f Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 18:40:45 +0800
Subject: [PATCH 20/22] FIX entries api bug

---
 app/views/repositories/_commit.json.jbuilder | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/views/repositories/_commit.json.jbuilder b/app/views/repositories/_commit.json.jbuilder
index 5ad8a1bce..888508839 100644
--- a/app/views/repositories/_commit.json.jbuilder
+++ b/app/views/repositories/_commit.json.jbuilder
@@ -1,6 +1,6 @@
 json.commit do
   json.sha commit['sha']
-  json.url EduSetting.get('host_name') + commit_repository_path(project.repository, commit['sha'])
+  # json.url EduSetting.get('host_name') + commit_repository_path(project.repository, commit['sha'])
   json.message commit['commit']['message']
   json.author commit['commit']['author']
   json.committer commit['commit']['committer']

From 193ba325ce2b3260da57de8c9b4bd526ac9fd156 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 22:24:06 +0800
Subject: [PATCH 21/22] FIX code bug

---
 app/controllers/projects_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb
index 739e9ec68..97753a5f5 100644
--- a/app/controllers/projects_controller.rb
+++ b/app/controllers/projects_controller.rb
@@ -2,7 +2,7 @@ class ProjectsController < ApplicationController
   include ApplicationHelper
   include OperateProjectAbilityAble
   include ProjectsHelper
-  before_action :require_login, except: %i[index branches group_type_list simple, show]
+  before_action :require_login, except: %i[index branches group_type_list simple show]
   before_action :load_project, except: %i[index group_type_list migrate]
   before_action :authorizate_user_can_edit_project!, only: %i[update]
   before_action :project_public?, only: %i[fork_users praise_users watch_users]

From 3becb4eabc07720618055f9485194d83c154d701 Mon Sep 17 00:00:00 2001
From: Jasder <2053003901@@qq.com>
Date: Wed, 12 Aug 2020 22:41:48 +0800
Subject: [PATCH 22/22] FIX projects routes for show action bug

---
 config/routes.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/routes.rb b/config/routes.rb
index a46d8d880..7012142cb 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -237,7 +237,7 @@ Rails.application.routes.draw do
         )
       end
 
-      resource :projects, path: '/' do
+      resource :projects, path: '/', except: [:show] do
         member do
           get :branches
           get :simple