Rework of milestones

Dmitriy Zaporozhets
1 parent b01f8b63
Showing 8 changed files with 72 additions and 34 deletions Show diff stats
app/assets/stylesheets/common.scss
app/controllers/milestones_controller.rb
app/models/milestone.rb
app/models/snippet.rb
app/views/issues/_show.html.haml
app/views/milestones/_milestone.html.haml
app/views/milestones/index.html.haml
app/views/milestones/show.html.haml
@@ -531,9 +531,14 @@ pre {
   }
 }
-.milestone .progress {
-  margin-bottom: 0;
-  margin-top: 4px;
+.milestone {
+  &.milestone-closed {
+    background: #eee;
+  }
+  .progress {
+    margin-bottom: 0;
+    margin-top: 4px;
+  }
 }
 .float-link {
@@ -12,11 +12,12 @@ class MilestonesController &lt; ProjectResourceController
   def index
     @milestones = case params[:f]
-                  when 'all'; @project.milestones
-                  else @project.milestones.active
+                  when 'all'; @project.milestones.order("closed, due_date DESC")
+                  when 'closed'; @project.milestones.closed.order("due_date DESC")
+                  else @project.milestones.active.order("due_date ASC")
                   end
-    @milestones = @milestones.includes(:project).order("due_date")
+    @milestones = @milestones.includes(:project)
     @milestones = @milestones.page(params[:page]).per(20)
   end
@@ -19,12 +19,19 @@ class Milestone &lt; ActiveRecord::Base
   has_many :issues
   has_many :merge_requests
+  scope :active, where(closed: false)
+  scope :closed, where(closed: true)
+
   validates :title, presence: true
   validates :project, presence: true
   validates :closed, inclusion: { in: [true, false] }
-  def self.active
-    where("due_date > ? OR due_date IS NULL", Date.today)
+  def expired?
+    if due_date
+      due_date < Date.today
+    else
+      false
+    end
   end
   def participants
@@ -52,4 +59,12 @@ class Milestone &lt; ActiveRecord::Base
   def expires_at
     "expires at #{due_date.stamp("Aug 21, 2011")}" if due_date
   end
+
+  def can_be_closed?
+    issues.count > 0 && open? && issues.opened.count.zero?
+  end
+
+  def open?
+    !closed
+  end
 end
@@ -22,7 +22,7 @@ class Snippet &lt; ActiveRecord::Base
   belongs_to :author, class_name: "User"
   has_many :notes, as: :noteable, dependent: :destroy
-  delegate :name, :email, to: :author, prefix: true
+  delegate :name, :email, to: :author, prefix: true, allow_nil: true
   validates :author, presence: true
   validates :project, presence: true
@@ -28,7 +28,7 @@
   %p= link_to_gfm truncate(issue.title, length: 100), project_issue_path(issue.project, issue), class: "row_title"
   %span.update-author
-    %small.cdark= "##{issue.id}"
+    %span.cdark= "##{issue.id}"
     - if issue.assignee
       assigned to #{issue.assignee_name}
     - else
-%li{class: "milestone", id: dom_id(milestone) }
+%li{class: "milestone milestone-#{milestone.closed ? 'closed' : 'open'}", id: dom_id(milestone) }
   .right
-    - if can? current_user, :admin_milestone, milestone.project
+    - if can?(current_user, :admin_milestone, milestone.project) and milestone.open?
       = link_to edit_project_milestone_path(milestone.project, milestone), class: "btn small edit-milestone-link grouped" do
         %i.icon-edit
         Edit
   %h4
     = link_to_gfm truncate(milestone.title, length: 100), project_milestone_path(milestone.project, milestone)
+    - if milestone.expired? and not milestone.closed
+      %span.cred (Expired)
     %small
       = milestone.expires_at
   .row
@@ -11,6 +11,9 @@
         %li{class: ("active" if (params[:f] == "active" || !params[:f]))}
           = link_to project_milestones_path(@project, f: "active") do
             Active
+        %li{class: ("active" if params[:f] == "closed")}
+          = link_to project_milestones_path(@project, f: "closed") do
+            Closed
         %li{class: ("active" if params[:f] == "all")}
           = link_to project_milestones_path(@project, f: "all") do
             All
@@ -19,7 +22,7 @@
       = render @milestones
       - if @milestones.present?
-        %li.bottom= paginate @milestones, remote: true, theme: "gitlab"
+        %li.bottom= paginate @milestones, theme: "gitlab"
       - else
         %li
           %h3.nothing_here_message Nothing to show here
-%h3.page_title
-  Milestone ##{@milestone.id}
-  %small
-    = @milestone.expires_at
+.row
+  .span6
+    %h3.page_title
+      Milestone ##{@milestone.id}
+      %small
+        = @milestone.expires_at
+    .back_link
+      = link_to project_milestones_path(@project) do
+        &larr; To milestones list
+  .span6
+    .right
+      - unless  @milestone.closed
+        = link_to new_project_issue_path(@project, issue: { milestone_id: @milestone.id }), class: "btn small grouped", title: "New Issue" do
+          %i.icon-plus
+          New Issue
+        = link_to 'Browse Issues', project_issues_path(@milestone.project, milestone_id: @milestone.id), class: "btn edit-milestone-link small grouped"
+        - if can?(current_user, :admin_milestone, @project)
+          = link_to edit_project_milestone_path(@project, @milestone), class: "btn small grouped" do
+            %i.icon-edit
+            Edit
-  %span.right
-    = link_to new_project_issue_path(@project, issue: { milestone_id: @milestone.id }), class: "btn small grouped", title: "New Issue" do
-      %i.icon-plus
-      New Issue
-    = link_to 'Browse Issues', project_issues_path(@milestone.project, milestone_id: @milestone.id), class: "btn edit-milestone-link small grouped"
-    - if can?(current_user, :admin_milestone, @project)
-      = link_to edit_project_milestone_path(@project, @milestone), class: "btn small grouped" do
-        %i.icon-edit
-        Edit
-.back_link
-  = link_to project_milestones_path(@project) do
-    &larr; To milestones list
+
+- if @milestone.can_be_closed?
+  %hr
+  %p
+    %span All issues for this milestone are closed. You may close milestone now.
+    = link_to 'Close Milestone', project_milestone_path(@project, @milestone, milestone: {closed: true }), method: :put, class: "btn small danger"
 .main_box
   .top_box_content
-    %h5
+    %h4.box-title
       - if @milestone.closed
-        .alert-message.error.status_info Closed
-      - else
-        .alert-message.success.status_info Open
+        .error.status_info Closed
+      - elsif @milestone.expired?
+        .error.status_info Expired
+
       = gfm escape_once(@milestone.title)
-      %small.right= @milestone.expires_at
   .middle_box_content
     %h5
@@ -34,6 +44,7 @@
         #{@milestone.closed_items_count} closed
         &ndash;
         #{@milestone.open_items_count} open
+      %span.right= @milestone.expires_at
     .progress.progress-info
       .bar{style: "width: #{@milestone.percent_complete}%;"}
@@ -43,6 +54,7 @@
       = preserve do
         = markdown @milestone.description
+
 .row
   .span6
     %table.milestone-issue-filter
	@@ -19,12 +19,19 @@ class Milestone < ActiveRecord::Base		@@ -19,12 +19,19 @@ class Milestone < ActiveRecord::Base
19	has_many :issues	19	has_many :issues
20	has_many :merge_requests	20	has_many :merge_requests
21		21
		22	+ scope :active, where(closed: false)
		23	+ scope :closed, where(closed: true)
		24	+
22	validates :title, presence: true	25	validates :title, presence: true
23	validates :project, presence: true	26	validates :project, presence: true
24	validates :closed, inclusion: { in: [true, false] }	27	validates :closed, inclusion: { in: [true, false] }
25		28
26	- def self.active
27	- where("due_date > ? OR due_date IS NULL", Date.today)	29	+ def expired?
		30	+ if due_date
		31	+ due_date < Date.today
		32	+ else
		33	+ false
		34	+ end
28	end	35	end
29		36
30	def participants	37	def participants
	@@ -52,4 +59,12 @@ class Milestone < ActiveRecord::Base		@@ -52,4 +59,12 @@ class Milestone < ActiveRecord::Base
52	def expires_at	59	def expires_at
53	"expires at #{due_date.stamp("Aug 21, 2011")}" if due_date	60	"expires at #{due_date.stamp("Aug 21, 2011")}" if due_date
54	end	61	end
		62	+
		63	+ def can_be_closed?
		64	+ issues.count > 0 && open? && issues.opened.count.zero?
		65	+ end
		66	+
		67	+ def open?
		68	+ !closed
		69	+ end
55	end	70	end
	@@ -22,7 +22,7 @@ class Snippet < ActiveRecord::Base		@@ -22,7 +22,7 @@ class Snippet < ActiveRecord::Base
22	belongs_to :author, class_name: "User"	22	belongs_to :author, class_name: "User"
23	has_many :notes, as: :noteable, dependent: :destroy	23	has_many :notes, as: :noteable, dependent: :destroy
24		24
25	- delegate :name, :email, to: :author, prefix: true	25	+ delegate :name, :email, to: :author, prefix: true, allow_nil: true
26		26
27	validates :author, presence: true	27	validates :author, presence: true
28	validates :project, presence: true	28	validates :project, presence: true
	@@ -28,7 +28,7 @@		@@ -28,7 +28,7 @@
28	%p= link_to_gfm truncate(issue.title, length: 100), project_issue_path(issue.project, issue), class: "row_title"	28	%p= link_to_gfm truncate(issue.title, length: 100), project_issue_path(issue.project, issue), class: "row_title"
29		29
30	%span.update-author	30	%span.update-author
31	- %small.cdark= "##{issue.id}"	31	+ %span.cdark= "##{issue.id}"
32	- if issue.assignee	32	- if issue.assignee
33	assigned to #{issue.assignee_name}	33	assigned to #{issue.assignee_name}
34	- else	34	- else