Fixed that links for relations in notifications do not include hostname (#15677).

git-svn-id: http://svn.redmine.org/redmine/trunk@12415 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Jean-Philippe Lang 2013-12-15 10:32:03 +00:00
parent c6fc1a8047
commit 13756eb3a8
3 changed files with 14 additions and 3 deletions

View File

@ -79,7 +79,8 @@ module ApplicationHelper
subject = truncate(subject, :length => options[:truncate])
end
end
s = link_to text, issue_path(issue), :class => issue.css_classes, :title => title
only_path = options[:only_path].nil? ? true : options[:only_path]
s = link_to text, issue_path(issue, :only_path => only_path), :class => issue.css_classes, :title => title
s << h(": #{subject}") if subject
s = h("#{issue.project} - ") + s if options[:project]
s

View File

@ -334,11 +334,11 @@ module IssuesHelper
if detail.value && !detail.old_value
rel_issue = Issue.visible.find_by_id(detail.value)
value = rel_issue.nil? ? "#{l(:label_issue)} ##{detail.value}" :
(no_html ? rel_issue : link_to_issue(rel_issue))
(no_html ? rel_issue : link_to_issue(rel_issue, :only_path => options[:only_path]))
elsif detail.old_value && !detail.value
rel_issue = Issue.visible.find_by_id(detail.old_value)
old_value = rel_issue.nil? ? "#{l(:label_issue)} ##{detail.old_value}" :
(no_html ? rel_issue : link_to_issue(rel_issue))
(no_html ? rel_issue : link_to_issue(rel_issue, :only_path => options[:only_path]))
end
label = l(detail.prop_key.to_sym)
end

View File

@ -113,6 +113,16 @@ class MailerTest < ActiveSupport::TestCase
end
end
def test_issue_edit_should_generate_url_with_hostname_for_relations
journal = Journal.new(:journalized => Issue.find(1), :user => User.find(1), :created_on => Time.now)
journal.details << JournalDetail.new(:property => 'relation', :prop_key => 'label_relates_to', :value => 2)
Mailer.deliver_issue_edit(journal)
assert_not_nil last_email
assert_select_email do
assert_select 'a[href=?]', 'http://mydomain.foo/issues/2', :text => 'Feature request #2'
end
end
def test_generated_links_with_prefix_and_no_relative_url_root
Setting.default_language = 'en'
relative_url_root = Redmine::Utils.relative_url_root