diff options
| author | tiagonbotelho <tiagonbotelho@hotmail.com> | 2016-03-11 17:46:14 +0000 |
|---|---|---|
| committer | tiagonbotelho <tiagonbotelho@hotmail.com> | 2016-03-12 18:08:01 +0000 |
| commit | 7530827ecae0596616623d1c4f7775b08d5ada3c (patch) | |
| tree | 57e4e09d7aebc77d1e85173a648babeb0a79d910 /app/finders/issuable_finder.rb | |
| parent | b9d13c11dee8f555b0d80fd5b9b6a42be7721461 (diff) | |
| download | gitlab-ce-7530827ecae0596616623d1c4f7775b08d5ada3c.tar.gz | |
fixes issues for mr acceptance
Diffstat (limited to 'app/finders/issuable_finder.rb')
| -rw-r--r-- | app/finders/issuable_finder.rb | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/app/finders/issuable_finder.rb b/app/finders/issuable_finder.rb index 410e6f6456c..d592bdd0eb5 100644 --- a/app/finders/issuable_finder.rb +++ b/app/finders/issuable_finder.rb @@ -94,10 +94,6 @@ class IssuableFinder params[:milestone_title].present? end - def upcoming? - params[:milestone_title] == 'Upcoming' - end - def filter_by_no_milestone? milestones? && params[:milestone_title] == Milestone::None.title end @@ -248,14 +244,17 @@ class IssuableFinder items end + def upcoming? + params[:milestone_title] == '#upcoming' && projects + end + def by_milestone(items) if milestones? if filter_by_no_milestone? items = items.where(milestone_id: [-1, nil]) elsif upcoming? - upcoming = Milestone.where(project_id: projects) - .where('due_date > ?', Time.now).order(due_date: :asc).first - items = items.joins(:milestone).where(milestone: { title: upcoming.title }) + upcoming = Milestone.upcoming(projects) + items = items.joins(:milestone).where(milestones: { title: upcoming.title }) else items = items.joins(:milestone).where(milestones: { title: params[:milestone_title] }) |