From ffdd4bc25199556bb471767aa0e965c75827efaf Mon Sep 17 00:00:00 2001 From: David Heinemeier Hansson Date: Sun, 17 Apr 2005 06:23:27 +0000 Subject: [PATCH] Added AssetTagHelper#image_path, AssetTagHelper#javascript_path, and AssetTagHelper#stylesheet_path #1110 [Larry Halff] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1176 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- actionpack/CHANGELOG | 2 + .../action_view/helpers/asset_tag_helper.rb | 32 +++++++++++-- .../test/template/asset_tag_helper_test.rb | 48 +++++++++++++++++++ 3 files changed, 77 insertions(+), 5 deletions(-) diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index f28143ffa8..c73aa707bc 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Added AssetTagHelper#image_path, AssetTagHelper#javascript_path, and AssetTagHelper#stylesheet_path #1110 [Larry Halff] + * Fixed url_for(nil) in functional tests #1116 [Alisdair McDiarmid] * Fixed error handling of broken layouts #1115 [Michael Schubert] diff --git a/actionpack/lib/action_view/helpers/asset_tag_helper.rb b/actionpack/lib/action_view/helpers/asset_tag_helper.rb index 55036a3ff4..f888e7a3e2 100644 --- a/actionpack/lib/action_view/helpers/asset_tag_helper.rb +++ b/actionpack/lib/action_view/helpers/asset_tag_helper.rb @@ -23,6 +23,13 @@ module ActionView ) end + # Returns path to a javascript asset. Example: + # + # javascript_path "xmlhr" # => /javascripts/xmlhr.js + def javascript_path(source) + compute_public_path(source, 'javascripts', 'js') + end + # Returns a script include tag per source given as argument. Examples: # # javascript_include_tag "xmlhr" # => @@ -33,11 +40,18 @@ module ActionView # def javascript_include_tag(*sources) sources.collect { |source| - source = compute_public_path(source, 'javascripts', 'js') + source = javascript_path(source) content_tag("script", "", "type" => "text/javascript", "src" => source) }.join("\n") end + # Returns path to a stylesheet asset. Example: + # + # stylesheet_path "style" # => /stylesheets/style.css + def stylesheet_path(source) + compute_public_path(source, 'stylesheets', 'css') + end + # Returns a css link tag per source given as argument. Examples: # # stylesheet_link_tag "style" # => @@ -48,11 +62,21 @@ module ActionView # def stylesheet_link_tag(*sources) sources.collect { |source| - source = compute_public_path(source, 'stylesheets', 'css') + source = stylesheet_path(source) tag("link", "rel" => "Stylesheet", "type" => "text/css", "media" => "screen", "href" => source) }.join("\n") end + # Returns path to an image asset. Example: + # + # The +src+ can be supplied as a... + # * full path, like "/my_images/image.gif" + # * file name, like "rss.gif", that gets expanded to "/images/rss.gif" + # * file name without extension, like "logo", that gets expanded to "/images/logo.png" + def image_path(source) + compute_public_path(source, 'images', 'png') + end + # Returns an image tag converting the +options+ instead html options on the tag, but with these special cases: # # * :alt - If no alt text is given, the file name part of the +src+ is used (capitalized and without the extension) @@ -65,7 +89,7 @@ module ActionView def image_tag(source, options = {}) options.symbolize_keys - options[:src] = compute_public_path(source, 'images', 'png') + options[:src] = image_path(source) options[:alt] ||= source.split("/").last.split(".").first.capitalize if options[:size] @@ -77,14 +101,12 @@ module ActionView end private - def compute_public_path(source, dir, ext) source = "/#{dir}/#{source}" unless source.include?("/") source = "#{source}.#{ext}" unless source.include?(".") source = "#{@request.relative_url_root}#{source}" source end - end end end diff --git a/actionpack/test/template/asset_tag_helper_test.rb b/actionpack/test/template/asset_tag_helper_test.rb index 10244e6b81..3d04abde67 100644 --- a/actionpack/test/template/asset_tag_helper_test.rb +++ b/actionpack/test/template/asset_tag_helper_test.rb @@ -29,16 +29,28 @@ class AssetTagHelperTest < Test::Unit::TestCase %(auto_discovery_link_tag(:rss, :action => "feed")) => %(), } + JavascriptPathToTag = { + %(javascript_path("xmlhr")) => %(/javascripts/xmlhr.js), + } + JavascriptIncludeToTag = { %(javascript_include_tag("xmlhr")) => %(), %(javascript_include_tag("common.javascript", "/elsewhere/cools")) => %(\n), } + StylePathToTag = { + %(stylesheet_path("style")) => %(/stylesheets/style.css), + } + StyleLinkToTag = { %(stylesheet_link_tag("style")) => %(), %(stylesheet_link_tag("random.styles", "/css/stylish")) => %(\n) } + ImagePathToTag = { + %(image_path("xml")) => %(/images/xml.png), + } + ImageLinkToTag = { %(image_tag("xml")) => %(Xml), %(image_tag("rss", :alt => "rss syndication")) => %(rss syndication), @@ -49,14 +61,26 @@ class AssetTagHelperTest < Test::Unit::TestCase AutoDiscoveryToTag.each { |method, tag| assert_equal(tag, eval(method)) } end + def test_javascript_path + JavascriptPathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end + def test_javascript_include JavascriptIncludeToTag.each { |method, tag| assert_equal(tag, eval(method)) } end + def test_style_path + StylePathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end + def test_style_link StyleLinkToTag.each { |method, tag| assert_equal(tag, eval(method)) } end + def test_image_path + ImagePathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end + def test_image_tag ImageLinkToTag.each { |method, tag| assert_equal(tag, eval(method)) } end @@ -91,15 +115,27 @@ class AssetTagHelperNonVhostTest < Test::Unit::TestCase %(auto_discovery_link_tag) => %(), } + JavascriptPathToTag = { + %(javascript_path("xmlhr")) => %(/calloboration/hieraki/javascripts/xmlhr.js), + } + JavascriptIncludeToTag = { %(javascript_include_tag("xmlhr")) => %(), %(javascript_include_tag("common.javascript", "/elsewhere/cools")) => %(\n), } + StylePathToTag = { + %(stylesheet_path("style")) => %(/calloboration/hieraki/stylesheets/style.css), + } + StyleLinkToTag = { %(stylesheet_link_tag("style")) => %(), %(stylesheet_link_tag("random.styles", "/css/stylish")) => %(\n) } + + ImagePathToTag = { + %(image_path("xml")) => %(/calloboration/hieraki/images/xml.png), + } ImageLinkToTag = { %(image_tag("xml")) => %(Xml), @@ -111,10 +147,18 @@ class AssetTagHelperNonVhostTest < Test::Unit::TestCase AutoDiscoveryToTag.each { |method, tag| assert_equal(tag, eval(method)) } end + def test_javascript_path + JavascriptPathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end + def test_javascript_include JavascriptIncludeToTag.each { |method, tag| assert_equal(tag, eval(method)) } end + def test_style_path + StylePathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end + def test_style_link StyleLinkToTag.each { |method, tag| assert_equal(tag, eval(method)) } end @@ -122,6 +166,10 @@ class AssetTagHelperNonVhostTest < Test::Unit::TestCase def test_image_tag assert_equal %(Gold), image_tag("gold", :size => "45x70") end + + def test_image_path + ImagePathToTag.each { |method, tag| assert_equal(tag, eval(method)) } + end def test_image_tag ImageLinkToTag.each { |method, tag| assert_equal(tag, eval(method)) }