@@ -53,7 +53,7 class Attachment < ActiveRecord::Base | |||||
53 | @@thumbnails_storage_path = File.join(Rails.root, "tmp", "thumbnails") |
|
53 | @@thumbnails_storage_path = File.join(Rails.root, "tmp", "thumbnails") | |
54 |
|
54 | |||
55 | before_create :files_to_final_location |
|
55 | before_create :files_to_final_location | |
56 |
after_ |
|
56 | after_commit :delete_from_disk, :on => :destroy | |
57 |
|
57 | |||
58 | # Returns an unsaved copy of the attachment |
|
58 | # Returns an unsaved copy of the attachment | |
59 | def copy(attributes=nil) |
|
59 | def copy(attributes=nil) |
@@ -47,6 +47,9 module RedmineApp | |||||
47 | # Do not include all helpers |
|
47 | # Do not include all helpers | |
48 | config.action_controller.include_all_helpers = false |
|
48 | config.action_controller.include_all_helpers = false | |
49 |
|
49 | |||
|
50 | # Do not supress errors in after_rollback and after_commit callbacks | |||
|
51 | config.active_record.raise_in_transactional_callbacks = true | |||
|
52 | ||||
50 | # XML parameter parser removed from core in Rails 4.0 |
|
53 | # XML parameter parser removed from core in Rails 4.0 | |
51 | # and extracted to actionpack-xml_parser gem |
|
54 | # and extracted to actionpack-xml_parser gem | |
52 | config.middleware.insert_after ActionDispatch::ParamsParser, ActionDispatch::XmlParamsParser |
|
55 | config.middleware.insert_after ActionDispatch::ParamsParser, ActionDispatch::XmlParamsParser |
@@ -23,6 +23,10 class AttachmentTest < ActiveSupport::TestCase | |||||
23 | fixtures :users, :projects, :roles, :members, :member_roles, |
|
23 | fixtures :users, :projects, :roles, :members, :member_roles, | |
24 | :enabled_modules, :issues, :trackers, :attachments |
|
24 | :enabled_modules, :issues, :trackers, :attachments | |
25 |
|
25 | |||
|
26 | # TODO: remove this with Rails 5 that supports after_commit callbacks | |||
|
27 | # in transactional fixtures (https://github.com/rails/rails/pull/18458) | |||
|
28 | self.use_transactional_fixtures = false | |||
|
29 | ||||
26 | class MockFile |
|
30 | class MockFile | |
27 | attr_reader :original_filename, :content_type, :content, :size |
|
31 | attr_reader :original_filename, :content_type, :content, :size | |
28 |
|
32 |
General Comments 0
You need to be logged in to leave comments.
Login now