postgres provider: use st_estimatedextent only when 'use estimated metadata' is enabled (fixes #21718)

jef-n · jef-n · commit 95bd7480d0f0 · 2019-04-02T15:59:51.000+02:00
diff --git a/src/providers/postgres/qgspostgresprovider.cpp b/src/providers/postgres/qgspostgresprovider.cpp
@@ -3237,7 +3237,7 @@ QgsRectangle QgsPostgresProvider::extent() const
     QString ext;
 
     // get the extents
-    if ( !mIsQuery && ( mUseEstimatedMetadata || mSqlWhereClause.isEmpty() ) )
+    if ( !mIsQuery && mUseEstimatedMetadata )
     {
       // do stats exists?
       sql = QStringLiteral( "SELECT count(*) FROM pg_stats WHERE schemaname=%1 AND tablename=%2 AND attname=%3" )

Original file line number	Diff line number	Diff line change
`@@ -3237,7 +3237,7 @@ QgsRectangle QgsPostgresProvider::extent() const`
`3237`	`3237`	`QString ext;`
`3238`	`3238`
`3239`	`3239`	`// get the extents`
`3240`		`- if ( !mIsQuery && ( mUseEstimatedMetadata \|\| mSqlWhereClause.isEmpty() ) )`
	`3240`	`+ if ( !mIsQuery && mUseEstimatedMetadata )`
`3241`	`3241`	`{`
`3242`	`3242`	`// do stats exists?`
`3243`	`3243`	`sql = QStringLiteral( "SELECT count(*) FROM pg_stats WHERE schemaname=%1 AND tablename=%2 AND attname=%3" )`