[Django] #27795: Audit force_text invocations

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
22 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Django] #27795: Audit force_text invocations

Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
               Reporter:  Claude     |          Owner:  Claude Paroz
  Paroz                              |
                   Type:             |         Status:  assigned
  Cleanup/optimization               |
              Component:  Utilities  |        Version:  master
               Severity:  Normal     |       Keywords:
           Triage Stage:  Accepted   |      Has patch:  0
    Needs documentation:  0          |    Needs tests:  0
Patch needs improvement:  0          |  Easy pickings:  0
                  UI/UX:  0          |
-------------------------------------+-------------------------------------
 `django.utils.encoding.force_text` is called many times in Django code for
 several use cases:
   - force the evaluation of a lazy string
   - convert bytes or a bytestring (PY2) to str
   - convert an object to its string representation

 A global audit is needed to clean that usage, also considering that Python
 3 native strings are more wildly used.
 I'm working on it currently.

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/050.b5d96b55658873d08573ab6146a54cb2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"ccfd1295f986cdf628d774937d0b38a14584721f" ccfd1295]:
 {{{
 #!CommitTicketReference repository=""
 revision="ccfd1295f986cdf628d774937d0b38a14584721f"
 Refs #27795 -- Prevented SafeText from losing safe status on str()

 This will allow to replace force_text() by str() in several places (as one
 of
 the features of force_text is to keep the safe status).
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.0b41ec8c5bf961a4797b2e011287fe83%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"2757209c9de3764364a8e8af39cefa2445312546" 2757209c]:
 {{{
 #!CommitTicketReference repository=""
 revision="2757209c9de3764364a8e8af39cefa2445312546"
 Refs #27795 -- Removed force_text from templatize function
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.ba3c7c9447cce34847cb9f71e1a032ec%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"3a148f958dddd97c1379081118c30fbede6b6bc4" 3a148f95]:
 {{{
 #!CommitTicketReference repository=""
 revision="3a148f958dddd97c1379081118c30fbede6b6bc4"
 Refs #27795 -- Removed force_text from the template layer

 Thanks Tim Graham for the review.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.7708b99b959d5b93b0b981b8fee68a19%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"8346680e1ca4a8ddc8190baf3f5f944f6418d5cf" 8346680e]:
 {{{
 #!CommitTicketReference repository=""
 revision="8346680e1ca4a8ddc8190baf3f5f944f6418d5cf"
 Refs #27795 -- Removed unneeded force_text calls

 Thanks Tim Graham for the review.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.c685350778fae0d88d5a9c6c0b028c4e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by GitHub <noreply@…>):

 In [changeset:"e5dce7b0fbd2965e524ce97114f501319ec2bb6f" e5dce7b0]:
 {{{
 #!CommitTicketReference repository=""
 revision="e5dce7b0fbd2965e524ce97114f501319ec2bb6f"
 Refs #27795 -- Removed unneeded force_text calls from the Oracle backend.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.3f2b33eb3fafa64e8fac563b26a58ae5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"301de774c21d055e9e5a7073e5bffdb52bc71079" 301de77]:
 {{{
 #!CommitTicketReference repository=""
 revision="301de774c21d055e9e5a7073e5bffdb52bc71079"
 Refs #27795 -- Replaced many force_text() with str()

 Thanks Tim Graham for the review.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.8038edf35820f9e54a8ce3a1b9e8d3ee%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"a8343fe7bf0ec237d8f77ba4e729ae093388b75f" a8343fe7]:
 {{{
 #!CommitTicketReference repository=""
 revision="a8343fe7bf0ec237d8f77ba4e729ae093388b75f"
 Refs #27795 -- Replaced force_text() with str() in feed generators
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.9e359e2a9728623b3e4a9b93ee344fa4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"b39aabc6c6b84cbe4005f3fdf3b9c0c4307afc76" b39aabc]:
 {{{
 #!CommitTicketReference repository=""
 revision="b39aabc6c6b84cbe4005f3fdf3b9c0c4307afc76"
 Refs #27795 -- Reworked get_or_create test erroneously mixing bytes and
 str.

 As CharField.to_python() now always calls str(), assigning bytes to a
 CharField is no longer correct usage. Doing so results in a warning:

   django/db/models/fields/__init__.py:1061: BytesWarning: str() on a bytes
 instance

 Use a unique constraint violation to trigger the database error instead.

 Warning introduced in 301de774c21d055e9e5a7073e5bffdb52bc71079.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.dfee0083022fb3ffdf40136c5f764dd2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"d842ada30524abf410a4de3fc1e539450173800f" d842ada]:
 {{{
 #!CommitTicketReference repository=""
 revision="d842ada30524abf410a4de3fc1e539450173800f"
 Refs #27795 -- Stopped converting integer format settings to str in
 JS/JSON i18n views

 Thanks Tim Graham for the review.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.5c59ced853a182a0e83fd100a0e86bee%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"f87256b15e375c6c57fe425c33ae23e116da3e84" f87256b1]:
 {{{
 #!CommitTicketReference repository=""
 revision="f87256b15e375c6c57fe425c33ae23e116da3e84"
 Refs #27795 -- Replaced force_text() with str() in contrib.messages

 Thanks Tim Graham for the review.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.fd9f3ff12a6ab5d062da90a4f0cfc60b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz <claude@…>):

 In [changeset:"45bbb9aae0e7af6cd16fa4fe410219d4d65f829c" 45bbb9aa]:
 {{{
 #!CommitTicketReference repository=""
 revision="45bbb9aae0e7af6cd16fa4fe410219d4d65f829c"
 Refs #27795 -- Removed force_text() usage in serialization framework
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.5a024603cd2a7a36f3031207b7f102e7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"6b2f8fb91c7adab123156e185db2efe31e41b628" 6b2f8fb9]:
 {{{
 #!CommitTicketReference repository=""
 revision="6b2f8fb91c7adab123156e185db2efe31e41b628"
 Refs #27795 -- Replaced force_text() usage in django.core.management.

 Use decode() since Popen.communicate() always returns bytes.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.1faaa4089037a2dff6bf1a7bca2bd406%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"16c5a334ff3ad9d8b3cd1314562c7af20a2a7c7d" 16c5a334]:
 {{{
 #!CommitTicketReference repository=""
 revision="16c5a334ff3ad9d8b3cd1314562c7af20a2a7c7d"
 Refs #27795 -- Replaced force_text/bytes() with decode()/encode() in
 password hashers.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.e2d644f082365e3d15b5878123be2cf5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Carlton Gibson <carlton.gibson@…>):

 In [changeset:"6a039800db7ce4aa4c1afecd26791c43ddde8920" 6a039800]:
 {{{
 #!CommitTicketReference repository=""
 revision="6a039800db7ce4aa4c1afecd26791c43ddde8920"
 Refs #27795 -- Replaced force_bytes() usage in django.core.cache.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.891967b63cb3579ab2d0568125ad8a7d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"b38532cd6be6ea91b47fd93e9cbaae4aa10015fc" b38532c]:
 {{{
 #!CommitTicketReference repository=""
 revision="b38532cd6be6ea91b47fd93e9cbaae4aa10015fc"
 Refs #27795 -- Replaced force_bytes() usage in django.core.signing.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.65a4ca6d27dbde1ec6e232a249f4b7e3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by GitHub <noreply@…>):

 In [changeset:"fa75b2cb512409116b6f1b5229d6f99074d8e452" fa75b2c]:
 {{{
 #!CommitTicketReference repository=""
 revision="fa75b2cb512409116b6f1b5229d6f99074d8e452"
 Refs #27795 -- Removed force_bytes/text() usage in tests.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.1dfe768a5f5e95476051777840f6bdcb%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"7d96f0c49ab750799860e42716d7105e11de44de" 7d96f0c4]:
 {{{
 #!CommitTicketReference repository=""
 revision="7d96f0c49ab750799860e42716d7105e11de44de"
 Refs #27795 -- Replaced force_bytes() usage in django.http.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.ed84f9c45ff6597a42663422bbcaf3e2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Jon Dufresne <jon.dufresne@…>):

 In [changeset:"968329b5c254fc0d77bde78f81b8bcd17655b14e" 968329b]:
 {{{
 #!CommitTicketReference repository=""
 revision="968329b5c254fc0d77bde78f81b8bcd17655b14e"
 Refs #27795 -- Replaced force_text() calls with str() in
 django.utils.html.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.f51a38791891a3070c958b26e8c6668c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27795: Audit force_text invocations

Django
In reply to this post by Django
#27795: Audit force_text invocations
-------------------------------------+-------------------------------------
     Reporter:  Claude Paroz         |                    Owner:  Claude
         Type:                       |  Paroz
  Cleanup/optimization               |                   Status:  assigned
    Component:  Utilities            |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"1f3852b708feb2a3e20bf7a65f6cab9366cf7296" 1f3852b]:
 {{{
 #!CommitTicketReference repository=""
 revision="1f3852b708feb2a3e20bf7a65f6cab9366cf7296"
 Refs #27795 -- Removed force_bytes() usage in
 django/template/loaders/cached.py.
 }}}

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

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/065.21601cc551994c22d7362bf585746742%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
12