@@ -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 => ' |
|
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"> |
|
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"> |
|
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 ' |
|
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 " |
|
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_s |
|
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