From 77262990b4d98c61880cc422900e7cebc03dfc83 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@orbi.tools>
Date: Fri, 5 Nov 2021 07:52:49 +0100
Subject: [PATCH] debugging component creator

---
 openpype/hosts/flame/hooks/pre_flame_setup.py    |  2 +-
 .../openpype_flame_to_ftrack.py                  | 16 ++++++++++------
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/openpype/hosts/flame/hooks/pre_flame_setup.py b/openpype/hosts/flame/hooks/pre_flame_setup.py
index 9043eb6cc6..718c4b574c 100644
--- a/openpype/hosts/flame/hooks/pre_flame_setup.py
+++ b/openpype/hosts/flame/hooks/pre_flame_setup.py
@@ -33,7 +33,7 @@ class FlamePrelaunch(PreLaunchHook):
         """Hook entry method."""
         project_doc = self.data["project_doc"]
         user_name = get_openpype_username()
-        hostname = socket.gethostname()
+        hostname = socket.gethostname()  # not returning wiretap host name
 
         self.log.debug("Collected user \"{}\"".format(user_name))
         self.log.info(pformat(project_doc))
diff --git a/openpype/hosts/flame/utility_scripts/openpype_flame_to_ftrack/openpype_flame_to_ftrack.py b/openpype/hosts/flame/utility_scripts/openpype_flame_to_ftrack/openpype_flame_to_ftrack.py
index c12c1afb72..e19402914d 100644
--- a/openpype/hosts/flame/utility_scripts/openpype_flame_to_ftrack/openpype_flame_to_ftrack.py
+++ b/openpype/hosts/flame/utility_scripts/openpype_flame_to_ftrack/openpype_flame_to_ftrack.py
@@ -452,7 +452,8 @@ class FtrackComponentCreator:
         self.session = session
         self._get_ftrack_location()
 
-    def create_comonent(self, parent, data, assetversion_entity=None):
+    def create_comonent(self, shot_entity, data, assetversion_entity=None):
+        self.shot_entity = shot_entity
         location = self._get_ftrack_location()
 
         file_path = data["file_path"]
@@ -489,7 +490,7 @@ class FtrackComponentCreator:
             asset_entity = self._get_asset({
                 "name": "plateReference",
                 "type": assettype_entity,
-                "parent": parent
+                "parent": self.shot_entity
             })
 
             # get or create assetversion entity from session
@@ -573,6 +574,7 @@ class FtrackComponentCreator:
         component_entity["metadata"] = existing_component_metadata
 
         if comp_data["name"] == "thumbnail":
+            self.shot_entity["thumbnail_id"] = component_entity["id"]
             assetversion_entity["thumbnail_id"] = component_entity["id"]
 
         self._commit()
@@ -595,7 +597,7 @@ class FtrackComponentCreator:
 
     def _get_assetversion(self, data):
         assetversion_entity = self.session.query(
-                self._query("AssetVersion", data)
+            self._query("AssetVersion", data)
         ).first()
 
         if assetversion_entity:
@@ -610,7 +612,7 @@ class FtrackComponentCreator:
 
     def _commit(self):
         try:
-            self.session._commit()
+            self.session.commit()
         except Exception:
             tp, value, tb = sys.exc_info()
             self.session.rollback()
@@ -618,6 +620,8 @@ class FtrackComponentCreator:
             six.reraise(tp, value, tb)
 
     def _get_ftrack_location(self, name=None):
+        name = name or self.default_location
+
         if name in self.ftrack_locations:
             return self.ftrack_locations[name]
 
@@ -751,7 +755,7 @@ def main_window(selection):
                 'parent': parent
             })
             try:
-                session._commit()
+                session.commit()
             except Exception:
                 tp, value, tb = sys.exc_info()
                 session.rollback()
@@ -989,7 +993,7 @@ def main_window(selection):
                     task_entity.create_note(task_description, user)
 
                 try:
-                    session._commit()
+                    session.commit()
                 except Exception:
                     tp, value, tb = sys.exc_info()
                     session.rollback()