Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
-+-
 Reporter:  treyh|Owner:  nobody
 Type:  Bug  |   Status:  closed
Component:  Database layer   |  Version:  1.2
  (models, ORM)  |   Resolution:  duplicate
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  1
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by russellm):

 * status:  new => closed
 * resolution:   => duplicate


Comment:

 Closing this one as a duplicate of #17541. It doesn't really matter which
 one we leave open, as long as there's only one ticket per problem.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
-+-
 Reporter:  treyh|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.2
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  1
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by stavros):

 As I said on the other ticket, please upgrade the severity of this. It has
 just caused entirely sane-looking code to delete over half our database
 rows. This needs to be fixed ASAP, the patch has been submitted over two
 years ago.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
-+-
 Reporter:  treyh|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.2
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  1
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by lukeplant):

 * ui_ux:   => 0


Comment:

 Ticket #17541 is a duplicate. I don't know which one to close at this
 point.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
-+-
   Reporter:  treyh  |  Owner:  nobody
   Type:  Bug| Status:  new
  Milestone: |  Component:  Database layer
Version:  1.2|  (models, ORM)
 Resolution: |   Severity:  Normal
   Triage Stage:  Accepted   |   Keywords:
Needs documentation:  0  |  Has patch:  1
Patch needs improvement:  1  |Needs tests:  0
 |  Easy pickings:  0
-+-
Changes (by elbarto):

 * owner:  elbarto => nobody
 * easy:   => 0


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
-+-
   Reporter:  treyh  |Owner:  elbarto
   Type:  Bug|   Status:  new
  Milestone: |Component:  Database layer
Version:  1.2|  (models, ORM)
 Resolution: | Severity:  Normal
   Triage Stage:  Accepted   | Keywords:
Needs documentation:  0  |Has patch:  1
Patch needs improvement:  1  |  Needs tests:  0
-+-
Changes (by julien):

 * needs_better_patch:  0 => 1


Comment:

 Could you combine the fix and tests into one single diff patch?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
---+
  Reporter:  treyh | Owner:  elbarto
Status:  new   | Milestone: 
 Component:  Database layer (models, ORM)  |   Version:  1.2
Resolution:|  Keywords: 
 Stage:  Accepted  | Has_patch:  1  
Needs_docs:  0 |   Needs_tests:  0  
Needs_better_patch:  0 |  
---+
Changes (by elbarto):

  * has_patch:  0 => 1

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
---+
  Reporter:  treyh | Owner:  elbarto
Status:  new   | Milestone: 
 Component:  Database layer (models, ORM)  |   Version:  1.2
Resolution:|  Keywords: 
 Stage:  Accepted  | Has_patch:  0  
Needs_docs:  0 |   Needs_tests:  0  
Needs_better_patch:  0 |  
---+
Changes (by elbarto):

  * owner:  nobody => elbarto

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
---+
  Reporter:  treyh | Owner:  nobody
Status:  new   | Milestone:
 Component:  Database layer (models, ORM)  |   Version:  1.2   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Changes (by russellm):

  * component:  Uncategorized => Database layer (models, ORM)
  * stage:  Unreviewed => Accepted

Comment:

 This is an edge case that exists when an object has been created, but not
 saved; at that point, the primary key of the object is NULL, so the
 reverse query does a lookup with that value.

 The fix here will be to prevent related object lookups when pk=None.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #15146: Reverse relations for unsaved objects include objects with NULL foreign key

[Django]

#15146: Reverse relations for unsaved objects include objects with NULL foreign 
key
+---
  Reporter:  treyh  | Owner:  nobody
Status:  new| Milestone:
 Component:  Uncategorized  |   Version:  1.2   
Resolution: |  Keywords:
 Stage:  Unreviewed | Has_patch:  0 
Needs_docs:  0  |   Needs_tests:  0 
Needs_better_patch:  0  |  
+---
Changes (by treyh):

  * summary:  Reverse relations include objects with NULL foreign key =>
  Reverse relations for unsaved objects include
  objects with NULL foreign key

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.