##// END OF EJS Templates
Don't prepend project name if the version is not shared....
Jean-Philippe Lang -
r12972:2e04614e218e
parent child
Show More
@@ -247,7 +247,7 module ApplicationHelper
247 end
247 end
248
248
249 def format_version_name(version)
249 def format_version_name(version)
250 if version.project == @project
250 if !version.shared? || version.project == @project
251 h(version)
251 h(version)
252 else
252 else
253 h("#{version.project} - #{version}")
253 h("#{version.project} - #{version}")
@@ -226,6 +226,11 class Version < ActiveRecord::Base
226 end
226 end
227 end
227 end
228
228
229 # Returns true if the version is shared, otherwise false
230 def shared?
231 sharing != 'none'
232 end
233
229 private
234 private
230
235
231 def load_issue_counts
236 def load_issue_counts
@@ -816,7 +816,7 class IssuesControllerTest < ActionController::TestCase
816 def test_index_with_fixed_version_column
816 def test_index_with_fixed_version_column
817 get :index, :set_filter => 1, :c => %w(fixed_version)
817 get :index, :set_filter => 1, :c => %w(fixed_version)
818 assert_select 'table.issues td.fixed_version' do
818 assert_select 'table.issues td.fixed_version' do
819 assert_select 'a[href=?]', '/versions/2', :text => 'eCookbook - 1.0'
819 assert_select 'a[href=?]', '/versions/2', :text => '1.0'
820 end
820 end
821 end
821 end
822
822
@@ -47,18 +47,18 class ProjectsHelperTest < ActionView::TestCase
47
47
48 def test_link_to_version
48 def test_link_to_version
49 User.current = User.find(1)
49 User.current = User.find(1)
50 assert_equal '<a href="/versions/5" title="07/01/2006">OnlineStore - Alpha</a>', link_to_version(Version.find(5))
50 assert_equal '<a href="/versions/5" title="07/01/2006">Alpha</a>', link_to_version(Version.find(5))
51 end
51 end
52
52
53 def test_link_to_version_without_effective_date
53 def test_link_to_version_without_effective_date
54 User.current = User.find(1)
54 User.current = User.find(1)
55 version = Version.find(5)
55 version = Version.find(5)
56 version.effective_date = nil
56 version.effective_date = nil
57 assert_equal '<a href="/versions/5">OnlineStore - Alpha</a>', link_to_version(version)
57 assert_equal '<a href="/versions/5">Alpha</a>', link_to_version(version)
58 end
58 end
59
59
60 def test_link_to_private_version
60 def test_link_to_private_version
61 assert_equal 'OnlineStore - Alpha', link_to_version(Version.find(5))
61 assert_equal 'Alpha', link_to_version(Version.find(5))
62 end
62 end
63
63
64 def test_link_to_version_invalid_version
64 def test_link_to_version_invalid_version
@@ -71,11 +71,20 class ProjectsHelperTest < ActionView::TestCase
71 end
71 end
72
72
73 def test_format_version_name
73 def test_format_version_name
74 assert_equal "eCookbook - 0.1", format_version_name(Version.find(1))
74 assert_equal "0.1", format_version_name(Version.find(1))
75 end
76
77 def test_format_version_name_for_shared_version_within_project_should_not_display_project_name
78 @project = Project.find(1)
79 version = Version.find(1)
80 version.sharing = 'system'
81 assert_equal "0.1", format_version_name(version)
75 end
82 end
76
83
77 def test_format_version_name_for_system_version
84 def test_format_version_name_for_shared_version_should_display_project_name
78 assert_equal "OnlineStore - Systemwide visible version", format_version_name(Version.find(7))
85 version = Version.find(1)
86 version.sharing = 'system'
87 assert_equal "eCookbook - 0.1", format_version_name(version)
79 end
88 end
80
89
81 def test_version_options_for_select_with_no_versions
90 def test_version_options_for_select_with_no_versions
General Comments 0
You need to be logged in to leave comments. Login now