From 14093da98f6c68b7a3e238d51c9c62cec7c16f2c Mon Sep 17 00:00:00 2001
From: Toke Stuart Jepsen <tokejepsen@gmail.com>
Date: Tue, 11 Aug 2020 21:55:11 +0100
Subject: [PATCH] Option to keep the review files.

- Disabled by default to maintain backwards compatibility.
- Open loading review image sequences into Nuke.
---
 pype/plugins/maya/create/create_review.py      | 2 ++
 pype/plugins/maya/publish/extract_playblast.py | 7 +++++--
 pype/plugins/nuke/load/load_sequence.py        | 2 +-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/pype/plugins/maya/create/create_review.py b/pype/plugins/maya/create/create_review.py
index 3e513032e1..c488a7559c 100644
--- a/pype/plugins/maya/create/create_review.py
+++ b/pype/plugins/maya/create/create_review.py
@@ -21,4 +21,6 @@ class CreateReview(avalon.maya.Creator):
         for key, value in animation_data.items():
             data[key] = value
 
+        data["keepImages"] = False
+
         self.data = data
diff --git a/pype/plugins/maya/publish/extract_playblast.py b/pype/plugins/maya/publish/extract_playblast.py
index 8d45f98b90..80a4dadf28 100644
--- a/pype/plugins/maya/publish/extract_playblast.py
+++ b/pype/plugins/maya/publish/extract_playblast.py
@@ -53,7 +53,6 @@ class ExtractPlayblast(pype.api.Extractor):
 
         preset['camera'] = camera
         preset['format'] = "image"
-        # preset['compression'] = "qt"
         preset['quality'] = 95
         preset['compression'] = "png"
         preset['start_frame'] = start
@@ -102,6 +101,10 @@ class ExtractPlayblast(pype.api.Extractor):
         if "representations" not in instance.data:
             instance.data["representations"] = []
 
+        tags = ["review"]
+        if not instance.data.get("keepImages"):
+            tags.append("delete")
+
         representation = {
             'name': 'png',
             'ext': 'png',
@@ -111,7 +114,7 @@ class ExtractPlayblast(pype.api.Extractor):
             "frameEnd": end,
             'fps': fps,
             'preview': True,
-            'tags': ['review', 'delete']
+            'tags': tags
         }
         instance.data["representations"].append(representation)
 
diff --git a/pype/plugins/nuke/load/load_sequence.py b/pype/plugins/nuke/load/load_sequence.py
index aa79d8736a..601e28c7c1 100644
--- a/pype/plugins/nuke/load/load_sequence.py
+++ b/pype/plugins/nuke/load/load_sequence.py
@@ -70,7 +70,7 @@ def loader_shift(node, frame, relative=True):
 class LoadSequence(api.Loader):
     """Load image sequence into Nuke"""
 
-    families = ["render2d", "source", "plate", "render", "prerender"]
+    families = ["render2d", "source", "plate", "render", "prerender", "review"]
     representations = ["exr", "dpx", "jpg", "jpeg", "png"]
 
     label = "Load sequence"