Wrong tool-tip for dmsf macro #545

app/models/dmsf_file.rb

@@ -452,20 +452,4 @@ class DmsfFile < ActiveRecord::Base
     false
   end
 
-  def tooltip
-    text = ''
-    if self.last_revision
-      if self.last_revision.description.present?
-        text = self.last_revision.description
-      end
-      if self.last_revision.comment.present?
-        if text.present?
-          text += '&#xA;'
-        end
-        text += self.last_revision.comment
-      end
-    end
-    text.html_safe
-  end
-
 end

app/models/dmsf_file_revision.rb

@@ -284,4 +284,14 @@ class DmsfFileRevision < ActiveRecord::Base
     end
   end
 
+  def tooltip
+    text = ''
+    text = self.description if self.description.present?
+    if self.comment.present?
+      text += '&#xA;' if text.present?
+      text += self.comment
+    end
+    text.html_safe
+  end
+
 end

app/views/dmsf/_file.html.erb

@@ -33,7 +33,7 @@
     file_view_url,
     :target => '_blank',
     :class => "icon icon-file #{DmsfHelper.filetype_css(file.name)}",
-    :title => file.tooltip,
+    :title => file.last_revision.try(:tooltip),
     'data-downloadurl' => "#{file.last_revision.detect_content_type}:#{h(file.name)}:#{file_view_url}") %>
   <div class="dmsf_filename" title="<%= l(:title_filename_for_download)%>"><%= h(link ? link.path : file.display_name) %></div>
   <%= '</span>'.html_safe if @tree_view %>

app/views/dmsf/_file_trash.html.erb

@@ -24,7 +24,7 @@
   :title => l(:title_check_for_restore_or_delete), :id => "file_#{id}") %></td>
 <td class="dmsf_title">  
   <%= content_tag(:span, h(title), 
-    :title => h(file.tooltip),
+    :title => h(file.last_revision.try(:tooltip)),
     :class => "icon icon-file #{DmsfHelper.filetype_css(file.name)}") %>
   <div class="dmsf_filename" title="<%= l(:title_filename_for_download)%>"><%= h(link ? link.path : file.display_name) %></div>
 </td>

lib/redmine_dmsf/macros.rb

@@ -40,7 +40,7 @@ Redmine::WikiFormatting::Macros.register do
       return link_to(h(args[1] ? args[1] : file.title),
         file_view_url,
         :target => '_blank',
-        :title => h(file.tooltip),
+        :title => h(revision.tooltip),
         'data-downloadurl' => "#{file.last_revision.detect_content_type}:#{h(file.name)}:#{file_view_url}")
     else
       raise l(:notice_not_authorized)
@@ -161,7 +161,7 @@ Redmine::WikiFormatting::Macros.register do
       end
       link_to(img,
         file_view_url, :target => '_blank',
-        :title => h(file.tooltip),
+        :title => h(file.last_revision.try(:tooltip)),
         'data-downloadurl' => "#{file.last_revision.detect_content_type}:#{h(file.name)}:#{file_view_url}")
     else
       raise "Document ID #{file_id} not found"