# HG changeset patch
# User Chris Cannam <chris.cannam@soundsoftware.ac.uk>
# Date 1350401263 -3600
# Node ID 5251072c02b6eb441f76a5d98b4b6f705bb92b9f
# Parent  c09e40c4a410ff8a47b36b949eace15e62f08b8c# Parent  910830dc222a956f65ad6c9bea7552c15c126b5a
Merge from branch cannam

diff -r c09e40c4a410 -r 5251072c02b6 app/controllers/attachments_controller.rb
--- a/app/controllers/attachments_controller.rb	Tue Oct 16 16:27:29 2012 +0100
+++ b/app/controllers/attachments_controller.rb	Tue Oct 16 16:27:43 2012 +0100
@@ -49,9 +49,9 @@
   end
 
   def download
-    if @attachment.container.is_a?(Version) || @attachment.container.is_a?(Project)
-      @attachment.increment_download
-    end
+    # cc: formerly this happened only if "@attachment.container.is_a?(Version)"
+    # or Project. Not good for us, we want to tally all downloads
+    @attachment.increment_download
 
     # images are sent inline
     send_file @attachment.diskfile, :filename => filename_for_content_disposition(@attachment.filename),