]> source.dussan.org Git - redmine.git/commitdiff
Adds a setting to limit the number of diff lines that should be displayed (default...
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Sun, 7 Dec 2008 15:21:40 +0000 (15:21 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Sun, 7 Dec 2008 15:21:40 +0000 (15:21 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2112 e93f8b46-1217-0410-a6f0-8f06a7374b81

37 files changed:
app/views/common/_diff.rhtml
app/views/settings/_general.rhtml
config/settings.yml
lang/bg.yml
lang/ca.yml
lang/cs.yml
lang/da.yml
lang/de.yml
lang/en.yml
lang/es.yml
lang/fi.yml
lang/fr.yml
lang/he.yml
lang/hu.yml
lang/it.yml
lang/ja.yml
lang/ko.yml
lang/lt.yml
lang/nl.yml
lang/no.yml
lang/pl.yml
lang/pt-br.yml
lang/pt.yml
lang/ro.yml
lang/ru.yml
lang/sk.yml
lang/sr.yml
lang/sv.yml
lang/th.yml
lang/tr.yml
lang/uk.yml
lang/vn.yml
lang/zh-tw.yml
lang/zh.yml
lib/redmine/unified_diff.rb
test/fixtures/diffs/subversion.diff [new file with mode: 0644]
test/unit/lib/redmine/unified_diff_test.rb [new file with mode: 0644]

index 07bd4745f3d2b3f9c8011fc0aaa8fdb56febdd0f..104845b4d98510442a88d70cc7f7f1bd2d15e894 100644 (file)
@@ -1,4 +1,5 @@
-<% Redmine::UnifiedDiff.new(diff, :type => diff_type).each do |table_file| -%>
+<% diff = Redmine::UnifiedDiff.new(diff, :type => diff_type, :max_lines => Setting.diff_max_lines_displayed.to_i) -%>
+<% diff.each do |table_file| -%>
 <div class="autoscroll">
 <% if diff_type == 'sbs' -%>
 <table class="filecontent CodeRay">
@@ -62,3 +63,5 @@
 
 </div>
 <% end -%>
+
+<%= l(:text_diff_truncated) if diff.truncated? %>
index 06db77b227ef7b89b17daa0aed43ec218a6506de..9889702a2bf6b91c33d9665931257e4eee7dff8a 100644 (file)
@@ -48,6 +48,9 @@
 <p><label><%= l(:setting_feeds_limit) %></label>
 <%= text_field_tag 'settings[feeds_limit]', Setting.feeds_limit, :size => 6 %></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>
+
 <p><label><%= l(:setting_gravatar_enabled) %></label>
 <%= check_box_tag 'settings[gravatar_enabled]', 1, Setting.gravatar_enabled? %><%= hidden_field_tag 'settings[gravatar_enabled]', 0 %></p>
 </div>
index 02f0b13b50304acd075d050899b3bea5e93785a9..50064450f408beb5401894ddab20146f7d2c2256 100644 (file)
@@ -61,6 +61,9 @@ protocol:
 feeds_limit:
   format: int
   default: 15
+diff_max_lines_displayed:
+  format: int
+  default: 1500
 enabled_scm:
   serialized: true
   default: 
index 9bce327c6c72a8a0e8407ef25ac6b9b199940628..5170a977e275867f97d6f69451b19e989012bc61 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 250bb1d8dc3c48465198fe252b728d381144331b..79398078bb6ea3ffbaff4dce838e10ea207d7a33 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages\r
 label_user_activity: "%s's activity"\r
 label_updated_time_by: Updated by %s %s ago\r
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'\r
+setting_diff_max_lines_displayed: Max number of diff lines displayed\r
index cb7f361bceb44887fdd532c8600fdf51104bfc3f..ba22ab12c830b3add94c0069291e9c00a289a09d 100644 (file)
@@ -699,3 +699,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index a8acfa49bae78d6840668a2fee469eef7d0d1d61..5b8ee5b89fe1e2ce4595cf06e452f42f4cf93411 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 82c91d991c9d4f07d61c0a8df9ca578ba9a9f680..93afc016b59a8c4c62696269850dea7bc991c6fa 100644 (file)
@@ -696,3 +696,5 @@ enumeration_doc_categories: Dokumentenkategorien
 enumeration_activities: Aktivitäten (Zeiterfassung)
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 256141d079dce201637649177ea34bc058fec40a..b2d7efc4f7aac7721396b762433e55c00a5bd54f 100644 (file)
@@ -223,6 +223,7 @@ setting_mail_handler_api_enabled: Enable WS for incoming emails
 setting_mail_handler_api_key: API key
 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
 
 permission_edit_project: Edit project
 permission_select_project_modules: Select project modules
@@ -670,6 +671,7 @@ text_enumeration_destroy_question: '%d objects are assigned to this value.'
 text_enumeration_category_reassign_to: 'Reassign them to this value:'
 text_email_delivery_not_configured: "Email delivery is not configured, and notifications are disabled.\nConfigure your SMTP server in config/email.yml and restart the application to enable them."
 text_repository_usernames_mapping: "Select or update the Redmine user mapped to each username found in the repository log.\nUsers with the same Redmine and repository username or email are automatically mapped."
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
 
 default_role_manager: Manager
 default_role_developper: Developer
index 0c773496d652df5c9173043a76f001a1c0037cc8..52d3c30aa69a99b10738b190a3814d700ae04589 100644 (file)
@@ -679,3 +679,5 @@ text_wiki_destroy_confirmation: ¿Seguro que quiere borrar el wiki y todo su con
 text_workflow_edit: Seleccionar un flujo de trabajo para actualizar
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index e917ec2a88cf757610d7dd43181c94c7b528db95..be0a46b067d131bc93f907606213b98e3b952227 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: Muokkaa omia viestejä
 permission_delete_own_messages: Poista omia viestejä
 label_user_activity: "Käyttäjän %s historia"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index ffc1b75f395a3f27c16b258d38c9c1c0041adee8..daa6f2f602489a564272a602d5d67c35b05f75ee 100644 (file)
@@ -223,6 +223,7 @@ setting_mail_handler_api_enabled: "Activer le WS pour la réception d'emails"
 setting_mail_handler_api_key: Clé de protection de l'API
 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
 
 permission_edit_project: Modifier le projet
 permission_select_project_modules: Choisir les modules
@@ -670,6 +671,7 @@ text_enumeration_destroy_question: 'Cette valeur est affectée à %d objets.'
 text_enumeration_category_reassign_to: 'Réaffecter les objets à cette valeur:'
 text_email_delivery_not_configured: "L'envoi de mail n'est pas configuré, les notifications sont désactivées.\nConfigurez votre serveur SMTP dans config/email.yml et redémarrez l'application pour les activer."
 text_repository_usernames_mapping: "Vous pouvez sélectionner ou modifier l'utilisateur Redmine associé à chaque nom d'utilisateur figurant dans l'historique du dépôt.\nLes utilisateurs avec le même identifiant ou la même adresse mail seront automatiquement associés."
+text_diff_truncated: '... Ce différentiel a été tronqué car il excède la taille maximale pouvant être affichée.'
 
 default_role_manager: Manager
 default_role_developper: Développeur
index da9160809c8b0952580cd4f725d029afbe7d1d29..552c42233aa27263a6277642e01e76a0f0c14c75 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index e3c649026ee607d821d20639b2e5fa37a5ef6592..24a9da4a5d40f71a1c08b6f7422845f72d53422f 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Saját üzenetek szerkesztése
 permission_delete_own_messages: Saját üzenetek törlése
 label_user_activity: "%s tevékenységei"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 3ae2a323df7cb02f881efccfe3cf8aa8a7f35278..c5efcb219f153a50e0595536862022e1cc49c2d6 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: Modifica propri messaggi
 permission_delete_own_messages: Elimina propri messaggi
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 71e50a794cab03df3312f37770b4c98c862e3076..e1dfa2d8a7c7140b7598ba94ad6068485b926748 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index a476966a36b3730f4c4d6e2d019a10fd510a0767..9d2fb6d8e6a9de3f11487ee68c5a0c22530494b0 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: 자기 메시지 편집
 permission_delete_own_messages: 자기 메시지 삭제
 label_user_activity: "%s의 활동"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 8a7db6b40d17bdfd6912047e963f635e2acf510c..cd83a1b22279aa8b6dc4c8c9120529b2348afa43 100644 (file)
@@ -696,3 +696,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index e70fd999711204331474d5b27f90d80de3a4dfdf..51b91a1a19601d90d9879127c3e7a5964835fa11 100644 (file)
@@ -696,3 +696,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index afad807871ca6fd3dc5c840f40820aa9284b9747..8cf2321f131b5c23123e8685265d737dcfb1da71 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 7aea9bbfda9e6b84dd4c6d6ab1558b0feded0a23..0591f7a25dce1fdcb0169574673e31bdae98f279 100644 (file)
@@ -713,3 +713,5 @@ text_workflow_edit: Zaznacz rolę i typ zagadnienia do edycji przepływu
 
 label_user_activity: "Aktywność: %s"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 5f202de50dc9be69d7772db9f610065051e8783d..72137889b84094de5a39f28facb9be9763f6e0ac 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Editar próprias mensagens
 permission_delete_own_messages: Excluir próprias mensagens
 label_user_activity: "Atividade de %s"
 label_updated_time_by: Updated by %s %s ago\r
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'\r
+setting_diff_max_lines_displayed: Max number of diff lines displayed\r
index 0df3cda381dbd05fc65ba6b68d2698d37c76ff37..25110002a03152630b05350f29a2f39d1f491580 100644 (file)
@@ -696,3 +696,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 27d9c8e160c00d060e28a48985eadd1b30caa691..72d8f871cc3e502df490dc3183af3adebf532f0a 100644 (file)
@@ -694,3 +694,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 4d7807cb12489e8790750a96105c633dbd310315..acf916006aed05303f2b955b75cf3697f2d4157f 100644 (file)
@@ -728,3 +728,5 @@ text_wiki_destroy_confirmation: Вы уверены, что хотите уда
 text_workflow_edit: Выберите роль и трекер для редактирования последовательности состояний
 
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 9cea7d0060584a33b68eecec8169057f639ce7ea..8f874356b138acae3b3db8bae5cd967b843c9a19 100644 (file)
@@ -700,3 +700,5 @@ permission_delete_own_messages: Delete own messages
 text_repository_usernames_mapping: "Select or update the Redmine user mapped to each username found in the repository log.\nUsers with the same Redmine and repository username or email are automatically mapped."
 label_user_activity: "%s's activity"\r
 label_updated_time_by: Updated by %s %s ago\r
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'\r
+setting_diff_max_lines_displayed: Max number of diff lines displayed\r
index 797e94f54b2780964012a605521dc6568a5e2972..4a4b6d083de81c7a083006653019c9625bed5f8a 100644 (file)
@@ -695,3 +695,5 @@ permission_delete_own_messages: Delete own messages
 text_repository_usernames_mapping: "Select or update the Redmine user mapped to each username found in the repository log.\nUsers with the same Redmine and repository username or email are automatically mapped."
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index f15febe70062d18853f9c5fe9add8f8b9328f3f2..5c7e5e0d285c0e63fcd10b7f2fa6b6564d459fc0 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 2689c988972c89d231003cecc71c06c5deb83329..a569a50d56d47ba8641a75f0a1639eea6f452f46 100644 (file)
@@ -697,3 +697,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index bdd971268a7fa71342505b3921f0ac7505e6c102..a587b42145797cfba1dc3ced2e61c1d144de3140 100644 (file)
@@ -695,3 +695,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 7b3c446f4d494fe97190b12cdf5ea7fc0c21a1a6..b5d7c9789ad9a641d63d43729f88a980f8bdc49c 100644 (file)
@@ -696,3 +696,5 @@ permission_edit_own_messages: Edit own messages
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 93a243ff3e61d3e41a041e75e06754788e650ab5..de5a4f058606722a985988f7cd29baef37cfdea6 100644 (file)
@@ -697,3 +697,5 @@ text_repository_usernames_mapping: "Chọn hoặc cập nhật ánh xạ ngườ
 permission_delete_own_messages: Delete own messages
 label_user_activity: "%s's activity"\r
 label_updated_time_by: Updated by %s %s ago\r
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'\r
+setting_diff_max_lines_displayed: Max number of diff lines displayed\r
index a23df295b95dd44efe61a65ba057245ff28f9020..f52f4d815f97129cc7a49d7dcd0939884b53036f 100644 (file)
@@ -696,3 +696,5 @@ default_activity_development: 開發
 enumeration_issue_priorities: 項目優先權
 enumeration_doc_categories: 文件分類
 enumeration_activities: 活動 (時間追蹤)
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 721d21719f5ff7fb43b9f41ffcf7e36843222ce8..428c6e67da7822d4ac112737d2f538e7174bbf75 100644 (file)
@@ -696,3 +696,5 @@ enumeration_issue_priorities: 问题优先级
 enumeration_doc_categories: 文档类别
 enumeration_activities: 活动(时间跟踪)
 label_updated_time_by: Updated by %s %s ago
+text_diff_truncated: '... This diff was truncated because it exceeds the maximum size that can be displayed.'
+setting_diff_max_lines_displayed: Max number of diff lines displayed
index 5408a1ef9375a15f233a5650c99322ec9a710d3e..bf4dec335e574b97fdf660dddce02e7c6f881df5 100644 (file)
@@ -19,8 +19,11 @@ module Redmine
   # Class used to parse unified diffs
   class UnifiedDiff < Array  
     def initialize(diff, options={})
+      options.assert_valid_keys(:type, :max_lines)
       diff_type = options[:type] || 'inline'
       
+      lines = 0
+      @truncated = false
       diff_table = DiffTable.new(diff_type)
       diff.each do |line|
         if line =~ /^(---|\+\+\+) (.*)$/
@@ -28,10 +31,17 @@ module Redmine
           diff_table = DiffTable.new(diff_type)
         end
         diff_table.add_line line
+        lines += 1
+        if options[:max_lines] && lines > options[:max_lines]
+          @truncated = true
+          break
+        end
       end
       self << diff_table unless diff_table.empty?
       self
     end
+    
+    def truncated?; @truncated; end
   end
 
   # Class that represents a file diff
diff --git a/test/fixtures/diffs/subversion.diff b/test/fixtures/diffs/subversion.diff
new file mode 100644 (file)
index 0000000..9b6c9d0
--- /dev/null
@@ -0,0 +1,79 @@
+Index: app/views/settings/_general.rhtml
+===================================================================
+--- app/views/settings/_general.rhtml  (revision 2094)
++++ app/views/settings/_general.rhtml  (working copy)
+@@ -48,6 +48,9 @@
+ <p><label><%= l(:setting_feeds_limit) %></label>
+ <%= text_field_tag 'settings[feeds_limit]', Setting.feeds_limit, :size => 6 %></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>
++
+ <p><label><%= l(:setting_gravatar_enabled) %></label>
+ <%= check_box_tag 'settings[gravatar_enabled]', 1, Setting.gravatar_enabled? %><%= hidden_field_tag 'settings[gravatar_enabled]', 0 %></p>
+ </div>
+Index: app/views/common/_diff.rhtml
+===================================================================
+--- app/views/common/_diff.rhtml       (revision 2111)
++++ app/views/common/_diff.rhtml       (working copy)
+@@ -1,4 +1,5 @@
+-<% Redmine::UnifiedDiff.new(diff, :type => diff_type).each do |table_file| -%>
++<% diff = Redmine::UnifiedDiff.new(diff, :type => diff_type, :max_lines => Setting.diff_max_lines_displayed.to_i) -%>
++<% diff.each do |table_file| -%>
+ <div class="autoscroll">
+ <% if diff_type == 'sbs' -%>
+ <table class="filecontent CodeRay">
+@@ -62,3 +63,5 @@
+ </div>
+ <% end -%>
++
++<%= l(:text_diff_truncated) if diff.truncated? %>
+Index: lang/lt.yml
+===================================================================
+--- config/settings.yml        (revision 2094)
++++ config/settings.yml        (working copy)
+@@ -61,6 +61,9 @@
+ feeds_limit:
+   format: int
+   default: 15
++diff_max_lines_displayed:
++  format: int
++  default: 1500
+ enabled_scm:
+   serialized: true
+   default: 
+Index: lib/redmine/unified_diff.rb
+===================================================================
+--- lib/redmine/unified_diff.rb        (revision 2110)
++++ lib/redmine/unified_diff.rb        (working copy)
+@@ -19,8 +19,11 @@
+   # Class used to parse unified diffs
+   class UnifiedDiff < Array  
+     def initialize(diff, options={})
++      options.assert_valid_keys(:type, :max_lines)
+       diff_type = options[:type] || 'inline'
+       
++      lines = 0
++      @truncated = false
+       diff_table = DiffTable.new(diff_type)
+       diff.each do |line|
+         if line =~ /^(---|\+\+\+) (.*)$/
+@@ -28,10 +31,17 @@
+           diff_table = DiffTable.new(diff_type)
+         end
+         diff_table.add_line line
++        lines += 1
++        if options[:max_lines] && lines > options[:max_lines]
++          @truncated = true
++          break
++        end
+       end
+       self << diff_table unless diff_table.empty?
+       self
+     end
++    
++    def truncated?; @truncated; end
+   end
+   # Class that represents a file diff
diff --git a/test/unit/lib/redmine/unified_diff_test.rb b/test/unit/lib/redmine/unified_diff_test.rb
new file mode 100644 (file)
index 0000000..5e6ba1a
--- /dev/null
@@ -0,0 +1,42 @@
+# Redmine - project management software
+# Copyright (C) 2006-2008  Jean-Philippe Lang
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+# 
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+# 
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+
+require File.dirname(__FILE__) + '/../../../test_helper'
+
+class Redmine::UnifiedDiffTest < Test::Unit::TestCase
+  
+  def setup
+  end
+
+  def test_subversion_diff
+    diff = Redmine::UnifiedDiff.new(read_diff_fixture('subversion.diff'))
+    # number of files
+    assert_equal 4, diff.size
+    assert diff.detect {|file| file.file_name =~ %r{^config/settings.yml}}
+  end
+  
+  def test_truncate_diff
+    diff = Redmine::UnifiedDiff.new(read_diff_fixture('subversion.diff'), :max_lines => 20)
+    assert_equal 2, diff.size
+  end
+
+  private
+  
+  def read_diff_fixture(filename)
+    File.new(File.join(File.dirname(__FILE__), '/../../../fixtures/diffs', filename)).read
+  end
+end