[processing][grass] Don't try to create skipped optional outputs

nyalldawson · nyalldawson · commit f03baa871aa3 · 2018-04-10T09:57:58.000+10:00
diff --git a/python/plugins/processing/algs/grass7/Grass7Algorithm.py b/python/plugins/processing/algs/grass7/Grass7Algorithm.py
@@ -637,6 +637,10 @@ def processOutputs(self, parameters, context, feedback):
 
         for out in self.destinationParameterDefinitions():
             outName = out.name()
+            if not outName in parameters:
+                # skipped output
+                continue
+
             if isinstance(out, QgsProcessingParameterRasterDestination):
                 self.exportRasterLayerFromParameter(outName, parameters, context)
             elif isinstance(out, QgsProcessingParameterVectorDestination):
@@ -688,8 +692,11 @@ def exportRasterLayerFromParameter(self, name, parameters, context, colorTable=T
         :param context: Algorithm context.
         :param colorTable: preserve color Table.
         """
-        fileName = os.path.normpath(
-            self.parameterAsOutputLayer(parameters, name, context))
+        fileName = self.parameterAsOutputLayer(parameters, name, context)
+        if not fileName:
+            return
+
+        fileName = os.path.normpath(fileName)
         grassName = '{}{}'.format(name, self.uniqueSuffix)
         outFormat = Grass7Utils.getRasterFormatFromFilename(fileName)
         createOpt = self.parameterAsString(parameters, self.GRASS_RASTER_FORMAT_OPT, context)
