#15799 closed Cleanup/optimization (fixed)
Document what exception should be raised when trying to open non-existent file
| Reported by: | Jonas H. | Owned by: | Anthony Kugel | 
|---|---|---|---|
| Component: | File uploads/storage | Version: | dev | 
| Severity: | Normal | Keywords: | |
| Cc: | Collin Anderson | Triage Stage: | Ready for checkin | 
| Has patch: | yes | Needs documentation: | no | 
| Needs tests: | no | Patch needs improvement: | no | 
| Easy pickings: | yes | UI/UX: | no | 
Description
What should custom storage backends raise if the file trying to open does not exist? (In Storage._open) That's not documented anywhere.
Change History (10)
comment:1 by , 15 years ago
| Triage Stage: | Unreviewed → Accepted | 
|---|---|
| Type: | Uncategorized → New feature | 
comment:4 by , 2 years ago
| Cc: | added | 
|---|---|
| Easy pickings: | set | 
| Needs documentation: | set | 
| Type: | New feature → Cleanup/optimization | 
FileNotFoundError was added in Python 3.3 / PEP 3151 and kinda fixed this issue.
Could maybe help to document raising exceptions like FileNotFoundError here https://docs.djangoproject.com/en/dev/howto/custom-file-storage/ , though probably doesn't need to be super thoroughly documented, as it mostly should be similar to Python's open(). 
comment:5 by , 2 years ago
| Owner: | changed from to | 
|---|---|
| Status: | new → assigned | 
comment:7 by , 2 years ago
| Patch needs improvement: | set | 
|---|
comment:8 by , 2 years ago
| Patch needs improvement: | unset | 
|---|---|
| Triage Stage: | Accepted → Ready for checkin | 
  Note:
 See   TracTickets
 for help on using tickets.
    
I can't find anywhere in Django that tries to catch an exception from calling .open() on a storage backend, and Django's FileSystemStorage doesn't catch the IOError and re-raise anything else, so at the moment I don't believe there is any such standard to document. For maximum compatibility with FileSystemStorage you'd want to raise IOError, I suppose.
I can see valid use cases (i.e. in reusable apps) for having some common exceptions standardized as part of the Storage interface, so accepting this as a feature request (not necessarily just for documentation, as FileSystemStorage should also use the common exception, and I'm not sure it should be IOError).