2012-04-23 18:06:14 +01:00
|
|
|
# -*- encoding : utf-8 -*-
|
2012-05-04 11:56:11 +01:00
|
|
|
class Projects::PullRequestsController < Projects::BaseController
|
2012-04-23 18:06:14 +01:00
|
|
|
before_filter :authenticate_user!
|
2012-05-30 19:08:07 +01:00
|
|
|
skip_before_filter :authenticate_user!, :only => [:index, :show] if APP_CONFIG['anonymous_access']
|
2012-04-23 18:06:14 +01:00
|
|
|
load_resource :project
|
2012-05-30 19:08:07 +01:00
|
|
|
|
2012-07-06 18:30:01 +01:00
|
|
|
load_and_authorize_resource :issue, :through => :project, :find_by => :serial_id, :parent => false, :except => :autocomplete_base_project
|
|
|
|
before_filter :load_pull, :except => :autocomplete_base_project
|
2012-04-23 18:06:14 +01:00
|
|
|
|
|
|
|
def new
|
2012-07-06 18:30:01 +01:00
|
|
|
base_project = (Project.find(params[:base_project_id]) if params[:base_project_id]) || PullRequest.default_base_project(@project)
|
|
|
|
authorize! :read, base_project
|
|
|
|
|
|
|
|
@pull = base_project.pull_requests.new
|
|
|
|
@pull.issue = base_project.issues.new
|
|
|
|
if pull_params && pull_params[:issue_attributes]
|
|
|
|
@pull.issue.title = pull_params[:issue_attributes][:title].presence
|
|
|
|
@pull.issue.body = pull_params[:issue_attributes][:body].presence
|
2012-05-24 18:10:49 +01:00
|
|
|
end
|
2012-04-28 18:28:57 +01:00
|
|
|
@pull.head_project = @project
|
2012-07-06 18:30:01 +01:00
|
|
|
@pull.base_ref = (pull_params[:base_ref].presence if pull_params) || @pull.base_project.default_branch
|
|
|
|
@pull.head_ref = params[:treeish].presence || (pull_params[:head_ref].presence if pull_params) || @pull.head_project.default_branch
|
|
|
|
|
2012-06-28 14:47:29 +01:00
|
|
|
@pull.check(false) # don't make event transaction
|
2012-05-29 11:09:37 +01:00
|
|
|
if @pull.status == 'already'
|
2012-06-09 10:09:38 +01:00
|
|
|
@pull.destroy
|
2012-05-24 18:10:49 +01:00
|
|
|
flash[:warning] = I18n.t('projects.pull_requests.up_to_date', :base_ref => @pull.base_ref, :head_ref => @pull.head_ref)
|
|
|
|
else
|
2012-05-31 12:23:30 +01:00
|
|
|
load_diff_commits_data
|
2012-05-23 17:09:11 +01:00
|
|
|
end
|
2012-04-23 18:06:14 +01:00
|
|
|
end
|
|
|
|
|
|
|
|
def create
|
2012-07-06 18:30:01 +01:00
|
|
|
unless pull_params
|
|
|
|
raise 'expect pull_request params' # for debug
|
|
|
|
redirect :back
|
|
|
|
end
|
|
|
|
base_project = Project.find(params[:base_project_id])
|
|
|
|
authorize! :read, base_project
|
|
|
|
|
|
|
|
@pull = base_project.pull_requests.new pull_params
|
|
|
|
@pull.issue.user, @pull.issue.project, @pull.head_project = current_user, base_project, @project
|
2012-04-28 18:28:57 +01:00
|
|
|
|
|
|
|
if @pull.save
|
2012-06-28 14:47:29 +01:00
|
|
|
@pull.check(false) # don't make event transaction
|
2012-05-29 11:09:37 +01:00
|
|
|
if @pull.status == 'already'
|
2012-05-23 17:09:11 +01:00
|
|
|
@pull.destroy
|
2012-06-09 10:09:38 +01:00
|
|
|
flash[:error] = I18n.t('projects.pull_requests.up_to_date', :base_ref => @pull.base_ref, :head_ref => @pull.head_ref)
|
2012-05-23 17:09:11 +01:00
|
|
|
render :new
|
|
|
|
else
|
2012-06-27 11:47:08 +01:00
|
|
|
@pull.save
|
2012-07-10 16:48:11 +01:00
|
|
|
redirect_to project_pull_request_path(@pull.base_project, @pull)
|
2012-05-23 17:09:11 +01:00
|
|
|
end
|
2012-04-28 18:28:57 +01:00
|
|
|
else
|
2012-05-23 17:09:11 +01:00
|
|
|
flash[:error] = t('flash.pull_request.save_error')
|
|
|
|
flash[:warning] = @pull.errors.full_messages.join('. ')
|
2012-07-10 17:10:12 +01:00
|
|
|
|
|
|
|
#@commits = @diff = @stats = nil
|
|
|
|
# FIXME wrong project or refs generate exception
|
|
|
|
@pull.check(false) # don't make event transaction
|
|
|
|
load_diff_commits_data
|
2012-04-28 18:28:57 +01:00
|
|
|
render :new
|
|
|
|
end
|
2012-04-23 18:06:14 +01:00
|
|
|
end
|
|
|
|
|
|
|
|
def update
|
2012-07-12 15:15:28 +01:00
|
|
|
if (action = params[:pull_request_action]) && %w(close reopen).include?(params[:pull_request_action])
|
|
|
|
if @pull.send("can_#{action}?")
|
|
|
|
@pull.set_user_and_time current_user
|
|
|
|
@pull.send(action)
|
|
|
|
@pull.check if @pull.open?
|
|
|
|
end
|
|
|
|
redirect_to project_pull_request_path(@pull.base_project, @pull)
|
|
|
|
else
|
|
|
|
render :nothing => true, :status => (@pull.update_attributes(params[:pull_request]) ? 200 : 500), :layout => false
|
|
|
|
end
|
2012-04-23 18:06:14 +01:00
|
|
|
end
|
|
|
|
|
2012-05-17 17:47:36 +01:00
|
|
|
def merge
|
2012-05-30 19:08:07 +01:00
|
|
|
@pull.check
|
2012-06-27 11:47:08 +01:00
|
|
|
unless @pull.merge!(current_user)
|
|
|
|
flash[:error] = t('flash.pull_request.save_error')
|
|
|
|
flash[:warning] = @pull.errors.full_messages.join('. ')
|
|
|
|
end
|
2012-07-10 17:58:39 +01:00
|
|
|
redirect_to project_pull_request_path(@pull.base_project, @pull)
|
2012-05-17 17:47:36 +01:00
|
|
|
end
|
|
|
|
|
|
|
|
def show
|
2012-05-31 12:23:30 +01:00
|
|
|
load_diff_commits_data
|
2012-05-17 17:47:36 +01:00
|
|
|
end
|
|
|
|
|
2012-07-06 18:30:01 +01:00
|
|
|
def autocomplete_base_project
|
|
|
|
#Maybe slow? ILIKE?
|
2012-05-14 18:09:18 +01:00
|
|
|
items = Project.accessible_by(current_ability, :membered)
|
2012-05-16 17:50:30 +01:00
|
|
|
items << PullRequest.default_base_project(@project)
|
2012-07-06 18:30:01 +01:00
|
|
|
logger.debug "items.count is #{items.count}"
|
2012-07-10 14:29:10 +01:00
|
|
|
items.select! {|e| Regexp.new(params[:term].downcase).match(e.name.downcase) && e.branches.count > 0}
|
2012-05-16 17:50:30 +01:00
|
|
|
items.uniq!
|
2012-07-06 18:30:01 +01:00
|
|
|
render :json => json_for_autocomplete_base(items)#, :full_name, [:branches])
|
2012-05-14 18:09:18 +01:00
|
|
|
|
2012-05-16 17:50:30 +01:00
|
|
|
end
|
|
|
|
|
2012-07-06 18:30:01 +01:00
|
|
|
protected
|
2012-05-16 17:50:30 +01:00
|
|
|
|
2012-07-06 18:30:01 +01:00
|
|
|
def pull_params
|
|
|
|
@pull_params ||= params[:pull_request].presence
|
2012-05-16 17:50:30 +01:00
|
|
|
end
|
|
|
|
|
2012-07-06 18:30:01 +01:00
|
|
|
def json_for_autocomplete_base items
|
|
|
|
items.collect do |project|
|
|
|
|
hash = {"id" => project.id.to_s, "label" => project.full_name, "value" => project.full_name}
|
|
|
|
hash[:refs] = project.branches_and_tags.map &:name
|
|
|
|
hash
|
2012-05-16 17:50:30 +01:00
|
|
|
end
|
|
|
|
end
|
2012-05-30 19:08:07 +01:00
|
|
|
|
|
|
|
def load_pull
|
2012-07-06 18:30:01 +01:00
|
|
|
@issue ||= @issues.first #FIXME! merge action create @issues?
|
2012-05-30 19:08:07 +01:00
|
|
|
if params[:action].to_sym != :index
|
2012-05-31 17:56:27 +01:00
|
|
|
@pull = @project.pull_requests.joins(:issue).where(:issues => {:id => @issue.id}).readonly(false).first
|
2012-05-30 19:08:07 +01:00
|
|
|
else
|
|
|
|
@pull_requests = @project.pull_requests
|
|
|
|
end
|
|
|
|
end
|
2012-05-31 12:23:30 +01:00
|
|
|
|
|
|
|
def load_diff_commits_data
|
|
|
|
repo = Grit::Repo.new(@pull.path)
|
2012-06-07 18:23:28 +01:00
|
|
|
@base_commit = @pull.common_ancestor
|
2012-05-31 12:23:30 +01:00
|
|
|
@head_commit = repo.commits(@pull.head_branch).first
|
|
|
|
|
2012-06-07 18:23:28 +01:00
|
|
|
@commits = repo.commits_between repo.commits(@pull.base_ref).first, @head_commit
|
|
|
|
|
|
|
|
@diff = @pull.diff repo, @base_commit, @head_commit
|
|
|
|
@stats = @pull.diff_stats repo, @base_commit, @head_commit
|
2012-05-31 12:23:30 +01:00
|
|
|
end
|
2012-04-23 18:06:14 +01:00
|
|
|
end
|