615ba0071a
This solves an N+1 query problem where we'd run multiple queries when getting the namespace owners of the displayed projects.
61 lines
1.5 KiB
Ruby
61 lines
1.5 KiB
Ruby
class Dashboard::ProjectsController < Dashboard::ApplicationController
|
|
include ParamsBackwardCompatibility
|
|
|
|
before_action :set_non_archived_param
|
|
before_action :default_sorting
|
|
|
|
def index
|
|
@projects = load_projects(params.merge(non_public: true)).page(params[:page])
|
|
|
|
respond_to do |format|
|
|
format.html
|
|
format.atom do
|
|
load_events
|
|
render layout: 'xml.atom'
|
|
end
|
|
format.json do
|
|
render json: {
|
|
html: view_to_html_string("dashboard/projects/_projects", locals: { projects: @projects })
|
|
}
|
|
end
|
|
end
|
|
end
|
|
|
|
def starred
|
|
@projects = load_projects(params.merge(starred: true))
|
|
.includes(:forked_from_project, :tags).page(params[:page])
|
|
|
|
@groups = []
|
|
|
|
respond_to do |format|
|
|
format.html
|
|
format.json do
|
|
render json: {
|
|
html: view_to_html_string("dashboard/projects/_projects", locals: { projects: @projects })
|
|
}
|
|
end
|
|
end
|
|
end
|
|
|
|
private
|
|
|
|
def default_sorting
|
|
params[:sort] ||= 'latest_activity_desc'
|
|
@sort = params[:sort]
|
|
end
|
|
|
|
def load_projects(finder_params)
|
|
ProjectsFinder
|
|
.new(params: finder_params, current_user: current_user)
|
|
.execute
|
|
.includes(:route, :creator, namespace: [:route, :owner])
|
|
end
|
|
|
|
def load_events
|
|
projects = load_projects(params.merge(non_public: true))
|
|
|
|
@events = EventCollection
|
|
.new(projects, offset: params[:offset].to_i, filter: event_filter)
|
|
.to_a
|
|
end
|
|
end
|