aboutsummaryrefslogtreecommitdiffstats
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
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
-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