Skip to content

Commit

Permalink
PostgreSQL Provider: Fix M detection
Browse files Browse the repository at this point in the history
Unlike Geometry(Z), GeometryM contains already M suffix.
It's useless to add M suffix here, since it will add a new name
and the geometry type name will be something like GeometryMM.

Fixes qgis#55223
  • Loading branch information
lbartoletti committed Feb 6, 2024
1 parent b6c7889 commit bf10057
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/providers/postgres/qgspostgresconn.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2354,7 +2354,11 @@ void QgsPostgresConn::retrieveLayerTypes( QVector<QgsPostgresLayerProperty *> &l
switch ( zmFlags )
{
case 1:
typeString.append( 'M' );
// Unlike Geometry(Z), GeometryM contains already M suffix.
// It's useless to add M suffix here,
// since it will add a new name and the geometry type name will be something like GeometryMM.
// see: https://github.com/qgis/QGIS/issues/55223
//typeString.append( 'M' );
break;
case 2:
typeString.append( 'Z' );
Expand Down

0 comments on commit bf10057

Please sign in to comment.