Opened 16 years ago
Closed 16 years ago
#4274 closed defect (fixed)
[PATCH] Folders for File > Open...
Reported by: | bastiK | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | Cc: |
Description
Since r2703 you can open images using File > Open...
As a consequence it is allowed to open a entire folder: Then all images are extracted recursively.
However if the folder contains no images, an empty image layer is silently created.
A better way would be to also load all other file types recursively. (And maybe give a warning if it would result in many layers being created.)
I would do this myself, but have I no time till mid January.
Attachments (1)
Change History (3)
by , 16 years ago
Attachment: | josm-empty-image-layer.diff added |
---|
comment:1 by , 16 years ago
Summary: | Folders for File > Open... → [PATCH] Folders for File > Open... |
---|
I added a patch to throw an exception if no image file was found.
This will display an error message and prevent the creation of an empty image layer.
Checking how many layers would be created by recursively processing all file types without actually creating them would probably require a check function in every FileImporter.
I think if we really want this, it might be better to do the directory recursion outside of the FileImporter subclasses.
comment:2 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
patch to avoid empty image layer