[IO-340] The use of file.exists() is failure prone. Where file.exists() is checked, it should be replaced with the following dual check: exists = ( file.exists() || file.length()>0 )

• Type: New Feature
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 2.5, 3.x
• Component/s: Utilities
• Labels: File exists false

Environment

All

• Assignee: Unassigned
• Reporter: Feeling Groovy
• Created: 03/Aug/12 19:34
• Updated: 08/Nov/16 18:04
• Resolved: 16/Apr/13 20:29

Description

The use of file.exists() is failure prone. In many cases the jvm can report that a file does not exist even when it does. This is due to the caching in the operating system.
Since file.exists() is prone to report false values, a second check can and should be made to file.length()>0.

Where file.exists() is checked, it should be replaced with the following dual check:

A file more reliably exists if: file.exists() || file.length()>0

Then if the file does not exist, file.length will be 0.