diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-07-16 16:42:23 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-07-16 16:42:23 +0000 |
commit | 5c2de4dfc95293a7965ff8cf980a310922fc4c05 (patch) | |
tree | 216137d4546e967ca7f4ce7fee328ac62ca93830 | |
parent | 570e1b1d626543738cec3dc3ed07da73a47a98dd (diff) | |
download | redmine-5c2de4dfc95293a7965ff8cf980a310922fc4c05.tar.gz redmine-5c2de4dfc95293a7965ff8cf980a310922fc4c05.zip |
Adds Attachment#title.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10012 e93f8b46-1217-0410-a6f0-8f06a7374b81
-rw-r--r-- | app/models/attachment.rb | 8 | ||||
-rw-r--r-- | test/unit/attachment_test.rb | 8 |
2 files changed, 16 insertions, 0 deletions
diff --git a/app/models/attachment.rb b/app/models/attachment.rb index 0dda33023..9aeee07d0 100644 --- a/app/models/attachment.rb +++ b/app/models/attachment.rb @@ -137,6 +137,14 @@ class Attachment < ActiveRecord::Base File.join(self.class.storage_path, disk_filename.to_s) end + def title + title = filename.to_s + if description.present? + title << " (#{description})" + end + title + end + def increment_download increment!(:downloads) end diff --git a/test/unit/attachment_test.rb b/test/unit/attachment_test.rb index c2a3e56b4..61ff048a7 100644 --- a/test/unit/attachment_test.rb +++ b/test/unit/attachment_test.rb @@ -150,6 +150,14 @@ class AttachmentTest < ActiveSupport::TestCase assert_equal 'cbb5b0f30978ba03731d61f9f6d10011', Attachment.disk_filename("test_accentué.ça")[13..-1] end + def test_title + a = Attachment.new(:filename => "test.png") + assert_equal "test.png", a.title + + a = Attachment.new(:filename => "test.png", :description => "Cool image") + assert_equal "test.png (Cool image)", a.title + end + def test_prune_should_destroy_old_unattached_attachments Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago) Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago) |