[#416] using a simpler approach to replace node with partial

4635a8c2 · Gregor Schmidt · e3f54d47 · 4635a8c2
Commit 4635a8c2 authored May 17, 2011 by Gregor Schmidt
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 7 deletions

watchers_controller.rb app/controllers/watchers_controller.rb +1 -7

No files found.
--- a/app/controllers/watchers_controller.rb
+++ b/app/controllers/watchers_controller.rb
@@ -86,10 +86,6 @@ private
      replace_selectors = ['#watcher']
    end

-    watcher_partial = lambda do
-      render_to_string(:partial => 'watchers/watchers', :locals => {:watched => @watched})
-    end
-
    respond_to do |format|
      format.html { redirect_to :back }
      format.js do
@@ -99,9 +95,7 @@ private

            case selector
            when '#watchers'
-              page.select('#watchers').each do |node|
-                node.update watcher_partial.call
-              end
+              page.replace_html 'watchers', :partial => 'watchers/watchers', :locals => {:watched => @watched}
            else
              page.select(selector).each do |node|
                options = {:replace => replace_selectors}