ogr: don't create spatial index for shapes after deletes. That was su…

…pposed to also run a repack and doesn't work without (fixes #10483, refs #8317, refs #8822)
qgis · Jun 13, 2014 · cf7e016 · cf7e016
1 parent 75f8269
commit cf7e016
Showing 1 changed file with 0 additions and 9 deletions.
diff --git a/src/core/qgsvectorlayereditbuffer.cpp b/src/core/qgsvectorlayereditbuffer.cpp
@@ -245,8 +245,6 @@ bool QgsVectorLayerEditBuffer::commitChanges( QStringList& commitErrors )
 
   QgsFields oldFields = L->pendingFields();
 
-  bool hadPendingDeletes = !mDeletedFeatureIds.isEmpty();
-
   //
   // delete attributes
   //
@@ -504,13 +502,6 @@ bool QgsVectorLayerEditBuffer::commitChanges( QStringList& commitErrors )
     }
   }
 
-  // for shapes run a REPACK after each transaction
-  // TODO: enhance provider interface to allow moving this there
-  if ( success && hadPendingDeletes && L->providerType() == "ogr" && L->storageType() == "ESRI Shapefile" )
-  {
-    provider->createSpatialIndex();
-  }
-
   if ( !success && provider->hasErrors() )
   {
     commitErrors << tr( "\n  Provider errors:" );