From e810020db524f7b032ddfd5952313f6ba62bfb20 Mon Sep 17 00:00:00 2001 From: ILLISIS <92364928+ILLISIS@users.noreply.github.com> Date: Sat, 17 Oct 2026 21:43:09 +1100 Subject: [PATCH] Fix research_sync echoing synced research back to the controller (#1111) The on_research_finished handler read ignore_research_finished from the research_sync global, which is the table of RCON functions and never has that field. The flag that sync_technologies and research_technology set lives on the script data, so the check never returned and research that came from the controller was sent back as newly finished. The handler now reads the flag from the script data. Co-authored-by: ILLISIS Co-authored-by: Claude Opus 5.5 (1M context) --- plugins/research_sync/module/sync.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/research_sync/module/sync.lua b/plugins/research_sync/module/sync.lua index 701f4f7e..23bf6301 100644 --- a/plugins/research_sync/module/sync.lua +++ b/plugins/research_sync/module/sync.lua @@ -114,7 +114,7 @@ end --- @param event EventData.on_research_finished sync.events[defines.events.on_research_finished] = function(event) local script_data = get_script_data() - if research_sync.ignore_research_finished then + if script_data.ignore_research_finished then return end