Opened 21 years ago

Closed 18 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 Gary Wilson <gary.wilson@…>, 19 years ago

Triage Stage: Ready for checkin → Accepted

Maybe the blank=True in not playing nice with core=True. The edit_inline stuff will be changing in the newforms-admin branch.

Anyhow, this is not ready for checkin.

comment:2 by clarice, 19 years ago

Cc: ashleigh added
Component: Admin interface → django.newforms
Keywords: ashleigh added
Triage Stage: Accepted → Ready for checkin
Version: → 0.91

comment:3 by alishia, 19 years ago

Component: django.newforms → Documentation
Triage Stage: Ready for checkin → Accepted
Version: 0.91 → other branch

comment:4 by Gary Wilson, 19 years ago

Cc: ashleigh removed
Component: Documentation → Admin interface
Keywords: ashleigh removed
Version: other branch → SVN

reverted spam

in reply to:  4 comment:5 by anonymous, 19 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 James Bennett, 19 years ago

Component: Database wrapper → Admin interface
Triage Stage: Design decision needed → Accepted

comment:7 by Chris Beaven, 19 years ago

Resolution: wontfix
Status: closed → reopened

Reverting close by anonymous

comment:8 by Brian Rosner, 19 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 Brian Rosner, 18 years ago

Resolution: → fixed
Status: reopened → closed

This is no longer a problem since the merge of newforms-admin in [7967].

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