fix: mark monitor as successful even if there are no new results (fix #…

…2918)
Bionus · Feb 26, 2023 · 366b1f5 · 366b1f5
1 parent 26e9c56
commit 366b1f5
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/lib/src/monitoring/monitoring-center.cpp b/src/lib/src/monitoring/monitoring-center.cpp
@@ -58,6 +58,7 @@ bool MonitoringCenter::checkMonitor(Monitor &monitor, const SearchQuery &search,
 	int newImages = 0;
 	QList<QSharedPointer<Image>> newImagesList;
 	QString state = "";
+	bool success = true;
 
 	for (Site *site : monitor.sites()) {
 		// Create a pack loader
@@ -92,6 +93,7 @@ bool MonitoringCenter::checkMonitor(Monitor &monitor, const SearchQuery &search,
 
 		const QString newState = countRun == 0 ? "empty" : (newImagesRun == 0 ? "finished" : "ok");
 		state = state.isEmpty() || state == newState ? newState : "mixed";
+		success = success && countRun != 0;
 	}
 
 	emit statusChanged(monitor, MonitoringStatus::Performing);
@@ -134,7 +136,7 @@ bool MonitoringCenter::checkMonitor(Monitor &monitor, const SearchQuery &search,
 
 	// Update monitor
 	monitor.setLastCheck(limit);
-	if (state == "ok") {
+	if (success) {
 		monitor.setLastSuccess(limit);
 	}
 	monitor.setLastState(state);