Skip to content

Commit edaae68

Browse files
authored
Merge pull request #347 from ataft/patch-3
Fix hard-coded "pkName" column in queries
2 parents c62fa58 + a9d1bcc commit edaae68

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

lib/discovery.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,10 @@ function mixinDiscovery(PostgreSQL) {
263263
var sql =
264264
'SELECT tc.table_schema AS "fkOwner", tc.constraint_name AS "fkName", tc.table_name AS "fkTableName",'
265265
+ ' kcu.column_name AS "fkColumnName", kcu.ordinal_position AS "keySeq",'
266-
+ ' ccu.table_schema AS "pkOwner", \'PK\' AS "pkName", '
266+
+ ' ccu.table_schema AS "pkOwner",'
267+
+ ' (SELECT constraint_name'
268+
+ ' FROM information_schema.table_constraints tc2'
269+
+ ' WHERE tc2.constraint_type = \'PRIMARY KEY\' and tc2.table_name=ccu.table_name limit 1) AS "pkName",'
267270
+ ' ccu.table_name AS "pkTableName", ccu.column_name AS "pkColumnName"'
268271
+ ' FROM information_schema.table_constraints tc'
269272
+ ' JOIN information_schema.key_column_usage AS kcu'
@@ -297,8 +300,10 @@ function mixinDiscovery(PostgreSQL) {
297300
PostgreSQL.prototype.buildQueryExportedForeignKeys = function(owner, table) {
298301
var sql = 'SELECT kcu.constraint_name AS "fkName", kcu.table_schema AS "fkOwner", kcu.table_name AS "fkTableName",'
299302
+ ' kcu.column_name AS "fkColumnName", kcu.ordinal_position AS "keySeq",'
300-
+ ' \'PK\' AS "pkName", ccu.table_schema AS "pkOwner",'
301-
+ ' ccu.table_name AS "pkTableName", ccu.column_name AS "pkColumnName"'
303+
+ ' (SELECT constraint_name'
304+
+ ' FROM information_schema.table_constraints tc2'
305+
+ ' WHERE tc2.constraint_type = \'PRIMARY KEY\' and tc2.table_name=ccu.table_name limit 1) AS "pkName",'
306+
+ ' ccu.table_schema AS "pkOwner", ccu.table_name AS "pkTableName", ccu.column_name AS "pkColumnName"'
302307
+ ' FROM'
303308
+ ' information_schema.constraint_column_usage ccu'
304309
+ ' JOIN information_schema.key_column_usage kcu'

0 commit comments

Comments
 (0)