From f168d0c60ad899ef3fbb5cdb6eda4824f2fcaa78 Mon Sep 17 00:00:00 2001 From: Martin Tranberg Date: Sun, 12 Apr 2026 09:53:58 +0200 Subject: [PATCH] refactor: remove dead is_breadcrumb parameter from _navigate_to_item_data (S-1) The S3 fix removed the only conditional that read is_breadcrumb. Remove the parameter from the signature and its kwarg from the one call site in the breadcrumb button handler. Co-Authored-By: Claude Sonnet 4.6 --- sharepoint_browser.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sharepoint_browser.py b/sharepoint_browser.py index 173d838..0335bb1 100644 --- a/sharepoint_browser.py +++ b/sharepoint_browser.py @@ -1644,7 +1644,7 @@ class SharePointApp(wx.Frame): btn.Bind(wx.EVT_BUTTON, self.load_sites) elif data: def on_click(e, d=data): - self._navigate_to_item_data(d, is_breadcrumb=True) + self._navigate_to_item_data(d) # Efter navigation, prøv at finde og vælge den i træet wx.CallAfter(self._sync_tree_selection_by_path, d["path"]) @@ -1853,7 +1853,7 @@ class SharePointApp(wx.Frame): self._navigate_to_item_data(data, tree_item=item) - def _navigate_to_item_data(self, data, tree_item=None, is_breadcrumb=False): + def _navigate_to_item_data(self, data, tree_item=None): try: # Race-condition beskyttelse: Hvis vi allerede er der, så stop if getattr(self, 'current_path', None) == data.get("path"):