Skip to content

Commit ec7c865

Browse files
committed
[BUGFIX] [PostgreSQL] Properly take into account 'name' type
1 parent 36694c1 commit ec7c865

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

src/providers/postgres/qgspostgresprovider.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1286,7 +1286,7 @@ bool QgsPostgresProvider::loadFields()
12861286
// PG 12 returns "name" type for some system table fields (e.g. information_schema.tables)
12871287
else if ( fieldTypeName == QLatin1String( "name" ) )
12881288
{
1289-
fieldSubType = QVariant::String;
1289+
fieldType = QVariant::String;
12901290
fieldSize = 63;
12911291
}
12921292
else

tests/src/python/test_provider_postgres.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3355,6 +3355,15 @@ def testPktLowerCase(self):
33553355
cur.execute(sql_pk)
33563356
self.assertEqual(cur.fetchall(), [('dep', 'character varying')])
33573357

3358+
def testNameType(self):
3359+
3360+
vl = QgsVectorLayer(self.dbconn + ' sslmode=disable key=\'"table_catalog","table_schema","table_name"\' table="information_schema"."tables" () sql=', 'test', 'postgres')
3361+
self.assertTrue(vl.isValid())
3362+
3363+
feat = next(vl.getFeatures())
3364+
self.assertTrue(feat.isValid())
3365+
self.assertIsNotNone(feat.attributes()[0])
3366+
33583367

33593368
class TestPyQgsPostgresProviderCompoundKey(QgisTestCase, ProviderTestCase):
33603369

0 commit comments

Comments
 (0)