mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
The image_tag helper is not converting the options hash to symbols (closes #3615) [agkr@pobox.com]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@3660 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
This commit is contained in:
parent
e57bd727df
commit
06dd7b8efa
2 changed files with 3 additions and 1 deletions
|
@ -131,7 +131,7 @@ module ActionView
|
|||
# * 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_tag(source, options = {})
|
||||
options.symbolize_keys
|
||||
options.symbolize_keys!
|
||||
|
||||
options[:src] = image_path(source)
|
||||
options[:alt] ||= File.basename(options[:src], '.*').split('.').first.capitalize
|
||||
|
|
|
@ -74,6 +74,7 @@ class AssetTagHelperTest < Test::Unit::TestCase
|
|||
%(image_tag("xml")) => %(<img alt="Xml" src="/images/xml.png" />),
|
||||
%(image_tag("rss", :alt => "rss syndication")) => %(<img alt="rss syndication" src="/images/rss.png" />),
|
||||
%(image_tag("gold", :size => "45x70")) => %(<img alt="Gold" height="70" src="/images/gold.png" width="45" />),
|
||||
%(image_tag("symbolize", "size" => "45x70")) => %(<img alt="Symbolize" height="70" src="/images/symbolize.png" width="45" />)
|
||||
}
|
||||
|
||||
def test_auto_discovery
|
||||
|
@ -174,6 +175,7 @@ class AssetTagHelperNonVhostTest < Test::Unit::TestCase
|
|||
%(image_tag("rss", :alt => "rss syndication")) => %(<img alt="rss syndication" src="/calloboration/hieraki/images/rss.png" />),
|
||||
%(image_tag("gold", :size => "45x70")) => %(<img alt="Gold" height="70" src="/calloboration/hieraki/images/gold.png" width="45" />),
|
||||
%(image_tag("http://www.example.com/images/icon.gif")) => %(<img alt="Icon" src="http://www.example.com/images/icon.gif" />),
|
||||
%(image_tag("symbolize", "size" => "45x70")) => %(<img alt="Symbolize" height="70" src="/calloboration/hieraki/images/symbolize.png" width="45" />)
|
||||
}
|
||||
|
||||
def test_auto_discovery
|
||||
|
|
Loading…
Reference in a new issue