diff --git a/app/helpers/journals_helper.rb b/app/helpers/journals_helper.rb index 4f21b30d..cf877243 100644 --- a/app/helpers/journals_helper.rb +++ b/app/helpers/journals_helper.rb @@ -16,13 +16,13 @@ # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. module JournalsHelper - def render_notes(journal, options={}) + def render_notes(issue, journal, options={}) content = '' - editable = journal.editable_by?(User.current) + editable = User.current.logged? && (User.current.allowed_to?(:edit_issue_notes, issue.project) || (journal.user == User.current && User.current.allowed_to?(:edit_own_issue_notes, issue.project))) links = [] if !journal.notes.blank? links << link_to_remote(image_tag('comment.png'), - { :url => {:controller => 'issues', :action => 'reply', :id => journal.journalized, :journal_id => journal} }, + { :url => {:controller => 'issues', :action => 'reply', :id => issue, :journal_id => journal} }, :title => l(:button_quote)) if options[:reply_links] links << link_to_in_place_notes_editor(image_tag('edit.png'), "journal-#{journal.id}-notes", { :controller => 'journals', :action => 'edit', :id => journal }, diff --git a/app/views/issues/_history.rhtml b/app/views/issues/_history.rhtml index ffbb2939..b57c543e 100644 --- a/app/views/issues/_history.rhtml +++ b/app/views/issues/_history.rhtml @@ -11,7 +11,7 @@
  • <%= show_detail(detail) %>
  • <% end %> - <%= render_notes(journal, :reply_links => reply_links) unless journal.notes.blank? %> + <%= render_notes(issue, journal, :reply_links => reply_links) unless journal.notes.blank? %> <%= call_hook(:view_issues_history_journal_bottom, { :journal => journal }) %> <% end %> diff --git a/app/views/issues/show.rhtml b/app/views/issues/show.rhtml index 3be765d0..5f6ba1ba 100644 --- a/app/views/issues/show.rhtml +++ b/app/views/issues/show.rhtml @@ -89,7 +89,7 @@ <% if @journals.any? %>

    <%=l(:label_history)%>

    -<%= render :partial => 'history', :locals => { :journals => @journals } %> +<%= render :partial => 'history', :locals => { :issue => @issue, :journals => @journals } %>
    <% end %>