Followups

qgis · Apr 26, 2018 · c3b6990 · c3b6990
1 parent c78bdae
commit c3b6990
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/src/providers/oracle/qgsoracleprovider.cpp b/src/providers/oracle/qgsoracleprovider.cpp
@@ -1301,7 +1301,7 @@ bool QgsOracleProvider::addFeatures( QgsFeatureList &flist, QgsFeatureSink::Flag
         QVariant value = attributevec.value( fieldId[i], QVariant() );
 
         QString v;
-        if ( !value.isValid() || value.isNull() )
+        if ( value.isNull() )
         {
           if ( mPrimaryKeyAttrs.contains( i ) && !defaultValues.at( i ).isEmpty() )
           {

diff --git a/tests/src/python/test_provider_oracle.py b/tests/src/python/test_provider_oracle.py
@@ -63,7 +63,6 @@ def execSQLCommand(self, sql, ignore_errors=False):
         self.assertTrue(query.exec_(sql), sql + ': ' + query.lastError().text())
         query.finish()
 
-    # disabled: WIP
     def getSource(self):
         # create temporary table for edit tests
         self.execSQLCommand('DROP TABLE "QGIS"."EDIT_DATA"', ignore_errors=True)
@@ -79,7 +78,6 @@ def getSource(self):
             'test', 'oracle')
         return vl
 
-    # disabled: WIP
     def getEditableLayer(self):
         return self.getSource()