Opened 6 years ago

Last modified 3 years ago

#15611 new Bug

Readonly fields not hidden during create

Reported by: coderanger Owned by:
Component: contrib.admin Version: 1.2
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: yes

Description

A field marked as readonly should be hidden when creating a new object from the admin.

Attachments (1)

django-ticket-15611.diff (8.8 KB) - added by coderanger 6 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 6 years ago by coderanger

Keywords: blocker added
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

comment:2 Changed 6 years ago by Russell Keith-Magee

Keywords: blocker removed
Triage Stage: UnreviewedAccepted

FYI: blocker doesn't mean "this is annoying and I want someone to fix it". It means "this is a problem so serious that we need to hold back the release".

This *certainly* isn't a blocker. It doesn't cause data loss. It doesn't cause crashes. It isn't a regression in behavior since the last release, and it isn't a bug in a new feature. It is, at best, an mild annoyance.

comment:3 Changed 6 years ago by coderanger

The flag was set on the request of Jannis ;-) The fix is done, going to make some tests tomorrow when I get down to the sprints.

Changed 6 years ago by coderanger

Attachment: django-ticket-15611.diff added

comment:4 Changed 6 years ago by coderanger

Owner: changed from coderanger to Jannis Leidel

comment:5 Changed 6 years ago by Luke Plant

Type: Bug

comment:6 Changed 6 years ago by Luke Plant

Severity: Normal

comment:7 Changed 5 years ago by Julien Phalip

UI/UX: set

comment:8 Changed 5 years ago by Julien Phalip

Easy pickings: unset
Triage Stage: AcceptedDesign decision needed

I'm not convinced that read-only fields should systematically and implicitly be hidden (or omitted) from the add form as, for example, one may want to display read-only default values. Instead I would prefer to let the admin offer more flexibility so that one could explicitly control what fields are displayed in the add/change forms on a per-request basis. For this reason I'm marking this ticket as DDN.

comment:9 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

comment:10 Changed 5 years ago by Jannis Leidel

Cc: Jannis Leidel removed
Owner: Jannis Leidel deleted

comment:11 Changed 4 years ago by Aymeric Augustin

Triage Stage: Design decision neededAccepted

The option to offer "more flexibility" didn't apper, and it doesn't look like it will be implemented any time soon. Can we proceed with the bugfix?

comment:12 Changed 3 years ago by Tim Graham

Has patch: set

comment:13 Changed 3 years ago by Tim Graham

Patch needs improvement: set

Patch doesn't apply cleanly.

Note: See TracTickets for help on using tickets.
Back to Top