From 5c2de4dfc95293a7965ff8cf980a310922fc4c05 2012-07-16 16:42:23
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: 2012-07-16 16:42:23
Subject: [PATCH] Adds Attachment#title.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10012 e93f8b46-1217-0410-a6f0-8f06a7374b81

---

diff --git a/app/models/attachment.rb b/app/models/attachment.rb
index 0dda330..9aeee07 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 c2a3e56..61ff048 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)