Don't hide entities from GM screen radar (#2824)

Bypass the early exit in BasicRadarRendering::renderOnRadar() for
LongRange-only traces if the radar system is flagged as GM
(GM screen, Spectate)

Author: oznogon

src/systems/missilesystem.cpp

@@ -361,6 +361,7 @@ void MissileSystem::spawnProjectile(sp::ecs::Entity source, MissileTubes::MountP
         trace.icon = mwd.radar_trace;
         trace.radius = 32.0f;
         trace.max_size = trace.min_size = 32 * (0.25f + 0.25f * category_modifier);
+        // LongRange intentionally omitted for gameplay
         trace.flags = RadarTrace::Rotate;
         // Exempt mines from LongRange restriction.
         if (tube.type_loaded == MW_Mine) trace.flags |= RadarTrace::LongRange;

src/systems/radar.cpp

@@ -15,7 +15,11 @@ std::vector<RadarRenderSystem::Handler> RadarRenderSystem::handlers;
 
 void BasicRadarRendering::renderOnRadar(sp::RenderTarget& renderer, sp::ecs::Entity entity, glm::vec2 screen_position, float scale, float rotation, RadarTrace& trace)
 {
-    if ((RadarRenderSystem::current_flags & RadarRenderSystem::FlagLongRange) && !(trace.flags & RadarTrace::LongRange))
+    // Exit early if the trace is flagged for LongRange and this is non-GM
+    // LongRange radar.
+    if ((RadarRenderSystem::current_flags & RadarRenderSystem::FlagLongRange)
+        && !(RadarRenderSystem::current_flags & RadarRenderSystem::FlagGM)
+        && !(trace.flags & RadarTrace::LongRange))
         return;
 
     auto scanstate = entity.getComponent<ScanState>();