Override checkParameterValues() method in align rasters processing al…

…gorithm to ensure that output files have been configured for all input rasters.
qgis · Oct 21, 2024 · ce35c42 · ce35c42
1 parent c2dd1aa
commit ce35c42
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/analysis/processing/qgsalgorithmalignrasters.cpp b/src/analysis/processing/qgsalgorithmalignrasters.cpp
@@ -62,6 +62,27 @@ QgsAlignRastersAlgorithm *QgsAlignRastersAlgorithm::createInstance() const
   return new QgsAlignRastersAlgorithm();
 }
 
+bool QgsAlignRastersAlgorithm::checkParameterValues( const QVariantMap &parameters, QgsProcessingContext &context, QString *message ) const
+{
+  const QVariant layersVariant = parameters.value( parameterDefinition( QStringLiteral( "LAYERS" ) )->name() );
+  const QList<QgsAlignRasterData::RasterItem> items = QgsProcessingParameterAlignRasterLayers::parameterAsItems( layersVariant, context );
+  bool unconfiguredLayers = false;
+  for ( const QgsAlignRasterData::RasterItem &item : items )
+  {
+    if ( item.outputFilename.isEmpty() )
+    {
+      unconfiguredLayers = true;
+      break;
+    }
+  }
+  if ( unconfiguredLayers )
+  {
+    *message = QObject::tr( "An output file is not configured for one or more input layers. Configure output files via 'Configure Raster…' under Input layers parameter." );
+    return false;
+  }
+  return QgsProcessingAlgorithm::checkParameterValues( parameters, context );
+}
+
 void QgsAlignRastersAlgorithm::initAlgorithm( const QVariantMap & )
 {
   addParameter( new QgsProcessingParameterAlignRasterLayers( QStringLiteral( "LAYERS" ), QObject::tr( "Input layers" ) ) );

diff --git a/src/analysis/processing/qgsalgorithmalignrasters.h b/src/analysis/processing/qgsalgorithmalignrasters.h
@@ -44,6 +44,7 @@ class QgsAlignRastersAlgorithm : public QgsProcessingAlgorithm
     QString groupId() const override;
     QString shortHelpString() const override;
     QgsAlignRastersAlgorithm *createInstance() const override SIP_FACTORY;
+    bool checkParameterValues( const QVariantMap &parameters, QgsProcessingContext &context, QString *message ) const override;
 
   protected: