oracle provider: accept number(0) as key

jef-n · jef-n · commit c0d9275772ca · 2013-01-22T21:43:33.000+01:00
diff --git a/src/providers/oracle/qgsoracleprovider.cpp b/src/providers/oracle/qgsoracleprovider.cpp
@@ -1235,7 +1235,10 @@ bool QgsOracleProvider::determinePrimaryKey()
 
         if ( idx >= 0 )
         {
-          if ( mAttributeFields[idx].type() == QVariant::Int || mAttributeFields[idx].type() == QVariant::LongLong )
+          const QgsField &fld = mAttributeFields[idx];
+          if ( fld.type() == QVariant::Int ||
+               fld.type() == QVariant::LongLong ||
+               ( fld.type() == QVariant::Double && fld.precision() == 0 ) )
           {
             if ( mUseEstimatedMetadata || uniqueData( mQuery, primaryKey ) )
             {

Original file line number	Diff line number	Diff line change
`@@ -1235,7 +1235,10 @@ bool QgsOracleProvider::determinePrimaryKey()`
`1235`	`1235`
`1236`	`1236`	`if ( idx >= 0 )`
`1237`	`1237`	`{`
`1238`		`- if ( mAttributeFields[idx].type() == QVariant::Int \|\| mAttributeFields[idx].type() == QVariant::LongLong )`
	`1238`	`+ const QgsField &fld = mAttributeFields[idx];`
	`1239`	`+ if ( fld.type() == QVariant::Int \|\|`
	`1240`	`+ fld.type() == QVariant::LongLong \|\|`
	`1241`	`+ ( fld.type() == QVariant::Double && fld.precision() == 0 ) )`
`1239`	`1242`	`{`
`1240`	`1243`	`if ( mUseEstimatedMetadata \|\| uniqueData( mQuery, primaryKey ) )`
`1241`	`1244`	`{`