#48 Caching documents in the issue form

app/views/dmsf_upload/_form.html.erb

@@ -21,8 +21,8 @@
 %>
 
 <span id="dmsf_attachments_fields">
-<% if defined?(container) && container && container.saved_attachments %>
-  <% container.saved_attachments.each_with_index do |attachment, i| %>
+<% if defined?(container) && container && container.saved_dmsf_attachments %>
+  <% container.saved_dmsf_attachments.each_with_index do |attachment, i| %>
     <span id="dmsf_attachments_p<%= i %>">
       <%= text_field_tag("dmsf_attachments[p#{i}][filename]", attachment.filename, :class => 'filename') +
               text_field_tag("dmsf_attachments[p#{i}][description]", attachment.description, :maxlength => 255, :placeholder => l(:label_optional_description), :class => 'description') +

lib/redmine_dmsf/hooks/controllers/issues_controller_hooks.rb

@@ -24,6 +24,10 @@ module RedmineDmsf
     
     class ControllerIssuesHook < RedmineDmsf::Hooks::Listener
 
+      def controller_issues_new_before_save(context={})
+        controller_issues_before_save(context)
+      end
+
       def controller_issues_new_after_save(context={})
         controller_issues_after_save(context)
         # Copy documents from the source issue
@@ -40,12 +44,24 @@ module RedmineDmsf
         end
       end
 
+      def controller_issues_edit_before_save(context={})
+        controller_issues_before_save(context)
+      end
+
       def controller_issues_edit_after_save(context={})
         controller_issues_after_save(context)
       end
 
       private
 
+      def controller_issues_before_save(context)
+        if context.is_a?(Hash)
+          issue = context[:issue]
+          params = context[:params]
+          issue.save_dmsf_attachments(params[:dmsf_attachments])
+        end
+      end
+
       def controller_issues_after_save(context)
         # Create attached documents
         if context.is_a?(Hash)

lib/redmine_dmsf/hooks/views/issue_view_hooks.rb

@@ -34,7 +34,7 @@ module RedmineDmsf
             html << '<p>'
             html << "<label>#{l(:label_document_plural)}</label>"
             html << context[:controller].send(:render_to_string,
-              {:partial => 'dmsf_upload/form', :locals => { :multiple => true }})
+              {:partial => 'dmsf_upload/form', :locals => { :container => issue, :multiple => true }})
             html << '</p>'
             html << '</div>'
             html.html_safe

lib/redmine_dmsf/patches/issue_patch.rb

@@ -32,6 +32,19 @@ module RedmineDmsf
 
       module InstanceMethods
 
+        def save_dmsf_attachments(dmsf_attachments)
+          @saved_dmsf_attachments = []
+          dmsf_attachments = dmsf_attachments.map(&:last)
+          dmsf_attachments.each do |dmsf_attachment|
+            a = Attachment.find_by_token(dmsf_attachment[:token])
+            @saved_dmsf_attachments << a if a
+          end
+        end
+
+        def saved_dmsf_attachments
+          @saved_dmsf_attachments || []
+        end
+
         def system_folder(create = false)
           parent = DmsfFolder.system.where(:project_id => self.project_id, :title => '.Issues').first
           if create && !parent