From 6e8405c143c684e77047c01339c189b50fac70b5 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Sun, 24 Oct 2021 19:29:34 +0200
Subject: [PATCH 01/13] Removed redundant variables

---
 python/InvenTreeLink/functions.py | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index 4bdd1a9..dd99eda 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -118,7 +118,7 @@ def load_config(ui: adsk.core.UserInterface):
             "next couple of input boxes.\n"
         )
 
-        def ask_user(line, context, default=""):
+        def ask_user(line, default=""):
             (value, cancelled) = ui.inputBox(
                 line, 
                 TITLE, 
@@ -131,7 +131,6 @@ def ask_user(line, context, default=""):
 
         address = ask_user(
             "Please enter the server address of the InvenTree instance.", 
-            "Instance address"
         )
 
         if address is None:
@@ -139,8 +138,7 @@ def ask_user(line, context, default=""):
             return False               
 
         token = ask_user(
-            "Please enter the user token:", 
-            "Authentication token", 
+            "Please enter the user token:"
         )
 
         if token is None:
@@ -148,8 +146,7 @@ def ask_user(line, context, default=""):
             return False
 
         part_category = ask_user(
-            "Please enter the part category's name were you would like the Part's to show up.", 
-            "Part Category",
+            "Please enter the part category's name were you would like the Part's to show up.",
             "plugin-test"
         )
 

From b3ae6c9e8794fb8187d6b776e1eb0de2a0534204 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 5 Nov 2021 15:29:06 +0100
Subject: [PATCH 02/13] Cleaned up generated conf.ini

---
 python/InvenTreeLink/functions.py | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index dd99eda..04f1842 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -154,15 +154,15 @@ def ask_user(line, default=""):
             ui.messageBox("Invalid part category", TITLE)
             return False
 
-        config_text = (
-            "[SERVER]\n"
-            "current = default\n"
-            "\n"
-            "[default]\n"
-            f"{config.CFG_ADDRESS} = {address}\n"
-            f"{config.CFG_TOKEN} = {token}\n"
-            f"{config.CFG_PART_CATEGORY} = {part_category}\n"            
-        )
+        config_text = '\n'.join((
+            "[SERVER]",
+            "current = default",
+            "",
+            "[default]",
+            f"{config.CFG_ADDRESS} = {address}",
+            f"{config.CFG_TOKEN} = {token}",
+            f"{config.CFG_PART_CATEGORY} = {part_category}",         
+        ))
 
         with open(config_path, "w") as f:
             f.write(config_text)

From 7b7d85c122d523435b0bd0db0f7d9a43a52f4e3c Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 5 Nov 2021 15:47:19 +0100
Subject: [PATCH 03/13] Added ImportPartCommand

---
 python/InvenTreeLink/InvenTreeLink.py         | 26 +++++++++++++++----
 .../commands/ImportPartCommand.py             | 23 ++++++++++++++++
 python/InvenTreeLink/config.py                |  7 ++++-
 3 files changed, 50 insertions(+), 6 deletions(-)
 create mode 100644 python/InvenTreeLink/commands/ImportPartCommand.py

diff --git a/python/InvenTreeLink/InvenTreeLink.py b/python/InvenTreeLink/InvenTreeLink.py
index 7a1ce0d..a5eafeb 100644
--- a/python/InvenTreeLink/InvenTreeLink.py
+++ b/python/InvenTreeLink/InvenTreeLink.py
@@ -17,6 +17,7 @@
     from .commands.SendBomCommand import SendBomCommand
     from .commands.SendBomOnlineCommand import SendBomOnlineCommand
     from .commands.SendStepCommand import SendStepCommand
+    from .commands.ImportPartCommand import ImportPartCommand
 
     # Commands
     my_addin.add_command(
@@ -26,7 +27,7 @@
             'cmd_description': 'Show the InvenTree part-details for the selected part',
             'cmd_id': config.DEF_SEND_PART,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': 'Commands',
+            'toolbar_panel_id': config.ToolbarPanelID.COMMANDS,
             'cmd_resources': config.DEF_SEND_PART,
             'command_visible': True,
             'command_promoted': False,
@@ -40,7 +41,7 @@
             'cmd_description': 'Generates a STEP file and attaches it to a part',
             'cmd_id': config.DEF_SEND_STEP,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': 'Commands',
+            'toolbar_panel_id': config.ToolbarPanelID.COMMANDS,
             'cmd_resources': config.DEF_SEND_BOM,
             'command_visible': True,
             'command_promoted': False,
@@ -56,7 +57,7 @@
             'cmd_description': 'Show the BOM overview palette',
             'cmd_id': config.DEF_SHOW_PALETTE,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.APP_PANEL,
+            'toolbar_panel_id': config.ToolbarPanelID.INVENTREE_LINK,
             'cmd_resources': 'ShowPalette',
             'command_visible': True,
             'command_promoted': True,
@@ -80,7 +81,7 @@
             'cmd_description': 'Load the BOM for the assembly in the current file',
             'cmd_id': config.DEF_SEND_BOM,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.APP_PANEL,
+            'toolbar_panel_id': config.ToolbarPanelID.INVENTREE_LINK,
             'cmd_resources': config.DEF_SEND_BOM,
             'command_visible': True,
             'command_promoted': False,
@@ -95,7 +96,22 @@
             'cmd_description': 'Fetch the InvenTree information for all BOM-parts',
             'cmd_id': config.DEF_SEND_ONLINE_STATE,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.APP_PANEL,
+            'toolbar_panel_id': config.ToolbarPanelID.INVENTREE_LINK,
+            'cmd_resources': config.DEF_SEND_ONLINE_STATE,
+            'command_visible': True,
+            'command_promoted': False,
+            'palette_id': config.ITEM_PALETTE,
+        }
+    )
+
+    my_addin.add_command(
+        'Import',
+        ImportPartCommand,
+        {
+            'cmd_description': 'Import a Part as STL',
+            'cmd_id': config.DEF_IMPORT_PART,
+            'workspace': 'FusionSolidEnvironment',
+            'toolbar_panel_id': config.ToolbarPanelID.PARTS,
             'cmd_resources': config.DEF_SEND_ONLINE_STATE,
             'command_visible': True,
             'command_promoted': False,
diff --git a/python/InvenTreeLink/commands/ImportPartCommand.py b/python/InvenTreeLink/commands/ImportPartCommand.py
new file mode 100644
index 0000000..da6bcca
--- /dev/null
+++ b/python/InvenTreeLink/commands/ImportPartCommand.py
@@ -0,0 +1,23 @@
+import adsk.core
+import adsk.fusion
+import adsk.cam
+
+import os
+from datetime import datetime
+
+from ..apper import apper
+from .. import functions
+from .. import helpers
+from .. import config
+
+
+class ImportPartCommand(apper.Fusion360CommandBase):
+
+    @apper.lib_import(config.lib_path)
+    def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
+        try:
+            ao = apper.AppObjects()
+
+        except Exception as _e:
+            config.app_tracking.capture_exception(_e)
+            raise _e
diff --git a/python/InvenTreeLink/config.py b/python/InvenTreeLink/config.py
index c70eb76..97e64bc 100644
--- a/python/InvenTreeLink/config.py
+++ b/python/InvenTreeLink/config.py
@@ -13,13 +13,13 @@
 
 # Magic numbers
 ITEM_PALETTE = 'InvenTreePalette'
-APP_PANEL = 'InvenTreeLink'
 
 DEF_SHOW_PALETTE = "ShowPalette"
 DEF_SEND_BOM = "SendBom"
 DEF_SEND_ONLINE_STATE = "SendOnlineState"
 DEF_SEND_PART = "SendPart"
 DEF_SEND_STEP = "SendStep"
+DEF_IMPORT_PART = "ImportPart"
 
 CFG_ADDRESS = 'address'
 CFG_TOKEN = 'token'
@@ -31,3 +31,8 @@
 INV_API = None  # API-connection
 CONFIG = {}  # Config section
 REF_CACHE = {}  # saves refs for reduced loading
+
+class ToolbarPanelID:
+    COMMANDS = "Commands"
+    INVENTREE_LINK = "Inventree Link"
+    PARTS = "Parts"
\ No newline at end of file

From b0fbb3097d1baa0e5cbda364b887e48d9da07766 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 5 Nov 2021 16:33:51 +0100
Subject: [PATCH 04/13] Little refactor to make it more clear

---
 python/InvenTreeLink/InvenTreeLink.py         |  8 +++----
 ...endBomCommand.py => GenerateBomCommand.py} |  8 +++----
 ...eCommand.py => LoadInventreeBomCommand.py} | 22 +++++++++++++------
 python/InvenTreeLink/functions.py             | 18 +++++++++------
 4 files changed, 34 insertions(+), 22 deletions(-)
 rename python/InvenTreeLink/commands/{SendBomCommand.py => GenerateBomCommand.py} (88%)
 rename python/InvenTreeLink/commands/{SendBomOnlineCommand.py => LoadInventreeBomCommand.py} (57%)

diff --git a/python/InvenTreeLink/InvenTreeLink.py b/python/InvenTreeLink/InvenTreeLink.py
index a5eafeb..46cb9c2 100644
--- a/python/InvenTreeLink/InvenTreeLink.py
+++ b/python/InvenTreeLink/InvenTreeLink.py
@@ -14,8 +14,8 @@
 
     from .commands.ShowPartCommand import ShowPartCommand
     from .commands.BOMOverviewCommand import BomOverviewPaletteShowCommand
-    from .commands.SendBomCommand import SendBomCommand
-    from .commands.SendBomOnlineCommand import SendBomOnlineCommand
+    from .commands.GenerateBomCommand import GenerateBomCommand
+    from .commands.LoadInventreeBomCommand import LoadInventreeBomCommand
     from .commands.SendStepCommand import SendStepCommand
     from .commands.ImportPartCommand import ImportPartCommand
 
@@ -76,7 +76,7 @@
     # Commands that need the palette
     my_addin.add_command(
         'Load BOM for assembly',
-        SendBomCommand,
+        GenerateBomCommand,
         {
             'cmd_description': 'Load the BOM for the assembly in the current file',
             'cmd_id': config.DEF_SEND_BOM,
@@ -91,7 +91,7 @@
 
     my_addin.add_command(
         'Get InvenTree Information',
-        SendBomOnlineCommand,
+        LoadInventreeBomCommand,
         {
             'cmd_description': 'Fetch the InvenTree information for all BOM-parts',
             'cmd_id': config.DEF_SEND_ONLINE_STATE,
diff --git a/python/InvenTreeLink/commands/SendBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
similarity index 88%
rename from python/InvenTreeLink/commands/SendBomCommand.py
rename to python/InvenTreeLink/commands/GenerateBomCommand.py
index 359d277..7e5ec3b 100644
--- a/python/InvenTreeLink/commands/SendBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -9,8 +9,8 @@
 from .. import config
 from .. import functions
 
-
-class SendBomCommand(apper.Fusion360CommandBase):
+# Loads the Bill of Materials from the current Fusion360 design.
+class GenerateBomCommand(apper.Fusion360CommandBase):
     def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
         try:
             # Get Reference to Palette
@@ -32,8 +32,8 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
 
                 palette.sendInfoToHTML(
                     config.DEF_SEND_BOM,
-                    '<p>{nbr} parts found in {time}</p>{table}'.format(
-                        nbr=len(config.BOM),
+                    '<p>{count} parts found in {time}</p>{table}'.format(
+                        count=len(config.BOM),
                         table=table_c,
                         time=datetime.now() - start
                     )
diff --git a/python/InvenTreeLink/commands/SendBomOnlineCommand.py b/python/InvenTreeLink/commands/LoadInventreeBomCommand.py
similarity index 57%
rename from python/InvenTreeLink/commands/SendBomOnlineCommand.py
rename to python/InvenTreeLink/commands/LoadInventreeBomCommand.py
index 45de1bb..759bfaa 100644
--- a/python/InvenTreeLink/commands/SendBomOnlineCommand.py
+++ b/python/InvenTreeLink/commands/LoadInventreeBomCommand.py
@@ -6,8 +6,8 @@
 from .. import config
 from .. import functions
 
-
-class SendBomOnlineCommand(apper.Fusion360CommandBase):
+# Loads the Bill of Materials that Inventree has of this Part.
+class LoadInventreeBomCommand(apper.Fusion360CommandBase):
     def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
         try:
             # Get Reference to Palette
@@ -22,14 +22,22 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                 )
 
                 # Work with it
-                inv_status = functions.inventree_get_part([a['id'] for a in config.BOM])
-                for a in config.BOM:
-                    a['status'] = inv_status[a['id']]
+                bom_parts = functions.inventree_get_part([item['id'] for item in config.BOM])
+                for item in config.BOM:
+                    part = bom_parts[item['id']]
+
+                    if part is not False:
+                        item['status'] = "<span style='color: green;'> Synced </span>"
+                    else:
+                        item['status'] = "<span style='color: red;'>Not synced</span>"
 
                 body = ''.join(
-                    ['<tr><td>%s</td><td>%s</td><td>%s</td></tr>' % (a['name'], a['instances'], a['status']) for a in config.BOM]
+                    [
+                        f"<tr> <td> {item['name']} </td> <td> {item['instances']} </td> <td> {item['status']} </td> </tr>"
+                        for item in config.BOM
+                    ]
                 )
-                table = '<div class="overflow-auto"><table class="table table-sm table-striped table-hover"><thead><tr><th scope="col">Name</th><th scope="col">Count</th><th scope="col">Is InvenTree</th></tr></thead><tbody>{body}</tbody></table></div>'.format(body=body)
+                table = '<div class="overflow-auto"><table class="table table-sm table-striped table-hover"><thead><tr><th scope="col">Name</th><th scope="col">Count</th><th scope="col">Status</th></tr></thead><tbody>{body}</tbody></table></div>'.format(body=body)
 
                 palette.sendInfoToHTML(
                     config.DEF_SEND_BOM,
diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index 04f1842..1e174cc 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -278,15 +278,17 @@ def extract_bom():
         bom = []
         for occ in occs:
             comp = occ.component
-            jj = 0
-            for bomI in bom:
-                if bomI['component'] == comp:
+            already_exists = False
+
+            # Go through the BOM for items previously added 
+            for item in bom:
+                if item['component'] == comp:
                     # Increment the instance count of the existing row.
-                    bomI['instances'] += 1
+                    item['instances'] += 1
+                    already_exists = True
                     break
-                jj += 1
 
-            if jj == len(bom):
+            if already_exists is False:
                 # Gather any BOM worthy values from the component
                 volume = 0
                 bodies = comp.bRepBodies
@@ -311,18 +313,20 @@ def component_info(comp, parent='#', comp_set=False):
     """ returns a node element """
     node = {
         'name': comp.name,
-        'nbr': comp.partNumber,
+        'IPN': comp.partNumber,
         'id': comp.id,
         'revision-id': comp.revisionId,
         'instances': 1,
         'parent': parent,
     }
+
     if comp_set:
         node['component'] = comp
     else:
         node['state'] = {'opened': True, 'checkbox_disabled': False}
         node["type"] = "4-root_component"
         node["text"] = comp.name
+
     return node
 
 

From b330a301b97458400f5ee5919773143b32a3382f Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 5 Nov 2021 17:35:20 +0100
Subject: [PATCH 05/13] Removed refreshing from online data to avoid confusion

---
 python/InvenTreeLink/InvenTreeLink.py         |  20 +--
 .../commands/BOMOverviewCommand.py            |   3 +
 .../commands/GenerateBomCommand.py            |  65 +++++--
 ...SendStepCommand.py => ImportStlCommand.py} |   6 +-
 .../commands/LoadInventreeBomCommand.py       |  48 -----
 .../commands/palette_html/palette.html        | 166 ++++++++++--------
 python/InvenTreeLink/config.py                |   2 -
 python/InvenTreeLink/functions.py             |  10 +-
 8 files changed, 164 insertions(+), 156 deletions(-)
 rename python/InvenTreeLink/commands/{SendStepCommand.py => ImportStlCommand.py} (92%)
 delete mode 100644 python/InvenTreeLink/commands/LoadInventreeBomCommand.py

diff --git a/python/InvenTreeLink/InvenTreeLink.py b/python/InvenTreeLink/InvenTreeLink.py
index 46cb9c2..cd4915b 100644
--- a/python/InvenTreeLink/InvenTreeLink.py
+++ b/python/InvenTreeLink/InvenTreeLink.py
@@ -15,8 +15,7 @@
     from .commands.ShowPartCommand import ShowPartCommand
     from .commands.BOMOverviewCommand import BomOverviewPaletteShowCommand
     from .commands.GenerateBomCommand import GenerateBomCommand
-    from .commands.LoadInventreeBomCommand import LoadInventreeBomCommand
-    from .commands.SendStepCommand import SendStepCommand
+    from .commands.ImportStlCommand import ImportStlCommand
     from .commands.ImportPartCommand import ImportPartCommand
 
     # Commands
@@ -36,7 +35,7 @@
 
     my_addin.add_command(
         'Upload STEP to attachments',
-        SendStepCommand,
+        ImportStlCommand,
         {
             'cmd_description': 'Generates a STEP file and attaches it to a part',
             'cmd_id': config.DEF_SEND_STEP,
@@ -89,21 +88,6 @@
         }
     )
 
-    my_addin.add_command(
-        'Get InvenTree Information',
-        LoadInventreeBomCommand,
-        {
-            'cmd_description': 'Fetch the InvenTree information for all BOM-parts',
-            'cmd_id': config.DEF_SEND_ONLINE_STATE,
-            'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.ToolbarPanelID.INVENTREE_LINK,
-            'cmd_resources': config.DEF_SEND_ONLINE_STATE,
-            'command_visible': True,
-            'command_promoted': False,
-            'palette_id': config.ITEM_PALETTE,
-        }
-    )
-
     my_addin.add_command(
         'Import',
         ImportPartCommand,
diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index e7b3947..34fe258 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -46,6 +46,9 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                 selections.add(entitiesByToken)  # TODO selection not working
                 helpers.get_cmd(ao, config.DEF_SEND_PART).execute()
 
+            elif html_args.action == 'UploadBom':
+                ao.ui.messageBox("Uploading bom")
+
             # TODO investigate ghost answers
             # else:
             #     raise NotImplementedError('unknown message received from HTML')
diff --git a/python/InvenTreeLink/commands/GenerateBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
index 7e5ec3b..072eb66 100644
--- a/python/InvenTreeLink/commands/GenerateBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -13,6 +13,15 @@
 class GenerateBomCommand(apper.Fusion360CommandBase):
     def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
         try:
+            ao = apper.AppObjects()
+            design = ao.product
+
+            if not design:
+                ao.ui.messageBox('No active design', 'Extract BOM')
+                return
+
+            root = design.rootComponent
+
             # Get Reference to Palette
             ao = apper.AppObjects()
             palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
@@ -25,22 +34,58 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                 )
 
                 start = datetime.now()
-                config.BOM = functions.extract_bom()
-                config.BOM_HIR = functions.make_component_tree()
-                body = ''.join(['<tr><td>%s</td><td>%s</td></tr>' % (a['name'], a['instances']) for a in config.BOM])
-                table_c = '<div class="overflow-auto"><table class="table table-sm table-striped table-hover"><thead><tr><th scope="col">Name</th><th scope="col">Count</th></tr></thead><tbody>{body}</tbody></table></div>'.format(body=body)
+
+                bom = functions.extract_bom()
+                unsynced_parts = [item for item in bom if item['synced'] is False]
+
+                component_tree = functions.make_component_tree()
+                
+                element_synced = "<span style='color: green'> Synced </span>"
+                element_not_synced = "<span style='color: red'> Not synced </span>"
+
+                header = ''.join((
+                    f"<h1>{root.name} </h1>"
+                ))
+
+                body = ''.join([
+                    ''.join((
+                        "<tr>"
+                        f"<td>{item['name']}</td>"
+                        f"<td>{item['instances']}</td>"
+                        f"<td>{element_synced if item['synced'] else element_not_synced}</td>"
+                        "</tr>" 
+                    ))
+                    for item in bom
+                ])
+
+                table_c = ''.join([
+                    "<div class='overflow-auto'>",
+                    "<table class='table table-sm table-striped table-hover'>",
+                    "<thead>",
+                    "<tr>",
+                    "<th scope='col'> Name </th>",
+                    "<th scope='col'> Count </th>",
+                    "<th scope='col'> Synced </th>",
+                    "</tr>",
+                    f"</thead><tbody>{body}</tbody></table></div>",
+                ])
+
+                complete = ''.join((
+                    header,
+                    f"<p> {len(bom)} parts found in {datetime.now() - start}</p>"
+                    f"{table_c}"
+                    "<button onclick='' class='btn btn-outline-secondary'> Sync All </button>" 
+                    "<button onclick='onClickUploadBom()' class='btn btn-outline-secondary'> Upload BOM </button>"    
+                ))
 
                 palette.sendInfoToHTML(
                     config.DEF_SEND_BOM,
-                    '<p>{count} parts found in {time}</p>{table}'.format(
-                        count=len(config.BOM),
-                        table=table_c,
-                        time=datetime.now() - start
-                    )
+                    complete
                 )
+
                 palette.sendInfoToHTML(
                     'SendTree',
-                    json.dumps(config.BOM_HIR)
+                    json.dumps(component_tree)
                 )
         except Exception as _e:
             config.app_tracking.capture_exception(_e)
diff --git a/python/InvenTreeLink/commands/SendStepCommand.py b/python/InvenTreeLink/commands/ImportStlCommand.py
similarity index 92%
rename from python/InvenTreeLink/commands/SendStepCommand.py
rename to python/InvenTreeLink/commands/ImportStlCommand.py
index ebe1067..58e6f28 100644
--- a/python/InvenTreeLink/commands/SendStepCommand.py
+++ b/python/InvenTreeLink/commands/ImportStlCommand.py
@@ -11,7 +11,7 @@
 from .. import config
 
 
-class SendStepCommand(apper.Fusion360CommandBase):
+class ImportStlCommand(apper.Fusion360CommandBase):
 
     @apper.lib_import(config.lib_path)
     def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
@@ -20,8 +20,6 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
         try:
             ao = apper.AppObjects()
 
-            ao.ui.messageBox("STEP FILE")
-
             if ao.ui.activeSelections.count == 1:
                 occ = adsk.fusion.Occurrence.cast(ao.ui.activeSelections[0].entity)
 
@@ -35,8 +33,6 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
 
                 self._write_step(temp_path, occ.component)
 
-                ao.ui.messageBox("File at {}, size: {}".format(temp_path, os.path.getsize(temp_path)))
-
                 part = functions.inventree_get_part(occ.component.id)
 
                 if part is False:
diff --git a/python/InvenTreeLink/commands/LoadInventreeBomCommand.py b/python/InvenTreeLink/commands/LoadInventreeBomCommand.py
deleted file mode 100644
index 759bfaa..0000000
--- a/python/InvenTreeLink/commands/LoadInventreeBomCommand.py
+++ /dev/null
@@ -1,48 +0,0 @@
-import adsk.core
-import adsk.fusion
-import adsk.cam
-
-from ..apper import apper
-from .. import config
-from .. import functions
-
-# Loads the Bill of Materials that Inventree has of this Part.
-class LoadInventreeBomCommand(apper.Fusion360CommandBase):
-    def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
-        try:
-            # Get Reference to Palette
-            ao = apper.AppObjects()
-            palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
-
-            # Send message to the HTML Page
-            if palette:
-                palette.sendInfoToHTML(
-                    config.DEF_SEND_BOM,
-                    '<br><br><br><div class="d-flex justify-content-center"><div class="spinner-border" role="status"><span class="visually-hidden">Loading...</span></div></div>'
-                )
-
-                # Work with it
-                bom_parts = functions.inventree_get_part([item['id'] for item in config.BOM])
-                for item in config.BOM:
-                    part = bom_parts[item['id']]
-
-                    if part is not False:
-                        item['status'] = "<span style='color: green;'> Synced </span>"
-                    else:
-                        item['status'] = "<span style='color: red;'>Not synced</span>"
-
-                body = ''.join(
-                    [
-                        f"<tr> <td> {item['name']} </td> <td> {item['instances']} </td> <td> {item['status']} </td> </tr>"
-                        for item in config.BOM
-                    ]
-                )
-                table = '<div class="overflow-auto"><table class="table table-sm table-striped table-hover"><thead><tr><th scope="col">Name</th><th scope="col">Count</th><th scope="col">Status</th></tr></thead><tbody>{body}</tbody></table></div>'.format(body=body)
-
-                palette.sendInfoToHTML(
-                    config.DEF_SEND_BOM,
-                    '{table}'.format(table=table)
-                )
-        except Exception as _e:
-            config.app_tracking.capture_exception(_e)
-            raise _e
diff --git a/python/InvenTreeLink/commands/palette_html/palette.html b/python/InvenTreeLink/commands/palette_html/palette.html
index 54453a2..2547636 100644
--- a/python/InvenTreeLink/commands/palette_html/palette.html
+++ b/python/InvenTreeLink/commands/palette_html/palette.html
@@ -1,83 +1,103 @@
 <!doctype html>
 <html lang="en" style="font-size: small;">
-    <head>
-        <meta charset="utf-8">
-        <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
 
-        <link href="static/css/bootstrap.min.css" rel="stylesheet">
-        <script src="static/js/bootstrap.min.js"></script>
+<head>
+    <meta charset="utf-8">
+    <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
 
-        <link href="static/css/themes/default/style.min.css" rel="stylesheet">
-        <script src="static/js/jquery-3.6.0.min.js"></script>
-        <script src="static/js/jstree.min.js"></script>
-    </head>
-    <body>
-        <div class="container-fluid">
-            <div class="card-group">
-                <div class="card card-default">
-                    <div class="card-header">
-                        <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse1" class="btn text-muted">more Information</a>
-                        </h4>
-                    </div>
-                    <div class="card-collapse collapsing" id="collapse1">
-                        <div class="card-body">
-                            <p id='p1'>Check the <a href ="https://matmair.github.io/F360-InvTree/">github</a> for more information</p>
-                        </div>
+    <link href="static/css/bootstrap.min.css" rel="stylesheet">
+    <script src="static/js/bootstrap.min.js"></script>
+
+    <link href="static/css/themes/default/style.min.css" rel="stylesheet">
+    <script src="static/js/jquery-3.6.0.min.js"></script>
+    <script src="static/js/jstree.min.js"></script>
+</head>
+
+<body>
+    <div class="container-fluid">
+        <div class="card-group">
+            <div class="card card-default">
+                <div class="card-header">
+                    <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse1"
+                            class="btn text-muted">more Information</a>
+                    </h4>
+                </div>
+                <div class="card-collapse collapsing" id="collapse1">
+                    <div class="card-body">
+                        <p id='p1'>Check the <a href="https://matmair.github.io/F360-InvTree/">github</a> for more
+                            information</p>
                     </div>
                 </div>
             </div>
+        </div>
 
-            <div class="btn-toolbar" role="toolbar" aria-label="Toolbar with button groups">
-                <div class="btn-group me-2" role="group" aria-label="First group">
-                    <button  type="button" onclick='sendGetInfo()' class="btn btn-outline-secondary"><img src="..\resources\SendBom\32x32.png"> Load BOM</button>
-                    <button type="button" onclick='sendGetBomOnline()' class="btn btn-outline-secondary"><img src="..\resources\SendOnlineState\32x32.png"> Refresh online data</button>
-                </div>
+        <div class="btn-toolbar" role="toolbar" aria-label="Toolbar with button groups">
+            <div class="btn-group me-2" role="group" aria-label="First group">
+                <button type="button" onclick='sendGetInfo()' class="btn btn-outline-secondary"><img
+                        src="..\resources\SendBom\32x32.png"> Refresh BOM </button>
+                <!-- <button type="button" onclick='sendGetBomOnline()' class="btn btn-outline-secondary"><img
+                        src="..\resources\SendOnlineState\32x32.png"> Refresh online data</button> -->
             </div>
-            <br>
+        </div>
+        <br>
 
-            <!-- Nav tabs -->
-            <ul class="nav nav-tabs" id="myTab" role="tablist">
-                <li class="nav-item" role="presentation">
-                    <button class="nav-link active" id="home-tab" data-bs-toggle="tab" data-bs-target="#home" type="button" role="tab">Home</button>
-                </li>
-                <li class="nav-item" role="presentation">
-                    <button class="nav-link" id="tree-tab" data-bs-toggle="tab" data-bs-target="#tree" type="button" role="tab">Structure</button>
-                </li>
-            </ul>
+        <!-- Nav tabs -->
+        <ul class="nav nav-tabs" id="myTab" role="tablist">
+            <li class="nav-item" role="presentation">
+                <button class="nav-link active" id="home-tab" data-bs-toggle="tab" data-bs-target="#home" type="button"
+                    role="tab">
+                    Home
+                </button>
+            </li>
 
-            <!-- Tab panes -->
-            <div class="tab-content">
-                <div class="tab-pane active" id="home" role="tabpanel" aria-labelledby="home-tab">
-                    <div id='BOM'>
-                        <p>Please load BOM.</p>
-                    </div>
-                </div>
+            <li class="nav-item" role="presentation">
+                <button class="nav-link" id="tree-tab" data-bs-toggle="tab" data-bs-target="#tree" type="button"
+                    role="tab">
+                    Structure
+                </button>
+            </li>
+        </ul>
 
-                <div class="tab-pane" id="tree" role="tabpanel" aria-labelledby="tree-tab">
-                    <p id="event_result">Load BOM to show BOM structure.</p>
-                    <div id="tree_div"></div>
+        <!-- Tab panes -->
+        <div class="tab-content">
+            <div class="tab-pane active" id="home" role="tabpanel" aria-labelledby="home-tab">
+                <div id='BOM'>
+                    <p>Please load BOM.</p>
                 </div>
             </div>
+
+            <div class="tab-pane" id="tree" role="tabpanel" aria-labelledby="tree-tab">
+                <p id="event_result">Load BOM to show BOM structure.</p>
+                <div id="tree_div"></div>
+            </div>
         </div>
-    </body>
+    </div>
+</body>
 
-    <script>
-        function sendGetInfo(){
-            var args = {};
-            adsk.fusionSendData('getBom', JSON.stringify(args));
-        }
-        function sendGetBomOnline(){
-            var args = {};
-            adsk.fusionSendData('getBomOnline', JSON.stringify(args));
-        }
-        function sendShowPart(id){
-            var args = {
-                id : id
-            };
-            adsk.fusionSendData('showPart', JSON.stringify(args));
-        }
+<script>
+    function sendGetInfo() {
+        var args = {};
+        adsk.fusionSendData('getBom', JSON.stringify(args));
+    }
+    
+    function sendGetBomOnline() {
+        var args = {};
+        adsk.fusionSendData('getBomOnline', JSON.stringify(args));
+    }
+    function sendShowPart(id) {
+        var args = {
+            id: id
+        };
+        adsk.fusionSendData('showPart', JSON.stringify(args));
+    }
+
+    function onClickUploadBom() {
+        var args = {};
+        adsk.fusionSendData('UploadBom', JSON.stringify(args));
+    }
 
-        window.fusionJavaScriptHandler = {handle: function(action, data){
+    window.fusionJavaScriptHandler = {
+        handle: function (action, data) {
             try {
                 if (action == 'SendBom') {
                     document.getElementById('BOM').innerHTML = data;
@@ -85,13 +105,13 @@ <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse
                 else if (action == 'SendTree') {
                     let data_in = JSON.parse(data);
                     $('#tree_div').jstree().on('changed.jstree', function (e, data) {
-                            var i, j, r = [];
-                            for(i = 0, j = data.selected.length; i < j; i++) {
-                                r.push(data.instance.get_node(data.selected[i]).text);
-                                sendShowPart(data.instance.get_node(data.selected[i]).id);
-                            }
-                            $('#event_result').html('Selected: ' + r.join(', '));
-                        });
+                        var i, j, r = [];
+                        for (i = 0, j = data.selected.length; i < j; i++) {
+                            r.push(data.instance.get_node(data.selected[i]).text);
+                            sendShowPart(data.instance.get_node(data.selected[i]).id);
+                        }
+                        $('#event_result').html('Selected: ' + r.join(', '));
+                    });
                     $('#tree_div').jstree(true).settings.core.data = data_in;
                     $('#tree_div').jstree(true).refresh();
                 }
@@ -106,6 +126,8 @@ <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse
                 console.log('exception caught with command: ' + action + ', data: ' + data);
             }
             return 'OK';
-        }};
-    </script>
+        }
+    };
+</script>
+
 </html>
\ No newline at end of file
diff --git a/python/InvenTreeLink/config.py b/python/InvenTreeLink/config.py
index 97e64bc..14c17c8 100644
--- a/python/InvenTreeLink/config.py
+++ b/python/InvenTreeLink/config.py
@@ -26,8 +26,6 @@
 CFG_PART_CATEGORY = 'part_category'
 
 # globals for reference
-BOM = []  # BOM-List
-BOM_HIR = []  # Hirarchical BOM
 INV_API = None  # API-connection
 CONFIG = {}  # Config section
 REF_CACHE = {}  # saves refs for reduced loading
diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index 1e174cc..04bd562 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -302,6 +302,15 @@ def extract_bom():
                 node['linked'] = occ.isReferencedComponent
                 bom.append(node)
 
+        bom_parts = inventree_get_part([item['id'] for item in bom])
+        for item in bom:
+            part = bom_parts[item['id']]
+
+            if part is not False:
+                item['synced'] = True # "<span style='color: green;'> Synced </span>"
+            else:
+                item['synced'] = False # "<span style='color: red;'> Not synced </span>"
+
         # Display the BOM
         return bom
     except Exception as _e:
@@ -338,7 +347,6 @@ def make_component_tree():
     node_list = []
 
     root_node = component_info(root)
-    root_node["type"] = "4-root_component"
     node_list.append(root_node)
 
     if root.occurrences.count > 0:

From d328c5f146d1ff24125fdd16a86640f8eb777dae Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 5 Nov 2021 17:55:13 +0100
Subject: [PATCH 06/13] Cleaned up UI

---
 python/InvenTreeLink/InvenTreeLink.py         | 65 ++++++++++---------
 ...{ShowPartCommand.py => EditPartCommand.py} | 62 +++++++++---------
 python/InvenTreeLink/config.py                |  2 +-
 3 files changed, 66 insertions(+), 63 deletions(-)
 rename python/InvenTreeLink/commands/{ShowPartCommand.py => EditPartCommand.py} (87%)

diff --git a/python/InvenTreeLink/InvenTreeLink.py b/python/InvenTreeLink/InvenTreeLink.py
index cd4915b..9ba57cb 100644
--- a/python/InvenTreeLink/InvenTreeLink.py
+++ b/python/InvenTreeLink/InvenTreeLink.py
@@ -12,41 +12,13 @@
     my_addin = apper.FusionApp(config.app_name, config.company_name, False)
     my_addin.root_path = config.app_path
 
-    from .commands.ShowPartCommand import ShowPartCommand
+    from .commands.EditPartCommand import EditPartCommand
     from .commands.BOMOverviewCommand import BomOverviewPaletteShowCommand
     from .commands.GenerateBomCommand import GenerateBomCommand
     from .commands.ImportStlCommand import ImportStlCommand
     from .commands.ImportPartCommand import ImportPartCommand
 
     # Commands
-    my_addin.add_command(
-        'Show part details',
-        ShowPartCommand,
-        {
-            'cmd_description': 'Show the InvenTree part-details for the selected part',
-            'cmd_id': config.DEF_SEND_PART,
-            'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.ToolbarPanelID.COMMANDS,
-            'cmd_resources': config.DEF_SEND_PART,
-            'command_visible': True,
-            'command_promoted': False,
-        }
-    )
-
-    my_addin.add_command(
-        'Upload STEP to attachments',
-        ImportStlCommand,
-        {
-            'cmd_description': 'Generates a STEP file and attaches it to a part',
-            'cmd_id': config.DEF_SEND_STEP,
-            'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.ToolbarPanelID.COMMANDS,
-            'cmd_resources': config.DEF_SEND_BOM,
-            'command_visible': True,
-            'command_promoted': False,
-            'palette_id': config.ITEM_PALETTE,
-        }
-    )
 
     # Palette
     my_addin.add_command(
@@ -89,14 +61,43 @@
     )
 
     my_addin.add_command(
-        'Import',
+        'Edit Part',
+        EditPartCommand,
+        {
+            'cmd_description': 'Show the InvenTree part-details for the selected part',
+            'cmd_id': config.DEF_SEND_PART,
+            'workspace': 'FusionSolidEnvironment',
+            'toolbar_panel_id': config.ToolbarPanelID.PART,
+            'cmd_resources': config.DEF_SEND_PART,
+            'command_visible': True,
+            'command_promoted': False,
+        }
+    )
+
+    my_addin.add_command(
+        'Import Part',
         ImportPartCommand,
         {
             'cmd_description': 'Import a Part as STL',
             'cmd_id': config.DEF_IMPORT_PART,
             'workspace': 'FusionSolidEnvironment',
-            'toolbar_panel_id': config.ToolbarPanelID.PARTS,
-            'cmd_resources': config.DEF_SEND_ONLINE_STATE,
+            'toolbar_panel_id': config.ToolbarPanelID.PART,
+            'cmd_resources': config.DEF_SEND_BOM,
+            'command_visible': True,
+            'command_promoted': False,
+            'palette_id': config.ITEM_PALETTE,
+        }
+    )
+
+    my_addin.add_command(
+        'Export STEP',
+        ImportStlCommand,
+        {
+            'cmd_description': 'Generates a STEP file and attaches it to a part',
+            'cmd_id': config.DEF_SEND_STEP,
+            'workspace': 'FusionSolidEnvironment',
+            'toolbar_panel_id': config.ToolbarPanelID.PART,
+            'cmd_resources': 'ShowPalette',
             'command_visible': True,
             'command_promoted': False,
             'palette_id': config.ITEM_PALETTE,
diff --git a/python/InvenTreeLink/commands/ShowPartCommand.py b/python/InvenTreeLink/commands/EditPartCommand.py
similarity index 87%
rename from python/InvenTreeLink/commands/ShowPartCommand.py
rename to python/InvenTreeLink/commands/EditPartCommand.py
index 2a6ff93..5440792 100644
--- a/python/InvenTreeLink/commands/ShowPartCommand.py
+++ b/python/InvenTreeLink/commands/EditPartCommand.py
@@ -10,11 +10,13 @@
 from ..functions import Fusion360Parameters
 
 
-class ShowPartCommand(apper.Fusion360CommandBase):
+class EditPartCommand(apper.Fusion360CommandBase):
     def on_input_changed(self, command, inputs, changed_input, input_values):
         try:
             ao = apper.AppObjects()
-            if ao.ui.activeSelections.count == 1:
+            if ao.ui.activeSelections.count == 1:              
+                # TODO: Support Design's
+
                 occ = adsk.fusion.Occurrence.cast(ao.ui.activeSelections[0].entity)
                 arg_id = changed_input.id
                 inp = inputs.command.commandInputs
@@ -84,21 +86,21 @@ def on_create(self, command, inputs):
             # Tabs
             tabCmdInput1 = inputs.addTabCommandInput('tab_1', 'Start')
             tab1ChildInputs = tabCmdInput1.children
-            tabCmdInput2 = inputs.addTabCommandInput('tab_2', 'Teil-Details')
-            tab2ChildInputs = tabCmdInput2.children
+            part_details_tab = inputs.addTabCommandInput('tab_2', 'Part Details')
+            part_details_children = part_details_tab.children
 
             # TextInputs for general information
-            tab2ChildInputs.addTextBoxCommandInput('text_id', 'id', 'id', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_name', 'name', 'name', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_description', 'description', 'description', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_opacity', 'opacity', 'opacity', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_partNumber', 'partNumber', 'partNumber', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_area', 'area', 'area', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_volume', 'volume', 'volume', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_mass', 'mass', 'mass', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_density', 'density', 'density', 1, True)
-            tab2ChildInputs.addTextBoxCommandInput('text_material', 'material', 'material', 1, True)
-            tableInput = tab2ChildInputs.addTableCommandInput('table', 'Table', 4, '1:1:1:1')
+            part_details_children.addTextBoxCommandInput('text_id', 'ID', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_name', 'Name', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_description', 'Description', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_opacity', 'Opacity', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_partNumber', 'Part Number', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_area', 'Area', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_volume', 'Volume', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_mass', 'Mass', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_density', 'Density', '', 1, True)
+            part_details_children.addTextBoxCommandInput('text_material', 'Material', '', 1, True)
+            tableInput = part_details_children.addTableCommandInput('table', 'Table', 4, '1:1:1:1')
             tableInput.isFullWidth = True
             tableInput.tablePresentationStyle = 2
 
@@ -107,31 +109,31 @@ def on_create(self, command, inputs):
             selectInput.addSelectionFilter('Occurrences')
             selectInput.setSelectionLimits(1, 1)
             # Buttons
-            tab1ChildInputs.addBoolValueInput('button_create', 'create part', False, 'commands/resources/ButtonCreate', True)
-            tab1ChildInputs.addBoolValueInput('button_refresh', 'refresh Information', False, 'commands/resources/SendOnlineState', True)
+            tab1ChildInputs.addBoolValueInput('button_create', 'Create part', False, 'commands/resources/ButtonCreate', True)
+            tab1ChildInputs.addBoolValueInput('button_refresh', 'Refresh', False, 'commands/resources/SendOnlineState', True)
 
             # TextInputs for InvenTree
             grpCmdInput1 = tab1ChildInputs.addGroupCommandInput('grp_1', 'General')
             grp1ChildInputs = grpCmdInput1.children
             # img = tab1ChildInputs.addImageCommandInput('text_part_image', 'image', 'blank_image.png')
             # img.isVisible = False  # TODO implement
-            grp1ChildInputs.addTextBoxCommandInput('text_part_name', 'name', 'name', 1, False)
-            grp1ChildInputs.addTextBoxCommandInput('text_part_ipn', 'IPN', 'IPN', 1, False)
-            grp1ChildInputs.addTextBoxCommandInput('text_part_description', 'description', 'description', 1, False)
-            grp1ChildInputs.addTextBoxCommandInput('text_part_notes', 'note', 'note', 2, False)
-            grp1ChildInputs.addTextBoxCommandInput('text_part_keywords', 'keywords', 'keywords', 1, False)
-            grp1ChildInputs.addTextBoxCommandInput('text_part_category', 'category', 'category', 1, True)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_name', 'Name', '', 1, False)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_ipn', 'IPN', '', 1, False)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_description', 'Description', '', 1, False)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_notes', 'Note', '', 2, False)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_keywords', 'Keywords', '', 1, False)
+            grp1ChildInputs.addTextBoxCommandInput('text_part_category', 'Category', 'category', 1, True)
             grp1ChildInputs.addTextBoxCommandInput('text_part_link', '', 'linktext', 1, True)
 
             grpCmdInput2 = tab1ChildInputs.addGroupCommandInput('grp_2', 'Settings')
             grp2ChildInputs = grpCmdInput2.children
-            grp2ChildInputs.addBoolValueInput('bool_part_virtual', 'virtual', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_template', 'template', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_assembly', 'assembly', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_component', 'component', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_trackable', 'trackable', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_purchaseable', 'purchaseable', True)
-            grp2ChildInputs.addBoolValueInput('bool_part_salable', 'salable', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_virtual', 'Virtual', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_template', 'Template', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_assembly', 'Assembly', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_component', 'Component', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_trackable', 'Trackable', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_purchaseable', 'Purchasable', True)
+            grp2ChildInputs.addBoolValueInput('bool_part_salable', 'Salable', True)
 
             grpCmdInput3 = tab1ChildInputs.addGroupCommandInput('grp_3', 'Supply')
             grp3ChildInputs = grpCmdInput3.children
diff --git a/python/InvenTreeLink/config.py b/python/InvenTreeLink/config.py
index 14c17c8..3139f1a 100644
--- a/python/InvenTreeLink/config.py
+++ b/python/InvenTreeLink/config.py
@@ -33,4 +33,4 @@
 class ToolbarPanelID:
     COMMANDS = "Commands"
     INVENTREE_LINK = "Inventree Link"
-    PARTS = "Parts"
\ No newline at end of file
+    PART = "Part"
\ No newline at end of file

From 731ddf92aa896a69319cc144cb7dd297d98d6ddb Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 12 Nov 2021 12:10:30 +0100
Subject: [PATCH 07/13] Added Sync All button to synchronize Fusion360 parts to
 InvenTree parts

---
 .../commands/BOMOverviewCommand.py            | 44 ++++++++++++++
 .../InvenTreeLink/commands/EditPartCommand.py | 50 +---------------
 .../commands/GenerateBomCommand.py            |  7 +--
 .../commands/palette_html/palette.html        |  5 ++
 python/InvenTreeLink/functions.py             | 12 ++--
 python/InvenTreeLink/helpers.py               | 59 +++++++++++++++++++
 6 files changed, 117 insertions(+), 60 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index 34fe258..00ca7ed 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -7,6 +7,7 @@
 from ..apper import apper
 from .. import config
 from .. import helpers
+from .. import functions
 
 
 # Class for a Fusion 360 Palette Command
@@ -46,9 +47,52 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                 selections.add(entitiesByToken)  # TODO selection not working
                 helpers.get_cmd(ao, config.DEF_SEND_PART).execute()
 
+            elif html_args.action == 'SyncAll':
+                from inventree.part import Part
+                ao.ui.messageBox("Sync All")
+                
+                bom = functions.extract_bom()
+                for item in bom:
+                    part = item['part'];
+
+                    if part == False:
+                        occ = item['occurence']
+
+                        item_list = Part.list(functions.inv_api(), IPN=occ.component.partNumber)
+                        if len(item_list) == 0:
+                            fusion_360_data = (
+                                f"ID: {occ.component.id}<br />"
+                                f"Area: {occ.component.physicalProperties.area}cm2<br />"
+                                f"Volume: {occ.physicalProperties.volume}cm3<br />"
+                                f"Mass: {occ.physicalProperties.mass}kg<br />"
+                                f"Density: {occ.physicalProperties.density}g/cm3<br />"
+                            )
+
+                            if occ.component.material and occ.component.material.name:
+                                fusion_360_data += f"Material: {occ.component.material.name}\n"
+
+                            text = (
+                                f"Part <b>{occ.component.partNumber} | {occ.component.name}</b> is not recognized by Inventree.<br />"
+                                "<b>Do you want to create a Part with the values provided by Fusion360?</b><br /><br />"
+                                f"<i>{fusion_360_data}</i>"
+                            )
+                            
+                            result = ao.ui.messageBox(text, "Sync All", 4, 1)
+                            if result == adsk.core.DialogResults.DialogYes:
+                                item['part'] = helpers.create_f360_part(occ, functions.config_ref(config.CFG_PART_CATEGORY))                     
+                            elif result == adsk.core.DialogResults.DialogNo:
+                                continue                          
+                            elif result == adsk.core.DialogResults.DialogCancel:
+                                return 
+                        elif len(item_list) == 1:
+                            # Just link it
+                            helpers.write_f360_parameters(part, occ)
+
             elif html_args.action == 'UploadBom':
                 ao.ui.messageBox("Uploading bom")
 
+
+
             # TODO investigate ghost answers
             # else:
             #     raise NotImplementedError('unknown message received from HTML')
diff --git a/python/InvenTreeLink/commands/EditPartCommand.py b/python/InvenTreeLink/commands/EditPartCommand.py
index 5440792..32a0b29 100644
--- a/python/InvenTreeLink/commands/EditPartCommand.py
+++ b/python/InvenTreeLink/commands/EditPartCommand.py
@@ -151,54 +151,8 @@ def on_create(self, command, inputs):
             helpers.error()
 
     # cstm fnc
-    @apper.lib_import(config.lib_path)
-    def part_create(self, occ, cat):
-        """ create part based on occurence """
-        from inventree.part import Part
-
-        ao = apper.AppObjects()
-
-        # build up args
-        part_kargs = {
-            'name': occ.component.name,
-            'description': occ.component.description if occ.component.description else 'None',
-            'IPN': occ.component.partNumber,
-            'active': True,
-            'virtual': False,
-        }
-        # add category if set
-        if cat:
-            part_kargs.update({'category': cat.pk})
-        # create part itself
-        part = Part.create(functions.inv_api(), part_kargs)
-        # check if part created - else raise error
-        if not part:
-            ao.ui.messageBox('Error occured during API-call')
-            return
-        elif not part.pk:
-            error_detail = [f'<strong>{a}</strong>\n{b[0]}' for a, b in part._data.items()]
-            ao.ui.messageBox(f'Error occured:<br><br>{"<br>".join(error_detail)}')
-            return
-
-        Fusion360Parameters.ID.value.create_parameter(part, occ.component.id)
-        Fusion360Parameters.AREA.value.create_parameter(part, occ.physicalProperties.area)
-        Fusion360Parameters.VOLUME.value.create_parameter(part, occ.physicalProperties.volume)
-        Fusion360Parameters.MASS.value.create_parameter(part, occ.physicalProperties.mass)
-        Fusion360Parameters.DENSITY.value.create_parameter(part, occ.physicalProperties.density)
-
-        if occ.component.material and occ.component.material.name:
-            Fusion360Parameters.MATERIAL.value.create_parameter(part, occ.component.material.name)
-
-        axis = ['x', 'y', 'z']
-        bb_min = {a: getattr(occ.boundingBox.minPoint, a) for a in axis}
-        bb_max = {a: getattr(occ.boundingBox.maxPoint, a) for a in axis}
-        bb = {a: bb_max[a] - bb_min[a] for a in axis}
-
-        Fusion360Parameters.BOUNDING_BOX_WIDTH.value.create_parameter(part, bb["x"])
-        Fusion360Parameters.BOUNDING_BOX_HEIGHT.value.create_parameter(part, bb["y"])
-        Fusion360Parameters.BOUNDING_BOX_DEPTH.value.create_parameter(part, bb["z"])
-
-        return part
+    def part_create(self, occ: adsk.fusion.Occurrence, cat: str):
+        return helpers.create_f360_part(occ, cat)
 
     def part_refresh(self, occ, inp, part):
         """ updates PartInfo command-inputs with values for supplied parts """
diff --git a/python/InvenTreeLink/commands/GenerateBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
index 072eb66..6b399b6 100644
--- a/python/InvenTreeLink/commands/GenerateBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -36,7 +36,6 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                 start = datetime.now()
 
                 bom = functions.extract_bom()
-                unsynced_parts = [item for item in bom if item['synced'] is False]
 
                 component_tree = functions.make_component_tree()
                 
@@ -50,9 +49,9 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                 body = ''.join([
                     ''.join((
                         "<tr>"
-                        f"<td>{item['name']}</td>"
+                        f"<td>{item['IPN']} | {item['name']}</td>"
                         f"<td>{item['instances']}</td>"
-                        f"<td>{element_synced if item['synced'] else element_not_synced}</td>"
+                        f"<td>{element_synced if item['part'] else element_not_synced}</td>"
                         "</tr>" 
                     ))
                     for item in bom
@@ -74,7 +73,7 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                     header,
                     f"<p> {len(bom)} parts found in {datetime.now() - start}</p>"
                     f"{table_c}"
-                    "<button onclick='' class='btn btn-outline-secondary'> Sync All </button>" 
+                    "<button onclick='onClickSyncAll()' class='btn btn-outline-secondary'> Sync All </button>" 
                     "<button onclick='onClickUploadBom()' class='btn btn-outline-secondary'> Upload BOM </button>"    
                 ))
 
diff --git a/python/InvenTreeLink/commands/palette_html/palette.html b/python/InvenTreeLink/commands/palette_html/palette.html
index 2547636..5d0ab6a 100644
--- a/python/InvenTreeLink/commands/palette_html/palette.html
+++ b/python/InvenTreeLink/commands/palette_html/palette.html
@@ -95,6 +95,11 @@ <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse
         var args = {};
         adsk.fusionSendData('UploadBom', JSON.stringify(args));
     }
+    
+    function onClickSyncAll() {
+        var args = {};
+        adsk.fusionSendData('SyncAll', JSON.stringify(args));
+    }
 
     window.fusionJavaScriptHandler = {
         handle: function (action, data) {
diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index 04bd562..cb072e2 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -260,7 +260,6 @@ def search(parameters, part_id):
     return search(parameters, part_id)
 # endregion
 
-
 # region bom functions
 def extract_bom():
     """ returns bom """
@@ -300,16 +299,13 @@ def extract_bom():
                 node = component_info(comp, comp_set=True)
                 node['volume'] = volume
                 node['linked'] = occ.isReferencedComponent
+                node['occurence'] = occ
+
                 bom.append(node)
 
         bom_parts = inventree_get_part([item['id'] for item in bom])
         for item in bom:
-            part = bom_parts[item['id']]
-
-            if part is not False:
-                item['synced'] = True # "<span style='color: green;'> Synced </span>"
-            else:
-                item['synced'] = False # "<span style='color: red;'> Not synced </span>"
+            item['part'] = bom_parts[item['id']]
 
         # Display the BOM
         return bom
@@ -318,7 +314,7 @@ def extract_bom():
         raise _e
 
 
-def component_info(comp, parent='#', comp_set=False):
+def component_info(comp: adsk.fusion.Component, parent='#', comp_set: bool = False):
     """ returns a node element """
     node = {
         'name': comp.name,
diff --git a/python/InvenTreeLink/helpers.py b/python/InvenTreeLink/helpers.py
index 2da2dc5..ca740ba 100644
--- a/python/InvenTreeLink/helpers.py
+++ b/python/InvenTreeLink/helpers.py
@@ -3,6 +3,8 @@
 import re
 import traceback
 
+import adsk
+
 from .apper import apper
 from . import config
 
@@ -44,3 +46,60 @@ def error(typ_str=None):
 def get_cmd(ao, key):
     ref_name = f'{config.company_name}_{config.app_name}_{key}'
     return ao.ui.commandDefinitions.itemById(ref_name)
+
+@apper.lib_import(config.lib_path)
+def create_f360_part(occ: adsk.fusion.Occurrence, cat: str):
+    """ create part based on occurence """
+    from inventree.part import Part
+    from . import functions
+    from .functions import Fusion360Parameters
+
+    ao = apper.AppObjects()
+
+    # build up args
+    part_kargs = {
+        'name': occ.component.name,
+        'description': occ.component.description if occ.component.description else 'None',
+        'IPN': occ.component.partNumber,
+        'active': True,
+        'virtual': False,
+    }
+    # add category if set
+    if cat:
+        part_kargs.update({'category': cat.pk})
+    # create part itself
+    part = Part.create(functions.inv_api(), part_kargs)
+    # check if part created - else raise error
+    if not part:
+        ao.ui.messageBox('Error occured during API-call')
+        return
+    elif not part.pk:
+        error_detail = [f'<strong>{a}</strong>\n{b[0]}' for a, b in part._data.items()]
+        ao.ui.messageBox(f'Error occured:<br><br>{"<br>".join(error_detail)}')
+        return
+
+    write_f360_parameters(part, occ)
+
+    return part
+
+@apper.lib_import(config.lib_path)
+def write_f360_parameters(part, occ: adsk.fusion.Occurrence):    
+    from .functions import Fusion360Parameters
+
+    Fusion360Parameters.ID.value.create_parameter(part, occ.component.id)
+    Fusion360Parameters.AREA.value.create_parameter(part, occ.physicalProperties.area)
+    Fusion360Parameters.VOLUME.value.create_parameter(part, occ.physicalProperties.volume)
+    Fusion360Parameters.MASS.value.create_parameter(part, occ.physicalProperties.mass)
+    Fusion360Parameters.DENSITY.value.create_parameter(part, occ.physicalProperties.density)
+
+    if occ.component.material and occ.component.material.name:
+        Fusion360Parameters.MATERIAL.value.create_parameter(part, occ.component.material.name)
+
+    axis = ['x', 'y', 'z']
+    bb_min = {a: getattr(occ.boundingBox.minPoint, a) for a in axis}
+    bb_max = {a: getattr(occ.boundingBox.maxPoint, a) for a in axis}
+    bb = {a: bb_max[a] - bb_min[a] for a in axis}
+
+    Fusion360Parameters.BOUNDING_BOX_WIDTH.value.create_parameter(part, bb["x"])
+    Fusion360Parameters.BOUNDING_BOX_HEIGHT.value.create_parameter(part, bb["y"])
+    Fusion360Parameters.BOUNDING_BOX_DEPTH.value.create_parameter(part, bb["z"])
\ No newline at end of file

From b06a12455c9b9ca71a93c3cbc3cad0631a746485 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 12 Nov 2021 14:36:32 +0100
Subject: [PATCH 08/13] Synced items with the Bill of Materials on InvenTree

---
 .../commands/BOMOverviewCommand.py            | 138 ++++++++++++++----
 python/InvenTreeLink/helpers.py               |  35 ++---
 2 files changed, 129 insertions(+), 44 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index 00ca7ed..62c1fb7 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -9,6 +9,10 @@
 from .. import helpers
 from .. import functions
 
+from inventree.part import Part
+from inventree.part import BomItem
+        
+from ..functions import inv_api, inventree_get_part
 
 # Class for a Fusion 360 Palette Command
 class BomOverviewPaletteShowCommand(apper.PaletteCommandBase):
@@ -23,7 +27,7 @@ def on_palette_execute(self, palette: adsk.core.Palette):
     # Run when ever a fusion event is fired from the corresponding web page
     def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
         data = json.loads(html_args.data)
-
+        
         try:
             ao = apper.AppObjects()
             palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
@@ -48,50 +52,95 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                 helpers.get_cmd(ao, config.DEF_SEND_PART).execute()
 
             elif html_args.action == 'SyncAll':
-                from inventree.part import Part
-                ao.ui.messageBox("Sync All")
+                
+                root = ao.product.rootComponent
+                root_part = inventree_get_part(root.id)
+
+                if root_part == False:      
+                    item_list = Part.list(inv_api(), IPN=root.partNumber)
+                    if len(item_list) == 0:
+                        result = propose_create_f360_part(ao, root, 3)
+
+                        if result == adsk.core.DialogResults.DialogYes:
+                            root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))               
+                        elif result == adsk.core.DialogResults.DialogNo:
+                            return 
+
+                    elif len(item_list) == 1:
+                        ao.ui.messageBox(f"Part was not linked yet but found Part for {item_list[0].IPN} | {item_list[0].name} so linking it.", "Sync All")
+                        root_part = item_list[0]
+
+                    elif len(item_list) > 1:
+                        ao.ui.messageBox(f"More than one item found with IPN <b>{root.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
+                        return
+                
+                root_part.save(data={ 
+                    'name': root.name,
+                    'IPN': root.partNumber,
+                    'description': root.description if root.description else 'None'
+                })
+                helpers.write_f360_parameters(root_part, root)  
                 
                 bom = functions.extract_bom()
                 for item in bom:
-                    part = item['part'];
+                    part = item['part']
 
                     if part == False:
-                        occ = item['occurence']
+                        component = item['occurence'].component
 
-                        item_list = Part.list(functions.inv_api(), IPN=occ.component.partNumber)
+                        item_list = Part.list(functions.inv_api(), IPN=component.partNumber)
                         if len(item_list) == 0:
-                            fusion_360_data = (
-                                f"ID: {occ.component.id}<br />"
-                                f"Area: {occ.component.physicalProperties.area}cm2<br />"
-                                f"Volume: {occ.physicalProperties.volume}cm3<br />"
-                                f"Mass: {occ.physicalProperties.mass}kg<br />"
-                                f"Density: {occ.physicalProperties.density}g/cm3<br />"
-                            )
-
-                            if occ.component.material and occ.component.material.name:
-                                fusion_360_data += f"Material: {occ.component.material.name}\n"
-
-                            text = (
-                                f"Part <b>{occ.component.partNumber} | {occ.component.name}</b> is not recognized by Inventree.<br />"
-                                "<b>Do you want to create a Part with the values provided by Fusion360?</b><br /><br />"
-                                f"<i>{fusion_360_data}</i>"
-                            )
-                            
-                            result = ao.ui.messageBox(text, "Sync All", 4, 1)
+                            result = propose_create_f360_part(ao, component, 4)                            
                             if result == adsk.core.DialogResults.DialogYes:
-                                item['part'] = helpers.create_f360_part(occ, functions.config_ref(config.CFG_PART_CATEGORY))                     
+                                part = helpers.create_f360_part(component, functions.config_ref(config.CFG_PART_CATEGORY))                     
                             elif result == adsk.core.DialogResults.DialogNo:
                                 continue                          
                             elif result == adsk.core.DialogResults.DialogCancel:
                                 return 
+                                
                         elif len(item_list) == 1:
+                            ao.ui.messageBox(f"Part was not linked yet but found Part for {item_list[0].ipn} | {item_list[0].name} so linking it.", "Sync All")
                             # Just link it
-                            helpers.write_f360_parameters(part, occ)
+                            part = item_list[0]
+
+                        elif len(item_list) > 1:
+                            ao.ui.messageBox(f"More than one item found with IPN <b>{component.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
+                            continue
+                        
+                        part.save(data={ 
+                            'name': component.name,
+                            'IPN': component.partNumber,
+                            'description': component.description if component.description else 'None'
+                        })
+                        helpers.write_f360_parameters(part, component)  
+
+                # Refresh this table                
+                if palette:
+                    command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
+                    command.execute()
 
             elif html_args.action == 'UploadBom':
                 ao.ui.messageBox("Uploading bom")
+                
+                root = ao.product.rootComponent
+                root_part = functions.inventree_get_part(root.id)
 
-
+                bom = functions.extract_bom()
+                if False in [item['part'] for item in bom] or root_part is False:
+                    ao.ui.messageBox("There's a Part not synced to InvenTree, unable to upload BOM.")
+                    return
+
+                if root_part.assembly is False:
+                    root_part.save({
+                        'assembly': True
+                    })
+
+                for item in bom:                    
+                    BomItem.create(inv_api(), {
+                        'part': root_part.pk,
+                        'quantity': item['instances'],
+                        'sub_part': item['part'].pk
+                    })
 
             # TODO investigate ghost answers
             # else:
@@ -103,3 +152,38 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
     # Handle any extra cleanup when user closes palette here
     def on_palette_close(self):
         pass
+
+def propose_create_f360_part(ao, component: adsk.fusion.Component, buttons):
+    physicalProperties = component.physicalProperties
+
+    fusion_360_data = (
+        f"Part Number: <b>{component.partNumber}</b><br />"
+        f"Name: <b>{component.name}</b><br />"
+        f"Description: <b>{component.description}</b><br /><br />"
+
+        f"ID: {component.id}<br />"
+        f"Area: {physicalProperties.area}cm2<br />"
+        f"Volume: {physicalProperties.volume}cm3<br />"
+        f"Mass: {physicalProperties.mass}kg<br />"
+        f"Density: {physicalProperties.density}g/cm3<br />"
+    )
+
+    if component.material and component.material.name:
+        fusion_360_data += f"Material: {component.material.name}<br />"
+
+    axis = ['x', 'y', 'z']
+    bb_min = {a: getattr(component.boundingBox.minPoint, a) for a in axis}
+    bb_max = {a: getattr(component.boundingBox.maxPoint, a) for a in axis}
+    bb = {a: bb_max[a] - bb_min[a] for a in axis}
+
+    fusion_360_data += (
+        f"Dimensions: {bb['x']}cm x {bb['y']}cm x {bb['z']}cm<br />"
+    )
+
+    text = (
+        f"Part <b>{component.partNumber} | {component.name}</b> is not recognized by Inventree.<br />"
+        "<b>Do you want to create a Part with the values provided by Fusion360?</b><br /><br />"
+        f"<i>{fusion_360_data}</i>"
+    )
+
+    return ao.ui.messageBox(text, "Sync All", buttons, 1)
\ No newline at end of file
diff --git a/python/InvenTreeLink/helpers.py b/python/InvenTreeLink/helpers.py
index ca740ba..29bab44 100644
--- a/python/InvenTreeLink/helpers.py
+++ b/python/InvenTreeLink/helpers.py
@@ -48,19 +48,18 @@ def get_cmd(ao, key):
     return ao.ui.commandDefinitions.itemById(ref_name)
 
 @apper.lib_import(config.lib_path)
-def create_f360_part(occ: adsk.fusion.Occurrence, cat: str):
+def create_f360_part(component: adsk.fusion.Component, cat: str):
     """ create part based on occurence """
     from inventree.part import Part
     from . import functions
-    from .functions import Fusion360Parameters
 
     ao = apper.AppObjects()
 
     # build up args
     part_kargs = {
-        'name': occ.component.name,
-        'description': occ.component.description if occ.component.description else 'None',
-        'IPN': occ.component.partNumber,
+        'name': component.name,
+        'description': component.description if component.description else 'None',
+        'IPN': component.partNumber,
         'active': True,
         'virtual': False,
     }
@@ -78,28 +77,30 @@ def create_f360_part(occ: adsk.fusion.Occurrence, cat: str):
         ao.ui.messageBox(f'Error occured:<br><br>{"<br>".join(error_detail)}')
         return
 
-    write_f360_parameters(part, occ)
+    write_f360_parameters(part, component)
 
     return part
 
 @apper.lib_import(config.lib_path)
-def write_f360_parameters(part, occ: adsk.fusion.Occurrence):    
+def write_f360_parameters(part, component: adsk.fusion.Component):    
     from .functions import Fusion360Parameters
 
-    Fusion360Parameters.ID.value.create_parameter(part, occ.component.id)
-    Fusion360Parameters.AREA.value.create_parameter(part, occ.physicalProperties.area)
-    Fusion360Parameters.VOLUME.value.create_parameter(part, occ.physicalProperties.volume)
-    Fusion360Parameters.MASS.value.create_parameter(part, occ.physicalProperties.mass)
-    Fusion360Parameters.DENSITY.value.create_parameter(part, occ.physicalProperties.density)
+    physicalProperties = component.physicalProperties
+
+    Fusion360Parameters.ID.value.create_parameter(part, component.id)
+    Fusion360Parameters.AREA.value.create_parameter(part, physicalProperties.area)
+    Fusion360Parameters.VOLUME.value.create_parameter(part, physicalProperties.volume)
+    Fusion360Parameters.MASS.value.create_parameter(part, physicalProperties.mass)
+    Fusion360Parameters.DENSITY.value.create_parameter(part, physicalProperties.density)
 
-    if occ.component.material and occ.component.material.name:
-        Fusion360Parameters.MATERIAL.value.create_parameter(part, occ.component.material.name)
+    if component.material and component.material.name:
+        Fusion360Parameters.MATERIAL.value.create_parameter(part, component.material.name)
 
     axis = ['x', 'y', 'z']
-    bb_min = {a: getattr(occ.boundingBox.minPoint, a) for a in axis}
-    bb_max = {a: getattr(occ.boundingBox.maxPoint, a) for a in axis}
+    bb_min = {a: getattr(component.boundingBox.minPoint, a) for a in axis}
+    bb_max = {a: getattr(component.boundingBox.maxPoint, a) for a in axis}
     bb = {a: bb_max[a] - bb_min[a] for a in axis}
 
     Fusion360Parameters.BOUNDING_BOX_WIDTH.value.create_parameter(part, bb["x"])
     Fusion360Parameters.BOUNDING_BOX_HEIGHT.value.create_parameter(part, bb["y"])
-    Fusion360Parameters.BOUNDING_BOX_DEPTH.value.create_parameter(part, bb["z"])
\ No newline at end of file
+    Fusion360Parameters.BOUNDING_BOX_DEPTH.value.create_parameter(part, bb["z"])

From d3d9f5cb318194b0381f207921c5db6d59274399 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 12 Nov 2021 16:00:37 +0100
Subject: [PATCH 09/13] Pre recurse function

---
 .../commands/BOMOverviewCommand.py            | 48 +++++++++++++++----
 python/InvenTreeLink/functions.py             | 28 ++++++-----
 python/InvenTreeLink/helpers.py               | 18 +++----
 3 files changed, 63 insertions(+), 31 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index 62c1fb7..94b6020 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -8,9 +8,6 @@
 from .. import config
 from .. import helpers
 from .. import functions
-
-from inventree.part import Part
-from inventree.part import BomItem
         
 from ..functions import inv_api, inventree_get_part
 
@@ -28,6 +25,9 @@ def on_palette_execute(self, palette: adsk.core.Palette):
     def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
         data = json.loads(html_args.data)
         
+        from inventree.part import Part
+        from inventree.part import BomItem
+
         try:
             ao = apper.AppObjects()
             palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
@@ -67,7 +67,6 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                             return 
 
                     elif len(item_list) == 1:
-                        ao.ui.messageBox(f"Part was not linked yet but found Part for {item_list[0].IPN} | {item_list[0].name} so linking it.", "Sync All")
                         root_part = item_list[0]
 
                     elif len(item_list) > 1:
@@ -99,7 +98,6 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                                 return 
                                 
                         elif len(item_list) == 1:
-                            ao.ui.messageBox(f"Part was not linked yet but found Part for {item_list[0].ipn} | {item_list[0].name} so linking it.", "Sync All")
                             # Just link it
                             part = item_list[0]
 
@@ -119,9 +117,7 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                     command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
                     command.execute()
 
-            elif html_args.action == 'UploadBom':
-                ao.ui.messageBox("Uploading bom")
-                
+            elif html_args.action == 'UploadBom':                
                 root = ao.product.rootComponent
                 root_part = functions.inventree_get_part(root.id)
 
@@ -186,4 +182,38 @@ def propose_create_f360_part(ao, component: adsk.fusion.Component, buttons):
         f"<i>{fusion_360_data}</i>"
     )
 
-    return ao.ui.messageBox(text, "Sync All", buttons, 1)
\ No newline at end of file
+    return ao.ui.messageBox(text, "Sync All", buttons, 1)
+
+def sync_all(ao, root: adsk.fusion.Component):
+    from inventree.part import Part
+    from inventree.part import BomItem
+
+    root_part = inventree_get_part(root.id)
+
+    if root_part == False:      
+        item_list = Part.list(inv_api(), IPN=root.partNumber)
+        if len(item_list) == 0:
+            result = propose_create_f360_part(ao, root, 3)
+
+            if result == adsk.core.DialogResults.DialogYes:
+                root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))               
+            elif result == adsk.core.DialogResults.DialogNo:
+                return 
+
+        elif len(item_list) == 1:
+            root_part = item_list[0]
+
+        elif len(item_list) > 1:
+            ao.ui.messageBox(f"More than one item found with IPN <b>{root.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
+            return
+    
+    root_part.save(data={ 
+        'name': root.name,
+        'IPN': root.partNumber,
+        'description': root.description if root.description else 'None'
+    })
+    helpers.write_f360_parameters(root_part, root)      
+
+    for occurrence in root.occurrences:
+        if occurrence.component:
+            sync_all(occurrence.component)
diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index cb072e2..81a2d78 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -31,23 +31,25 @@ def create_template(self):
         })
 
     @apper.lib_import(config.lib_path)
-    def create_parameter(self, part, data):
+    def set(self, part, data):
         from inventree.base import Parameter
+        
+        params = Parameter.list(inv_api(), 
+            part=part.pk,
+            template=self.pk
+        )
 
-        Parameter.create(inv_api(), {'part': part.pk, 'template': self.pk, 'data': data})
+        if len(params) == 0:    
+            Parameter.create(inv_api(), {'part': part.pk, 'template': self.pk, 'data': data})
+        else:
+            param = params[0]
+            param.save({
+                "data": data
+            })
 
     @apper.lib_import(config.lib_path)
-    def update_parameter(self, part, data):
-        from inventree.base import Parameter
-
-        param = Parameter.list(inv_api(), {
-            "part": part.pk,
-            "template": self.pk
-        })[0]
-
-        param.save({
-            "data": data
-        })
+    def has_parameter(self, part):
+        test = ""  
 
     __PART_TEMPLATE_CACHE = {}
 
diff --git a/python/InvenTreeLink/helpers.py b/python/InvenTreeLink/helpers.py
index 29bab44..82832f2 100644
--- a/python/InvenTreeLink/helpers.py
+++ b/python/InvenTreeLink/helpers.py
@@ -87,20 +87,20 @@ def write_f360_parameters(part, component: adsk.fusion.Component):
 
     physicalProperties = component.physicalProperties
 
-    Fusion360Parameters.ID.value.create_parameter(part, component.id)
-    Fusion360Parameters.AREA.value.create_parameter(part, physicalProperties.area)
-    Fusion360Parameters.VOLUME.value.create_parameter(part, physicalProperties.volume)
-    Fusion360Parameters.MASS.value.create_parameter(part, physicalProperties.mass)
-    Fusion360Parameters.DENSITY.value.create_parameter(part, physicalProperties.density)
+    Fusion360Parameters.ID.value.set(part, component.id)
+    Fusion360Parameters.AREA.value.set(part, physicalProperties.area)
+    Fusion360Parameters.VOLUME.value.set(part, physicalProperties.volume)
+    Fusion360Parameters.MASS.value.set(part, physicalProperties.mass)
+    Fusion360Parameters.DENSITY.value.set(part, physicalProperties.density)
 
     if component.material and component.material.name:
-        Fusion360Parameters.MATERIAL.value.create_parameter(part, component.material.name)
+        Fusion360Parameters.MATERIAL.value.set(part, component.material.name)
 
     axis = ['x', 'y', 'z']
     bb_min = {a: getattr(component.boundingBox.minPoint, a) for a in axis}
     bb_max = {a: getattr(component.boundingBox.maxPoint, a) for a in axis}
     bb = {a: bb_max[a] - bb_min[a] for a in axis}
 
-    Fusion360Parameters.BOUNDING_BOX_WIDTH.value.create_parameter(part, bb["x"])
-    Fusion360Parameters.BOUNDING_BOX_HEIGHT.value.create_parameter(part, bb["y"])
-    Fusion360Parameters.BOUNDING_BOX_DEPTH.value.create_parameter(part, bb["z"])
+    Fusion360Parameters.BOUNDING_BOX_WIDTH.value.set(part, bb["x"])
+    Fusion360Parameters.BOUNDING_BOX_HEIGHT.value.set(part, bb["y"])
+    Fusion360Parameters.BOUNDING_BOX_DEPTH.value.set(part, bb["z"])

From e20b830a17583065f2dfe15dafdc2650f934a39a Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 12 Nov 2021 17:32:32 +0100
Subject: [PATCH 10/13] Made recursive syncing

---
 .../commands/BOMOverviewCommand.py            | 131 ++++++------------
 .../commands/GenerateBomCommand.py            |   3 +-
 python/InvenTreeLink/helpers.py               |  17 ++-
 3 files changed, 54 insertions(+), 97 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index 94b6020..f0c6631 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -54,90 +54,13 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
             elif html_args.action == 'SyncAll':
                 
                 root = ao.product.rootComponent
-                root_part = inventree_get_part(root.id)
-
-                if root_part == False:      
-                    item_list = Part.list(inv_api(), IPN=root.partNumber)
-                    if len(item_list) == 0:
-                        result = propose_create_f360_part(ao, root, 3)
-
-                        if result == adsk.core.DialogResults.DialogYes:
-                            root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))               
-                        elif result == adsk.core.DialogResults.DialogNo:
-                            return 
-
-                    elif len(item_list) == 1:
-                        root_part = item_list[0]
-
-                    elif len(item_list) > 1:
-                        ao.ui.messageBox(f"More than one item found with IPN <b>{root.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
-                        return
-                
-                root_part.save(data={ 
-                    'name': root.name,
-                    'IPN': root.partNumber,
-                    'description': root.description if root.description else 'None'
-                })
-                helpers.write_f360_parameters(root_part, root)  
-                
-                bom = functions.extract_bom()
-                for item in bom:
-                    part = item['part']
-
-                    if part == False:
-                        component = item['occurence'].component
-
-                        item_list = Part.list(functions.inv_api(), IPN=component.partNumber)
-                        if len(item_list) == 0:
-                            result = propose_create_f360_part(ao, component, 4)                            
-                            if result == adsk.core.DialogResults.DialogYes:
-                                part = helpers.create_f360_part(component, functions.config_ref(config.CFG_PART_CATEGORY))                     
-                            elif result == adsk.core.DialogResults.DialogNo:
-                                continue                          
-                            elif result == adsk.core.DialogResults.DialogCancel:
-                                return 
-                                
-                        elif len(item_list) == 1:
-                            # Just link it
-                            part = item_list[0]
-
-                        elif len(item_list) > 1:
-                            ao.ui.messageBox(f"More than one item found with IPN <b>{component.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
-                            continue
-                        
-                        part.save(data={ 
-                            'name': component.name,
-                            'IPN': component.partNumber,
-                            'description': component.description if component.description else 'None'
-                        })
-                        helpers.write_f360_parameters(part, component)  
+                sync_all(ao, root)
 
                 # Refresh this table                
                 if palette:
                     command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
                     command.execute()
 
-            elif html_args.action == 'UploadBom':                
-                root = ao.product.rootComponent
-                root_part = functions.inventree_get_part(root.id)
-
-                bom = functions.extract_bom()
-                if False in [item['part'] for item in bom] or root_part is False:
-                    ao.ui.messageBox("There's a Part not synced to InvenTree, unable to upload BOM.")
-                    return
-
-                if root_part.assembly is False:
-                    root_part.save({
-                        'assembly': True
-                    })
-
-                for item in bom:                    
-                    BomItem.create(inv_api(), {
-                        'part': root_part.pk,
-                        'quantity': item['instances'],
-                        'sub_part': item['part'].pk
-                    })
-
             # TODO investigate ghost answers
             # else:
             #     raise NotImplementedError('unknown message received from HTML')
@@ -184,7 +107,7 @@ def propose_create_f360_part(ao, component: adsk.fusion.Component, buttons):
 
     return ao.ui.messageBox(text, "Sync All", buttons, 1)
 
-def sync_all(ao, root: adsk.fusion.Component):
+def sync_all(ao, root: adsk.fusion.Component, updated={}):
     from inventree.part import Part
     from inventree.part import BomItem
 
@@ -193,27 +116,53 @@ def sync_all(ao, root: adsk.fusion.Component):
     if root_part == False:      
         item_list = Part.list(inv_api(), IPN=root.partNumber)
         if len(item_list) == 0:
-            result = propose_create_f360_part(ao, root, 3)
-
-            if result == adsk.core.DialogResults.DialogYes:
-                root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))               
-            elif result == adsk.core.DialogResults.DialogNo:
-                return 
+            root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))
 
         elif len(item_list) == 1:
             root_part = item_list[0]
 
         elif len(item_list) > 1:
-            ao.ui.messageBox(f"More than one item found with IPN <b>{root.partNumber}. Please resolve this in InvenTree for now.", "Sync All")
+            ao.ui.messageBox((
+                f"Part <i>{root.name}</i> does not have a unique IPN <b>{root.partNumber}</b>.<br />"
+                "<b>Please resolve this in InvenTree for now.</b>"
+            ), "Sync All")
             return
     
+    # Delete previous bom
+    for item in root_part.getBomItems():
+        item.delete() 
+
     root_part.save(data={ 
         'name': root.name,
         'IPN': root.partNumber,
-        'description': root.description if root.description else 'None'
+        'description': root.description if root.description else 'None',
+        # If the part has more than one occurences, it's probably an assembly.
+        # However, if you marked it Purchasable in InvenTree, it will not be an assembly.
+        'assembly': len(root.occurrences) > 0 and not root_part.purchaseable
     })
     helpers.write_f360_parameters(root_part, root)      
-
-    for occurrence in root.occurrences:
-        if occurrence.component:
-            sync_all(occurrence.component)
+       
+    if root_part.assembly:
+        instance_count = {}
+        for occurrence in root.occurrences:
+            if occurrence.component:
+                if str(occurrence.component.id) in instance_count:
+                    instance_count[occurrence.component.id] += 1
+                else:
+                    instance_count[occurrence.component.id] = 1
+
+                if not str(occurrence.component.id) in updated:
+                    sync_all(ao, occurrence.component, updated)
+                    updated[str(occurrence.component.id)] = True     
+
+        for id in instance_count:
+            part = inventree_get_part(id)    
+
+            if part is not False:
+                BomItem.create(inv_api(), {
+                    'part': root_part.pk,
+                    'quantity': instance_count[id],
+                    'sub_part': part.pk
+                })
+            else:
+                ao.ui.messageBox(f"Make sure you don't have duplicate IPN + names in child drawings of {root_part.name}!", "")
diff --git a/python/InvenTreeLink/commands/GenerateBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
index 6b399b6..89d1d64 100644
--- a/python/InvenTreeLink/commands/GenerateBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -73,8 +73,7 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                     header,
                     f"<p> {len(bom)} parts found in {datetime.now() - start}</p>"
                     f"{table_c}"
-                    "<button onclick='onClickSyncAll()' class='btn btn-outline-secondary'> Sync All </button>" 
-                    "<button onclick='onClickUploadBom()' class='btn btn-outline-secondary'> Upload BOM </button>"    
+                    "<button onclick='onClickSyncAll()' class='btn btn-outline-secondary'> Sync All </button>"  
                 ))
 
                 palette.sendInfoToHTML(
diff --git a/python/InvenTreeLink/helpers.py b/python/InvenTreeLink/helpers.py
index 82832f2..ef4f7e2 100644
--- a/python/InvenTreeLink/helpers.py
+++ b/python/InvenTreeLink/helpers.py
@@ -84,14 +84,23 @@ def create_f360_part(component: adsk.fusion.Component, cat: str):
 @apper.lib_import(config.lib_path)
 def write_f360_parameters(part, component: adsk.fusion.Component):    
     from .functions import Fusion360Parameters
+    import math
 
     physicalProperties = component.physicalProperties
 
     Fusion360Parameters.ID.value.set(part, component.id)
-    Fusion360Parameters.AREA.value.set(part, physicalProperties.area)
-    Fusion360Parameters.VOLUME.value.set(part, physicalProperties.volume)
-    Fusion360Parameters.MASS.value.set(part, physicalProperties.mass)
-    Fusion360Parameters.DENSITY.value.set(part, physicalProperties.density)
+
+    if not math.isnan(physicalProperties.area):
+        Fusion360Parameters.AREA.value.set(part, physicalProperties.area)
+
+    if not math.isnan(physicalProperties.volume):
+        Fusion360Parameters.VOLUME.value.set(part, physicalProperties.volume)
+
+    if not math.isnan(physicalProperties.mass):
+        Fusion360Parameters.MASS.value.set(part, physicalProperties.mass)
+
+    if not math.isnan(physicalProperties.density):
+        Fusion360Parameters.DENSITY.value.set(part, physicalProperties.density)
 
     if component.material and component.material.name:
         Fusion360Parameters.MATERIAL.value.set(part, component.material.name)

From 0b245059bb6df2db5b58164654213d4178801a63 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 19 Nov 2021 17:38:42 +0100
Subject: [PATCH 11/13] Further improved BOM loading into InvenTree

Added threads to keep the UI smooth while doing heavy operations
Added some nice UI in general
---
 .../commands/BOMOverviewCommand.py            | 96 ++++++++++++++++---
 .../commands/GenerateBomCommand.py            |  7 +-
 .../commands/palette_html/palette.html        |  5 +
 python/InvenTreeLink/config.py                |  1 +
 4 files changed, 94 insertions(+), 15 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index f0c6631..aeda060 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -11,6 +11,9 @@
         
 from ..functions import inv_api, inventree_get_part
 
+import threading
+import typing
+
 # Class for a Fusion 360 Palette Command
 class BomOverviewPaletteShowCommand(apper.PaletteCommandBase):
 
@@ -52,14 +55,12 @@ def on_html_event(self, html_args: adsk.core.HTMLEventArgs):
                 helpers.get_cmd(ao, config.DEF_SEND_PART).execute()
 
             elif html_args.action == 'SyncAll':
-                
                 root = ao.product.rootComponent
-                sync_all(ao, root)
 
-                # Refresh this table                
-                if palette:
-                    command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
-                    command.execute()
+                # sync_all_thread(ao, root)
+
+                t = threading.Thread(target=sync_all_thread, args=(ao, root))
+                t.start()
 
             # TODO investigate ghost answers
             # else:
@@ -106,10 +107,71 @@ def propose_create_f360_part(ao, component: adsk.fusion.Component, buttons):
     )
 
     return ao.ui.messageBox(text, "Sync All", buttons, 1)
+  
+
+@apper.lib_import(config.lib_path)
+def sync_all_thread(ao, root: adsk.fusion.Component):
+    palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
+
+    log_html = []
+    def log(message: str, color: str = "black"):
+        log_html.append(f'<span style="color: {color};">{message}</span>')
 
-def sync_all(ao, root: adsk.fusion.Component, updated={}):
+        palette.sendInfoToHTML(
+            config.DEF_SYNC_LOG,
+            '<br />'.join(log_html)
+        )  
+        
+    palette.sendInfoToHTML(
+        config.DEF_SEND_BOM,
+        (
+            '<div id="loading">'
+                '<br><br><br>'
+                '<div class="d-flex justify-content-center">'
+                    '<div class="spinner-border" role="status"> </div>'
+                    '</div>'
+
+                    '<div id="ignore_warnings" style="display: none;">'         
+                        '<div class="d-flex justify-content-center">'   
+                        '<button onclick="" class="btn btn-outline-secondary"> Ignore warnings and Sync </button>'
+                    '</div>'
+                '</div>'
+            
+                '<div class="d-flex justify-content-center">'
+                    '<b> Syncing <span id="status">...</span> </b>'
+                '</div>'
+
+                '<br />'
+            '</div>'
+
+            '<div class="d-flex justify-content-center">'
+                '<div id="sync_log"> </div>'
+            '</div'
+        )
+    )  
+
+    if sync_all(ao, root, log) is True:
+        ao.ui.messageBox('<br />'.join(log_html), "Synchronization")
+    
+    command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
+    command.execute()
+
+COLOR_WARNING = "rgb(255,145,0)"
+    
+@apper.lib_import(config.lib_path)
+def sync_all(ao, root: adsk.fusion.Component, log, visited={}, warning_raised = False):
     from inventree.part import Part
     from inventree.part import BomItem
+    
+    palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
+    palette.sendInfoToHTML(
+        "exec",
+        f"document.getElementById('status').innerHTML = '{root.name}';"
+    )  
+
+    if root.name.lower() == root.partNumber.lower():
+        log(f"Warning: {root.name}'s name is the same as it's part number!", COLOR_WARNING)
+        warning_raised = True
 
     root_part = inventree_get_part(root.id)
 
@@ -122,10 +184,11 @@ def sync_all(ao, root: adsk.fusion.Component, updated={}):
             root_part = item_list[0]
 
         elif len(item_list) > 1:
-            ao.ui.messageBox((
-                f"Part <i>{root.name}</i> does not have a unique IPN <b>{root.partNumber}</b>.<br />"
+            log((
+                f"Part <i>{root.name}</i> does not have a unique (IPN = <b>'{root.partNumber}'</b>).<br />"
                 "<b>Please resolve this in InvenTree for now.</b>"
-            ), "Sync All")
+            ), "red")
+            warning_raised = True
             return
     
     # Delete previous bom
@@ -143,6 +206,7 @@ def sync_all(ao, root: adsk.fusion.Component, updated={}):
     helpers.write_f360_parameters(root_part, root)      
        
     if root_part.assembly:
+        instance_name = {}
         instance_count = {}
         for occurrence in root.occurrences:
             if occurrence.component:
@@ -150,10 +214,11 @@ def sync_all(ao, root: adsk.fusion.Component, updated={}):
                     instance_count[occurrence.component.id] += 1
                 else:
                     instance_count[occurrence.component.id] = 1
+                    instance_name[occurrence.component.id] = occurrence.component.name
 
-                if not str(occurrence.component.id) in updated:
-                    sync_all(ao, occurrence.component, updated)
-                    updated[str(occurrence.component.id)] = True     
+                if not str(occurrence.component.id) in visited:
+                    sync_all(ao, occurrence.component, log, visited, warning_raised)
+                    visited[str(occurrence.component.id)] = True     
 
         for id in instance_count:
             part = inventree_get_part(id)    
@@ -165,4 +230,7 @@ def sync_all(ao, root: adsk.fusion.Component, updated={}):
                     'sub_part': part.pk
                 })
             else:
-                ao.ui.messageBox(f"Make sure you don't have duplicate IPN + names in child drawings of {root_part.name}!", "")
+                log(f"Unable to get part for {instance_name[id]}")                
+                warning_raised = True
+
+    return warning_raised
diff --git a/python/InvenTreeLink/commands/GenerateBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
index 89d1d64..a39945a 100644
--- a/python/InvenTreeLink/commands/GenerateBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -9,6 +9,8 @@
 from .. import config
 from .. import functions
 
+import threading
+
 # Loads the Bill of Materials from the current Fusion360 design.
 class GenerateBomCommand(apper.Fusion360CommandBase):
     def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.CommandInputs, args, input_values):
@@ -27,7 +29,7 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
             palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
 
             # Send message to the HTML Page
-            if palette:
+            def create_bom_thread():
                 palette.sendInfoToHTML(
                     config.DEF_SEND_BOM,
                     '<br><br><br><div class="d-flex justify-content-center"><div class="spinner-border" role="status"><span class="visually-hidden">Loading...</span></div></div>'
@@ -85,6 +87,9 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
                     'SendTree',
                     json.dumps(component_tree)
                 )
+
+            t = threading.Thread(target=create_bom_thread, args=())
+            t.start()
         except Exception as _e:
             config.app_tracking.capture_exception(_e)
             raise _e
diff --git a/python/InvenTreeLink/commands/palette_html/palette.html b/python/InvenTreeLink/commands/palette_html/palette.html
index 5d0ab6a..f97aaf1 100644
--- a/python/InvenTreeLink/commands/palette_html/palette.html
+++ b/python/InvenTreeLink/commands/palette_html/palette.html
@@ -120,6 +120,11 @@ <h4 class="card-title">InvenTreeBom <a data-bs-toggle="collapse" href="#collapse
                     $('#tree_div').jstree(true).settings.core.data = data_in;
                     $('#tree_div').jstree(true).refresh();
                 }
+                else if(action == "SyncLog") {
+                    document.getElementById("sync_log").innerHTML = data;
+                } else if(action == "exec") {
+                    eval(data);
+                }
                 else if (action == 'debugger') {
                     debugger;
                 }
diff --git a/python/InvenTreeLink/config.py b/python/InvenTreeLink/config.py
index 3139f1a..ce65dcd 100644
--- a/python/InvenTreeLink/config.py
+++ b/python/InvenTreeLink/config.py
@@ -20,6 +20,7 @@
 DEF_SEND_PART = "SendPart"
 DEF_SEND_STEP = "SendStep"
 DEF_IMPORT_PART = "ImportPart"
+DEF_SYNC_LOG = "SyncLog"
 
 CFG_ADDRESS = 'address'
 CFG_TOKEN = 'token'

From e5ec2505ddaff411b8fd6b62036bbe68eed8f747 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 10 Dec 2021 18:18:50 +0100
Subject: [PATCH 12/13] Fixed several issues with retrieving the part by
 Fusion360 ID, which was not the fault of how it originally was.

---
 python/InvenTreeLink/InvenTreeLink.py         |  59 +++++++-
 .../commands/BOMOverviewCommand.py            | 143 +++++++++++++-----
 .../InvenTreeLink/commands/EditPartCommand.py |  12 +-
 .../commands/GenerateBomCommand.py            |  80 +++++-----
 python/InvenTreeLink/functions.py             |  77 ++++++++--
 python/InvenTreeLink/helpers.py               |   8 +-
 6 files changed, 286 insertions(+), 93 deletions(-)

diff --git a/python/InvenTreeLink/InvenTreeLink.py b/python/InvenTreeLink/InvenTreeLink.py
index 9ba57cb..5f4c5a7 100644
--- a/python/InvenTreeLink/InvenTreeLink.py
+++ b/python/InvenTreeLink/InvenTreeLink.py
@@ -3,6 +3,8 @@
 
 import os
 
+import threading
+
 try:
     from .apper import apper
     from . import config
@@ -17,7 +19,7 @@
     from .commands.GenerateBomCommand import GenerateBomCommand
     from .commands.ImportStlCommand import ImportStlCommand
     from .commands.ImportPartCommand import ImportPartCommand
-
+    
     # Commands
 
     # Palette
@@ -114,6 +116,61 @@
     
     functions.init_Fusion360()
 
+    # class MyDocumentActivatedHandler(adsk.core.DocumentEventHandler):
+    #     def __init__(self):
+    #         super().__init__()
+    #     def notify(self, args):
+    #         eventArgs = adsk.core.DocumentEventArgs.cast(args)
+
+    #         # Code to react to the event.
+    #         ui.messageBox('In MyDocumentActivatedHandler event handler.\ndocument: {}'.format(eventArgs.document.name))   
+        
+    # class MyDocumentSavedHandler(adsk.core.DocumentEventHandler):
+    #     def __init__(self):
+    #         super().__init__()
+
+    #     def notify(self, args):
+    #         eventArgs = adsk.core.DocumentEventArgs.cast(args)
+
+    #         # Code to react to the event.
+    #         ui.messageBox('In MyDocumentSavedHandler event handler.')   
+
+    # onDocumentSaved = MyDocumentSavedHandler()
+    # app.documentSaved.add(onDocumentSaved)
+    
+    # onDocumentActivated = MyDocumentActivatedHandler()
+    # app.documentActivated.add(onDocumentActivated)
+
+    # def correct_inventree_names_thread():
+    #     # Recursively correct the names to inventree names
+    #     def correct_inventree_names(component):
+    #         part = functions.inventree_get_part(component.id)
+
+    #         if part:           
+    #             if component.name.lower() != part.name.lower():
+    #                 print(f"Correcting name of '{component.name}' to '{part.name}'")
+    #                 component.name = part.name
+                    
+    #             if component.partNumber.lower() != part.IPN.lower():
+    #                 print(f"Correcting IPN of '{component.partNumber}' to '{part.IPN}'")
+    #                 component.partNumber = part.IPN
+
+    #         for occurrence in component.occurrences:
+    #             if occurrence.component:
+    #                 correct_inventree_names(occurrence.component)
+
+    #     ao = apper.AppObjects()
+    #     root = ao.product.rootComponent
+
+    #     for occurrence in root.occurrences:
+    #         if occurrence.component:
+    #             correct_inventree_names(occurrence.component)  
+
+    # t = threading.Thread(target=correct_inventree_names_thread)
+    # t.start()
+
+    print("InvenTreeLink started.")
+
 except:  # noqa: E722
     app = adsk.core.Application.get()
     ui = app.userInterface
diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index aeda060..722582f 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -13,6 +13,7 @@
 
 import threading
 import typing
+import sys
 
 # Class for a Fusion 360 Palette Command
 class BomOverviewPaletteShowCommand(apper.PaletteCommandBase):
@@ -111,6 +112,8 @@ def propose_create_f360_part(ao, component: adsk.fusion.Component, buttons):
 
 @apper.lib_import(config.lib_path)
 def sync_all_thread(ao, root: adsk.fusion.Component):
+    print("Starting sync_all thread.")
+
     palette = ao.ui.palettes.itemById(config.ITEM_PALETTE)
 
     log_html = []
@@ -130,15 +133,10 @@ def log(message: str, color: str = "black"):
                 '<div class="d-flex justify-content-center">'
                     '<div class="spinner-border" role="status"> </div>'
                     '</div>'
-
-                    '<div id="ignore_warnings" style="display: none;">'         
-                        '<div class="d-flex justify-content-center">'   
-                        '<button onclick="" class="btn btn-outline-secondary"> Ignore warnings and Sync </button>'
-                    '</div>'
                 '</div>'
             
                 '<div class="d-flex justify-content-center">'
-                    '<b> Syncing <span id="status">...</span> </b>'
+                    '<p> Syncing <i><b><span id="status"></span></b></i>...</p>'
                 '</div>'
 
                 '<br />'
@@ -150,16 +148,21 @@ def log(message: str, color: str = "black"):
         )
     )  
 
-    if sync_all(ao, root, log) is True:
-        ao.ui.messageBox('<br />'.join(log_html), "Synchronization")
+    visited = dict()
+    sync_all(ao, root, log, None, visited, False)
     
     command = helpers.get_cmd(ao, config.DEF_SEND_BOM)
     command.execute()
 
+    visited.clear()
+
+    print("Stopping thread...")
+    sys.exit()
+
 COLOR_WARNING = "rgb(255,145,0)"
     
 @apper.lib_import(config.lib_path)
-def sync_all(ao, root: adsk.fusion.Component, log, visited={}, warning_raised = False):
+def sync_all(ao, root: adsk.fusion.Component, log, parent = None, visited={}, warning_raised = False):
     from inventree.part import Part
     from inventree.part import BomItem
     
@@ -170,40 +173,86 @@ def sync_all(ao, root: adsk.fusion.Component, log, visited={}, warning_raised =
     )  
 
     if root.name.lower() == root.partNumber.lower():
-        log(f"Warning: {root.name}'s name is the same as it's part number!", COLOR_WARNING)
-        warning_raised = True
+        # The name is the same as the part number, remove the part number.
+        root.partNumber = ""
 
-    root_part = inventree_get_part(root.id)
 
-    if root_part == False:      
-        item_list = Part.list(inv_api(), IPN=root.partNumber)
-        if len(item_list) == 0:
-            root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))
+    parent_info = f" in {parent.name}" if parent else ''
+    #print(f"Updating {root.name}{parent_info}...")
 
-        elif len(item_list) == 1:
-            root_part = item_list[0]
+    root_part = inventree_get_part(root.id)
+    if root_part == False:
+        # The part is not found using the fusion 360 ID.
+        print(f"Unable to find {root.name} by Fusion 360 ID {root.id}.")
+
+        if not root.partNumber or root.partNumber.startswith("Component"):
+            try:                
+                # If the partnumber is set by Fusion360, remove it for InvenTree
+                root.partNumber = ""
+            except:
+                pass
+            # The part does not have an IPN set, however this is no problem because it will be found
+            # using the Fusion 360 ID.
+            root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))
+            print(f"{root.name} does not have it's IPN set, so created a new part.")
 
-        elif len(item_list) > 1:
             log((
-                f"Part <i>{root.name}</i> does not have a unique (IPN = <b>'{root.partNumber}'</b>).<br />"
-                "<b>Please resolve this in InvenTree for now.</b>"
-            ), "red")
-            warning_raised = True
-            return
+                f"<i><b>{root.name}</b></i>{parent_info} does not have a IPN set"
+            ), "warn")
+        else:
+            # The part number is not empty, so search InvenTree for an existing part.
+            item_list = Part.list(inv_api(), IPN=root.partNumber, has_ipn=True)
+
+            if len(item_list) == 0:              
+                # If the partnumber is not recognized by InvenTree, remove it for Fusion360
+                root.partNumber = ""
+
+                print(f"Item {root.name}{parent_info} is not matched by part number, but will be created with an empty IPN.")
+
+                # There are no matches with this IPN, create the part.
+                root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))
+            elif len(item_list) == 1:
+                #print(f"Item {root.name} is singly matched by part number.")
+                # There's a single match by IPN, it must be the part we want.
+                root_part = item_list[0]
+            elif len(item_list) > 1:
+                print(f"Item {root.name}{parent_info} is not matched by Fusion360 ID, but found {len(item_list)} items matching IPN {root.partNumber}")
+
+                log((
+                    f"Skipping part <i><b>{root.name}</b></i>{parent_info} because it does not have a unique IPN. (IPN = <b>'{root.partNumber}'</b>)"
+                ), "red")
+
+                return
+
+    # If there's an inventree name, it will be copied and set.
+    # If the part was created right before this, it was created with the 
+    # part name of Fusion360, so this would have no side effect.
+    new_name = root_part.name
+    new_IPN = root_part.IPN
+
+    # Exception is thrown when trying to edit root component name.
+    try:
+        root.name = new_name
+    except:
+        pass
     
-    # Delete previous bom
-    for item in root_part.getBomItems():
-        item.delete() 
+    root.partNumber = new_IPN
+    
+    if not root_part.assembly:
+        # Delete previous bom
+        for item in root_part.getBomItems():
+            item.delete() 
+
+    #print(f"Before save: {new_name} was an assembly: {root_part.assembly} with {len(root.occurrences)} occs")
 
     root_part.save(data={ 
-        'name': root.name,
-        'IPN': root.partNumber,
-        'description': root.description if root.description else 'None',
-        # If the part has more than one occurences, it's probably an assembly.
-        # However, if you marked it Purchasable in InvenTree, it will not be an assembly.
-        'assembly': len(root.occurrences) > 0 and not root_part.purchaseable
+        'name': new_name,
+        'IPN': new_IPN,
+        'description': root.description if root.description else f'Fusion360 Name: {root.name}',
     })
     helpers.write_f360_parameters(root_part, root)      
+    
+    #print(f"After save: {new_name} is an assembly: {root_part.assembly} with {len(root.occurrences)} occs")
        
     if root_part.assembly:
         instance_name = {}
@@ -211,17 +260,27 @@ def sync_all(ao, root: adsk.fusion.Component, log, visited={}, warning_raised =
         for occurrence in root.occurrences:
             if occurrence.component:
                 if str(occurrence.component.id) in instance_count:
-                    instance_count[occurrence.component.id] += 1
+                    instance_count[str(occurrence.component.id)] += 1
+                else:
+                    instance_count[str(occurrence.component.id)] = 1
+                    instance_name[str(occurrence.component.id)] = occurrence.component.name
+
+                if str(occurrence.component.id) in visited:
+                    #print(f"Skipping {occurrence.component.name} because it's already visited.") 
+                    noop = True
                 else:
-                    instance_count[occurrence.component.id] = 1
-                    instance_name[occurrence.component.id] = occurrence.component.name
+                    sync_all(ao, occurrence.component, log, root, visited, warning_raised)
+                    visited[str(occurrence.component.id)] = True    
 
-                if not str(occurrence.component.id) in visited:
-                    sync_all(ao, occurrence.component, log, visited, warning_raised)
-                    visited[str(occurrence.component.id)] = True     
 
+        palette.sendInfoToHTML(
+            "exec",
+            f"document.getElementById('status').innerHTML = '{root.name}'s Bill of Materials';"
+        )  
+        parts = inventree_get_part([id for id in instance_count])
+        print(f"Creating Bill of Materials for {new_name}... ")
         for id in instance_count:
-            part = inventree_get_part(id)    
+            part = parts[id]
 
             if part is not False:
                 BomItem.create(inv_api(), {
@@ -233,4 +292,8 @@ def sync_all(ao, root: adsk.fusion.Component, log, visited={}, warning_raised =
                 log(f"Unable to get part for {instance_name[id]}")                
                 warning_raised = True
 
+        print(f"Done creating Bill of Materials for {new_name}.")
+        
+    #print(f"Updated part {root.name} ({root.id})")
+
     return warning_raised
diff --git a/python/InvenTreeLink/commands/EditPartCommand.py b/python/InvenTreeLink/commands/EditPartCommand.py
index 32a0b29..5f8ec6d 100644
--- a/python/InvenTreeLink/commands/EditPartCommand.py
+++ b/python/InvenTreeLink/commands/EditPartCommand.py
@@ -51,6 +51,8 @@ def getText(text_name, obj, item, data):
                     # compare
                     if not getattr(obj, item) == value:
                         data[item] = value
+                    # Return the value so it can be set.
+                    return value
 
                 def getValue(text_name, obj, item, data):
                     value = inp.itemById(text_name).value
@@ -60,8 +62,14 @@ def getValue(text_name, obj, item, data):
 
                 if part:
                     _data = {}
-                    getText('text_part_name', part, 'name', _data)
-                    getText('text_part_ipn', part, 'IPN', _data)
+                    
+                    try:
+                        occ.component.name = getText('text_part_name', part, 'name', _data)
+                    except:
+                        pass
+
+                    occ.component.partNumber = getText('text_part_ipn', part, 'IPN', _data)
+                    
                     getText('text_part_description', part, 'description', _data)
                     getText('text_part_notes', part, 'notes', _data)
                     getText('text_part_keywords', part, 'keywords', _data)
diff --git a/python/InvenTreeLink/commands/GenerateBomCommand.py b/python/InvenTreeLink/commands/GenerateBomCommand.py
index a39945a..488beb6 100644
--- a/python/InvenTreeLink/commands/GenerateBomCommand.py
+++ b/python/InvenTreeLink/commands/GenerateBomCommand.py
@@ -30,64 +30,76 @@ def on_execute(self, command: adsk.core.Command, command_inputs: adsk.core.Comma
 
             # Send message to the HTML Page
             def create_bom_thread():
+                loading_html = (
+                    '<br><br><br>'
+
+                    '<div class="d-flex justify-content-center">'
+                        '<div class="spinner-border" role="status">'
+                            '<span class="visually-hidden">Loading...</span>'
+                        '</div>'
+                    '</div>'                
+            
+                    '<div class="d-flex justify-content-center">'
+                        '<p> Retrieving BOM...</p>'
+                    '</div>'
+                )
+
                 palette.sendInfoToHTML(
                     config.DEF_SEND_BOM,
-                    '<br><br><br><div class="d-flex justify-content-center"><div class="spinner-border" role="status"><span class="visually-hidden">Loading...</span></div></div>'
+                    loading_html
                 )
 
                 start = datetime.now()
-
                 bom = functions.extract_bom()
-
-                component_tree = functions.make_component_tree()
                 
                 element_synced = "<span style='color: green'> Synced </span>"
                 element_not_synced = "<span style='color: red'> Not synced </span>"
 
-                header = ''.join((
+                header_html = (
                     f"<h1>{root.name} </h1>"
-                ))
+                )
 
                 body = ''.join([
-                    ''.join((
+                    (
                         "<tr>"
-                        f"<td>{item['IPN']} | {item['name']}</td>"
-                        f"<td>{item['instances']}</td>"
-                        f"<td>{element_synced if item['part'] else element_not_synced}</td>"
+                            f"<td>{item['IPN']} | {item['name']}</td>"
+                            f"<td>{item['instances']}</td>"
+                            f"<td>{element_synced if item['part'] else element_not_synced}</td>"
                         "</tr>" 
-                    ))
+                    )
                     for item in bom
                 ])
 
-                table_c = ''.join([
-                    "<div class='overflow-auto'>",
-                    "<table class='table table-sm table-striped table-hover'>",
-                    "<thead>",
-                    "<tr>",
-                    "<th scope='col'> Name </th>",
-                    "<th scope='col'> Count </th>",
-                    "<th scope='col'> Synced </th>",
-                    "</tr>",
-                    f"</thead><tbody>{body}</tbody></table></div>",
-                ])
+                table_html = (
+                    "<div class='overflow-auto'>"
+                        "<table class='table table-sm table-striped table-hover'>"
+                            "<thead>"
+                                "<tr>"
+                                "<th scope='col'> Name </th>"
+                                "<th scope='col'> Count </th>"
+                                "<th scope='col'> Synced </th>"
+                                "</tr>"
+                            "</thead>"
+                            "<tbody>"
+                                    f"{body}"
+                            "</tbody>"
+                        "</table>"
+                    "</div>"
+                )
 
-                complete = ''.join((
-                    header,
+                complete_html = (
+                    f"{header_html}"
                     f"<p> {len(bom)} parts found in {datetime.now() - start}</p>"
-                    f"{table_c}"
-                    "<button onclick='onClickSyncAll()' class='btn btn-outline-secondary'> Sync All </button>"  
-                ))
-
-                palette.sendInfoToHTML(
-                    config.DEF_SEND_BOM,
-                    complete
+                    f"{table_html}"
+                    "<button onclick='onClickSyncAll()' class='btn btn-outline-secondary'> Sync All </button>"
                 )
-
+                
                 palette.sendInfoToHTML(
-                    'SendTree',
-                    json.dumps(component_tree)
+                    config.DEF_SEND_BOM,
+                    json.dumps(complete_html)
                 )
 
+            #create_bom_thread()
             t = threading.Thread(target=create_bom_thread, args=())
             t.start()
         except Exception as _e:
diff --git a/python/InvenTreeLink/functions.py b/python/InvenTreeLink/functions.py
index 81a2d78..3e4cb4b 100644
--- a/python/InvenTreeLink/functions.py
+++ b/python/InvenTreeLink/functions.py
@@ -241,25 +241,74 @@ def inventree_get_part(part_id):
     from inventree.part import Part
     from inventree.base import Parameter
 
-    def search(parameters, part_id):
-        try:
-            part = [a.part for a in parameters if a._data['data'] == part_id]
-            if len(part) == 1:
-                return Part(inv_api(), part[0])
-            return False
-        except Exception as _e:
-            config.app_tracking.capture_exception(_e)
-            raise Exception from _e
+    def find(inv_parameters, part_id):
+        inv_part_id = [param.part for param in inv_parameters if str(param.data) == str(part_id)]
+
+        if len(inv_part_id) == 1:
+            #print(f"inventree_get_part(): Found the perfect match.")
+            return Part(inv_api(), inv_part_id[0])
+        elif len(inv_part_id) > 0:
+            print(f"inventree_get_part(): Warning: {part_id} multiple ({len(inv_part_id)}) matches:")
+            for id in inv_part_id:
+                p = Part(inv_api(), id)
+                print(f"Part: {p.IPN} | {p.name}")
+            print()
+        #else:                
+            #print(f"inventree_get_part(): Warning: {part_id} has no ({len(inv_part_id)}) matches!")
+
+        return False
 
-    parameters = Parameter.list(inv_api())
-    if not parameters:
-        parameters = []
+    # Get all fusion 360 named parts.
+    parameters = Parameter.list(inv_api(), template=Fusion360Parameters.ID.value.pk)
     if type(part_id) in (list, tuple):
+        # Process the list or tuple.
         result = {}
+
         for cur_id in part_id:
-            result[cur_id] = search(parameters, cur_id)
+            result[cur_id] = find(parameters, cur_id)
+
         return result
-    return search(parameters, part_id)
+    else:
+        # Just a single id.
+        return find(parameters, part_id)
+
+    # def search(parameters, part_id):
+    #     try:
+    #         for param in parameters:
+    #             print("Paramater")
+
+    #         part = [a.part for a in parameters if a._data['data'] == part_id]
+    #         if len(part) == 1:
+    #             return Part(inv_api(), part[0])
+    #         elif len(part) > 0:
+    #             print(f"inventree_get_part(): Warning: {part_id} multiple ({len(part)}) matches:")
+    #             for id in part:
+    #                 p = Part(inv_api(), id)
+    #                 print(f"Part: {p.IPN} | {p.name}")
+    #             print()
+
+    #         else:                
+    #             print(f"inventree_get_part(): Warning: {part_id} has no ({len(part)}) matches!")
+
+    #         return False
+    #     except Exception as _e:
+    #         config.app_tracking.capture_exception(_e)
+    #         raise Exception from _e
+
+    # parameters = Parameter.list(inv_api())
+
+    # if not parameters:
+    #     parameters = []
+
+    # if type(part_id) in (list, tuple):
+    #     result = {}
+
+    #     for cur_id in part_id:
+    #         result[cur_id] = search(parameters, cur_id)
+
+    #     return result
+
+    # return search(parameters, part_id)
 # endregion
 
 # region bom functions
diff --git a/python/InvenTreeLink/helpers.py b/python/InvenTreeLink/helpers.py
index ef4f7e2..d5402c7 100644
--- a/python/InvenTreeLink/helpers.py
+++ b/python/InvenTreeLink/helpers.py
@@ -102,8 +102,12 @@ def write_f360_parameters(part, component: adsk.fusion.Component):
     if not math.isnan(physicalProperties.density):
         Fusion360Parameters.DENSITY.value.set(part, physicalProperties.density)
 
-    if component.material and component.material.name:
-        Fusion360Parameters.MATERIAL.value.set(part, component.material.name)
+    try:
+        if component.material and component.material.name:
+            Fusion360Parameters.MATERIAL.value.set(part, component.material.name)
+    except:
+        # For some weird reason an assetInst exception gets thrown here.
+        pass
 
     axis = ['x', 'y', 'z']
     bb_min = {a: getattr(component.boundingBox.minPoint, a) for a in axis}

From 4a8968d101d8463b3f1167967abd83eb39fd2179 Mon Sep 17 00:00:00 2001
From: Guusggg <Guus.geurkink@gmail.com>
Date: Fri, 17 Dec 2021 14:17:39 +0100
Subject: [PATCH 13/13] Update BOMOverviewCommand.py

---
 python/InvenTreeLink/commands/BOMOverviewCommand.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/python/InvenTreeLink/commands/BOMOverviewCommand.py b/python/InvenTreeLink/commands/BOMOverviewCommand.py
index 722582f..54bb57e 100644
--- a/python/InvenTreeLink/commands/BOMOverviewCommand.py
+++ b/python/InvenTreeLink/commands/BOMOverviewCommand.py
@@ -183,7 +183,7 @@ def sync_all(ao, root: adsk.fusion.Component, log, parent = None, visited={}, wa
     root_part = inventree_get_part(root.id)
     if root_part == False:
         # The part is not found using the fusion 360 ID.
-        print(f"Unable to find {root.name} by Fusion 360 ID {root.id}.")
+        #print(f"Unable to find {root.name} by Fusion 360 ID {root.id}.")
 
         if not root.partNumber or root.partNumber.startswith("Component"):
             try:                
@@ -212,7 +212,7 @@ def sync_all(ao, root: adsk.fusion.Component, log, parent = None, visited={}, wa
                 # There are no matches with this IPN, create the part.
                 root_part = helpers.create_f360_part(root, functions.config_ref(config.CFG_PART_CATEGORY))
             elif len(item_list) == 1:
-                #print(f"Item {root.name} is singly matched by part number.")
+                print(f"Item {root.name} is singly matched by part number.")
                 # There's a single match by IPN, it must be the part we want.
                 root_part = item_list[0]
             elif len(item_list) > 1:
@@ -277,6 +277,7 @@ def sync_all(ao, root: adsk.fusion.Component, log, parent = None, visited={}, wa
             "exec",
             f"document.getElementById('status').innerHTML = '{root.name}'s Bill of Materials';"
         )  
+
         parts = inventree_get_part([id for id in instance_count])
         print(f"Creating Bill of Materials for {new_name}... ")
         for id in instance_count: