#11218 closed (fixed)
typo: forgot to rename an occurrence (2)
| Reported by: | Yuri Baburov | Owned by: | nobody |
|---|---|---|---|
| Component: | Database layer (models, ORM) | Version: | 1.1-beta |
| Severity: | Keywords: | ||
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
No variable "results" in the function, seems there's no test to check this, but my PyDev Extensions nailed it down
Please check the flow correctness, as I don't know what is going on there.
diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py
index 394e30b..cc8c264 100644
--- a/django/db/models/sql/query.py
+++ b/django/db/models/sql/query.py
@@ -2362,7 +2362,7 @@ class BaseQuery(object):
return cursor
if result_type == SINGLE:
if self.ordering_aliases:
- return cursor.fetchone()[:-len(results.ordering_aliases)]
+ return cursor.fetchone()[:-len(self.ordering_aliases)]
return cursor.fetchone()
# The MULTI case.
Attachments (1)
Change History (6)
by , 16 years ago
| Attachment: | 11218.patch added |
|---|
comment:1 by , 16 years ago
| Has patch: | set |
|---|
comment:2 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:3 by , 16 years ago
comment:4 by , 16 years ago
Note:
See TracTickets
for help on using tickets.
silly typo fix (not verified flow correctness)