Adjust exclusion of SA muons from LRT running

Rather than building SA muons and then rejecting them, this change simply ensures that when making LRT muons, SA muons are not built at all.

Reconstruction/MuonIdentification/MuonCombinedBaseTools/src/MuonCreatorTool.cxx

@@ -137,10 +137,12 @@ namespace MuonCombined {
       }
       ATH_MSG_DEBUG("Creation of Muon from InDetCandidates done");
     }
-    for( auto can : resolvedMuonCandidates )  {
-      ATH_MSG_DEBUG("New MuonCandidate");
-      create(*can,outputData);
-      ATH_MSG_DEBUG("Creation of Muon from MuonCandidates done");
+    if (!m_requireIDTracks){ //only build SA muons if ID tracks are not required
+      for( auto can : resolvedMuonCandidates )  {
+	ATH_MSG_DEBUG("New MuonCandidate");
+	create(*can,outputData);
+	ATH_MSG_DEBUG("Creation of Muon from MuonCandidates done");
+      }
     }
 
     // Add alignment effects on tracks to ME and CB tracks
@@ -320,13 +322,6 @@ namespace MuonCombined {
     // check if there is a cluster container, if yes collect the cells around the muon and fill
     // Etcore variables for muon
     if(m_useCaloCells) collectCells(*muon,outputData.clusterContainer,caloExtension.get());
-    if (m_requireIDTracks){
-      if (!muon->trackParticle(xAOD::Muon::TrackParticleType:: InnerDetectorTrackParticle)){
-	ATH_MSG_DEBUG("The muon does not have any associated ID track although it should have. Discard it");
-	outputData.muonContainer->pop_back();
-	return nullptr;
-      }
-    }
 
     return muon;
   }