Use sqlite3.Row factory in Flaskr (#588) #625

Merged
merged 1 commit into from Jan 25, 2013

Projects

None yet

2 participants

Contributor

As pointed out in issue #588 sqlite3.Row should be used instead of using casting to dict(). Also altered the "Easy Querying" Patterns example to include the more correct way to return rows as dicts. Did not touch Tutorial examples ("Views"), as these are not up to date with the current Flaskr code, and the "Show Entries" section points out the "Easy Querying" section on how to convert to a dict().

@soulseekah soulseekah Use sqlite3.Row factory in Flaskr
As pointed out in issue #588 sqlite3.Row should be used instead of
using casting to dict(). Also altered the "Easy Querying" Patterns
example to include the more correct way to return rows as dicts.
Did not touch Tutorial examples ("Views"), as these are not up to
date with the current Flaskr code, and the "Show Entries" section
points out the "Easy Querying" section on how to convert to a
dict().
82b29c0
Contributor

thanks!

@kennethreitz kennethreitz merged commit 3fdcefb into pallets:master Jan 25, 2013

1 check passed

default The Travis build passed
Details
@DasIch DasIch referenced this pull request Jun 19, 2013
Closed

sqlite dict cursors #588

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment