[BUGFIX] [PostgreSQL] Properly take into account 'name' type

qgis · Oct 21, 2024 · e2a0e3e · e2a0e3e
1 parent 36694c1
commit e2a0e3e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
@@ -1286,7 +1286,7 @@ bool QgsPostgresProvider::loadFields()
       // PG 12 returns "name" type for some system table fields (e.g. information_schema.tables)
       else if ( fieldTypeName == QLatin1String( "name" ) )
       {
-        fieldSubType = QVariant::String;
+        fieldType = QVariant::String;
         fieldSize = 63;
       }
       else

diff --git a/tests/src/python/test_provider_postgres.py b/tests/src/python/test_provider_postgres.py
@@ -3355,6 +3355,15 @@ def testPktLowerCase(self):
         cur.execute(sql_pk)
         self.assertEqual(cur.fetchall(), [('dep', 'character varying')])
 
+    def testNameType(self):
+
+        vl = QgsVectorLayer(self.dbconn + ' sslmode=disable key=\'"table_catalog","table_schema","table_name"\' table="information_schema"."tables" () sql=', 'test', 'postgres')
+        self.assertTrue(vl.isValid())
+
+        feat = next(vl.getFeatures())
+        self.assertTrue(feat.isValid())
+        self.assertIsNotNone(feat.attributes()[0])
+
 
 class TestPyQgsPostgresProviderCompoundKey(QgisTestCase, ProviderTestCase):