Skip Menu |

This queue is for tickets about the SQL-Interp CPAN distribution.

Report information
The Basics
Id: 31488
Status: open
Priority: 0/
Queue: SQL-Interp
People
Owner: Nobody in particular
Requestors: MARKSTOS [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.05
Fixed in: (no value)

History Show all quoted text
  Thu Dec 13 10:57:50 2007 MARKSTOS [...] cpan.org - Ticket created
Subject: column names with spaces in them not handled.
Inserting a column with a name of "Job Title" fails because the column name is not quoted. We should confirm that the SQL standard is to use double quotes around the column name, and then start doing that in all cases. Mark
  Thu Dec 13 11:32:27 2007 MARKSTOS [...] cpan.org - Status changed from 'new' to 'open'
  Thu Dec 13 11:34:01 2007 MARKSTOS [...] cpan.org - Correspondence added
On Thu Dec 13 10:57:50 2007, MARKSTOS wrote: Show quoted text
> Inserting a column with a name of "Job Title" fails because the column > name is not quoted. > > We should confirm that the SQL standard is to use double quotes around > the column name, and then start doing that in all cases.
I found that "Foo" is not the same as Foo in PostgreSQL, which becomes lower-cased ( "foo" ). I think the approach we should take here is adopt the "quote_char" and "name_sep" options from SQL::Abstract, which makes quoting column names an option, so the feature is available, and backwards compatibility is maintained.