Re: [Django] #34286: Class based views cause 404 on :view: directive.

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
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
-------------------------------------+-------------------------------------
Changes (by Rahmat Faisal):

* owner: nobody => Rahmat Faisal
* status: new => assigned

Comment:

gonna look

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

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
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 Rahmat Faisal):

Replying to [comment:2 Rahmat Faisal]:
> gonna look

Yeah if u have views for example

{{{
class About(Views):
...
}}}

and in the docs you write
{{{
:view:`app_label.views.About`
}}}

gonna change to app_label.views.about and return 404

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

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
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 Rahmat Faisal):

the problem from admindocs utils function
https://github.com/django/django/blob/main/django/contrib/admindocs/utils.py#L59,
that makes convert into lowercase

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

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
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 Rahmat Faisal):

* has_patch: 0 => 1

Comment:

patch this PR https://github.com/django/django/pull/16517

--
Ticket URL: <https://code.djangoproject.com/ticket/34286#comment:5>

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Mariusz Felisiak):

* needs_tests: 0 => 1

--
Ticket URL: <https://code.djangoproject.com/ticket/34286#comment:6>

[Django]

#34286: Class based views cause 404 on :view: directive.
-------------------------------------+-------------------------------------
Reporter: Alex Life | Owner: Rahmat
| Faisal
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: 4.1
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 1 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Rahmat Faisal):

add simple test case for :view:

--
Ticket URL: <https://code.djangoproject.com/ticket/34286#comment:7>