Opened 7 years ago

Closed 7 years ago

#9633 closed (wontfix)

Using adaptor objects as query parameters with sqlite3

Reported by: mdh Owned by: nobody
Component: Database layer (models, ORM) Version: 1.0
Severity: Keywords:
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


And now for my final stumbling block within the Django core on the road to using GeoDjango with SQLite:

It is handy to be able to pass an adaptor object as a query parameter that will ultimately end up in the hands of cursor.execute(), which can convert the adaptor to a string to insert into the SQL. This is explicitly supported in (at least) the postgresql back-end, where it’s handled by [source:django/trunk/django/db/backends/postgresql/ UnicodeCursorWrapper.format_params()]. GeoDjango currently takes advantage of this when it uses [source:django/trunk/django/contrib/gis/db/backend/postgis/ PostGISAdaptor] to wrap, well, pretty much any geospatial entity en route to PostGIS.

The sqlite3 back-end could easily support something similar, but it doesn’t: if you try the corresponding trick with SQLite, you get to enjoy “InterfaceError: Error binding parameter 0 - probably unsupported type.” There may be a completely different way to address the underlying problem, which I’d certainly be very open to hearing about, but I’m personally inclined towards doing things in the corresponding ways for each back-end as much as possible, so this is the approach I’ve taken for now..

I’ve attached a fairly simple patch that takes what is clearly one of many possible similar approaches to achieving the corresponding thing for sqlite3, as well as quick test to see if your favorite back-end supports query parameter adaptors.

Attachments (2)

9633-adaptor-query-params-sqlite3.diff (2.6 KB) - added by mdh 7 years ago.
A first patch to support adaptor objects as query parameters with sqlite3
9633-adaptor-query-params-sqlite3-v2.diff (2.6 KB) - added by jbronn 7 years ago.
clean up of initial patch

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by mdh

A first patch to support adaptor objects as query parameters with sqlite3

comment:1 Changed 7 years ago by jacob

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Design decision needed

Changed 7 years ago by jbronn

clean up of initial patch

comment:2 Changed 7 years ago by jbronn

  • Resolution set to wontfix
  • Status changed from new to closed

This patch was not necessary for SpatiaLite support. Just had to add a __conform__ method to the [browser:django/trunk/django/contrib/gis/db/backend/spatialite/ SpatiaLite adaptor].

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