[processing] Refactor Fields algorithm: avoid unnecessary confirmatio…

…n dialogs
qgis · Apr 30, 2018 · 387e724 · 387e724
1 parent fca5238
commit 387e724
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/python/plugins/processing/algs/qgis/ui/FieldsMappingPanel.py b/python/plugins/processing/algs/qgis/ui/FieldsMappingPanel.py
@@ -325,6 +325,8 @@ def setDelegate(self, column_name, delegate):
             delegate)
 
     def setLayer(self, layer):
+        if self.model.layer() == layer:
+            return
         self.model.setLayer(layer)
         if layer is None:
             return

diff --git a/python/plugins/processing/tools/general.py b/python/plugins/processing/tools/general.py
@@ -154,4 +154,5 @@ def execAlgorithmDialog(algOrName, parameters={}):
         canvas.setMapTool(prevMapTool)
 
     results = dlg.results()
+    dlg.close()
     return results