[Django] #24125: TemplateDetailView in admindocs requires a single Django templates engine

13 views
Skip to first unread message

Django

unread,
Jan 11, 2015, 1:12:42 PM1/11/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: | Version: master
contrib.admindocs |
Severity: Release | Keywords: multiple-template-
blocker | engines
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
`TemplateDetailView.get_context_data()` calls `Engine.get_default()` which
raises `ImproperlyConfigured` if zero or several Django templates engines
are configured.

This limitation should either be lifted or documented. Gracefully showing
an error message would already be better than failing with an uncaught
exception.

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

Django

unread,
Jan 12, 2015, 3:31:19 PM1/12/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: contrib.admindocs | Version: master
Severity: Release blocker | Resolution:
Keywords: multiple-template- | Triage Stage:
engines | Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Aymeric Augustin <aymeric.augustin@…>):

In [changeset:"511a53b3142551a1bc3093ed1b6655f57634f510"]:
{{{
#!CommitTicketReference repository=""
revision="511a53b3142551a1bc3093ed1b6655f57634f510"
Avoided exceptions in admindocs' template detail view.

This is marginally better than crashing when several Django template
engines are configured in a project.

Refs #24125.
}}}

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

Django

unread,
Jan 15, 2015, 10:32:54 AM1/15/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines 1.8 |
Has patch: 0 | Needs documentation: 0

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

* keywords: multiple-template-engines => multiple-template-engines 1.8
* severity: Release blocker => Normal
* stage: Unreviewed => Accepted


Comment:

This view seems pretty obscure and of limited use. I'm going to demote
from a release blocker and tag it "1.8" instead so we make sure to
document the limitation if no one cares to write a patch by then.

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

Django

unread,
Jan 15, 2015, 11:40:56 AM1/15/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines 1.8 |
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by aaugustin):

Yes, since it doesn't crash anymore, it isn't a blocker.

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

Django

unread,
Feb 16, 2015, 10:07:33 AM2/16/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

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

* keywords: multiple-template-engines 1.8 => multiple-template-engines


Comment:

I don't think so many people will hit this issue that it's even worth
documenting.

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

Django

unread,
Feb 16, 2015, 10:11:00 AM2/16/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner: nobody
Type: Bug | Status: new
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by aaugustin):

Fine.

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

Django

unread,
Jun 4, 2015, 6:28:01 AM6/4/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner:
| kairichard
Type: Bug | Status: assigned

Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

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

* owner: nobody => kairichard
* status: new => assigned


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

Django

unread,
Jun 8, 2015, 5:18:41 PM6/8/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner:
| kairichard
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by aaugustin):

As duplicate ticket #24957 shows, at least one person has noticed.

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

Django

unread,
Jun 8, 2015, 9:03:22 PM6/8/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner:
| kairichard
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by timgraham):

It was the person working on this ticket who didn't realize the purpose of
it.

--
Ticket URL: <https://code.djangoproject.com/ticket/24125#comment:8>

Django

unread,
Jun 11, 2015, 8:28:30 PM6/11/15
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: aaugustin | Owner:
| kairichard
Type: Bug | Status: assigned
Component: contrib.admindocs | Version: master
Severity: Normal | Resolution:
Keywords: multiple-template- | Triage Stage: Accepted
engines |
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:"3c593ba79e5bdee46419e2aa6b14e7ce8bc01758" 3c593ba7]:
{{{
#!CommitTicketReference repository=""
revision="3c593ba79e5bdee46419e2aa6b14e7ce8bc01758"
Refs #24125 -- Added admin_docs tests for multiple template engines.

This adds a test for 511a53b3142551a1bc3093ed1b6655f57634f510
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/24125#comment:9>

Django

unread,
Dec 31, 2021, 1:56:34 AM12/31/21
to django-...@googlegroups.com
#24125: TemplateDetailView in admindocs requires a single Django templates engine
-------------------------------------+-------------------------------------
Reporter: Aymeric Augustin | Owner: Kai
| Richard König
Type: Bug | Status: closed
Component: contrib.admindocs | Version: dev
Severity: Normal | Resolution: wontfix

Keywords: multiple-template- | Triage Stage: Accepted
engines |
Has patch: 0 | Needs documentation: 0

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

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


Comment:

Replying to [comment:4 Tim Graham]:


> I don't think so many people will hit this issue that it's even worth
documenting.

I agree with Tim.

--
Ticket URL: <https://code.djangoproject.com/ticket/24125#comment:10>

Reply all
Reply to author
Forward
0 new messages