Invalid module references in the documentation
This ticket is for David Smith's PR which covers a few cases where the module references are incorrect in the documentation. I created the ticket to help signal to others that this can be picked up.
I used Claude to determine the following errors still need to be fixed. The rest from David's work were merged in other changes.
File: docs/internals/deprecation.txt:945
Current (wrong): django.template.backends.base.Template.render()
Should be: django.template.base.Template.render()
────────────────────────────────────────
File: docs/ref/templates/api.txt:37
Current (wrong): django.template.backends.base.Template.render
Should be: django.template.base.Template.render
────────────────────────────────────────
File: docs/releases/1.10.txt:1320
Current (wrong): django.template.backends.base.Template.render
Should be: django.template.base.Template.render
────────────────────────────────────────
File: docs/topics/templates.txt:369
Current (wrong): django.template.backends.base
Should be: django.template.base
────────────────────────────────────────
File: docs/releases/2.0.txt:283
Current (wrong): django.db.backends.base.DatabaseWrapper.execute_wrapper
Should be: django.db.backends.base.base.BaseDatabaseWrapper.execute_wrapper
────────────────────────────────────────
File: docs/topics/db/instrumentation.txt:99
Current (wrong): django.db.backends.base.DatabaseWrapper
Should be: django.db.backends.base.base.BaseDatabaseWrapper
Change History
(5)
| Patch needs improvement: |
set
|
| Owner: |
changed from Django Sprints to David Smith
|
| Patch needs improvement: |
unset
|
| Triage Stage: |
Unreviewed → Accepted
|
| Triage Stage: |
Accepted → Ready for checkin
|
| Resolution: |
→ fixed
|
| Status: |
assigned → closed
|
Thanks Tim for the ticket, I'm accepting for the
execute_wrapper()reference. While working through the ticket to understand the issue, I rebased PR 16294 onto main and cleaned it up, then pushed the result to the PR, following Tim's 2022 review comments.The cache changes were already merged in c3862735cd. I dropped the template changes: as Tim pointed out in review,
django.template.base.Templateisn't theTemplatereturned by each backend.topics/templates.txtdocuments the backend-agnosticrender(context=None, request=None)protocol, and retargeting it would make the deprecation and 1.10 notes andref/templates/api.txtpoint at the DTL engine'srender(context)instead.