From e969cd188395b34b447978ff44989fb0d210ff01 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=B4me=20LORRAIN?= <jerome.lorrain@protonmail.com>
Date: Tue, 12 Oct 2021 16:41:02 +0200
Subject: [PATCH] Add photoshop extractReview plugin options for
 representations tags

---
 .../plugins/publish/extract_review.py         |  9 +++--
 openpype/plugins/publish/extract_burnin.py    |  3 +-
 .../defaults/project_settings/photoshop.json  | 12 +++++++
 .../schema_project_photoshop.json             | 34 ++++++++++++++++++-
 4 files changed, 54 insertions(+), 4 deletions(-)

diff --git a/openpype/hosts/photoshop/plugins/publish/extract_review.py b/openpype/hosts/photoshop/plugins/publish/extract_review.py
index 1c53c3a2ef9..8c4d05b282a 100644
--- a/openpype/hosts/photoshop/plugins/publish/extract_review.py
+++ b/openpype/hosts/photoshop/plugins/publish/extract_review.py
@@ -17,6 +17,10 @@ class ExtractReview(openpype.api.Extractor):
     hosts = ["photoshop"]
     families = ["review"]
 
+    # Extract Options
+    jpg_options = None
+    mov_options = None
+
     def process(self, instance):
         staging_dir = self.staging_dir(instance)
         self.log.info("Outputting image to {}".format(staging_dir))
@@ -53,7 +57,8 @@ def process(self, instance):
             "name": "jpg",
             "ext": "jpg",
             "files": output_image,
-            "stagingDir": staging_dir
+            "stagingDir": staging_dir,
+            "tags": self.jpg_options['tags']
         })
         instance.data["stagingDir"] = staging_dir
 
@@ -97,7 +102,7 @@ def process(self, instance):
             "frameEnd": 1,
             "fps": 25,
             "preview": True,
-            "tags": ["review", "ftrackreview"]
+            "tags": self.mov_options['tags']
         })
 
         # Required for extract_review plugin (L222 onwards).
diff --git a/openpype/plugins/publish/extract_burnin.py b/openpype/plugins/publish/extract_burnin.py
index 207e696fb14..06eb85c593c 100644
--- a/openpype/plugins/publish/extract_burnin.py
+++ b/openpype/plugins/publish/extract_burnin.py
@@ -45,7 +45,8 @@ class ExtractBurnin(openpype.api.Extractor):
         "aftereffects",
         "tvpaint",
         "webpublisher",
-        "aftereffects"
+        "aftereffects",
+        "photoshop"
         # "resolve"
     ]
     optional = True
diff --git a/openpype/settings/defaults/project_settings/photoshop.json b/openpype/settings/defaults/project_settings/photoshop.json
index 4c36e4bd49a..36c30bad6cb 100644
--- a/openpype/settings/defaults/project_settings/photoshop.json
+++ b/openpype/settings/defaults/project_settings/photoshop.json
@@ -17,6 +17,18 @@
                 "png",
                 "jpg"
             ]
+        },
+        "ExtractReview": {
+            "jpg_options": {
+                "tags": [
+                ]
+            },
+            "mov_options": {
+                "tags": [
+                    "review",
+                    "ftrackreview"
+                ]
+            }
         }
     },
     "workfile_builder": {
diff --git a/openpype/settings/entities/schemas/projects_schema/schema_project_photoshop.json b/openpype/settings/entities/schemas/projects_schema/schema_project_photoshop.json
index 3b65f08ac42..6f5577650c4 100644
--- a/openpype/settings/entities/schemas/projects_schema/schema_project_photoshop.json
+++ b/openpype/settings/entities/schemas/projects_schema/schema_project_photoshop.json
@@ -60,7 +60,39 @@
                             "object_type": "text"
                         }
                     ]
-                }                
+                },
+                {
+                    "type": "dict",
+                    "collapsible": true,
+                    "key": "ExtractReview",
+                    "label": "Extract Review",
+                    "children": [
+                        {
+                            "type": "dict",
+                            "collapsible": false,
+                            "key": "jpg_options",
+                            "label": "Extracted jpg Options",
+                            "children": [
+                                {
+                                    "type": "schema",
+                                    "name": "schema_representation_tags"
+                                }
+                            ]
+                        },
+                        {
+                            "type": "dict",
+                            "collapsible": false,
+                            "key": "mov_options",
+                            "label": "Extracted mov Options",
+                            "children": [
+                                {
+                                    "type": "schema",
+                                    "name": "schema_representation_tags"
+                                }
+                            ]
+                        }
+                    ]
+                }
             ]
         },
         {