[Django] #33195: Class-based urlize()

9 views
Skip to first unread message

Django

unread,
Oct 14, 2021, 1:33:05 PM10/14/21
to django-...@googlegroups.com
#33195: Class-based urlize()
------------------------------------------------+------------------------
Reporter: Claude Paroz | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Utilities | Version: dev
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 |
------------------------------------------------+------------------------
I had a use case for using a customized urlize() utility and was annoyed
by monolithic implementation of the Django urlize() utility.

I'm suggesting a class-based implementation though a following patch.

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

Django

unread,
Oct 14, 2021, 2:24:01 PM10/14/21
to django-...@googlegroups.com
#33195: Class-based urlize()
-------------------------------------+-------------------------------------

Reporter: Claude Paroz | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Utilities | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Claude Paroz):

* has_patch: 0 => 1


Comment:

[https://github.com/django/django/pull/14993 PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/33195#comment:1>

Django

unread,
Oct 15, 2021, 12:57:33 AM10/15/21
to django-...@googlegroups.com
#33195: Class-based urlize()
-------------------------------------+-------------------------------------
Reporter: Claude Paroz | Owner: Claude
Type: | Paroz
Cleanup/optimization | Status: assigned
Component: Utilities | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* owner: nobody => Claude Paroz
* status: new => assigned
* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/33195#comment:2>

Django

unread,
Oct 15, 2021, 3:21:36 AM10/15/21
to django-...@googlegroups.com
#33195: Class-based urlize()
-------------------------------------+-------------------------------------
Reporter: Claude Paroz | Owner: Claude
Type: | Paroz
Cleanup/optimization | Status: assigned
Component: Utilities | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/33195#comment:3>

Django

unread,
Oct 15, 2021, 3:56:30 AM10/15/21
to django-...@googlegroups.com
#33195: Class-based urlize()
-------------------------------------+-------------------------------------
Reporter: Claude Paroz | Owner: Claude
Type: | Paroz
Cleanup/optimization | Status: closed
Component: Utilities | Version: dev
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak <felisiak.mariusz@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"e567670b1abe61af4acfaa6a6a7e92a7acfa8b00" e567670b]:
{{{
#!CommitTicketReference repository=""
revision="e567670b1abe61af4acfaa6a6a7e92a7acfa8b00"
Fixed #33195 -- Refactored urlize() based on a class.

This allows easier customization/
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/33195#comment:4>

Reply all
Reply to author
Forward
0 new messages