commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bernd Eckenfels (JIRA)" <>
Subject [jira] [Commented] (VFS-585) CompressedFileFileObject should override exists()
Date Thu, 29 Oct 2015 19:51:27 GMT


Bernd Eckenfels commented on VFS-585:

Using the file type (IMAGINARY == non existing) is used by all file providers, this is how
VFS works: the file type is determined on doAttach and exists() is only a helper to check
for the type. (Only HdfsFileObject overwrites AbstractFileObject#exists() - with basically
the same function).

So the problem is more that it does not implement doAttach() and the getType() purely works
on the file name.

> CompressedFileFileObject should override exists()
> -------------------------------------------------
>                 Key: VFS-585
>                 URL:
>             Project: Commons VFS
>          Issue Type: Bug
>    Affects Versions: 2.0
>            Reporter: Eduard Wirch
> {{CompressedFileFileObject}} does not implement {{exists()}}. This leads to inheriting
the implementation of {{AbstractFileObject}} which always returns {{true}} if the type is
not {{IMAGINARY}}. That's completely useless in production.
> Make {{exists()}} return {{true}} only if the archive and the compressed file exist.
> Alternative: at least existence of the archive should be checked.

This message was sent by Atlassian JIRA

View raw message