#12997 closed (fixed)
Add Metadata Targets to QuerySet API Reference
| Reported by: | Derek Willis | Owned by: | Derek Willis |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | yes |
| Easy pickings: | no | UI/UX: | no |
Description
I've switched titles to targets in the QuerySet API reference, per http://docs.djangoproject.com/en/dev/internals/documentation/#todo
Attachments (3)
Change History (13)
by , 16 years ago
| Attachment: | 12997.patch added |
|---|
comment:1 by , 16 years ago
| Component: | Uncategorized → Documentation |
|---|---|
| Status: | new → assigned |
comment:2 by , 16 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
by , 16 years ago
| Attachment: | 12997-r12842.diff added |
|---|
comment:3 by , 16 years ago
| milestone: | 1.3 |
|---|
Problem is that if we remove the titles, the methods don't appear in the local HTML TOC (shown in the left sidebar) and so we loose the useful ability of easily jumping to an arbitrary method section.
I modified the patch to have both the title and new crossref metatdata markup, this makes the method headers look a bit ugly (content generated by the new crossref metatdata markup appear below the titles and is identical) but oh well, it seems we have two requirements (easy navigation, desirability of more metadata markup) whose current results when pursued together aren't ideal in this case.
Removing milestone = 1.3. Justin committed some related changes in r12808 so IMHO there is hope we can get this in 1.2.
by , 16 years ago
| Attachment: | 12997-1.1.X-r13132.diff added |
|---|
Patch for 1.1.X branch updated to apply cleanly as of now
comment:4 by , 16 years ago
| milestone: | → 1.2 |
|---|---|
| Triage Stage: | Accepted → Ready for checkin |
comment:5 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:6 by , 16 years ago
comment:7 by , 15 years ago
| Patch needs improvement: | set |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
| Triage Stage: | Ready for checkin → Accepted |
Looks like these methods are appearing in the django.db.models namespace rather than django.db.models.QuerySet. We need another currentmodule declaration (django.db.models.QuerySet) or else each target needs "QuerySet." prefix:
.. method:: QuerySet.filter(**kwargs)
The patch in #11800 does the latter. With the current setup, a link would look like "django.db.models.defer" whereas I think they should look like "django.db.models.QuerySet.defer".
comment:8 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Patch updated wiht some missed methods, updated to r12842 and modified to no remove the title markup.