Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
-+-
Reporter: Joseph Metzinger |Owner: Neeraj
| Kumar
Type: Bug | Status: closed
Component: Forms| Version: 4.2
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1| Needs documentation: 0
Needs tests: 0| Patch needs improvement: 0
Easy pickings: 0|UI/UX: 0
-+-
Changes (by Mariusz Felisiak ):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"eed096574fea5c9d82d0dc5952ad439dfde13718" eed0965]:
{{{
#!CommitTicketReference repository=""
revision="eed096574fea5c9d82d0dc5952ad439dfde13718"
Fixed #32210 -- Fixed model inlines with to_field that has a default.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:19>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/010701892f7f6a1f-668f8a27-cbbb-4779-b71c-8bc7f076ddef-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value -+- Reporter: Joseph Metzinger |Owner: Neeraj | Kumar Type: Bug | Status: assigned Component: Forms| Version: 4.2 Severity: Normal | Resolution: Keywords: | Triage Stage: Ready for | checkin Has patch: 1| Needs documentation: 0 Needs tests: 0| Patch needs improvement: 0 Easy pickings: 0|UI/UX: 0 -+- Changes (by Mariusz Felisiak): * needs_better_patch: 1 => 0 * needs_tests: 1 => 0 * stage: Accepted => Ready for checkin -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:18> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/010701892f3780ec-cb07fc61-c9ae-4339-8410-6670b200e34f-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 4.2 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 1 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Mariusz Felisiak): * needs_better_patch: 0 => 1 * needs_tests: 0 => 1 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:17> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/0107018924cc6aed-75eea0d7-2cad-4889-ae32-e85642ecac39-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 4.2 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Natalia Bidart): * needs_tests: 1 => 0 Comment: Resetting the PR flags to ensure the new PR gets picked up in the review queue. -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:16> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070188d3a3aad9-3496ce61-dcd5-44bd-943f-80b8d6b8c1e7-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 4.2 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Neeraj Kumar): * version: 3.1 => 4.2 Comment: Any update about this issue? Added PR related to this issue. This issue still exists in the latest version of Django. -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:15> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070188a510180f-19048e29-940d-405b-96fd-7e98ec961577-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Comment (by Bhuvnesh): [https://github.com/django/django/pull/16950 PR] -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:14> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/0107018892791b70-14e28833-345b-4918-a850-e05b4e328e53-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Carlton Gibson): * needs_tests: 0 => 1 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:13> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070183acd32203-3f7578c2-0a2b-430c-a339-fddf1ae1b336-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Carlton Gibson): * needs_tests: 1 => 0 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:12> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182caee6430-d2b14133-daea-437e-bb28-6b13130e9ad4-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Carlton Gibson): * needs_tests: 0 => 1 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:11> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182ca7859a0-2130bdc8-76c5-4ef7-901d-47e2cd7691cf-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Neeraj Kumar): * needs_better_patch: 1 => 0 * needs_tests: 1 => 0 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:10> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182c99b544b-7e293572-9c3c-47c9-afd6-ee995f9dc9ed-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 1 | Patch needs improvement: 1 Easy pickings: 0 |UI/UX: 0 --+ Changes (by David Wobrock): * cc: David Wobrock (added) * needs_better_patch: 0 => 1 * needs_tests: 0 => 1 -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:9> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182c75ba078-65d227a2-e437-4f9e-b068-27797b9a242d-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Neeraj Kumar): * cc: Neeraj Kumar (added) -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:8> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182c72641cd-b99893b1-c569-4bac-afb4-8fffccb92b92-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: Neeraj Kumar Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 1 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Neeraj Kumar): * has_patch: 0 => 1 Comment: Check PR for this issue. [https://github.com/django/django/pull/15983] -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:7> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182c724069a-f36cfd69-1466-41a5-9aec-2a9b26d37811-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: NeErAj KuMaR Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 0 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by NeErAj KuMaR): * owner: (none) => NeErAj KuMaR -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:6> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182c6f7dd42-23c641a2-056a-4611-a975-0e306b0eef1b-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
--+
Reporter: Joseph Metzinger |Owner: (none)
Type: Bug | Status: assigned
Component: Forms | Version: 3.1
Severity: Normal| Resolution:
Keywords:| Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 |UI/UX: 0
--+
Comment (by NeErAj KuMaR):
The below code set id value **None** when formsets is_valid calling. so
need to stop set id value None when that field is not model's pk as
UUIDField.
{{{
if self.instance._state.adding:
if kwargs.get("to_field") is not None:
to_field =
self.instance._meta.get_field(kwargs["to_field"])
else:
to_field = self.instance._meta.pk
if to_field.has_default():
setattr(self.instance, to_field.attname, None)
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:5>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/01070182c6f6a91e-e5266323-4d62-4b69-8bb1-5295dc74c1c3-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: (none) Type: Bug | Status: assigned Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 0 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by NeErAj KuMaR): * owner: nobody => (none) * status: new => assigned -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:4> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/01070182b74f39da-de27b407-27d2-4448-96b4-49d90e9d47e1-00%40eu-central-1.amazonses.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
--+
Reporter: Joseph Metzinger |Owner: nobody
Type: Bug | Status: new
Component: Forms | Version: 3.1
Severity: Normal| Resolution:
Keywords:| Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 |UI/UX: 0
--+
Comment (by kamilturek):
Looks like it's not directly related to formsets' validation but
instantiating formset's forms.
{{{
(Pdb) new_object.id
UUID('06048350-3ad9-45f5-bca3-d08d795d7231')
(Pdb) formsets[0].forms
[]
(Pdb) new_object.id
(Pdb)
}}}
{{{
(Pdb) new_object.id
UUID('652863a7-ccc8-4a18-9390-6fb77aa4bafa')
(Pdb) formsets[0]._construct_form(0)
(Pdb) new_object.id
(Pdb)
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/067.cde659736b71b14088ee443c5525fad6%40djangoproject.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value --+ Reporter: Joseph Metzinger |Owner: (none) Type: Bug | Status: new Component: Forms | Version: 3.1 Severity: Normal| Resolution: Keywords:| Triage Stage: Accepted Has patch: 0 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 --+ Changes (by Blackeyeforreal): * owner: Blackeyeforreal => (none) * status: assigned => new -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:4> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.8cbfd3de5806737f44007eff36b39535%40djangoproject.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value -+- Reporter: Joseph Metzinger |Owner: | Blackeyeforreal Type: Bug | Status: assigned Component: Forms| Version: 3.1 Severity: Normal | Resolution: Keywords: | Triage Stage: Accepted Has patch: 0| Needs documentation: 0 Needs tests: 0| Patch needs improvement: 0 Easy pickings: 0|UI/UX: 0 -+- Changes (by Blackeyeforreal): * owner: nobody => Blackeyeforreal * status: new => assigned -- Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:3> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.bfcbd18cfc310c836cc1063d24f01ebf%40djangoproject.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
--+
Reporter: Joseph Metzinger |Owner: nobody
Type: Bug | Status: new
Component: Forms | Version: 3.1
Severity: Normal| Resolution:
Keywords:| Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 |UI/UX: 0
--+
Changes (by Mariusz Felisiak):
* component: contrib.admin => Forms
* stage: Unreviewed => Accepted
Comment:
Thanks for the detailed report. I was able to reproduce this issue. It
looks that formsets' validation mutates the main object, because
`new_object.id` is not empty before the
[https://github.com/django/django/blob/fe9c7ded2996364f853c524b4421274717d89d5f/django/contrib/admin/options.py#L1584
all_valid()] call:
{{{
>>> new_object.id
e13fd82c-c3fc-42dc-ac12-577a4412ba51
>>> all_valid(formsets)
True
>>> new_object.id
None
}}}
Reproduced at ead37dfb580136cc27dbd487a1f1ad90c9235d15.
--
Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/067.bbda04fa66a0bfe013b8170f959f5e6f%40djangoproject.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
---+--
Reporter: joetheone |Owner: nobody
Type: Bug| Status: new
Component: contrib.admin | Version: 3.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 |UI/UX: 0
---+--
Description changed by joetheone:
Old description:
> Hello,
>
> I am a long time django user, first time bug reporter, so please let me
> know if I need to do anything else to help get this bug fixed :)
>
> I am using Django 3.1.3 and python 3.8.5 and have cerated a toy project
> to illustrate the bug. I have the following models:
>
> {{{
> class UUIDModel(models.Model):
> pkid = models.BigAutoField(primary_key=True, editable=False)
> id = models.UUIDField(default=uuid.uuid4, editable=False,
> unique=True)
>
> class Meta:
> abstract = True
>
> class Thing(UUIDModel):
> name = models.CharField(max_length=191)
>
> class SubThing(models.Model):
> name = models.CharField(max_length=191)
> thing = models.ForeignKey(
> 'bugapp.Thing',
> to_field='id',
> on_delete = models.CASCADE,
> related_name='subthings',
> )
> }}}
>
> And the following admin.py file:
>
> {{{
> class SubThingInline(admin.StackedInline):
> model = SubThing
>
> @admin.register(Thing)
> class ThingAdmin(admin.ModelAdmin):
> list_display = ('name',)
> ordering = ('pkid',)
> #search_fields = ('name',)
> inlines = (SubThingInline,)
> }}}
>
> When logging into the admin, if you delete all of the entries for
> "subthings", add a name, and save the model, it will work. As soon as you
> try to add a subthing alongside the main Thing, it fails with the
> following exception:
>
> https://dpaste.com/8EU4FF6RW
>
> It shows that the value of "id" in the Thing model is being set to null.
>
> I believe this is a bug in django.
>
> Thanks!
New description:
Hello,
I am a long time django user, first time bug reporter, so please let me
know if I need to do anything else to help get this bug fixed :)
I am using Django 3.1.3 and python 3.8.5 and have cerated a toy project to
illustrate the bug. I have the following models:
{{{
class UUIDModel(models.Model):
pkid = models.BigAutoField(primary_key=True, editable=False)
id = models.UUIDField(default=uuid.uuid4, editable=False, unique=True)
class Meta:
abstract = True
class Thing(UUIDModel):
name = models.CharField(max_length=191)
class SubThing(models.Model):
name = models.CharField(max_length=191)
thing = models.ForeignKey(
'bugapp.Thing',
to_field='id',
on_delete = models.CASCADE,
related_name='subthings',
)
}}}
And the following admin.py file:
{{{
class SubThingInline(admin.StackedInline):
model = SubThing
@admin.register(Thing)
class ThingAdmin(admin.ModelAdmin):
list_display = ('name',)
ordering = ('pkid',)
inlines = (SubThingInline,)
}}}
When logging into the admin, if you delete all of the entries for
"subthings", add a name, and save the model, it will work. As soon as you
try to add a subthing alongside the main Thing, it fails with the
following exception:
https://dpaste.com/8EU4FF6RW
It shows that the value of "id" in the Thing model is being set to null.
I believe this is a bug in django.
Thanks!
--
--
Ticket URL: <https://code.djangoproject.com/ticket/32210#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/067.81d110bb051a125496af0edad0c6b98a%40djangoproject.com.
Re: [Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value ---+-- Reporter: joetheone |Owner: nobody Type: Bug| Status: new Component: contrib.admin | Version: 3.1 Severity: Normal | Resolution: Keywords: | Triage Stage: Unreviewed Has patch: 0 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 |UI/UX: 0 ---+-- Changes (by joetheone): * Attachment "bugdemo.tbz" added. I made a toy project containing all the code to reproduce the error and zipped it. -- Ticket URL: <https://code.djangoproject.com/ticket/32210> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.e5f3e95a1a1eeef1faa8434cbde7d617%40djangoproject.com.
[Django] #32210: Django Admin with Inlines not using UUIDField default value
#32210: Django Admin with Inlines not using UUIDField default value
-+
Reporter: joetheone | Owner: nobody
Type: Bug| Status: new
Component: contrib.admin |Version: 3.1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 |Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-+
Hello,
I am a long time django user, first time bug reporter, so please let me
know if I need to do anything else to help get this bug fixed :)
I am using Django 3.1.3 and python 3.8.5 and have cerated a toy project to
illustrate the bug. I have the following models:
{{{
class UUIDModel(models.Model):
pkid = models.BigAutoField(primary_key=True, editable=False)
id = models.UUIDField(default=uuid.uuid4, editable=False, unique=True)
class Meta:
abstract = True
class Thing(UUIDModel):
name = models.CharField(max_length=191)
class SubThing(models.Model):
name = models.CharField(max_length=191)
thing = models.ForeignKey(
'bugapp.Thing',
to_field='id',
on_delete = models.CASCADE,
related_name='subthings',
)
}}}
And the following admin.py file:
{{{
class SubThingInline(admin.StackedInline):
model = SubThing
@admin.register(Thing)
class ThingAdmin(admin.ModelAdmin):
list_display = ('name',)
ordering = ('pkid',)
#search_fields = ('name',)
inlines = (SubThingInline,)
}}}
When logging into the admin, if you delete all of the entries for
"subthings", add a name, and save the model, it will work. As soon as you
try to add a subthing alongside the main Thing, it fails with the
following exception:
https://dpaste.com/8EU4FF6RW
It shows that the value of "id" in the Thing model is being set to null.
I believe this is a bug in django.
Thanks!
--
Ticket URL: <https://code.djangoproject.com/ticket/32210>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/052.d77be4ae2db9e74d335efb95161fe93b%40djangoproject.com.
