close temp files after upload

CNVS-11519

this fixes an error introduced in 
https://gerrit.instructure.com/#/c/28234/

test plan:
-try to upload a zipped directory containing lots
of files (for example, there is a dropbox link in
CNVS-11519 containing a file you can use
-while the upload is still in progress, try to upload
the same in a different course
-check that all files are uploaded, in each course

Change-Id: I373dfebe5dbc1f21e681d35e227037cb4a316676
Reviewed-on: https://gerrit.instructure.com/31933
Reviewed-by: Cody Cutrer <cody@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
QA-Review: August Thornton <august@instructure.com>
Product-Review: Anthus Williams <awilliams@instructure.com>
This commit is contained in:
Anthus Williams 2014-03-14 15:06:42 -06:00
parent 13005bcd84
commit bf208212d9
1 changed files with 2 additions and 0 deletions

View File

@ -54,6 +54,8 @@ class FileInContext
destroy_files(@attachment.handle_duplicates(allow_rename ? :rename : :overwrite, :caller_will_destroy => true))
@attachment
ensure
uploaded_data.close if uploaded_data
end
end