#7769 closed (fixed)
[patch] TemporaryUploadedFile is missing a few methods/params
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | HTTP handling | Version: | dev |
| Severity: | Keywords: | #2070-fixes | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
seek on TemporaryUploadedFile is missing the whence parameter.
TemporaryUploadedFile and InMemoryUploadedFile are missing tell() methods, which has been added for convenience.
Also renames file on InMemoryUploadedFile for consistency with TemporaryUploadedFile.
Attachments (2)
Change History (9)
by , 17 years ago
| Attachment: | uploaded_file.diff added |
|---|
by , 17 years ago
| Attachment: | uploaded_file.2.diff added |
|---|
comment:1 by , 17 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 17 years ago
| Summary: | [patch] Missing seek() whence parameter in TemporaryUploadedFile + add tell() → [patch] TemporaryUploadedFile is missing a few methods/params |
|---|
comment:3 by , 17 years ago
This patch fixes an issue I had, but I'd like to mention that it still has a bug:
SimpleUploadedFile is a subclass of InMemoryUploadedFile, so it uses its tell() and seek() functions, which expect a _file member, but SimpleUploadedFile calls it file, so those functions don't work for a SimpleUploadedFile.
comment:4 by , 17 years ago
| milestone: | → 1.0 |
|---|
#7900 was closed at a dupe, as it only looked at the missing tell method and therefore was a subset of this ticket. Also setting this one as milestone 1.0.
comment:5 by , 17 years ago
One question though. Since this ticket is adding new features, can it actually be considered for 1.0? #7900 was set to 1.0, that's why I just set this one as well, but a core dev might think otherwise.
comment:6 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Removed write from InMemoryTemporaryFile