aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
diff options
context:
space:
mode:
authorErik St. Martin <alakriti@gmail.com>2010-01-23 16:10:44 -0500
committerStefan Penner <stefan.penner@gmail.com>2010-01-27 12:44:32 -0600
commit7bf5aef907a90a9896c05ae3c6ed371a70aa6b8f (patch)
tree2a8399b87992e5ca860ff0de8317365e006542e7 /actionpack
parent255066b6c7a5877de20e9aef9b58f886c7e66e13 (diff)
downloadrails-7bf5aef907a90a9896c05ae3c6ed371a70aa6b8f.tar.gz
rails-7bf5aef907a90a9896c05ae3c6ed371a70aa6b8f.tar.bz2
rails-7bf5aef907a90a9896c05ae3c6ed371a70aa6b8f.zip
fixed current tests that were failing do to module location change, this is temporary to fix state of repo till all the new tests are complete
Diffstat (limited to 'actionpack')
-rw-r--r--actionpack/lib/action_view/helpers/ajax_helper.rb2
-rw-r--r--actionpack/test/template/ajax_helper_test.rb11
2 files changed, 10 insertions, 3 deletions
diff --git a/actionpack/lib/action_view/helpers/ajax_helper.rb b/actionpack/lib/action_view/helpers/ajax_helper.rb
index 54915e3c89..6f7f3750bc 100644
--- a/actionpack/lib/action_view/helpers/ajax_helper.rb
+++ b/actionpack/lib/action_view/helpers/ajax_helper.rb
@@ -574,6 +574,8 @@ module ActionView
if !options && url.is_a?(Hash) && url.key?(:url)
url, options = url.delete(:url), url
end
+ options = {} if options.nil?
+
set_callbacks(options, options[:html] ||= {})
super
diff --git a/actionpack/test/template/ajax_helper_test.rb b/actionpack/test/template/ajax_helper_test.rb
index 80d1f42327..4ca130d010 100644
--- a/actionpack/test/template/ajax_helper_test.rb
+++ b/actionpack/test/template/ajax_helper_test.rb
@@ -1,7 +1,6 @@
require "abstract_unit"
class AjaxTestCase < ActiveSupport::TestCase
- include ActionView::Helpers::AjaxHelper
include ActionView::Helpers::TagHelper
def url_for(url)
@@ -34,6 +33,8 @@ class AjaxTestCase < ActiveSupport::TestCase
end
class LinkToRemoteTest < AjaxTestCase
+ include ActionView::Helpers::AjaxHelperCompat
+
def link(options = {})
link_to_remote("Delete this post", "/blog/destroy/3", options)
end
@@ -79,7 +80,7 @@ class LinkToRemoteTest < AjaxTestCase
end
class LegacyLinkToRemoteTest < AjaxTestCase
- include ActionView::Helpers::AjaxHelper::Rails2Compatibility
+ include ActionView::Helpers::AjaxHelperCompat
def link(options)
link_to_remote("Delete this post", "/blog/destroy/3", options)
@@ -98,6 +99,8 @@ class LinkToRemoteTest < AjaxTestCase
end
class ButtonToRemoteTest < AjaxTestCase
+ include ActionView::Helpers::AjaxHelperCompat
+
def button(options, html = {})
button_to_remote("Remote outpost", options, html)
end
@@ -110,7 +113,7 @@ class ButtonToRemoteTest < AjaxTestCase
end
class LegacyButtonToRemoteTest < ButtonToRemoteTest
- include ActionView::Helpers::AjaxHelper::Rails2Compatibility
+ include ActionView::Helpers::AjaxHelperCompat
assert_callbacks_work do |callback|
button(callback => "undoRequestCompleted(request)")
@@ -119,6 +122,8 @@ class ButtonToRemoteTest < AjaxTestCase
end
class ObserveFieldTest < AjaxTestCase
+ include ActionView::Helpers::AjaxHelperCompat
+
def url_for(hash)
"/blog/update"
end