Merged in feature/PYPE-645_comment_to_ftrack (pull request #463)

Feature/PYPE-645 comment to ftrack

Approved-by: Milan Kolar <milan@orbi.tools>

pype/plugins/ftrack/publish/integrate_ftrack_api.py

@@ -77,6 +77,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
         info_msg = "Created new {entity_type} with data: {data}"
         info_msg += ", metadata: {metadata}."
 
+        used_asset_versions = []
         # Iterate over components and publish
         for data in instance.data.get("ftrackComponentsList", []):
 
@@ -386,3 +387,14 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
                     tp, value, tb = sys.exc_info()
                     session.rollback()
                     six.reraise(tp, value, tb)
+
+            if assetversion_entity not in used_asset_versions:
+                used_asset_versions.append(assetversion_entity)
+
+        asset_versions_key = "ftrackIntegratedAssetVersions"
+        if asset_versions_key not in instance.data:
+            instance.data[asset_versions_key] = []
+
+        for asset_version in used_asset_versions:
+            if asset_version not in instance.data[asset_versions_key]:
+                instance.data[asset_versions_key].append(asset_version)

pype/plugins/ftrack/publish/integrate_ftrack_note.py

@@ -0,0 +1,51 @@
+import sys
+import pyblish.api
+import six
+
+
+class IntegrateFtrackNote(pyblish.api.InstancePlugin):
+    """Create comments in Ftrack."""
+
+    # Must be after integrate asset new
+    order = pyblish.api.IntegratorOrder + 0.4999
+    label = "Integrate Ftrack note"
+    families = ["ftrack"]
+    optional = True
+
+    def process(self, instance):
+        comment = (instance.context.data.get("comment") or "").strip()
+        if not comment:
+            self.log.info("Comment is not set.")
+            return
+
+        self.log.debug("Comment is set to {}".format(comment))
+
+        asset_versions_key = "ftrackIntegratedAssetVersions"
+        asset_versions = instance.data.get(asset_versions_key)
+        if not asset_versions:
+            self.log.info("There are any integrated AssetVersions")
+            return
+
+        session = instance.context.data["ftrackSession"]
+        user = session.query(
+            "User where username is \"{}\"".format(session.api_user)
+        ).first()
+        if not user:
+            self.log.warning(
+                "Was not able to query current User {}".format(
+                    session.api_user
+                )
+            )
+
+        for asset_version in asset_versions:
+            asset_version.create_note(comment, author=user)
+
+            try:
+                session.commit()
+                self.log.debug("Note added to AssetVersion \"{}\"".format(
+                    str(asset_version)
+                ))
+            except Exception:
+                tp, value, tb = sys.exc_info()
+                session.rollback()
+                six.reraise(tp, value, tb)

pype/plugins/global/publish/collect_comment.py

@@ -15,4 +15,5 @@ class CollectComment(pyblish.api.ContextPlugin):
     order = pyblish.api.CollectorOrder
 
     def process(self, context):
-        context.data["comment"] = ""
+        comment = (context.data.get("comment") or "").strip()
+        context.data["comment"] = comment

pype/plugins/standalonepublisher/publish/collect_matchmove.py

@@ -21,7 +21,7 @@ class CollectMatchmovePublish(pyblish.api.InstancePlugin):
 
     label = "Collect Matchmove - SA Publish"
     order = pyblish.api.CollectorOrder
-    family = ["matchmove"]
+    families = ["matchmove"]
     hosts = ["standalonepublisher"]
 
     def process(self, instance):