[Django] #33063: Use string module instead of writing all of characters

8 views

Skip to first unread message

Django

unread,

Aug 27, 2021, 1:46:14 AM8/27/21

to django-...@googlegroups.com

#33063: Use string module instead of writing all of characters
------------------------------------------------+------------------------
Reporter: sammiee5311 | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Utilities | Version: 3.2
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------------------+------------------------
From

{{{
#!div style="font-size: 80%"
Code highlighting:
{{{#!python
RANDOM_STRING_CHARS =
'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789'
}}}
}}}

To
{{{
#!div style="font-size: 80%"
Code highlighting:
{{{#!python
RANDOM_STRING_CHARS = string.ascii_letters + string.digits
}}}
}}}

It looks more clean with string module than writing all characters.

--
Ticket URL: <https://code.djangoproject.com/ticket/33063>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,

Aug 27, 2021, 2:13:16 AM8/27/21

to django-...@googlegroups.com

#33063: Use string module instead of writing all of characters

-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

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

Comment:

The length of alphabet is important in few places, that's why I would
prefer the current form. I don't think it's worth changing.