Make assignHeaderVersion virtual to allow to detector to change version

Author: shahor02

Detectors/Base/include/DetectorsBase/CTFCoderBase.h

@@ -57,6 +57,9 @@ class CTFCoderBase
 
   virtual void createCoders(const std::vector<char>& bufVec, o2::ctf::CTFCoderBase::OpType op) = 0;
 
+  // detector coder need to redefine this method if uses no default version, see comment in the cxx file
+  virtual void assignDictVersion(CTFDictHeader& h) const;
+
   template <typename CTF>
   std::vector<char> readDictionaryFromFile(const std::string& dictPath, bool mayFail = false);
 
@@ -112,12 +115,6 @@ class CTFCoderBase
 
  protected:
   std::string getPrefix() const { return o2::utils::Str::concat_string(mDet.getName(), "_CTF: "); }
-  void assignDictVersion(CTFDictHeader& h) const
-  {
-    if (mExtHeader.isValidDictTimeStamp()) {
-      h = mExtHeader;
-    }
-  }
 
   void checkDictVersion(const CTFDictHeader& h) const;
 

Detectors/Base/src/CTFCoderBase.cxx

@@ -30,6 +30,21 @@ void CTFCoderBase::checkDictVersion(const CTFDictHeader& h) const
   }
 }
 
+// Assign version of the dictionary which will be stored in the data (including dictionary data during dictionary creation)
+// In case detector CTFCoder uses non-defaul dict. version, it should redefine this method in order to assign the version
+// it needs ONLY when the external dictionary is not provided
+void CTFCoderBase::assignDictVersion(CTFDictHeader& h) const
+{
+  if (mExtHeader.isValidDictTimeStamp()) {
+    h = mExtHeader;
+  }
+  // detector code may exten it by
+  //  else {
+  //    h.majorVersion = <A>;
+  //    h.minorVersion = <B>;
+  //  }
+}
+
 void CTFCoderBase::updateTimeDependentParams(ProcessingContext& pc)
 {
   if (mLoadDictFromCCDB) {