Re: [Django] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
---+
  Reporter:  msaelices | Owner:  msaelices
Status:  closed| Milestone:   
 Component:  django.contrib.admin  |   Version:  1.2  
Resolution:  wontfix   |  Keywords:   
 Stage:  Unreviewed| Has_patch:  0
Needs_docs:  0 |   Needs_tests:  0
Needs_better_patch:  0 |  
---+
Changes (by russellm):

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

Comment:

 I'm with Alex - I don't see the use case here. THe list of objects to be
 deleted isn't a matter of debate or control; it's a direct consequence of
 the delete cascading rules.

 If you want to advocate for this change, please start a discussion on
 django-dev.

-- 
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] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
---+
  Reporter:  msaelices | Owner:  msaelices
Status:  new   | Milestone:   
 Component:  django.contrib.admin  |   Version:  1.2  
Resolution:|  Keywords:   
 Stage:  Unreviewed| Has_patch:  0
Needs_docs:  0 |   Needs_tests:  0
Needs_better_patch:  0 |  
---+
Comment (by Alex):

 Why?  What's the usecase here.

-- 
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] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
---+
  Reporter:  msaelices | Owner:  msaelices
Status:  new   | Milestone:   
 Component:  django.contrib.admin  |   Version:  1.2  
Resolution:|  Keywords:   
 Stage:  Unreviewed| Has_patch:  0
Needs_docs:  0 |   Needs_tests:  0
Needs_better_patch:  0 |  
---+
Changes (by msaelices):

  * owner:  nobody => msaelices

-- 
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] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
---+
  Reporter:  msaelices | Owner:  nobody
Status:  new   | Milestone:
 Component:  django.contrib.admin  |   Version:  1.2   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Comment (by msaelices):

 Note that the only lines I've changed in patch was this:
 {{{
 #!python
 def get_deleted_objects(objs, opts, user, admin_site, using):
 """ ... """
 collector = NestedObjects(using=using)
 }}}

 To this:
 {{{
 #!python

 def get_deleted_objects(objs, opts, user, admin_site, using,
 collector_class=NestedObjects):
 """ ... """
 collector = collector_class(using=using)
 }}}

 The other changes (marked in green and red in patch) are due because I
 have to move up the {{{NestedObjects}}} implementation to be accesible by
 the {{{get_deleted_objects}}} function.

-- 
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] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
---+
  Reporter:  msaelices | Owner:  nobody
Status:  new   | Milestone:
 Component:  django.contrib.admin  |   Version:  1.2   
Resolution:|  Keywords:
 Stage:  Unreviewed| Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Changes (by msaelices):

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

Comment:

 With this patch you can do something like this:
 {{{
 #!python

 class FooModelAdmin(admin.ModelAdmin):
 # ...
 def delete_view(self, request, object_id, extra_context=None):
 # ...

 (deleted_objects, perms_needed, protected) = get_deleted_objects(
 [obj], opts, request.user, self.admin_site, using,
 collector_class=CustomNestedObjects)

 # ...
 }}}

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

[Django] #15140: Allow redefining object to delete collector in Django admin

2011-01-21 Thread Django 
#15140: Allow redefining object to delete collector in Django admin
--+-
 Reporter:  msaelices |   Owner:  nobody
   Status:  new   |   Milestone:
Component:  django.contrib.admin  | Version:  1.2   
 Keywords:|   Stage:  Unreviewed
Has_patch:  0 |  
--+-
 {{{ModelAdmin}}} should be able to redefining easily the objects to delete
 collector.

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