Refactor dataset picker invalidation and reset

Introduce helper lambdas to centralize dataset picker state updates: invalidateFilters() to invalidate dependent filters and resetAuxiliaryDatasets() to clear aux picker selections (setCurrentIndex(-1)). Connect the position picker signal to run the invalidation and connect position dataset change signals to reset auxiliary pickers so color/size/opacity selections are cleared when position datasets change. This removes duplicated calls and ensures auxiliary pickers don't hold stale selections.

Author: ThomasKroes

src/DatasetsAction.cpp

@@ -42,15 +42,24 @@ DatasetsAction::DatasetsAction(QObject* parent, const QString& title) :
 
     setupDatasetPickerActions(scatterplotPlugin);
 
-    connect(&_positionDatasetPickerAction, &DatasetPickerAction::datasetPicked, [this, scatterplotPlugin](Dataset<DatasetImpl> pickedDataset) -> void {
+    const auto invalidateFilters = [this, scatterplotPlugin]() -> void {
         _colorDatasetPickerAction.invalidateFilter();
         _pointSizeDatasetPickerAction.invalidateFilter();
         _pointOpacityDatasetPickerAction.invalidateFilter();
+    };
+
+    connect(&_positionDatasetPickerAction, &DatasetPickerAction::datasetPicked, [this, scatterplotPlugin, invalidateFilters](Dataset<DatasetImpl> pickedDataset) -> void {
+        invalidateFilters();
     });
 
-    _colorDatasetPickerAction.invalidateFilter();
-    _pointSizeDatasetPickerAction.invalidateFilter();
-    _pointOpacityDatasetPickerAction.invalidateFilter();
+    const auto resetAuxilliaryDatasets = [this]() -> void {
+        _colorDatasetPickerAction.setCurrentIndex(-1);
+		_pointSizeDatasetPickerAction.setCurrentIndex(-1);
+		_pointOpacityDatasetPickerAction.setCurrentIndex(-1);
+	};
+
+    connect(&scatterplotPlugin->getPositionDataset(), &Dataset<Points>::changed, this, resetAuxilliaryDatasets);
+    connect(&scatterplotPlugin->getPositionSourceDataset(), &Dataset<Points>::changed, this, resetAuxilliaryDatasets);
 }
 
 void DatasetsAction::connectToPublicAction(WidgetAction* publicAction, bool recursive)