Adds a setting to limit the size of text attachments that can be displayed by…

Adds a setting to limit the size of text attachments that can be displayed by the file viewer (default set to 512KB). Larger files are sent for download. git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2503 e93f8b46-1217-0410-a6f0-8f06a7374b81

Adds a setting to limit the size of text attachments that can be displayed by…
Adds a setting to limit the size of text attachments that can be displayed by the file viewer (default set to 512KB). Larger files are sent for download. git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2503 e93f8b46-1217-0410-a6f0-8f06a7374b81
66afc8c0 · Jean-Philippe Lang · 2bd419f2 · 66afc8c0 · 66afc8c0 · 66afc8c0
Commit 66afc8c0 authored Feb 21, 2009 by Jean-Philippe Lang
37 changed files
--- a/app/controllers/attachments_controller.rb
+++ b/app/controllers/attachments_controller.rb
@@ -26,7 +26,7 @@ class AttachmentsController < ApplicationController
    if @attachment.is_diff?
      @diff = File.new(@attachment.diskfile, "rb").read
      render :action => 'diff'
-    elsif @attachment.is_text?
+    elsif @attachment.is_text? && @attachment.filesize <= Setting.setting_file_max_size_displayed.to_i.kilobyte
      @content = File.new(@attachment.diskfile, "rb").read
      render :action => 'file'
    else

--- a/app/views/settings/_general.rhtml
+++ b/app/views/settings/_general.rhtml
@@ -33,6 +33,9 @@
 <p><label><%= l(:setting_feeds_limit) %></label>
 <%= text_field_tag 'settings[feeds_limit]', Setting.feeds_limit, :size => 6 %></p>

+<p><label><%= l(:setting_file_max_size_displayed) %></label>
+<%= text_field_tag 'settings[setting_file_max_size_displayed]', Setting.setting_file_max_size_displayed, :size => 6 %> KB</p>
+
 <p><label><%= l(:setting_diff_max_lines_displayed) %></label>
 <%= text_field_tag 'settings[diff_max_lines_displayed]', Setting.diff_max_lines_displayed, :size => 6 %></p>
 </div>

--- a/config/locales/bg.yml
+++ b/config/locales/bg.yml
@@ -771,3 +771,4 @@ bg:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/ca.yml
+++ b/config/locales/ca.yml
@@ -772,3 +772,4 @@ ca:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/cs.yml
+++ b/config/locales/cs.yml
@@ -776,3 +776,4 @@ cs:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/da.yml
+++ b/config/locales/da.yml
@@ -803,3 +803,4 @@ da:
  general_csv_decimal_separator: '.'
  permission_edit_own_time_entries: Edit own time logs
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/de.yml
+++ b/config/locales/de.yml
@@ -804,3 +804,4 @@ de:
  button_create_and_continue: Create and continue
  text_custom_field_possible_values_info: 'One line for each value'
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -278,6 +278,7 @@ en:
  setting_sequential_project_identifiers: Generate sequential project identifiers
  setting_gravatar_enabled: Use Gravatar user icons
  setting_diff_max_lines_displayed: Max number of diff lines displayed
+  setting_file_max_size_displayed: Max size of text files displayed inline
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
  
  permission_edit_project: Edit project

--- a/config/locales/es.yml
+++ b/config/locales/es.yml
@@ -824,3 +824,4 @@ es:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/fi.yml
+++ b/config/locales/fi.yml
@@ -814,3 +814,4 @@ fi:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/fr.yml
+++ b/config/locales/fr.yml
@@ -311,6 +311,7 @@ fr:
  setting_sequential_project_identifiers: Générer des identifiants de projet séquentiels
  setting_gravatar_enabled: Afficher les Gravatar des utilisateurs
  setting_diff_max_lines_displayed: Nombre maximum de lignes de diff affichées
+  setting_file_max_size_displayed: Taille maximum des fichiers texte affichés en ligne
  setting_repository_log_display_limit: "Nombre maximum de revisions affichées sur l'historique d'un fichier"
  
  permission_edit_project: Modifier le projet

--- a/config/locales/gl.yml
+++ b/config/locales/gl.yml
@@ -804,3 +804,4 @@ gl:
  button_create_and_continue: Create and continue
  text_custom_field_possible_values_info: 'One line for each value'
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/he.yml
+++ b/config/locales/he.yml
@@ -787,3 +787,4 @@ he:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/hu.yml
+++ b/config/locales/hu.yml
@@ -809,3 +809,4 @@
  label_display: Megmutat
  field_editable: Szerkeszthető
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/it.yml
+++ b/config/locales/it.yml
@@ -790,3 +790,4 @@ it:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/ja.yml
+++ b/config/locales/ja.yml
@@ -802,3 +802,4 @@ ja:
  label_display: 表示
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/ko.yml
+++ b/config/locales/ko.yml
@@ -833,3 +833,4 @@ ko:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/lt.yml
+++ b/config/locales/lt.yml
@@ -814,3 +814,4 @@ lt:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/nl.yml
+++ b/config/locales/nl.yml
@@ -759,3 +759,4 @@ nl:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/no.yml
+++ b/config/locales/no.yml
@@ -777,3 +777,4 @@
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/pl.yml
+++ b/config/locales/pl.yml
@@ -807,3 +807,4 @@ pl:
  button_create_and_continue: Create and continue
  text_custom_field_possible_values_info: 'One line for each value'
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/pt-BR.yml
+++ b/config/locales/pt-BR.yml
@@ -810,3 +810,4 @@ pt-BR:
  label_display: Exibição
  field_editable: Editável
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/pt.yml
+++ b/config/locales/pt.yml
@@ -795,3 +795,4 @@ pt:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/ro.yml
+++ b/config/locales/ro.yml
@@ -817,3 +817,4 @@ ro:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/ru.yml
+++ b/config/locales/ru.yml
@@ -902,3 +902,4 @@ ru:
  field_editable: Редактируемый
  
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/sk.yml
+++ b/config/locales/sk.yml
@@ -775,3 +775,4 @@ sk:
  button_create_and_continue: Create and continue
  text_custom_field_possible_values_info: 'One line for each value'
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/sl.yml
+++ b/config/locales/sl.yml
@@ -773,3 +773,4 @@ sl:
  button_create_and_continue: Create and continue
  text_custom_field_possible_values_info: 'One line for each value'
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/sr.yml
+++ b/config/locales/sr.yml
@@ -798,3 +798,4 @@
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/sv.yml
+++ b/config/locales/sv.yml
@@ -831,3 +831,4 @@ sv:
  enumeration_doc_categories: Dokumentkategorier
  enumeration_activities: Aktiviteter (tidsuppföljning)
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/th.yml
+++ b/config/locales/th.yml
@@ -774,3 +774,4 @@ th:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/tr.yml
+++ b/config/locales/tr.yml
@@ -811,3 +811,4 @@ tr:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/uk.yml
+++ b/config/locales/uk.yml
@@ -773,3 +773,4 @@ uk:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/vi.yml
+++ b/config/locales/vi.yml
@@ -843,3 +843,4 @@ vi:
  label_display: Display
  field_editable: Editable
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/zh-TW.yml
+++ b/config/locales/zh-TW.yml
@@ -805,3 +805,4 @@
  enumeration_doc_categories: 文件分類
  enumeration_activities: 活動 (時間追蹤)
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/locales/zh.yml
+++ b/config/locales/zh.yml
@@ -805,3 +805,4 @@ zh:
  enumeration_doc_categories: 文档类别
  enumeration_activities: 活动（时间跟踪）
  setting_repository_log_display_limit: Maximum number of revisions displayed on file log
+  setting_file_max_size_displayed: Max size of text files displayed inline
--- a/config/settings.yml
+++ b/config/settings.yml
@@ -61,6 +61,11 @@ protocol:
 feeds_limit:
  format: int
  default: 15
+# Maximum size of files that can be displayed
+# inline through the file viewer (in KB)
+setting_file_max_size_displayed:
+  format: int
+  default: 512
 diff_max_lines_displayed:
  format: int
  default: 1500

--- a/test/functional/attachments_controller_test.rb
+++ b/test/functional/attachments_controller_test.rb
@@ -53,12 +53,23 @@ class AttachmentsControllerTest < Test::Unit::TestCase
    get :show, :id => 5
    assert_response :success
    assert_template 'diff'
+    assert_equal 'text/html', @response.content_type
  end
  
  def test_show_text_file
    get :show, :id => 4
    assert_response :success
    assert_template 'file'
+    assert_equal 'text/html', @response.content_type
+  end
+  
+  def test_show_text_file_should_send_if_too_big
+    Setting.setting_file_max_size_displayed = 512
+    Attachment.find(4).update_attribute :filesize, 754.kilobyte
+    
+    get :show, :id => 4
+    assert_response :success
+    assert_equal 'application/x-ruby', @response.content_type
  end
  
  def test_show_other