Changes between Initial Version and Version 1 of Ticket #14760, comment 8
- Timestamp:
- Feb 14, 2011, 1:48:27 PM (14 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #14760, comment 8
initial v1 1 So, this ticket report and comments seem to be actually about three different issues about the admin ''Edit view'' of a model with inlines of instances of a model that has a FK to it and contains a !FileField or !ImageField:1 So, this ticket report and comments seem to be actually about three different issues involving: 2 2 3 1. Pressing the "Save as" button reports back a validation error and the following: 4 a. The image/file fields of inlines already existing in the DB/FS will be empty and show "This field is required." message error. 3 * The admin ''Edit view'' of a model with inlines of instances of a model that has a FK to it and that contains a !FileField or !ImageField. 4 * A !ModelAdmin with `save_as=True`. 5 6 The issues are: 7 8 1. Pressing the "Save as" button reports back a validation error and: 9 a. The image/file fields of inlines already existing in the DB/FS are empty and show "This field is required." message error. 5 10 b. Values of other fields of the already existing inlines are preserved. 6 11 c. All the information filled by the user in extra inlines is lost. … … 8 13 2. If, in the resulting admin ''Add view'', new files are selected from the local FS for the fields reporting validation errors. You get a `'ValueError?: invalid literal for int() with base 10'` error. 9 14 10 3. When the error of item 1 happens, the ''save_as_new'' button gets replaced with a ''save_and_add_another'' 15 3. When the error of item 1 happens, the ''save_as_new'' button gets replaced with a ''save_and_add_another'' button. 11 16 12 17 I propose to limit this ticket to item 1. 13 18 14 Item 2 is already covered by #13223. 19 Item 2 is already covered by #13223. That ticket seems to suggest the problem isn't specific to inlines containing Image/!FileField's. 15 20 16 21 Item 3 is already covered by #5681, please see ticket:5681#comment:4 for a different opinion about it.