diff --git a/app/controllers/dmsf_files_controller.rb b/app/controllers/dmsf_files_controller.rb index ee2dc557..a589590c 100644 --- a/app/controllers/dmsf_files_controller.rb +++ b/app/controllers/dmsf_files_controller.rb @@ -52,8 +52,13 @@ class DmsfFilesController < ApplicationController access.action = DmsfFileRevisionAccess::DownloadAction access.save! member = Member.where(:user_id => User.current.id, :project_id => @file.project.id).first + if member && !member.title_format.nil? && !member.title_format.empty? + title_format = member.title_format + else + title_format = Setting.plugin_redmine_dmsf['dmsf_global_title_format'] + end send_file(@revision.disk_file, - :filename => filename_for_content_disposition(@revision.formatted_name(member ? member.title_format : nil)), + :filename => filename_for_content_disposition(@revision.formatted_name(title_format)), :type => @revision.detect_content_type, :disposition => 'inline') rescue DmsfAccessError => e @@ -84,8 +89,13 @@ class DmsfFilesController < ApplicationController access.action = DmsfFileRevisionAccess::DownloadAction access.save! member = Member.where(:user_id => User.current.id, :project_id => @file.project.id).first + if member && !member.title_format.nil? && !member.title_format.empty? + title_format = member.title_format + else + title_format = Setting.plugin_redmine_dmsf['dmsf_global_title_format'] + end send_file(@revision.disk_file, - :filename => filename_for_content_disposition(@revision.formatted_name(member ? member.title_format : nil)), + :filename => filename_for_content_disposition(@revision.formatted_name(title_format)), :type => @revision.detect_content_type, :disposition => 'attachment') rescue DmsfAccessError => e diff --git a/app/models/dmsf_file.rb b/app/models/dmsf_file.rb index 76b437a4..824bc68b 100644 --- a/app/models/dmsf_file.rb +++ b/app/models/dmsf_file.rb @@ -403,10 +403,17 @@ class DmsfFile < ActiveRecord::Base end def display_name - if self.name.length > 50 - return "#{self.name[0, 25]}...#{self.name[-25, 25]}" + member = Member.where(:user_id => User.current.id, :project_id => self.project.id).first + if member && !member.title_format.nil? && !member.title_format.empty? + title_format = member.title_format + else + title_format = Setting.plugin_redmine_dmsf['dmsf_global_title_format'] end - self.name + fname = formatted_name(title_format) + if fname.length > 50 + return "#{fname[0, 25]}...#{fname[-25, 25]}" + end + fname end def image? diff --git a/app/models/dmsf_file_revision.rb b/app/models/dmsf_file_revision.rb index 6df8987e..5a8a165e 100644 --- a/app/models/dmsf_file_revision.rb +++ b/app/models/dmsf_file_revision.rb @@ -258,13 +258,14 @@ class DmsfFileRevision < ActiveRecord::Base else filename = self.name end - format.sub!('%t', self.title) - format.sub!('%f', filename) - format.sub!('%d', self.updated_at.strftime('%Y%m%d%H%M%S')) - format.sub!('%v', self.version) - format.sub!('%i', self.dmsf_file.id.to_s) - format.sub!('%r', self.id.to_s) - format + ext + format2 = format.dup + format2.sub!('%t', self.title) + format2.sub!('%f', filename) + format2.sub!('%d', self.updated_at.strftime('%Y%m%d%H%M%S')) + format2.sub!('%v', self.version) + format2.sub!('%i', self.dmsf_file.id.to_s) + format2.sub!('%r', self.id.to_s) + format2 + ext end def self.create_digest(path) diff --git a/app/models/dmsf_link.rb b/app/models/dmsf_link.rb index 1e67e143..89807d04 100644 --- a/app/models/dmsf_link.rb +++ b/app/models/dmsf_link.rb @@ -98,7 +98,7 @@ class DmsfLink < ActiveRecord::Base def path if self.target_type == DmsfFile.model_name.to_s - path = self.target_file.dmsf_path.map { |element| element.is_a?(DmsfFile) ? element.name : element.title }.join('/') if self.target_file + path = self.target_file.dmsf_path.map { |element| element.is_a?(DmsfFile) ? element.display_name : element.title }.join('/') if self.target_file else path = self.target_folder ? self.target_folder.dmsf_path_str : '' end