From 46bfbd28506be577532cb68f319a3051e63957ee Mon Sep 17 00:00:00 2001
From: Allan Ihsan <allan.ihsan@gmail.com>
Date: Mon, 27 Jun 2022 06:13:35 +0300
Subject: [PATCH] Make appropriate feature fixes.

---
 .../maya/plugins/create/create_review.py      |  8 +++---
 .../maya/plugins/publish/collect_review.py    |  4 +--
 .../maya/plugins/publish/extract_playblast.py | 26 ++++++++++++-------
 3 files changed, 23 insertions(+), 15 deletions(-)

diff --git a/openpype/hosts/maya/plugins/create/create_review.py b/openpype/hosts/maya/plugins/create/create_review.py
index 331f0818eb..83b7f34d82 100644
--- a/openpype/hosts/maya/plugins/create/create_review.py
+++ b/openpype/hosts/maya/plugins/create/create_review.py
@@ -15,8 +15,8 @@ class CreateReview(plugin.Creator):
     keepImages = False
     isolate = False
     imagePlane = True
-    resolutionWidth = 0
-    resolutionHeight = 0
+    attrWidth = 0
+    attrHeight = 0
     transparency = [
         "preset",
         "simple",
@@ -35,8 +35,8 @@ class CreateReview(plugin.Creator):
         for key, value in animation_data.items():
             data[key] = value
 
-        data["resolutionWidth"] = self.resolutionWidth
-        data["resolutionHeight"] = self.resolutionHeight
+        data["attrWidth"] = self.attrWidth
+        data["attrHeight"] = self.attrHeight
         data["isolate"] = self.isolate
         data["keepImages"] = self.keepImages
         data["imagePlane"] = self.imagePlane
diff --git a/openpype/hosts/maya/plugins/publish/collect_review.py b/openpype/hosts/maya/plugins/publish/collect_review.py
index 0769747205..83c4760535 100644
--- a/openpype/hosts/maya/plugins/publish/collect_review.py
+++ b/openpype/hosts/maya/plugins/publish/collect_review.py
@@ -71,8 +71,8 @@ class CollectReview(pyblish.api.InstancePlugin):
                 data['handles'] = instance.data.get('handles', None)
                 data['step'] = instance.data['step']
                 data['fps'] = instance.data['fps']
-                data['resolutionWidth'] = instance.data['resolutionWidth']
-                data['resolutionHeight'] = instance.data['resolutionHeight']
+                data['attrWidth'] = instance.data['attrWidth']
+                data['attrHeight'] = instance.data['attrHeight']
                 data["isolate"] = instance.data["isolate"]
                 cmds.setAttr(str(instance) + '.active', 1)
                 self.log.debug('data {}'.format(instance.context[i].data))
diff --git a/openpype/hosts/maya/plugins/publish/extract_playblast.py b/openpype/hosts/maya/plugins/publish/extract_playblast.py
index 7cd829dafb..6f7dd5e16a 100644
--- a/openpype/hosts/maya/plugins/publish/extract_playblast.py
+++ b/openpype/hosts/maya/plugins/publish/extract_playblast.py
@@ -1,6 +1,7 @@
 import os
 import glob
 import contextlib
+
 import clique
 import capture
 
@@ -56,23 +57,30 @@ class ExtractPlayblast(openpype.api.Extractor):
         width_preset = capture_presets["Resolution"]["width"]
         height_preset = capture_presets["Resolution"]["height"]
         # Set resolution variables from asset values
-        instance_width = instance.context.data.get("resolutionWidth")
-        instance_height = instance.context.data.get("resolutionHeight")
+        asset_width = instance.data.get("resolutionWidth")
+        asset_height = instance.data.get("resolutionHeight")
+        review_instance_width = instance.data.get("attrWidth")
+        review_instance_height = instance.data.get("attrHeight")
         preset['camera'] = camera
 
         # Tests if project resolution is set,
         # if it is a value other than zero, that value is
         # used, if not then the asset resolution is
         # used
-        if width_preset != 0:
-            preset["width"] = width_preset
-        else:
-            preset['width'] = instance_width
 
-        if height_preset != 0:
+        if review_instance_width != 0:
+            preset['width'] = review_instance_width
+        elif width_preset == 0:
+            preset['width'] = asset_width
+        elif width_preset != 0:
+            preset['width'] = width_preset
+
+        if review_instance_height != 0:
+            preset['height'] = review_instance_height
+        elif height_preset == 0:
+            preset['height'] = asset_height
+        elif height_preset != 0:
             preset['height'] = height_preset
-        else:
-            preset['height'] = instance_height
 
         preset['start_frame'] = start
         preset['end_frame'] = end