fixes #18421 : Keep options AUTO_REPACK=OFF when reloading data in ogr provider

troopa81 · nyalldawson · commit 627230c10ad6 · 2019-02-20T11:48:11.000+10:00
diff --git a/src/providers/ogr/qgsogrprovider.cpp b/src/providers/ogr/qgsogrprovider.cpp
@@ -4261,7 +4261,7 @@ void QgsOgrProvider::open( OpenMode mode )
   if ( !openReadOnly )
   {
     QStringList options;
-    if ( mode == OpenModeForceUpdateRepackOff )
+    if ( mode == OpenModeForceUpdateRepackOff || ( mDeferRepack && OpenModeSameAsCurrent ) )
     {
       options << "AUTO_REPACK=OFF";
     }
@@ -6595,4 +6595,3 @@ QGISEXTERN QgsTransaction *createTransaction( const QString &connString )
 
   return new QgsOgrTransaction( connString, ds );
 }
-
diff --git a/tests/src/python/test_provider_shapefile.py b/tests/src/python/test_provider_shapefile.py
@@ -484,6 +484,34 @@ def testDeleteShapes(self):
 
         vl = None
 
+    def testDontRepackOnReload(self):
+        ''' Test fix for #18421 '''
+
+        tmpdir = tempfile.mkdtemp()
+        self.dirs_to_cleanup.append(tmpdir)
+        srcpath = os.path.join(TEST_DATA_DIR, 'provider')
+        for file in glob.glob(os.path.join(srcpath, 'shapefile.*')):
+            shutil.copy(os.path.join(srcpath, file), tmpdir)
+        datasource = os.path.join(tmpdir, 'shapefile.shp')
+
+        vl = QgsVectorLayer('{}|layerid=0'.format(datasource), 'test', 'ogr')
+        feature_count = vl.featureCount()
+        # Start an iterator that will open a new connection
+        iterator = vl.getFeatures()
+        next(iterator)
+
+        # Delete another feature while in update mode
+        vl.dataProvider().enterUpdateMode()
+        vl.dataProvider().reloadData()
+        vl.dataProvider().deleteFeatures([0])
+
+        # Test that repacking has not been done (since in update mode)
+        ds = osgeo.ogr.Open(datasource)
+        self.assertTrue(ds.GetLayer(0).GetFeatureCount() == feature_count)
+        ds = None
+
+        vl = None
+
     def testRepackUnderFileLocks(self):
         ''' Test fix for #15570 and #15393 '''
 

Original file line number	Diff line number	Diff line change
`@@ -4261,7 +4261,7 @@ void QgsOgrProvider::open( OpenMode mode )`
`4261`	`4261`	`if ( !openReadOnly )`
`4262`	`4262`	`{`
`4263`	`4263`	`QStringList options;`
`4264`		`- if ( mode == OpenModeForceUpdateRepackOff )`
	`4264`	`+ if ( mode == OpenModeForceUpdateRepackOff \|\| ( mDeferRepack && OpenModeSameAsCurrent ) )`
`4265`	`4265`	`{`
`4266`	`4266`	`options << "AUTO_REPACK=OFF";`
`4267`	`4267`	`}`
`@@ -6595,4 +6595,3 @@ QGISEXTERN QgsTransaction *createTransaction( const QString &connString )`
`6595`	`6595`
`6596`	`6596`	`return new QgsOgrTransaction( connString, ds );`
`6597`	`6597`	`}`
`6598`		`-`