From 24077be0b882c706324e1e84322eef45647bc0dd Mon Sep 17 00:00:00 2001
From: Piotr Konopka <piotr.jan.konopka@cern.ch>
Date: Wed, 6 Nov 2024 08:53:06 +0100
Subject: [PATCH] ITS Track Check returns Null Quality by default

Initializing Quality result with 0 had an effect of producing a Quality without a name and with level 0, which is treated as "not worse than Good".
This does not seem like an intentional behaviour, so I am proposing to use the default Null Quality instead.
Precautions will be taken in the framework, so that it cannot be done anymore.
---
 Modules/ITS/src/ITSTrackCheck.cxx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Modules/ITS/src/ITSTrackCheck.cxx b/Modules/ITS/src/ITSTrackCheck.cxx
index 5489e0cbeb..5bba810093 100644
--- a/Modules/ITS/src/ITSTrackCheck.cxx
+++ b/Modules/ITS/src/ITSTrackCheck.cxx
@@ -33,7 +33,7 @@ Quality ITSTrackCheck::check(std::map<std::string, std::shared_ptr<MonitorObject
   mEtaRatio = o2::quality_control_modules::common::getFromConfig<float>(mCustomParameters, "EtaRatio", mEtaRatio);
   mPhiRatio = o2::quality_control_modules::common::getFromConfig<float>(mCustomParameters, "PhiRatio", mPhiRatio);
 
-  Quality result = 0;
+  Quality result;
   Int_t id = 0;
   std::map<std::string, std::shared_ptr<MonitorObject>>::iterator iter;
   for (iter = moMap->begin(); iter != moMap->end(); ++iter) {