Re: Unicode SlugField design decision (#16501)

2014-11-05 Thread Marc Tamlyn
Even if the backend is storing using an ascii backed data type, it should still return unicode data to Django anyway, so I'm not sure there would be a difference. Marc On 6 November 2014 01:18, Michael Manfre wrote: > > > On Wed, Nov 5, 2014 at 7:53 PM, Shai Berger wrote: > >> On Wednesday 05

Re: Unicode SlugField design decision (#16501)

2014-11-05 Thread Michael Manfre
On Wed, Nov 5, 2014 at 7:53 PM, Shai Berger wrote: > On Wednesday 05 November 2014 17:29:20 Michael Manfre wrote: > > We can change the internal type of SlugField for option 1 based upon > > whether or not the field should be unicode. Whether or not an existing > > backend stores SlugField in an

Re: Unicode SlugField design decision (#16501)

2014-11-05 Thread Shai Berger
On Wednesday 05 November 2014 17:29:20 Michael Manfre wrote: > We can change the internal type of SlugField for option 1 based upon > whether or not the field should be unicode. Whether or not an existing > backend stores SlugField in an ascii char datatype shouldn't dictate > whether we go with Sl

Re: Unicode SlugField design decision (#16501)

2014-11-05 Thread Michael Manfre
We can change the internal type of SlugField for option 1 based upon whether or not the field should be unicode. Whether or not an existing backend stores SlugField in an ascii char datatype shouldn't dictate whether we go with SlugField(unicode=True) or UnicodeSlugField(). Let me rephrase the ques

Re: Unicode SlugField design decision (#16501)

2014-11-05 Thread Marc Tamlyn
It feels to me more like an option to SlugField and I feel UnicodeSlugField is a bit ugly. If however we find an example where Michael's point is valid (an external 3rd party backend which uses ascii chars for SlugField now) then we should go with that. On 5 November 2014 03:20, Michael Manfre wr

Re: Unicode SlugField design decision (#16501)

2014-11-04 Thread Michael Manfre
Should the unicode validated field (either options) have a different internal type for the sake of 3rd party database backends? I'm not sure if any currently store SlugFIeld in an ascii char datatype and would need an easy way to differentiate. Regards, Michael Manfre On Tue, Nov 4, 2014 at 8:27

Unicode SlugField design decision (#16501)

2014-11-04 Thread Tim Graham
I wanted to solicit feedback on the two approaches to supporting unicode in SlugField: 1. Added a unicode argument to models.SlugField and forms.SlugField. https://github.com/django/django/pull/1979 2. Added models.UnicodeSlugField and forms.UnicodeSlugField. https://github.com/django/django/pu