cfwheels-dbmigrate-plugin icon indicating copy to clipboard operation
cfwheels-dbmigrate-plugin copied to clipboard

Published 20 hours ago • verified publisher icon tdm00

Postgres - QuoteTableName

Open malpaso opened this issue 13 years ago • 2 comments

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

malpaso avatar Jan 01 '12 05:01 malpaso

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.

jordanclark avatar Apr 02 '13 17:04 jordanclark

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 [email protected]:

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

tdm00 avatar Apr 02 '13 21:04 tdm00