Ticket #119 (closed defect: wontfix)

Opened 6 years ago

Last modified 3 years ago

PostGIS missing quotes for columns in queries

Reported by: jbirch Owned by:
Priority: major Milestone: 3.4.0
Component: PostGIS Provider Version: 3.2.2
Severity: 3 Keywords: quoting
Cc: riccardo.vignotto@… External ID:

Description

You can see in this example that the column names are not being quoted properly in the SQL statements to preserve case:


DECLARE crsselcmd_8122f830936479525e7e775d958cd3e1 CURSOR FOR SELECT "public"."testtab".geom,"public"."testtab".geom_name,"public"."testtab".geom_description FROM "public"."testtab"


Change History

Changed 6 years ago by mloskot

  • status changed from new to assigned

I improved a little two functions responsible for quoting SQL names and values (r3318).

Changed 5 years ago by gregboone

  • milestone changed from 3.3.0 to 3.3.1

Changed 5 years ago by mloskot

  • cc riccardo.vignotto@… added
  • keywords quoting added

Recently, Riccardo Vignotto has reported similar problems when using FDO PostGIS provider from GEOMAP GROUP website ( http://www.geomapgis.com/web/geomap-group/fdo/index.html):

I see that if you query like this 'SELECT colUMN FROM TaBLe' postgre convert the query to 'SELECT column FROM table'. But if you use double quotes like this 'SELECT "colUMN" FROM "TaBLe"' it does not lower the case of characters. From log files I see that you don’t use double quotes when querying the db. So this is my suggestion for the next versions of the FDO provider that you develop.

I've asked Riccardo to check if he uses a patched version (r3318) of the provider.

Riccardo, I've added your email to CC of this ticket to keep you notified about news. I hope you don't mind.

Changed 4 years ago by mloskot

  • owner mloskot deleted
  • status changed from assigned to new

Changed 3 years ago by gregboone

  • status changed from new to closed
  • resolution set to wontfix

Further PostGIS Provider development has been routed to the PostgreSQL Provider. Advise using that provider.

Note: See TracTickets for help on using tickets.