From f238641bf26ea24f186195a80b980255b60d6425 Mon Sep 17 00:00:00 2001
From: mmusich <marco.musich@cern.ch>
Date: Wed, 31 Jan 2024 05:29:05 +0100
Subject: [PATCH] EgammaHLTHcalVarProducerFromRecHit: fix HcalPFCuts consumes
 pattern

---
 .../plugins/EgammaHLTHcalVarProducerFromRecHit.cc  | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTHcalVarProducerFromRecHit.cc b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTHcalVarProducerFromRecHit.cc
index be6fd133ed3e4..e721892bd06df 100644
--- a/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTHcalVarProducerFromRecHit.cc
+++ b/RecoEgamma/EgammaHLTProducers/plugins/EgammaHLTHcalVarProducerFromRecHit.cc
@@ -41,7 +41,6 @@ class EgammaHLTHcalVarProducerFromRecHit : public edm::global::EDProducer<> {
   explicit EgammaHLTHcalVarProducerFromRecHit(const edm::ParameterSet &);
 
 public:
-  void beginRun(edm::Run const &, edm::EventSetup const &);
   void produce(edm::StreamID, edm::Event &, const edm::EventSetup &) const final;
   static void fillDescriptions(edm::ConfigurationDescriptions &descriptions);
 
@@ -75,7 +74,6 @@ class EgammaHLTHcalVarProducerFromRecHit : public edm::global::EDProducer<> {
   //Get HCAL thresholds from GT
   edm::ESGetToken<HcalPFCuts, HcalPFCutsRcd> hcalCutsToken_;
   bool cutsFromDB;
-  HcalPFCuts const *hcalCuts = nullptr;
 };
 
 EgammaHLTHcalVarProducerFromRecHit::EgammaHLTHcalVarProducerFromRecHit(const edm::ParameterSet &config)
@@ -124,7 +122,7 @@ EgammaHLTHcalVarProducerFromRecHit::EgammaHLTHcalVarProducerFromRecHit(const edm
   }
 
   if (cutsFromDB) {
-    hcalCutsToken_ = esConsumes<HcalPFCuts, HcalPFCutsRcd, edm::Transition::BeginRun>(edm::ESInputTag("", "withTopo"));
+    hcalCutsToken_ = esConsumes<HcalPFCuts, HcalPFCutsRcd>(edm::ESInputTag("", "withTopo"));
   }
 }
 
@@ -155,12 +153,6 @@ void EgammaHLTHcalVarProducerFromRecHit::fillDescriptions(edm::ConfigurationDesc
   descriptions.add("hltEgammaHLTHcalVarProducerFromRecHit", desc);
 }
 
-void EgammaHLTHcalVarProducerFromRecHit::beginRun(edm::Run const &run, edm::EventSetup const &iSetup) {
-  if (cutsFromDB) {
-    hcalCuts = &iSetup.getData(hcalCutsToken_);
-  }
-}
-
 void EgammaHLTHcalVarProducerFromRecHit::produce(edm::StreamID,
                                                  edm::Event &iEvent,
                                                  const edm::EventSetup &iSetup) const {
@@ -214,6 +206,10 @@ void EgammaHLTHcalVarProducerFromRecHit::produce(edm::StreamID,
                                                            iSetup.getData(hcalChannelQualityToken_),
                                                            iSetup.getData(hcalSevLvlComputerToken_),
                                                            iSetup.getData(caloTowerConstituentsMapToken_));
+    const HcalPFCuts *hcalCuts{nullptr};
+    if (cutsFromDB) {
+      hcalCuts = &iSetup.getData(hcalCutsToken_);
+    }
 
     if (useSingleTower_) {
       if (doEtSum_) {  //this is cone-based HCAL isolation with single tower based footprint removal