@@ -60,6 +60,10 class Version < ActiveRecord::Base | |||
|
60 | 60 | project.present? && project.attachments_visible?(*args) |
|
61 | 61 | end |
|
62 | 62 | |
|
63 | def attachments_deletable?(usr=User.current) | |
|
64 | project.present? && project.attachments_deletable?(usr) | |
|
65 | end | |
|
66 | ||
|
63 | 67 | def start_date |
|
64 | 68 | @start_date ||= fixed_issues.minimum('start_date') |
|
65 | 69 | end |
@@ -374,6 +374,16 class AttachmentsControllerTest < ActionController::TestCase | |||
|
374 | 374 | end |
|
375 | 375 | end |
|
376 | 376 | |
|
377 | def test_destroy_version_attachment_with_issue_tracking_disabled | |
|
378 | Project.find(1).disable_module! :issue_tracking | |
|
379 | set_tmp_attachments_directory | |
|
380 | @request.session[:user_id] = 2 | |
|
381 | assert_difference 'Attachment.count', -1 do | |
|
382 | delete :destroy, :id => 9 | |
|
383 | assert_response 302 | |
|
384 | end | |
|
385 | end | |
|
386 | ||
|
377 | 387 | def test_destroy_without_permission |
|
378 | 388 | set_tmp_attachments_directory |
|
379 | 389 | assert_no_difference 'Attachment.count' do |
General Comments 0
You need to be logged in to leave comments.
Login now