jro_apps/redmine Commit - r5200:e3dae9ddbf46

Cleaner way to handle the replacement of watch links (#8071)....

Jean-Philippe Lang -

r5200:e3dae9ddbf46

parent child

Context file:

r5200:e3dae9ddbf46

Collapse all files

app/controllers/watchers_controller.rb +5 -13

-            # redMine - project management software
+            # Redmine - project management software
-            # Copyright (C) 2006-2007  Jean-Philippe Lang
+            # Copyright (C) 2006-2011  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
               def set_watcher(user, watching)
                 @watched.set_watcher(user, watching)
-                if params[:replace].present?
-                  if params[:replace].is_a? Array
-                    replace_ids = params[:replace]
-                  else
-                    replace_ids = [params[:replace]]
-                  end
-                else
-                  replace_ids = ['watcher']
-                end
                 respond_to do |format|
                   format.html { redirect_to :back }
                   format.js do
                     render(:update) do |page|
-                      replace_ids.each do |replace_id|
+                      c = watcher_css(@watched)
-                        page.replace_html replace_id, watcher_link(@watched, user, :replace => replace_ids)
+                      page.select(".#{c}").each do |item|
+                        page.replace_html item, watcher_link(@watched, user)
                       end
                     end
                   end

app/helpers/watchers_helper.rb +11 -16

-            # redMine - project management software
+            # Redmine - project management software
-            # Copyright (C) 2006-2007  Jean-Philippe Lang
+            # Copyright (C) 2006-2011  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             module WatchersHelper
-              # Valid options
+              def watcher_tag(object, user, options={})
-              # * :id - the element id
+                content_tag("span", watcher_link(object, user), :class => watcher_css(object))
-              # * :replace - a string or array of element ids that will be
-              #   replaced
-              def watcher_tag(object, user, options={:replace => 'watcher'})
-                id = options[:id]
-                id ||= options[:replace] if options[:replace].is_a? String
-                content_tag("span", watcher_link(object, user, options), :id => id)
               end
-              # Valid options
+              def watcher_link(object, user)
-              # * :replace - a string or array of element ids that will be
-              #   replaced
-              def watcher_link(object, user, options={:replace => 'watcher'})
                 return '' unless user && user.logged? && object.respond_to?('watched_by?')
                 watched = object.watched_by?(user)
                 url = {:controller => 'watchers',
                        :action => (watched ? 'unwatch' : 'watch'),
                        :object_type => object.class.to_s.underscore,
-                       :object_id => object.id,
+                       :object_id => object.id}
-                       :replace => options[:replace]}
                 link_to_remote((watched ? l(:button_unwatch) : l(:button_watch)),
                                {:url => url},
                                :href => url_for(url),
               end
+              # Returns the css class used to identify watch links for a given +object+
+              def watcher_css(object)
+                "#{object.class.to_s.underscore}-#{object.id}-watcher"
+              end
               # Returns a comma separated list of users watching the given object
               def watchers_list(object)
                 remove_allowed = User.current.allowed_to?("delete_#{object.class.name.underscore}_watchers".to_sym, object.project)

app/views/issues/_action_menu.rhtml +1 -2

             <div class="contextual">
             <%= link_to_if_authorized(l(:button_update), {:controller => 'issues', :action => 'edit', :id => @issue }, :onclick => 'showAndScrollTo("update", "notes"); return false;', :class => 'icon icon-edit', :accesskey => accesskey(:edit)) %>
             <%= link_to_if_authorized l(:button_log_time), {:controller => 'timelog', :action => 'new', :issue_id => @issue}, :class => 'icon icon-time-add' %>
-            <% replace_watcher ||= 'watcher' %>
+            <%= watcher_tag(@issue, User.current) %>
-            <%= watcher_tag(@issue, User.current, {:id => replace_watcher, :replace => ['watcher','watcher2']}) %>
             <%= link_to_if_authorized l(:button_duplicate), {:controller => 'issues', :action => 'new', :project_id => @project, :copy_from => @issue }, :class => 'icon icon-duplicate' %>
             <%= link_to_if_authorized l(:button_copy), {:controller => 'issue_moves', :action => 'new', :id => @issue, :copy_options => {:copy => 't'}}, :class => 'icon icon-copy' %>
             <%= link_to_if_authorized l(:button_move), {:controller => 'issue_moves', :action => 'new', :id => @issue}, :class => 'icon icon-move' %>

app/views/issues/show.rhtml +1 -1

             <div style="clear: both;"></div>
-            <%= render :partial => 'action_menu', :locals => {:replace_watcher => 'watcher2' } %>
+            <%= render :partial => 'action_menu' %>
             <div style="clear: both;"></div>
             <% if authorize_for('issues', 'edit') %>

test/functional/watchers_controller_test.rb +3 -24

             # Redmine - project management software
-            # Copyright (C) 2006-2008  Jean-Philippe Lang
+            # Copyright (C) 2006-2011  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
                 assert_difference('Watcher.count') do
                   xhr :post, :watch, :object_type => 'issue', :object_id => '1'
                   assert_response :success
-                  assert_select_rjs :replace_html, 'watcher'
+                  assert @response.body.include?('$$(".issue-1-watcher")')
                 end
                 assert Issue.find(1).watched_by?(User.find(3))
               end
                 end
               end
-              def test_watch_with_multiple_replacements
-                @request.session[:user_id] = 3
-                assert_difference('Watcher.count') do
-                  xhr :post, :watch, :object_type => 'issue', :object_id => '1', :replace => ['watch_item_1','watch_item_2']
-                  assert_response :success
-                  assert_select_rjs :replace_html, 'watch_item_1'
-                  assert_select_rjs :replace_html, 'watch_item_2'
-                end
-              end
               def test_unwatch
                 @request.session[:user_id] = 3
                 assert_difference('Watcher.count', -1) do
                   xhr :post, :unwatch, :object_type => 'issue', :object_id => '2'
                   assert_response :success
-                  assert_select_rjs :replace_html, 'watcher'
+                  assert @response.body.include?('$$(".issue-2-watcher")')
-                end
-                assert !Issue.find(1).watched_by?(User.find(3))
-              end
-              def test_unwatch_with_multiple_replacements
-                @request.session[:user_id] = 3
-                assert_difference('Watcher.count', -1) do
-                  xhr :post, :unwatch, :object_type => 'issue', :object_id => '2', :replace => ['watch_item_1', 'watch_item_2']
-                  assert_response :success
-                  assert_select_rjs :replace_html, 'watch_item_1'
-                  assert_select_rjs :replace_html, 'watch_item_2'
                 end
                 assert !Issue.find(1).watched_by?(User.find(3))
               end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages