Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#16293 closed New feature (fixed)

Patch database documentation to explain how to return fetchall() as a dict rather than list.

Reported by: Cal Leeming Owned by: nobody
Component: Documentation Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Full discussion about this topic can be found in:

http://groups.google.com/group/django-developers/browse_thread/thread/72c91698cba2ea5b

So far we have +2 votes and -1 vote.

-1 vote:

The thing is, this is a DB API snippet, not a Django snippet 
specifically.  If Django were a DB API toolbox, then it might make 
sense to include it in some form or other.  But it's not, so in the 
interest of keeping things relatively tidy I'm a -0 on this. 

+2 vote:

It is often said here that Django ORM is designed to do 80% of the 
stuff, the rest can be done using raw SQL. So, giving pointers to 
users how to perform the raw SQL as painlessly as possible is 
something Django documentation should do. 

Could we get some more feedback on this??

Cal

Attachments (1)

16293-1.patch (1.2 KB ) - added by Cal Leeming 13 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 by Brillgen Developers, 13 years ago

+1 - I don't personally have a need for this particular case yet, however I would support having more documentation as long as it met the quality and style guidelines of the project.

Django does provide and mention a way of getting raw db access, this documentation makes sense to put there. It couldn't really hurt.

comment:2 by Jacob, 13 years ago

Triage Stage: UnreviewedAccepted

There's almost never a reason not to add something to the docs as long as its done in a sensible way. Cal, can you work up a patch with some suggested language and find a good place to work it in?

comment:3 by Jacob, 13 years ago

Type: UncategorizedNew feature

by Cal Leeming, 13 years ago

Attachment: 16293-1.patch added

comment:4 by Cal Leeming, 13 years ago

Has patch: set
Needs tests: set

Patched, could someone please verify if this is suitable?

comment:5 by James Bennett, 13 years ago

Resolution: fixed
Status: newclosed

In [16808]:

Fixed #16293: Document a way to return dicts with column names from a DB cursor.

comment:6 by James Bennett, 13 years ago

In [16809]:

[1.3.X] Fixed #16293: Document a way to return dicts with column names from a DB cursor. Backport of [16808] from trunk.

Note: See TracTickets for help on using tickets.
Back to Top