@@ -1,3 +1,4 @@ +* [#83] Added mail_to HTML helper. * [#82] Added ACL support to mack-distributed * [#81] Fixed sessions working with redirects in testing * [#80] Distributed objects present both the DRb::DRbObject inspect and the original object's inspect when asked. CHANGELOG @@ -124,6 +124,34 @@ module Mack # content_tag(:form, options, &block) end + # Builds a mailto href. By default it will generate + # JavaScript to help prevent phishing. To turn this off + # pass in the option :format => :plain + # + # mail_to("Saul Frami", "frami.saul@klocko.ca") # => + # <script>document.write(String.fromCharCode(60,97,32,104,114,101, + # 102,61,34,109,97,105,108,116,111,58,102,114,97,109,105,46,115,97,117, + # 108,64,107,108,111,99,107,111,46,99,97,34,62,83,97,117,108,32,70,114,97,109,105,60,47,97,62));</script> + def mail_to(text, email_address = nil, options = {}) + email_address = text if email_address.blank? + options = {:format => :js}.merge(options) + format = options[:format] + options - [:format] + link = link_to(text, "mailto:#{email_address}", options) + if format == :js + y = '' + link.size.times {y << 'a'} + js_link = "<script>" + c_code = [] + link.each_byte {|c| c_code << c} + js_link << "document.write(String.fromCharCode(#{c_code.join(",")}));" + js_link << "</script>" + return js_link + else + return link + end + end + private def build_options(options) opts = "" lib/mack/view_helpers/html_helpers.rb @@ -13,6 +13,22 @@ end describe Mack::ViewHelpers::HtmlHelpers do include HTMLSpecHelpers + describe "mail_to" do + + it "should use the 'text' parameter for the email parameter if one isn't given" do + mail_to("iii.blick.randy@langworthtowne.co.uk").should == %{<script>document.write(String.fromCharCode(60,97,32,104,114,101,102,61,34,109,97,105,108,116,111,58,105,105,105,46,98,108,105,99,107,46,114,97,110,100,121,64,108,97,110,103,119,111,114,116,104,116,111,119,110,101,46,99,111,46,117,107,34,62,105,105,105,46,98,108,105,99,107,46,114,97,110,100,121,64,108,97,110,103,119,111,114,116,104,116,111,119,110,101,46,99,111,46,117,107,60,47,97,62));</script>}.strip + end + + it "should build a javascript link by default" do + mail_to("Randy Blick III", "iii.blick.randy@langworthtowne.co.uk").should == %{<script>document.write(String.fromCharCode(60,97,32,104,114,101,102,61,34,109,97,105,108,116,111,58,105,105,105,46,98,108,105,99,107,46,114,97,110,100,121,64,108,97,110,103,119,111,114,116,104,116,111,119,110,101,46,99,111,46,117,107,34,62,82,97,110,100,121,32,66,108,105,99,107,32,73,73,73,60,47,97,62));</script>}.strip + end + + it "should generate a 'plain' version of the link if specified" do + mail_to("Randy Blick III", "iii.blick.randy@langworthtowne.co.uk", :format => :plain).should == link_to("Randy Blick III", "mailto:iii.blick.randy@langworthtowne.co.uk") + end + + end + describe "submit_tag" do it "should build a simple submit tag" do test/unit/view_helpers/html_helpers_spec.rb c9ff2bb597018796f83858b7fe6e0f166e285db9 Mark Bates mark@markbates.com http://github.com/markbates/mack/commit/5df6fe20c63f83b9ace55dcd6a92d58d8df31ec0 5df6fe20c63f83b9ace55dcd6a92d58d8df31ec0 2008-08-08T11:16:02-07:00 2008-08-08T11:16:02-07:00 Added mail_to HTML helper. [#83 state:resolved] e9f9ed03dc977788582485be34abc7a76094a137 Mark Bates mark@markbates.com