Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cleanup temp files after import/export #464

Merged
merged 8 commits into from
Jul 22, 2015
Merged

Cleanup temp files after import/export #464

merged 8 commits into from
Jul 22, 2015

Conversation

mvi-eea-sk
Copy link

Cleaning up all temp files after:

  • importing pipeline from zip file
  • importing dpu templates from zip file
  • importing dpu templates from jar file
  • replacing existing dpu template
  • exporting pipeline
  • exporting (all) dpu templates

Notes:

  • there was no normal way to cleanup after exporting file, because Vaadin didn't provide a way to catch the "after finishing download" event. So the returned InputStream had to be replaced with DeletingFileInputStream, which cleans up the after it is closed.

Resolves #373

@mvi-eea-sk mvi-eea-sk changed the title #373 Cleanup temp files after import/export Cleanup temp files after import/export Jul 16, 2015
@skrchnavy skrchnavy added this to the Release v2.2.0 milestone Jul 16, 2015
@tomas-knap tomas-knap assigned mvi-eea-sk and unassigned tomas-knap Jul 22, 2015
@mvi-eea-sk
Merge branch 'develop' into feature/delete_temp_file_after_import_export
84d49e2 
Conflicts:
	frontend/src/main/java/cz/cuni/mff/xrg/odcs/frontend/gui/components/DPUTemplatesExport.java
	frontend/src/main/java/cz/cuni/mff/xrg/odcs/frontend/gui/dialog/PipelineExport.java
@mvi-eea-sk
Merge branch 'develop' into feature/delete_temp_file_after_import_export
a410f8b 
Conflicts:
	frontend/src/main/resources/frontend-messages.properties
	frontend/src/main/resources/frontend-messages_sk.properties
@mvi-eea-sk mvi-eea-sk merged commit a410f8b into develop Jul 22, 2015
@mvi-eea-sk mvi-eea-sk deleted the feature/delete_temp_file_after_import_export branch July 22, 2015 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mvi-eea-sk mvi-eea-sk

Labels
priority: Normal resolution: fixed severity: bug status: resolved
Projects
None yet
Milestone
Release v2.1.2
Development

Successfully merging this pull request may close these issues.
3 participants
@mvi-eea-sk @tomas-knap @skrchnavy