Re: [Django] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
---+
  Reporter:  rui.coe...@gmail.com  | Owner:  nobody
Status:  closed| Milestone:
 Component:  Uncategorized |   Version:  1.1   
Resolution:  duplicate |  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  1 |  
---+
Changes (by russellm):

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

Comment:

 Closing as a duplicate of #6191.

-- 
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-upda...@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] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
---+
  Reporter:  rui.coe...@gmail.com  | Owner:  nobody
Status:  new   | Milestone:
 Component:  Uncategorized |   Version:  1.1   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  1 |  
---+
Comment (by carljm):

 Replying to [comment:2 Alex]:
 > This patch a) still has the mutable default value, and b) includes
 django.contrib.contenttypes in the ORM, we want to work to reducing that
 dependency, not increasing it.

 Unfortunately the ORM already depends on contrib.contenttypes in at least
 one other place:
 source:/django/trunk/django/db/models/sql/subqueries.py#29 - agree that we
 should be looking to get rid of that, not add more dependency.

 This patch would be eclipsed by the patch on #6191, which replaces the
 admin's related-objects-finding with delegation to the actual ORM code.
 I'm looking at what it would take to fix this as part of #6191.

-- 
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-upda...@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] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
---+
  Reporter:  rui.coe...@gmail.com  | Owner:  nobody
Status:  new   | Milestone:
 Component:  Uncategorized |   Version:  1.1   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  1 |  
---+
Comment (by anonymous):

 Replying to [comment:2 Alex]:
 > This patch a) still has the mutable default value, and b) includes
 django.contrib.contenttypes in the ORM, we want to work to reducing that
 dependency, not increasing it.

 a) Which mutable default value are you talking about, is it "objs_seen",
 if yes, you suggest to start this value with what and why ??

 b) Humm, ok, i will see what i can do to avoid that. If you have a
 suggestion i would appreciate.

-- 
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-upda...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=.

Re: [Django] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
---+
  Reporter:  rui.coe...@gmail.com  | Owner:  nobody
Status:  new   | Milestone:
 Component:  Uncategorized |   Version:  1.1   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  1 |  
---+
Comment (by Alex):

 This patch a) still has the mutable default value, and b) includes
 django.contrib.contenttypes in the ORM, we want to work to reducing that
 dependency, not increasing it.

-- 
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-upda...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=.

Re: [Django] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
---+
  Reporter:  rui.coe...@gmail.com  | Owner:  nobody
Status:  new   | Milestone:
 Component:  Uncategorized |   Version:  1.1   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  1 |  
---+
Changes (by Alex):

  * needs_better_patch:  => 1
  * needs_tests:  => 0
  * needs_docs:  => 0

Comment:

 This patch uses a mutable default value which is bad practice and will
 cause it to fail when called more than once.

-- 
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-upda...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=.

[Django] #12025: Show generic relation on deleted objects

[Django]

#12025: Show generic relation on deleted objects
--+-
 Reporter:  rui.coe...@gmail.com  |   Owner:  nobody
   Status:  new   |   Milestone:
Component:  Uncategorized | Version:  1.1   
 Keywords:|   Stage:  Unreviewed
Has_patch:  0 |  
--+-
 Generic foreign key relations are deleted but not shown in confirmation
 page.

-- 
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
-~--~~~~--~~--~--~---