# HG changeset patch
# User Chris Cannam
# Date 1350398172 -3600
# Node ID 352539ac7b434b1f001835150bf26208f1ab6b4d
# Parent  42e2437766c25d9276bda891ae1221d5f7c3d377# Parent  35782bf2eb588cc1dfd8a62b2df22aadca38c172
Merge from branch "bug_521"

diff -r 42e2437766c2 -r 352539ac7b43 app/controllers/attachments_controller.rb
--- a/app/controllers/attachments_controller.rb	Wed Aug 29 15:55:57 2012 +0100
+++ b/app/controllers/attachments_controller.rb	Tue Oct 16 15:36:12 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),