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.

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

In Memory File Zipping #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

In Memory File Zipping #1

wants to merge 1 commit into from

Conversation

jachaos
Copy link

@jachaos jachaos commented Mar 12, 2015

Replace the code to zip the files in memory instead of writing them to
the disk first and some minor code adjustment and cleanup.

I was working on this code to submit files through the IDES Gateway, when creating several files or the same file multiples times it becomes bit messy having all these files in the destination folder, generating just the final zip file without the need of creating the "temporary" files first would produce a cleaner result. And the user can always find the files inside the .Zip file in case he wants to take a look to them.

Also moved the Creation of the Metadata XML file to the XMLManager class.

Replace the code to zip the files in memory instead of writting them to
the disk first and some minor code adjustment and cleanup.
@MrGeneric
Copy link

Much more elegant than my solution of programatically deleting the leftover files, nice work. Would be nice to see this merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants