Mailer.deliver_attachments_add(@attachments) if !@attachments.empty? and Permission.find_by_controller_and_action(params[:controller], params[:action]).mail_enabled?
redirect_to :controller => 'projects', :action => 'list_files', :id => @project
end
- @versions = @project.versions
+ @versions = @project.versions.sort
end
def list_files
- @versions = @project.versions
+ @versions = @project.versions.sort
end
# Show changelog for @project
def changelog
@trackers = Tracker.find(:all, :conditions => ["is_in_chlog=?", true], :order => 'position')
- retrieve_selected_tracker_ids(@trackers)
-
- @fixed_issues = @project.issues.find(:all,
- :include => [ :fixed_version, :status, :tracker ],
- :conditions => [ "#{IssueStatus.table_name}.is_closed=? and #{Issue.table_name}.tracker_id in (#{@selected_tracker_ids.join(',')}) and #{Issue.table_name}.fixed_version_id is not null", true],
- :order => "#{Version.table_name}.effective_date DESC, #{Issue.table_name}.id DESC"
- ) unless @selected_tracker_ids.empty?
- @fixed_issues ||= []
+ retrieve_selected_tracker_ids(@trackers)
+ @versions = @project.versions.sort
end
def roadmap
@trackers = Tracker.find(:all, :conditions => ["is_in_roadmap=?", true], :order => 'position')
- retrieve_selected_tracker_ids(@trackers)
- conditions = ("1" == params[:completed] ? nil : [ "#{Version.table_name}.effective_date > ?", Date.today])
-
- @versions = @project.versions.find(:all,
- :conditions => conditions,
- :order => "#{Version.table_name}.effective_date ASC"
- )
+ retrieve_selected_tracker_ids(@trackers)
+ conditions = ("1" == params[:completed] ? nil : [ "#{Version.table_name}.effective_date > ? OR #{Version.table_name}.effective_date IS NULL", Date.today])
+ @versions = @project.versions.find(:all, :conditions => conditions).sort
end
def activity
@available_filters["assigned_to_id"] = { :type => :list_optional, :order => 4, :values => user_values }
@available_filters["author_id"] = { :type => :list, :order => 5, :values => user_values }
@available_filters["category_id"] = { :type => :list_optional, :order => 6, :values => @project.issue_categories.collect{|s| [s.name, s.id.to_s] } }
- @available_filters["fixed_version_id"] = { :type => :list_optional, :order => 7, :values => @project.versions.collect{|s| [s.name, s.id.to_s] } }
+ @available_filters["fixed_version_id"] = { :type => :list_optional, :order => 7, :values => @project.versions.sort.collect{|s| [s.name, s.id.to_s] } }
unless @project.children.empty?
@available_filters["subproject_id"] = { :type => :list_one_or_more, :order => 13, :values => @project.children.collect{|s| [s.name, s.id.to_s] } }
end
validates_presence_of :name
validates_uniqueness_of :name, :scope => [:project_id]
- validates_format_of :effective_date, :with => /^\d{4}-\d{2}-\d{2}$/, :message => :activerecord_error_not_a_date
+ validates_format_of :effective_date, :with => /^\d{4}-\d{2}-\d{2}$/, :message => :activerecord_error_not_a_date, :allow_nil => true
def start_date
effective_date
effective_date && effective_date <= Date.today
end
+ # Versions are sorted by effective_date
+ # Those with no effective_date are at the end, sorted by name
+ def <=>(version)
+ if self.effective_date
+ version.effective_date ? (self.effective_date <=> version.effective_date) : -1
+ else
+ version.effective_date ? 1 : (self.name <=> version.name)
+ end
+ end
+
private
def check_integrity
raise "Can't delete version" if self.fixed_issues.find(:first)
<% end %>
</div>
-<% if @
fixed_issues.empty? %><p><i><%= l(:label_no_data) %></i></p><% end %>
+<% if @
versions.empty? %><p><i><%= l(:label_no_data) %></i></p><% end %>
-<% ver_id = nil
- @fixed_issues.each do |issue| %>
- <% unless ver_id == issue.fixed_version_id %>
- <% if ver_id %></ul><% end %>
- <h3 class="icon22 icon22-package"><%= issue.fixed_version.name %></h3>
- <p><%= format_date(issue.fixed_version.effective_date) %><br />
- <%=h issue.fixed_version.description %></p>
+<% @versions.each do |version| %>
+ <a name="<%= version.name %>"><h3 class="icon22 icon22-package"><%= version.name %></h3></a>
+ <% if version.completed? %>
+ <p><%= format_date(version.effective_date) %></p>
+ <% elsif version.effective_date %>
+ <p><strong><%=l(:label_roadmap_due_in)%> <%= distance_of_time_in_words Time.now, version.effective_date %> (<%= format_date(version.effective_date) %>)</strong></p>
+ <% end %>
+ <p><%=h version.description %></p>
+ <% issues = version.fixed_issues.find(:all,
+ :include => [:status, :tracker],
+ :conditions => ["#{IssueStatus.table_name}.is_closed=? AND #{Issue.table_name}.tracker_id in (#{@selected_tracker_ids.join(',')})", true],
+ :order => "#{Tracker.table_name}.position")
+ %>
+ <% if !issues.empty? %>
<ul>
- <% ver_id = issue.fixed_version_id
- end %>
- <li><%= link_to_issue issue %>: <%=h issue.subject %></li>
+ <% issues.each do |issue| %>
+ <li><%= link_to_issue(issue) %>: <%=h issue.subject %></li>
+ <% end %>
+ </ul>
+ <% end %>
<% end %>
<a name="<%= version.name %>"><h3 class="icon22 icon22-package"><%= version.name %></h3></a>
<% if version.completed? %>
<p><%= format_date(version.effective_date) %></p>
- <% else %>
+ <% elsif version.effective_date %>
<p><strong><%=l(:label_roadmap_due_in)%> <%= distance_of_time_in_words Time.now, version.effective_date %> (<%= format_date(version.effective_date) %>)</strong></p>
<% end %>
<p><%=h version.description %></p>
<table class="list">
<thead><th><%= l(:label_version) %></th><th><%= l(:field_effective_date) %></th><th><%= l(:field_description) %></th><th style="width:15%"></th><th style="width:15%"></th></thead>
<tbody>
-<% for version in @project.versions %>
+<% for version in @project.versions.sort %>
<tr class="<%= cycle 'odd', 'even' %>">
<td><%=h version.name %></td>
<td align="center"><%= format_date(version.effective_date) %></td>
<!--[form:version]-->
<p><%= f.text_field :name, :size => 20, :required => true %></p>
<p><%= f.text_field :description, :size => 60 %></p>
-<p><%= f.text_field :effective_date, :size => 10, :required => true %><%= calendar_for('version_effective_date') %></p>
+<p><%= f.text_field :effective_date, :size => 10 %><%= calendar_for('version_effective_date') %></p>
<!--[eoform:version]-->
</div>
--- /dev/null
+class AllowNullVersionEffectiveDate < ActiveRecord::Migration
+ def self.up
+ change_column :versions, :effective_date, :date, :default => nil
+ end
+
+ def self.down
+ raise IrreversibleMigration
+ end
+end
id: 2\r
description: Stable release\r
effective_date: 2006-07-19\r
+versions_003: \r
+ created_on: 2006-07-19 21:00:33 +02:00\r
+ name: "2.0"\r
+ project_id: 1\r
+ updated_on: 2006-07-19 21:00:33 +02:00\r
+ id: 3\r
+ description: Future version\r
+ effective_date: \r
+
\ No newline at end of file
get :changelog, :id => 1\r
assert_response :success\r
assert_template 'changelog'\r
- assert_not_nil assigns(:fixed_issues)\r
+ assert_not_nil assigns(:versions)\r
end
def test_roadmap
projects / redmine.git / commitdiff