From ce79fe36d6dbb0fcb85675d24b091b98921f9fa2 Mon Sep 17 00:00:00 2001 From: dragonmacher <48328597+dragonmacher@users.noreply.github.com> Date: Thu, 2 Jul 2026 15:42:52 -0400 Subject: [PATCH] GP-6999 - Fixed Fid menu action enablement --- .../java/ghidra/feature/fid/plugin/FidPlugin.java | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/Ghidra/Features/FunctionID/src/main/java/ghidra/feature/fid/plugin/FidPlugin.java b/Ghidra/Features/FunctionID/src/main/java/ghidra/feature/fid/plugin/FidPlugin.java index 5863e181cd..1f24aec69d 100644 --- a/Ghidra/Features/FunctionID/src/main/java/ghidra/feature/fid/plugin/FidPlugin.java +++ b/Ghidra/Features/FunctionID/src/main/java/ghidra/feature/fid/plugin/FidPlugin.java @@ -121,7 +121,7 @@ public class FidPlugin extends ProgramPlugin implements ChangeListener { .buildAndInstall(tool); new ActionBuilder("Detach attached FidDb", getName()) - .enabledWhen(ac -> enabledForUserFidFiles()) + .enabledWhen(ac -> fidFileManager.hasUserFidFiles()) .onAction(ac -> removeFidFile()) .menuPath(ToolConstants.MENU_TOOLS, FUNCTION_ID_NAME, "Detach attached FidDb...") .menuGroup(MENU_GROUP_1, "4") @@ -130,7 +130,6 @@ public class FidPlugin extends ProgramPlugin implements ChangeListener { .buildAndInstall(tool); new ActionBuilder("Populate FidDb from programs", getName()) - .enabledWhen(ac -> enabledForUserFidFiles()) .onAction(ac -> { fidFileManager.load(); PopulateFidDialog populateFidDialog = new PopulateFidDialog(tool, service); @@ -153,15 +152,6 @@ public class FidPlugin extends ProgramPlugin implements ChangeListener { return fidFileManager.hasFidFiles(); } - private boolean enabledForUserFidFiles() { - if (!fidFileManager.hasLoadedFidFiles()) { - // We haven't loaded Fid files yet. Since we don't know if we can enable, return true - // so users can at least try to perform the action. - return true; - } - return fidFileManager.hasUserFidFiles(); - } - /** * Method to select which known FID databases are currently active * during search.