diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index 2d772dc1199f36d0affbc1e9f6b5877e18d9dd9e..1dad31d9e6905fb2bb7da3c2d1bbef21ad985383 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Make auto_link match urls with a port number specified. [Marcel Molina Jr.] + * Added support for toggling visual effects to ScriptaculousHelper::visual_effect, #3323. [Thomas Fuchs] * Update to script.aculo.us to 1.5.0 rev. 3343 [Thomas Fuchs] diff --git a/actionpack/lib/action_view/helpers/text_helper.rb b/actionpack/lib/action_view/helpers/text_helper.rb index 2c38ffb3454d65a142dbbf98adffc6a874850a0e..1b1ca386c26e4e5c8243b4f3bafb1e17f6772605 100644 --- a/actionpack/lib/action_view/helpers/text_helper.rb +++ b/actionpack/lib/action_view/helpers/text_helper.rb @@ -324,7 +324,7 @@ def set_cycle(name, cycle_object) (?:www\.) # www.* ) ( - ([\w]+[=?&\/.-]?)* # url segment + ([\w]+:?[=?&\/.-]?)* # url segment \w+[\/]? # url tail (?:\#\w*)? # trailing anchor ) diff --git a/actionpack/test/template/text_helper_test.rb b/actionpack/test/template/text_helper_test.rb index 5bfe0709d9315ee13522523183d9d0b78001385a..37c81e12a682d7d47af93ffbd0b04097cacf2063 100644 --- a/actionpack/test/template/text_helper_test.rb +++ b/actionpack/test/template/text_helper_test.rb @@ -128,6 +128,8 @@ def test_auto_linking link3_result = %{#{link3_raw}} link4_raw = 'http://foo.example.com/controller/action?parm=value&p2=v2#anchor123' link4_result = %{#{link4_raw}} + link5_raw = 'http://foo.example.com:3000/controller/action' + link5_result = %{#{link5_raw}} assert_equal %(hello #{email_result}), auto_link("hello #{email_raw}", :email_addresses) assert_equal %(Go to #{link_result}), auto_link("Go to #{link_raw}", :urls) @@ -152,6 +154,7 @@ def test_auto_linking assert_equal %(

Go to #{link3_result}. seriously, #{link3_result}? i think I'll say hello to #{email_result}. instead.

), auto_link(%(

Go to #{link3_raw}. seriously, #{link3_raw}? i think I'll say hello to #{email_raw}. instead.

)) assert_equal %(

Link #{link4_result}

), auto_link("

Link #{link4_raw}

") assert_equal %(

#{link4_result} Link

), auto_link("

#{link4_raw} Link

") + assert_equal %(

#{link5_result} Link

), auto_link("

#{link5_raw} Link

") end def test_auto_link_at_eol