@@ -24,7 +24,16 class Mailer < ActionMailer::Base | |||||
24 | include Redmine::I18n |
|
24 | include Redmine::I18n | |
25 |
|
25 | |||
26 | def self.default_url_options |
|
26 | def self.default_url_options | |
27 |
|
|
27 | options = {:protocol => Setting.protocol} | |
|
28 | if Setting.host_name.to_s =~ /\A(https?\:\/\/)?(.+?)(\:(\d+))?(\/.+)?\z/i | |||
|
29 | host, port, prefix = $2, $4, $5 | |||
|
30 | options.merge!({ | |||
|
31 | :host => host, :port => port, :script_name => prefix | |||
|
32 | }) | |||
|
33 | else | |||
|
34 | options[:host] = Setting.host_name | |||
|
35 | end | |||
|
36 | options | |||
28 | end |
|
37 | end | |
29 |
|
38 | |||
30 | # Builds a mail for notifying to_users and cc_users about a new issue |
|
39 | # Builds a mail for notifying to_users and cc_users about a new issue |
@@ -119,6 +119,24 class MailerTest < ActiveSupport::TestCase | |||||
119 | end |
|
119 | end | |
120 | end |
|
120 | end | |
121 |
|
121 | |||
|
122 | def test_generated_links_with_port_and_prefix | |||
|
123 | with_settings :host_name => '10.0.0.1:81/redmine', :protocol => 'http' do | |||
|
124 | Mailer.test_email(User.find(1)).deliver | |||
|
125 | mail = last_email | |||
|
126 | assert_not_nil mail | |||
|
127 | assert_include 'http://10.0.0.1:81/redmine', mail_body(mail) | |||
|
128 | end | |||
|
129 | end | |||
|
130 | ||||
|
131 | def test_generated_links_with_port | |||
|
132 | with_settings :host_name => '10.0.0.1:81', :protocol => 'http' do | |||
|
133 | Mailer.test_email(User.find(1)).deliver | |||
|
134 | mail = last_email | |||
|
135 | assert_not_nil mail | |||
|
136 | assert_include 'http://10.0.0.1:81', mail_body(mail) | |||
|
137 | end | |||
|
138 | end | |||
|
139 | ||||
122 | def test_issue_edit_should_generate_url_with_hostname_for_relations |
|
140 | def test_issue_edit_should_generate_url_with_hostname_for_relations | |
123 | journal = Journal.new(:journalized => Issue.find(1), :user => User.find(1), :created_on => Time.now) |
|
141 | journal = Journal.new(:journalized => Issue.find(1), :user => User.find(1), :created_on => Time.now) | |
124 | journal.details << JournalDetail.new(:property => 'relation', :prop_key => 'label_relates_to', :value => 2) |
|
142 | journal.details << JournalDetail.new(:property => 'relation', :prop_key => 'label_relates_to', :value => 2) |
General Comments 0
You need to be logged in to leave comments.
Login now