Link to diary comments using anchors, rather than showing them alone

Neither way is perfect, but this is less surprising than seeing a diary
entry with only one comment visible.

diff --git a/app/controllers/diary_entry_controller.rb b/app/controllers/diary_entry_controller.rb
index 88febbe2fe96c520a75dcf04696121be5a627fba..9e0fd4991bd4f62b6ec60e295efc44cf69aa6b49 100644 (file)
--- a/app/controllers/diary_entry_controller.rb
+++ b/app/controllers/diary_entry_controller.rb
@@ -186,9 +186,6 @@ class DiaryEntryController < ApplicationController
     @entry = @this_user.diary_entries.visible.where(:id => params[:id]).first
     if @entry
       @title = t "diary_entry.view.title", :user => params[:display_name], :title => @entry.title
-      if params[:comment_id]
-        @reported_comment = DiaryComment.where(:id => params[:comment_id])
-      end
     else
       @title = t "diary_entry.no_such_entry.title", :id => params[:id]
       render :action => "no_such_entry", :status => :not_found

diff --git a/app/helpers/issues_helper.rb b/app/helpers/issues_helper.rb
index 27a51c0fbc4ce66cfebc9d2247e2bf79bafb16cf..8452cbd608331921246f1c2ce0e3aabe907bf90a 100644 (file)
--- a/app/helpers/issues_helper.rb
+++ b/app/helpers/issues_helper.rb
@@ -7,7 +7,7 @@ module IssuesHelper
     when "User"
       link_to reportable.display_name.to_s, :controller => reportable.class.name.underscore, :action => :view, :display_name => reportable.display_name
     when "DiaryComment"
-      link_to "#{reportable.diary_entry.title}, Comment id ##{reportable.id}", :controller => reportable.diary_entry.class.name.underscore, :action => :view, :display_name => reportable.diary_entry.user.display_name, :id => reportable.diary_entry.id, :comment_id => reportable.id
+      link_to "#{reportable.diary_entry.title}, Comment id ##{reportable.id}", :controller => reportable.diary_entry.class.name.underscore, :action => :view, :display_name => reportable.diary_entry.user.display_name, :id => reportable.diary_entry.id, :anchor => "comment#{reportable.id}"
     when "Changeset"
       link_to "Changeset ##{reportable.id}", :controller => :browse, :action => :changeset, :id => reportable.id
     when "Note"
@@ -39,7 +39,7 @@ module IssuesHelper
     when "User"
       link_to "Show Instance", :controller => reportable.class.name.underscore, :action => :view, :display_name => reportable.display_name
     when "DiaryComment"
-      link_to "Show Instance", :controller => reportable.diary_entry.class.name.underscore, :action => :view, :display_name => reportable.diary_entry.user.display_name, :id => reportable.diary_entry.id, :comment_id => reportable.id
+      link_to "Show Instance", :controller => reportable.diary_entry.class.name.underscore, :action => :view, :display_name => reportable.diary_entry.user.display_name, :id => reportable.diary_entry.id, :anchor => "comment#{reportable.id}"
     when "Changeset"
       link_to "Show Instance", :controller => :browse, :action => :changeset, :id => reportable.id
     when "Note"

diff --git a/app/views/diary_entry/view.html.erb b/app/views/diary_entry/view.html.erb
index 5079074e844fbc745a60304f15185af581fbe0bc..3c2264d3e58fb849a8793ef1e0aee9da806cd315 100644 (file)
--- a/app/views/diary_entry/view.html.erb
+++ b/app/views/diary_entry/view.html.erb
@@ -10,11 +10,7 @@
 
 <a id="comments"></a>
 <div class='comments'>
-  <% if @reported_comment %>
-    <%= render :partial => 'diary_comment', :collection => @reported_comment %>
-  <% else %>
-    <%= render :partial => 'diary_comment', :collection => @entry.visible_comments %>
-  <% end %>
+<%= render :partial => 'diary_comment', :collection => @entry.visible_comments %>
 </div>
 <%= if_logged_in(:div) do %>
   <h3 id="newcomment"><%= t 'diary_entry.view.leave_a_comment' %></h3>