search

bigpresh / Dancer-Plugin-Database

Dancer::Plugin::Database - easy database support for Dancer applications
http://search.cpan.org/dist/Dancer-Plugin-Database
37 stars 36 forks source link

Table names with schemes #33

Closed mscolly closed 11 years ago

mscolly commented 11 years ago

The quoting of table names is too simplistic. Using quick_select for example, if the tablename is schema.table, then the whole string is quoted. The DB then tries to get a table called "schema.table" instead of "schema"."table" and replies that it doesn't exist.

bigpresh commented 11 years ago

Ah, yes, that's a good point. I guess I'll need to split on /\./, quote each part separately, then concatenate the result.

I'll look into fixing that up ASAP - thanks for the report!

bigpresh commented 11 years ago

This problem should be fixed by 761ab14; testing a little further, then will get a new release out :)

bigpresh commented 11 years ago

Fix will be going out in a dev release for tester feedback shortly, then a new stable release in the next few days.

Sorry again for the wait!