Opened 20 years ago
Closed 17 years ago
#575 closed defect (fixed)
edit_inline don't save child data when a "blank=True" field is not set
| Reported by: | pylorca | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | major | Keywords: | nfa-fixed |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
when I try to insert or modify throught the admin interface, child data is not saved when there is a "blank=True" field and it's not set.
example:
model:
class Person(meta.Model):
name = meta.CharField(maxlength=30)
lastName = meta.CharField(maxlength=30)
class Phone:
person = meta.ForeignKey(Person, edit_inline=meta.STACKED, num_in_admin=2)
number = meta.CharField(maxlength=30, core=True)
description = meta.CharField(maxlength=100, blank=True, core=True)
if I set a Phone number and I don't set description, this don't throw any error, but the Phone data is not saved, but this would be saved if I set the description too
Change History (9)
comment:1 by , 18 years ago
| Triage Stage: | Ready for checkin → Accepted |
|---|
comment:2 by , 18 years ago
| Cc: | added |
|---|---|
| Component: | Admin interface → django.newforms |
| Keywords: | ashleigh added |
| Triage Stage: | Accepted → Ready for checkin |
| Version: | → 0.91 |
comment:3 by , 18 years ago
| Component: | django.newforms → Documentation |
|---|---|
| Triage Stage: | Ready for checkin → Accepted |
| Version: | 0.91 → other branch |
follow-up: 5 comment:4 by , 18 years ago
| Cc: | removed |
|---|---|
| Component: | Documentation → Admin interface |
| Keywords: | ashleigh removed |
| Version: | other branch → SVN |
reverted spam
comment:5 by , 18 years ago
| Component: | Admin interface → Database wrapper |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
| Triage Stage: | Accepted → Design decision needed |
Replying to gwilson:
reverted spam
comment:6 by , 18 years ago
| Component: | Database wrapper → Admin interface |
|---|---|
| Triage Stage: | Design decision needed → Accepted |
comment:7 by , 18 years ago
| Resolution: | wontfix |
|---|---|
| Status: | closed → reopened |
Reverting close by anonymous
comment:8 by , 18 years ago
| Keywords: | nfa-fixed added |
|---|
This is no longer a problem with newforms-admin. core=True usage is completely gone. Tagging with nfa-fixed to be closed when newforms-admin is merged.
comment:9 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
This is no longer a problem since the merge of newforms-admin in [7967].
Maybe the
blank=Truein not playing nice withcore=True. Theedit_inlinestuff will be changing in the newforms-admin branch.Anyhow, this is not ready for checkin.