Remove getDet in CaloFutureElectron

This MR removes the usage of getDet in CaloFutureElectron and makes corresponding modifications in CaloFutureECorrection, as described in issue of Rec Issue388.

This MR needs to be tested in the same time with the MR in Rec MR3094 in Rec

patch generated by https://gitlab.cern.ch/lhcb/LHCb/-/jobs/24113435

CaloFuture/CaloFutureInterfaces/include/CaloFutureInterfaces/ICaloFutureElectron.h

@@ -46,14 +46,13 @@ namespace LHCb::Calo::Interfaces {
 
     virtual hypoPairStruct getElectronBrem( LHCb::ProtoParticle const& proto ) const = 0;
 
-    virtual State                 caloState( ProtoParticle const& proto, IGeometryInfo const& geometry ) const    = 0;
-    virtual State                 closestState( ProtoParticle const& proto, IGeometryInfo const& geometry ) const = 0;
-    virtual Momentum              bremMomentum( ProtoParticle const& proto ) const                                = 0;
-    virtual std::optional<double> caloTrajectoryL( ProtoParticle const& proto, IGeometryInfo const& geometry,
-                                                   CaloPlane::Plane refPlane = CaloPlane::ShowerMax ) const       = 0;
+    virtual State    caloState( const DeCalorimeter& detCalo, ProtoParticle const& proto,
+                                IGeometryInfo const& geometry ) const = 0;
+    virtual Momentum bremMomentum( ProtoParticle const& proto ) const = 0;
 
     // interface that does not require ProtoParticle or Hypo, just basic track and cluster...
-    virtual State caloState( const LHCb::Track&, IGeometryInfo const& geometry ) const = 0;
+    virtual State caloState( const DeCalorimeter& detCalo, const LHCb::Track&,
+                             IGeometryInfo const& geometry ) const = 0;
 
     // functions implemented in terms of other functions...
     double eOverP( ProtoParticle const& proto ) const {