Fix crash

nyalldawson · nyalldawson · commit f0e01d81f0b8 · 2018-01-25T15:47:42.000+11:00
diff --git a/src/core/processing/qgsprocessingalgorithm.cpp b/src/core/processing/qgsprocessingalgorithm.cpp
@@ -107,12 +107,12 @@ void QgsProcessingAlgorithm::setProvider( QgsProcessingProvider *provider )
 {
   mProvider = provider;
 
-  if ( !mProvider->supportsNonFileBasedOutput() )
+  if ( mProvider && !mProvider->supportsNonFileBasedOutput() )
   {
     // need to update all destination parameters to turn off non file based outputs
     Q_FOREACH ( const QgsProcessingParameterDefinition *definition, mParameters )
     {
-      if ( definition->isDestination() && mProvider )
+      if ( definition->isDestination() )
       {
         const QgsProcessingDestinationParameter *destParam = static_cast< const QgsProcessingDestinationParameter *>( definition );
         const_cast< QgsProcessingDestinationParameter *>( destParam )->setSupportsNonFileBasedOutput( false );

Original file line number	Diff line number	Diff line change
`@@ -107,12 +107,12 @@ void QgsProcessingAlgorithm::setProvider( QgsProcessingProvider *provider )`
`107`	`107`	`{`
`108`	`108`	`mProvider = provider;`
`109`	`109`
`110`		`- if ( !mProvider->supportsNonFileBasedOutput() )`
	`110`	`+ if ( mProvider && !mProvider->supportsNonFileBasedOutput() )`
`111`	`111`	`{`
`112`	`112`	`// need to update all destination parameters to turn off non file based outputs`
`113`	`113`	`Q_FOREACH ( const QgsProcessingParameterDefinition *definition, mParameters )`
`114`	`114`	`{`
`115`		`- if ( definition->isDestination() && mProvider )`
	`115`	`+ if ( definition->isDestination() )`
`116`	`116`	`{`
`117`	`117`	`const QgsProcessingDestinationParameter destParam = static_cast< const QgsProcessingDestinationParameter >( definition );`
`118`	`118`	`const_cast< QgsProcessingDestinationParameter *>( destParam )->setSupportsNonFileBasedOutput( false );`