Re: [Django] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  gabrielhurley
Status:  assigned  | Milestone:  1.2  
 Component:  django.contrib.admin  |   Version:  SVN  
Resolution:|  Keywords:   
 Stage:  Accepted  | Has_patch:  1
Needs_docs:  0 |   Needs_tests:  1
Needs_better_patch:  0 |  
---+
Changes (by kmtracey):

  * needs_tests:  0 => 1
  * stage:  Ready for checkin => Accepted

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  gabrielhurley
Status:  assigned  | Milestone:  1.2  
 Component:  django.contrib.admin  |   Version:  SVN  
Resolution:|  Keywords:   
 Stage:  Ready for checkin | Has_patch:  1
Needs_docs:  0 |   Needs_tests:  0
Needs_better_patch:  0 |  
---+
Changes (by jacob):

  * stage:  Accepted => Ready for checkin

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  gabrielhurley
Status:  assigned  | Milestone:  1.2  
 Component:  django.contrib.admin  |   Version:  SVN  
Resolution:|  Keywords:   
 Stage:  Accepted  | Has_patch:  1
Needs_docs:  0 |   Needs_tests:  0
Needs_better_patch:  0 |  
---+
Changes (by gabrielhurley):

  * owner:  nobody => gabrielhurley
  * status:  new => assigned
  * has_patch:  0 => 1

Comment:

 Capitalize works fine for international characters as long as the string
 is a unicode string:

 {{{
 >>> print u'ñ'.capitalize()
 Ñ
 }}}

 Even though it's probably overkill for a small patch, I ran some timeit
 tests to see how various solutions to this problem compared:

 {{{
 >>> import timeit
 >>> t1 = """\
 ... name = u'el_ñino_season'
 ... name.capitalize().replace('_', ' ')
 ... """
 >>> timeit.timeit(t1)
 2.1008538794735085
 >>> t2 = """\
 ... name = u'el_ñino_season'
 ... name.upper().replace('_', ' ')
 ... """
 >>> timeit.timeit(t2)
 2.2086846233250319
 >>> t3 = """\
 ... name = u'el_ñino_season'
 ... name = name.upper()
 ... name.replace('_', ' ')
 ... """
 >>> timeit.timeit(t3)
 2.1825877311222515
 >>> t4 = """\
 ... name = u'el_ñino_season'
 ... if not name:
 ... return u''
 ... name.replace('_', ' ').capitalize()
 ... """
 >>> timeit.timeit(t4)
 2.1088863905887484
 }}}

 My patch reflects option 4, which is consistently fast and handles the
 case of name being 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-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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Comment (by yedpodtrzitko):

 You're right, I didn't try it. Sorry.

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Changes (by ales_zoulek):

 * cc: ales.zou...@gmail.com (added)

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Comment (by ales_zoulek):

 yedpodtrzitko: Yep, but neither does 'ř'.upper()

 So no difference there...

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Comment (by yedpodtrzitko):

 AFAIK capitalizce() doesn't work on non-ascii chars:


 {{{
 y...@deflorenc ~/skript/codegolf $ python
 Python 2.6.4 (r264:75706, Feb  2 2010, 11:10:56)
 [GCC 4.4.2] on linux2
 Type "help", "copyright", "credits" or "license" for more information.
 >>> print 'ř'.capitalize()
 ř

 }}}

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Comment (by ales_zoulek):

 Is there a specific reason not to use .capitalize()
 - That means that pretty_name(some_NamE) is translated to "Some NamE" not
 to "Some name"?

-- 
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] #12434: django.contrib.admin does not work with blank short_description

[Django]

#12434: django.contrib.admin does not work with blank short_description
---+
  Reporter:  anonymous | Owner:  nobody
Status:  new   | Milestone:  1.2   
 Component:  django.contrib.admin  |   Version:  SVN   
Resolution:|  Keywords:
 Stage:  Accepted  | Has_patch:  0 
Needs_docs:  0 |   Needs_tests:  0 
Needs_better_patch:  0 |  
---+
Changes (by russellm):

  * needs_better_patch:  => 0
  * needs_docs:  => 0
  * stage:  Unreviewed => Accepted
  * needs_tests:  => 0
  * milestone:  => 1.2

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