search

tdm00 / cfwheels-dbmigrate-plugin

ColdFusion on Wheels DBMigrate plugin fork
https://github.com/tdm00/cfwheels-dbmigrate-plugin/wiki
21 stars 19 forks source link

Postgres - QuoteTableName #43

Open malpaso opened 12 years ago

malpaso commented 12 years ago

The 'quoteTableName' function (Abstract.cfc) needs to be moved to the 'PostgreSQL' adapter as Postgres requires double quotes.

jordanclark commented 11 years ago

My fork/pull request should fix this issue, apparently if you quote PostgreSQL table names they become case-sensitive, so now the adapter will check if the name needs the double quotes before adding them.

tdm00 commented 11 years ago

Jordan,

Go ahead and include these issue numbers in your commit messages and when I accept the pull-request they will automatically be closed. For more info see https://github.com/blog/1386-closing-issues-via-commit-messages

Go ahead and update the README.md with your name as a contributor and the CHANGELOG.md with your list of changes. This will be for version 1.2.0.

On Tue, Apr 2, 2013 at 1:50 PM, jordanclark notifications@github.comwrote:

My fork/pull request should fix this issue, apparently if you quote PostgreSQL table names they become case-sensitive, so now the adapter will check if the name needs the double quotes before adding them.

— Reply to this email directly or view it on GitHubhttps://github.com/tdm00/cfwheels-dbmigrate-plugin/issues/43#issuecomment-15791116 .

Troy Murray