summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2015-11-16 19:55:58 +0100
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2015-11-16 19:55:58 +0100
commitb093f50986b6dcd0e4caf33d3c96831155e71db8 (patch)
treeed37394c7b2479a66c494768c9af339b6dc7c29d /app
parent32f1a7196817b1073327c607905ee40b9140e6df (diff)
downloadgitlab-ce-b093f50986b6dcd0e4caf33d3c96831155e71db8.tar.gz
Some code and doc improvementsglobal-milestones
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Diffstat (limited to 'app')
-rw-r--r--app/models/global_milestone.rb4
-rw-r--r--app/views/dashboard/milestones/show.html.haml2
-rw-r--r--app/views/groups/milestones/show.html.haml2
3 files changed, 6 insertions, 2 deletions
diff --git a/app/models/global_milestone.rb b/app/models/global_milestone.rb
index f96e9d41c94..1321ccd963f 100644
--- a/app/models/global_milestone.rb
+++ b/app/models/global_milestone.rb
@@ -94,4 +94,8 @@ class GlobalMilestone
def closed_merge_requests
merge_requests.values_at("closed", "merged", "locked").compact.flatten
end
+
+ def complete?
+ total_items_count == closed_items_count
+ end
end
diff --git a/app/views/dashboard/milestones/show.html.haml b/app/views/dashboard/milestones/show.html.haml
index 580db613ed4..83077a398bd 100644
--- a/app/views/dashboard/milestones/show.html.haml
+++ b/app/views/dashboard/milestones/show.html.haml
@@ -8,7 +8,7 @@
Milestone #{@milestone.title}
%hr
-- if (@milestone.total_items_count == @milestone.closed_items_count) && @milestone.active?
+- if @milestone.complete? && @milestone.active?
.alert.alert-success
%span All issues for this milestone are closed. You may close the milestone now.
diff --git a/app/views/groups/milestones/show.html.haml b/app/views/groups/milestones/show.html.haml
index 716e93f558b..d161259e4aa 100644
--- a/app/views/groups/milestones/show.html.haml
+++ b/app/views/groups/milestones/show.html.haml
@@ -16,7 +16,7 @@
= link_to 'Reopen Milestone', group_milestone_path(@group, @milestone.safe_title, title: @milestone.title, milestone: {state_event: :activate }), method: :put, class: "btn btn-sm btn-grouped btn-reopen"
%hr
-- if (@milestone.total_items_count == @milestone.closed_items_count) && @milestone.active?
+- if @milestone.complete? && @milestone.active?
.alert.alert-success
%span All issues for this milestone are closed. You may close the milestone now.