fix #3451: don't recheck saved primary key column for views when 'use estimates' is on

jef · jef · commit 3cc6f3f9a33a · 2011-01-29T20:39:52.000Z
git-svn-id: http://svn.osgeo.org/qgis/trunk@15107 c8812cc2-4d05-0410-92ff-de0c093fc19c
diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
@@ -1253,9 +1253,10 @@ QString QgsPostgresProvider::getPrimaryKey()
           }
 
           // mPrimaryKeyDefault stays null and is retrieved later on demand
+          // if mUseEstimatedMetadata is on assume that the already keyfield is still unique
 
           if (( type != "int4" && type != "oid" ) ||
-              !uniqueData( mQuery, primaryKey ) )
+              ( !mUseEstimatedMetadata && !uniqueData( mQuery, primaryKey ) ) )
           {
             primaryKey = "";
           }

Original file line number	Diff line number	Diff line change
`@@ -1253,9 +1253,10 @@ QString QgsPostgresProvider::getPrimaryKey()`
`1253`	`1253`	`}`
`1254`	`1254`
`1255`	`1255`	`// mPrimaryKeyDefault stays null and is retrieved later on demand`
	`1256`	`+ // if mUseEstimatedMetadata is on assume that the already keyfield is still unique`
`1256`	`1257`
`1257`	`1258`	`if (( type != "int4" && type != "oid" ) \|\|`
`1258`		`- !uniqueData( mQuery, primaryKey ) )`
	`1259`	`+ ( !mUseEstimatedMetadata && !uniqueData( mQuery, primaryKey ) ) )`
`1259`	`1260`	`{`
`1260`	`1261`	`primaryKey = "";`
`1261`	`1262`	`}`