You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to django-d...@googlegroups.com
Hello,
what do You think about response_change method in django/contrib/options.py (master)? Maybe this place must be refactored? The code here is pretty much the same, so we can create one method and run each if/elif through it.
Jelena
Tim Graham
unread,
Oct 3, 2014, 7:27:03 PM10/3/14
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to django-d...@googlegroups.com
Thanks for the suggestion, Jelena. I'd be happy to look at a patch, although my initial instinct is that while there is some duplication, adding a method might actually created unneeded complexity and make things somewhat less readable. Feel free to send a pull request if you take a stab at it and think that it's an improvement.