Remove link_to_function / button_to_function from ActionView::Helpers::JavascriptHelper were deprecated.
This commit is contained in:
parent
8b80d72327
commit
825f4df23c
@ -70,48 +70,6 @@ def javascript_tag(content_or_options_with_block = nil, html_options = {}, &bloc
|
||||
def javascript_cdata_section(content) #:nodoc:
|
||||
"\n//#{cdata_section("\n#{content}\n//")}\n".html_safe
|
||||
end
|
||||
|
||||
# Returns a button whose +onclick+ handler triggers the passed JavaScript.
|
||||
#
|
||||
# The helper receives a name, JavaScript code, and an optional hash of HTML options. The
|
||||
# name is used as button label and the JavaScript code goes into its +onclick+ attribute.
|
||||
# If +html_options+ has an <tt>:onclick</tt>, that one is put before +function+.
|
||||
#
|
||||
# button_to_function "Greeting", "alert('Hello world!')", class: "ok"
|
||||
# # => <input class="ok" onclick="alert('Hello world!');" type="button" value="Greeting" />
|
||||
#
|
||||
def button_to_function(name, function=nil, html_options={})
|
||||
message = "button_to_function is deprecated and will be removed from Rails 4.1. We recommend using Unobtrusive JavaScript instead. " +
|
||||
"See http://guides.rubyonrails.org/working_with_javascript_in_rails.html#unobtrusive-javascript"
|
||||
ActiveSupport::Deprecation.warn message
|
||||
|
||||
onclick = "#{"#{html_options[:onclick]}; " if html_options[:onclick]}#{function};"
|
||||
|
||||
tag(:input, html_options.merge(:type => 'button', :value => name, :onclick => onclick))
|
||||
end
|
||||
|
||||
# Returns a link whose +onclick+ handler triggers the passed JavaScript.
|
||||
#
|
||||
# The helper receives a name, JavaScript code, and an optional hash of HTML options. The
|
||||
# name is used as the link text and the JavaScript code goes into the +onclick+ attribute.
|
||||
# If +html_options+ has an <tt>:onclick</tt>, that one is put before +function+. Once all
|
||||
# the JavaScript is set, the helper appends "; return false;".
|
||||
#
|
||||
# The +href+ attribute of the tag is set to "#" unless +html_options+ has one.
|
||||
#
|
||||
# link_to_function "Greeting", "alert('Hello world!')", class: "nav_link"
|
||||
# # => <a class="nav_link" href="#" onclick="alert('Hello world!'); return false;">Greeting</a>
|
||||
#
|
||||
def link_to_function(name, function, html_options={})
|
||||
message = "link_to_function is deprecated and will be removed from Rails 4.1. We recommend using Unobtrusive JavaScript instead. " +
|
||||
"See http://guides.rubyonrails.org/working_with_javascript_in_rails.html#unobtrusive-javascript"
|
||||
ActiveSupport::Deprecation.warn message
|
||||
|
||||
onclick = "#{"#{html_options[:onclick]}; " if html_options[:onclick]}#{function}; return false;"
|
||||
href = html_options[:href] || '#'
|
||||
|
||||
content_tag(:a, name, html_options.merge(:href => href, :onclick => onclick))
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -42,48 +42,6 @@ def test_escape_javascript_with_safebuffer
|
||||
assert_instance_of ActiveSupport::SafeBuffer, escape_javascript(ActiveSupport::SafeBuffer.new(given))
|
||||
end
|
||||
|
||||
def test_button_to_function
|
||||
assert_deprecated do
|
||||
assert_dom_equal %(<input type="button" onclick="alert('Hello world!');" value="Greeting" />),
|
||||
button_to_function("Greeting", "alert('Hello world!')")
|
||||
end
|
||||
end
|
||||
|
||||
def test_button_to_function_with_onclick
|
||||
assert_deprecated do
|
||||
assert_dom_equal "<input onclick=\"alert('Goodbye World :('); alert('Hello world!');\" type=\"button\" value=\"Greeting\" />",
|
||||
button_to_function("Greeting", "alert('Hello world!')", :onclick => "alert('Goodbye World :(')")
|
||||
end
|
||||
end
|
||||
|
||||
def test_button_to_function_without_function
|
||||
assert_deprecated do
|
||||
assert_dom_equal "<input onclick=\";\" type=\"button\" value=\"Greeting\" />",
|
||||
button_to_function("Greeting")
|
||||
end
|
||||
end
|
||||
|
||||
def test_link_to_function
|
||||
assert_deprecated do
|
||||
assert_dom_equal %(<a href="#" onclick="alert('Hello world!'); return false;">Greeting</a>),
|
||||
link_to_function("Greeting", "alert('Hello world!')")
|
||||
end
|
||||
end
|
||||
|
||||
def test_link_to_function_with_existing_onclick
|
||||
assert_deprecated do
|
||||
assert_dom_equal %(<a href="#" onclick="confirm('Sanity!'); alert('Hello world!'); return false;">Greeting</a>),
|
||||
link_to_function("Greeting", "alert('Hello world!')", :onclick => "confirm('Sanity!')")
|
||||
end
|
||||
end
|
||||
|
||||
def test_function_with_href
|
||||
assert_deprecated do
|
||||
assert_dom_equal %(<a href="http://example.com/" onclick="alert('Hello world!'); return false;">Greeting</a>),
|
||||
link_to_function("Greeting", "alert('Hello world!')", :href => 'http://example.com/')
|
||||
end
|
||||
end
|
||||
|
||||
def test_javascript_tag
|
||||
self.output_buffer = 'foo'
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user