From a44e5143a36be05bafe722909d5c3589d4fa5992 Mon Sep 17 00:00:00 2001
From: Suzanne Soy <jsmaniac.github@suzanne.soy>
Date: Sat, 15 Apr 2023 14:54:04 +0100
Subject: [PATCH] When the cache is outdated, there will be attempts to fetch
 the icons of workbenches which don't exist anymore. Added a conditional to
 prevent a crash in this case.

---
 ResultsToolbar.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ResultsToolbar.py b/ResultsToolbar.py
index c992846..d0ab287 100644
--- a/ResultsToolbar.py
+++ b/ResultsToolbar.py
@@ -47,7 +47,9 @@ def subToolAction(nfo):
   print('Tool ' + toolPath + ' not found, was it offered by an extension that is no longer present?')
 
 def toolbarToolTip(nfo, setParent):
-  return '<p>Show the ' + nfo['text'] + ' toolbar</p><p>This toolbar appears in the following workbenches: <ul>' + ''.join(['<li>' + Serialize.iconToHTML(QtGui.QIcon(FreeCADGui.listWorkbenches()[wb].Icon)) + wb + '</li>' for wb in nfo['action']['workbenches']]) + '</ul></p>'
+  workbenches = FreeCADGui.listWorkbenches()
+  in_workbenches = ['<li>' + (Serialize.iconToHTML(QtGui.QIcon(workbenches[wb].Icon)) if wb in workbenches else '? ') + wb + '</li>' for wb in nfo['action']['workbenches']]
+  return '<p>Show the ' + nfo['text'] + ' toolbar</p><p>This toolbar appears in the following workbenches: <ul>' + ''.join(in_workbenches) + '</ul></p>'
 
 def subToolToolTip(nfo, setParent):
   return Serialize.iconToHTML(nfo['icon'], 32) + '<p>' + nfo['toolTip'] + '</p>'