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

Handle Zip files on upload #905

Open
wants to merge 10 commits into
base: use-files-directly
Choose a base branch
from

Conversation

trevorgerhardt
Copy link
Member

@trevorgerhardt trevorgerhardt commented Oct 27, 2023

Extract zips into their individual files before handling. Refactors controllers that handle file uploads to use the new utilities.

@trevorgerhardt trevorgerhardt linked an issue Oct 28, 2023 that may be closed by this pull request
@trevorgerhardt trevorgerhardt added feature t0 Time level 0: think hours labels Oct 28, 2023
@trevorgerhardt trevorgerhardt marked this pull request as ready for review October 28, 2023 02:20
@trevorgerhardt trevorgerhardt marked this pull request as draft October 28, 2023 02:47
@trevorgerhardt trevorgerhardt added t1 Time level 1: think days and removed t0 Time level 0: think hours labels Oct 30, 2023
@trevorgerhardt trevorgerhardt marked this pull request as ready for review October 31, 2023 10:34
We use Apache FileUpload to assist with handling incoming files. We also use a DiskItemFactory that persists each incoming file to disk immediately. This means that we could represent each incoming file as a java.io.File instead of a FileItem. java.io.File is a much more common way of handling a file and easier to work with.
Extract them into their individual files before handling like normal files. Refactors all controllers that handle file uploads to use the new utilities.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature t1 Time level 1: think days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support uploading Shapefiles as a single .zip
1 participant