Merge pull request #4303 from ynput/bugfix/OP-4679_Issue-Comment-on-S…

…tandalone-and-tray-Publisher General: missing comment on standalone and tray publisher
ynput · Jan 12, 2023 · 554c72c · 554c72c
2 parents e150510 + f854de4
commit 554c72c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
@@ -129,8 +129,8 @@ def process(self, instance):
             if not note_text.solved:
                 self.log.warning((
                     "Note template require more keys then can be provided."
-                    "\nTemplate: {}\nData: {}"
-                ).format(template, format_data))
+                    "\nTemplate: {}\nMissing values for keys:{}\nData: {}"
+                ).format(template, note_text.missing_keys, format_data))
                 continue
 
             if not note_text:

diff --git a/openpype/plugins/publish/collect_comment.py b/openpype/plugins/publish/collect_comment.py
@@ -73,7 +73,9 @@ class CollectComment(
     """
 
     label = "Collect Instance Comment"
-    order = pyblish.api.CollectorOrder + 0.49
+    # TODO change to CollectorOrder after Pyblish is purged
+    # Pyblish allows modifying comment after collect phase
+    order = pyblish.api.ExtractorOrder - 0.49
 
     def process(self, context):
         context_comment = self.cleanup_comment(context.data.get("comment"))