Commit b7e49d80 authored by Valery Sizov's avatar Valery Sizov

Fix: ci projects order

parent 2e9a7032
...@@ -5,38 +5,32 @@ module Ci ...@@ -5,38 +5,32 @@ module Ci
before_action :authenticate_user!, except: [:build, :badge, :index, :show] before_action :authenticate_user!, except: [:build, :badge, :index, :show]
before_action :authenticate_public_page!, only: :show before_action :authenticate_public_page!, only: :show
before_action :project, only: [:build, :integration, :show, :badge, :edit, :update, :destroy, :toggle_shared_runners, :dumped_yaml] before_action :project, only: [:build, :integration, :show, :badge, :edit, :update, :destroy, :toggle_shared_runners, :dumped_yaml]
before_action :authorize_access_project!, except: [:build, :gitlab, :badge, :index, :show, :new, :create] before_action :authorize_access_project!, except: [:build, :badge, :index, :show, :new, :create]
before_action :authorize_manage_project!, only: [:edit, :integration, :update, :destroy, :toggle_shared_runners, :dumped_yaml] before_action :authorize_manage_project!, only: [:edit, :integration, :update, :destroy, :toggle_shared_runners, :dumped_yaml]
before_action :authenticate_token!, only: [:build] before_action :authenticate_token!, only: [:build]
before_action :no_cache, only: [:badge] before_action :no_cache, only: [:badge]
protect_from_forgery except: :build protect_from_forgery except: :build
layout 'ci/project', except: [:index, :gitlab] layout 'ci/project', except: :index
def index def index
@projects = Ci::Project.ordered_by_last_commit_date.public_only.page(params[:page]) unless current_user
end
def gitlab
@limit, @offset = (params[:limit] || PROJECTS_BATCH).to_i, (params[:offset] || 0).to_i @limit, @offset = (params[:limit] || PROJECTS_BATCH).to_i, (params[:offset] || 0).to_i
@page = @offset == 0 ? 1 : (@offset / @limit + 1) @page = @offset == 0 ? 1 : (@offset / @limit + 1)
@gl_projects = current_user.authorized_projects if current_user
@gl_projects = @gl_projects.where("name LIKE ?", "%#{params[:search]}%") if params[:search] @projects = ProjectListBuilder.new.execute(current_user, params[:search])
@gl_projects = @gl_projects.page(@page).per(@limit)
@projects = Ci::Project.where(gitlab_id: @gl_projects.map(&:id)).ordered_by_last_commit_date @projects = @projects.page(@page).per(@limit)
@total_count = @gl_projects.size
@gl_projects = @gl_projects.where.not(id: @projects.map(&:gitlab_id)) @total_count = @projects.size
end
respond_to do |format| respond_to do |format|
format.json do format.json do
pager_json("ci/projects/gitlab", @total_count) pager_json("ci/projects/index", @total_count)
end end
format.html
end end
rescue
@error = 'Failed to fetch GitLab projects'
end end
def show def show
......
- @gl_projects.sort_by(&:name_with_namespace).each do |project|
%tr.light
%td
= project.name_with_namespace
%td
%small Not added to CI
%td
%td
- if Ci::Project.already_added?(project)
%strong.cgreen
Added
- else
= form_tag ci_projects_path do
= hidden_field_tag :project, project.to_json(methods: [:name_with_namespace, :path_with_namespace, :ssh_url_to_repo])
= submit_tag 'Add project to CI', class: 'btn btn-default btn-sm'
- last_commit = project.last_commit - if project.gitlab_ci_project
%tr.alert{class: commit_status_alert_class(last_commit) } - ci_project = project.gitlab_ci_project
- last_commit = ci_project.last_commit
%tr.alert{class: commit_status_alert_class(last_commit) }
%td %td
= link_to [:ci, project] do = link_to [:ci, ci_project] do
= project.name = ci_project.name
%td %td
- if last_commit - if last_commit
#{last_commit.status} (#{commit_link(last_commit)}) #{last_commit.status} (#{commit_link(last_commit)})
- if project.last_commit_date - if ci_project.last_commit_date
= time_ago_in_words project.last_commit_date = time_ago_in_words ci_project.last_commit_date
ago ago
- else - else
No builds yet No builds yet
%td %td
- if project.public - if ci_project.public
%i.fa.fa-globe %i.fa.fa-globe
Public Public
- else - else
%i.fa.fa-lock %i.fa.fa-lock
Private Private
%td %td
= project.commits.count = ci_project.commits.count
- else
%tr.light
%td
= project.name_with_namespace
%td
%small Not added to CI
%td
%td
= form_tag ci_projects_path do
= hidden_field_tag :project, project.to_json(methods: [:name_with_namespace, :path_with_namespace, :ssh_url_to_repo])
= submit_tag 'Add project to CI', class: 'btn btn-default btn-sm'
\ No newline at end of file
...@@ -5,13 +5,7 @@ ...@@ -5,13 +5,7 @@
.input-group-addon .input-group-addon
%i.fa.fa-search %i.fa.fa-search
:coffeescript :coffeescript
$('.ci-search-form').submit -> $('.ci-search-form').submit ->
NProgress.start() CiPager.init "#{ci_projects_path}" + "?search=" + query, #{Ci::ProjectsController::PROJECTS_BATCH}, false
query = $('.ci-search-form .search-input').val()
$.get '#{gitlab_ci_projects_path}', { search: query }, (data) ->
$(".projects").html data.html
NProgress.done()
CiPager.init "#{gitlab_ci_projects_path}" + "?search=" + query, #{Ci::ProjectsController::PROJECTS_BATCH}, false
false false
- if @offset == 0
.gray-content-block.clearfix.light.second-block
.pull-left.fetch-status
- if params[:search].present?
by keyword: "#{params[:search]}",
#{@total_count} projects, #{@projects.size} of them added to CI
.wide-table-holder
%table.table.projects-table.content-list
%thead
%tr
%th Project Name
%th Last commit
%th Access
%th Commits
= render @projects
= render "gl_projects"
%p.text-center.hide.loading
%i.fa.fa-refresh.fa-spin
- else
= render @projects
= render "gl_projects"
- if current_user - if current_user
- if @offset > 0
= render @projects
- else
.gray-content-block.top-block .gray-content-block.top-block
= render "search" = render "search"
.projects .projects
%p.fetch-status.light .gray-content-block.clearfix.light.second-block
.pull-left.fetch-status
- if params[:search].present?
by keyword: "#{params[:search]}",
#{@total_count} projects
.wide-table-holder
%table.table.projects-table.content-list
%thead
%tr
%th Project Name
%th Last commit
%th Access
%th Commits
= render @projects
%p.text-center.hide.loading
%i.fa.fa-refresh.fa-spin %i.fa.fa-refresh.fa-spin
:coffeescript :coffeescript
$.get '#{gitlab_ci_projects_path}', (data) -> CiPager.init "#{ci_projects_path}", #{Ci::ProjectsController::PROJECTS_BATCH}, false
$(".projects").html data.html
CiPager.init "#{gitlab_ci_projects_path}", #{Ci::ProjectsController::PROJECTS_BATCH}, false
- else - else
= render 'public' = render 'public'
module Ci
class ProjectListBuilder
def execute(current_user, search = nil)
projects = current_user.authorized_projects
projects = projects.search(search) if search
projects.
joins("LEFT JOIN ci_projects ON projects.id = ci_projects.gitlab_id
LEFT JOIN #{last_commit_subquery} AS last_commit ON #{Ci::Project.table_name}.id = last_commit.project_id").
reorder("ci_projects.id is NULL ASC,
CASE WHEN last_commit.committed_at IS NULL THEN 1 ELSE 0 END,
last_commit.committed_at DESC")
end
private
def last_commit_subquery
"(SELECT project_id, MAX(committed_at) committed_at FROM #{Ci::Commit.table_name} GROUP BY project_id)"
end
end
end
...@@ -84,7 +84,7 @@ describe Ci::ProjectsController do ...@@ -84,7 +84,7 @@ describe Ci::ProjectsController do
end end
it "searches projects" do it "searches projects" do
xhr :get, :gitlab, { search: "str", format: "js" }.with_indifferent_access xhr :get, :index, { search: "str", format: "json" }.with_indifferent_access
expect(response).to be_success expect(response).to be_success
expect(response.code).to eq('200') expect(response.code).to eq('200')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment