Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#12111 closed (fixed)

contrib.gis - postgis adaptor spurious warnings about geometry escaping in SQL

Reported by: mattrussell Owned by: nobody
Component: GIS Version: 1.1
Severity: Keywords: quoting escape geometry sql
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

When queries which include a geometry are run, PostgreSQL/postgis complains about the escaping of the geometry WKB string in the query:

WARNING: nonstandard use of
in a string literal at character 144

HINT: Use the escape string syntax for backslashes, e.g., E'
'.

LOG: statement: SELECT COUNT(*) FROM "routes" WHERE ST_Within("routes"."start_point", ST_GeomFromWKB('
001
003
000
000
000
001
000
000
000
005
000
000
000
217V
321
232&
311
031
300S)
324


360
020J@
217V
321
232&
311
031
300@z
330=
021pK@@LQ
312
262
370
003@@z
330=
021pK@@LQ
312
262
370
003@S)
324


360
020J@
217V
321
232&
311
031
300S)
324


360
020J@', 4326)))

WARNING: nonstandard use of
in a string literal at character 144

HINT: Use the escape string syntax for backslashes, e.g., E'
'.

This is a very small patch to get rid of this warning.

SELECT version();

PostgreSQL 8.4.0 on i686-redhat-linux-gnu, compiled by GCC gcc (GCC) 4.1.2 20080704 (Red Hat 4.1.2-44), 32-bit

SELECT postgis_full_version();

POSTGIS="1.3.6" GEOS="3.0.0-CAPI-1.4.1" USE_STATS

This should be future proof, see the note here, in the PostgreSQL 8.1 manual.

Attachments (1)

django-contrib-gis-db-backend-postgis-adaptor.patch (610 bytes) - added by mattrussell 6 years ago.
Patch which gets rid of warnings from PostgreSQL/postgis for escaping geometries

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by mattrussell

Patch which gets rid of warnings from PostgreSQL/postgis for escaping geometries

comment:1 Changed 6 years ago by mattrussell

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Summary changed from contrig.gis - postgis adaptor spurious warnings about geometry escaping in SQL to contrib.gis - postgis adaptor spurious warnings about geometry escaping in SQL

comment:2 Changed 6 years ago by Alex

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

Fixed in r11726.

comment:3 Changed 6 years ago by jbronn

This patch will not be backported to 1.1.X because it will break compatibility with PostgreSQL versions prior to 8.1 (and compatibility was ensured for versions "8.X" in docs). Users on versions 8.0 and below are recommended to set escape_string_warning = off in their settings.

comment:4 Changed 6 years ago by jbronn

And by 'settings' I mean postgresql.conf.

comment:5 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

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