oracle provider: fix retrieval of column comments for geometryless tables (fixes #15853)

jef-n · jef-n · commit f2e3d5308d2b · 2016-11-16T11:54:31.000+01:00
(cherry picked from commit a62fdb0)
diff --git a/src/providers/oracle/qgsoracleprovider.cpp b/src/providers/oracle/qgsoracleprovider.cpp
@@ -582,13 +582,14 @@ bool QgsOracleProvider::loadFields()
 
     qry.finish();
 
-    if ( exec( qry, QString( "SELECT column_name,comments FROM all_col_comments t WHERE t.owner=%1 AND t.table_name=%2 AND t.column_name<>%3" )
+    if ( exec( qry, QString( "SELECT column_name,comments FROM all_col_comments t WHERE t.owner=%1 AND t.table_name=%2" )
                .arg( quotedValue( mOwnerName ) )
-               .arg( quotedValue( mTableName ) )
-               .arg( quotedValue( mGeometryColumn ) ) ) )
+               .arg( quotedValue( mTableName ) ) ) )
     {
       while ( qry.next() )
       {
+	if( qry.value( 0 ).toString() == mGeometryColumn )
+	  continue;
         comments.insert( qry.value( 0 ).toString(), qry.value( 1 ).toString() );
       }
     }
@@ -2153,7 +2154,7 @@ bool QgsOracleProvider::setSubsetString( const QString& theSQL, bool updateFeatu
   }
   qry.finish();
 
-  if ( mPrimaryKeyType == pktInt && !uniqueData( mQuery, mAttributeFields.at( mPrimaryKeyAttrs[0] ).name() ) )
+  if ( mPrimaryKeyType == pktInt && !mUseEstimatedMetadata && !uniqueData( mQuery, mAttributeFields.at( mPrimaryKeyAttrs[0] ).name() ) )
   {
     mSqlWhereClause = prevWhere;
     return false;

Original file line number	Diff line number	Diff line change
`@@ -582,13 +582,14 @@ bool QgsOracleProvider::loadFields()`
`582`	`582`
`583`	`583`	`qry.finish();`
`584`	`584`
`585`		`- if ( exec( qry, QString( "SELECT column_name,comments FROM all_col_comments t WHERE t.owner=%1 AND t.table_name=%2 AND t.column_name<>%3" )`
	`585`	`+ if ( exec( qry, QString( "SELECT column_name,comments FROM all_col_comments t WHERE t.owner=%1 AND t.table_name=%2" )`
`586`	`586`	`.arg( quotedValue( mOwnerName ) )`
`587`		`- .arg( quotedValue( mTableName ) )`
`588`		`- .arg( quotedValue( mGeometryColumn ) ) ) )`
	`587`	`+ .arg( quotedValue( mTableName ) ) ) )`
`589`	`588`	`{`
`590`	`589`	`while ( qry.next() )`
`591`	`590`	`{`
	`591`	`+ if( qry.value( 0 ).toString() == mGeometryColumn )`
	`592`	`+ continue;`
`592`	`593`	`comments.insert( qry.value( 0 ).toString(), qry.value( 1 ).toString() );`
`593`	`594`	`}`
`594`	`595`	`}`
`@@ -2153,7 +2154,7 @@ bool QgsOracleProvider::setSubsetString( const QString& theSQL, bool updateFeatu`
`2153`	`2154`	`}`
`2154`	`2155`	`qry.finish();`
`2155`	`2156`
`2156`		`- if ( mPrimaryKeyType == pktInt && !uniqueData( mQuery, mAttributeFields.at( mPrimaryKeyAttrs[0] ).name() ) )`
	`2157`	`+ if ( mPrimaryKeyType == pktInt && !mUseEstimatedMetadata && !uniqueData( mQuery, mAttributeFields.at( mPrimaryKeyAttrs[0] ).name() ) )`
`2157`	`2158`	`{`
`2158`	`2159`	`mSqlWhereClause = prevWhere;`
`2159`	`2160`	`return false;`