From 0c296ae14db22f7b961b7c8784cafee2ac2d254e Mon Sep 17 00:00:00 2001
From: Milan Kolar <milan@pype.club>
Date: Fri, 13 Mar 2020 23:51:44 +0100
Subject: [PATCH] rename startFrameReview to frameStartFtrack

---
 pype/plugins/ftrack/publish/integrate_ftrack_instances.py | 4 ++--
 pype/plugins/maya/publish/extract_quicktime.py            | 8 ++------
 .../plugins/standalonepublisher/publish/extract_review.py | 4 ++--
 3 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/pype/plugins/ftrack/publish/integrate_ftrack_instances.py b/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
index 78583b0a2f..ec57f46d61 100644
--- a/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
+++ b/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
@@ -73,9 +73,9 @@ class IntegrateFtrackInstance(pyblish.api.InstancePlugin):
                 '''
                 start_frame = 0
                 end_frame = 1
-                if 'endFrameReview' in comp and 'startFrameReview' in comp:
+                if 'frameEndFtrack' in comp and 'frameStartFtrack' in comp:
                     end_frame += (
-                        comp['endFrameReview'] - comp['startFrameReview']
+                        comp['frameEndFtrack'] - comp['frameStartFtrack']
                     )
                 else:
                     end_frame += (
diff --git a/pype/plugins/maya/publish/extract_quicktime.py b/pype/plugins/maya/publish/extract_quicktime.py
index 94b5a716a2..29d6b78051 100644
--- a/pype/plugins/maya/publish/extract_quicktime.py
+++ b/pype/plugins/maya/publish/extract_quicktime.py
@@ -33,17 +33,13 @@ class ExtractQuicktime(pype.api.Extractor):
 
         # if start and end frames cannot be determined, get them
         # from Maya timeline
-        start = instance.data.get("startFrameReview")
-        end = instance.data.get("endFrameReview")
+        start = instance.data.get("frameStartFtrack")
+        end = instance.data.get("frameEndFtrack")
         if start is None:
             start = cmds.playbackOptions(query=True, animationStartTime=True)
         if end is None:
             end = cmds.playbackOptions(query=True, animationEndTime=True)
         self.log.info("start: {}, end: {}".format(start, end))
-        handles = instance.data.get("handles", 0)
-        if handles:
-            start -= handles
-            end += handles
 
         # get cameras
         camera = instance.data['review_camera']
diff --git a/pype/plugins/standalonepublisher/publish/extract_review.py b/pype/plugins/standalonepublisher/publish/extract_review.py
index 66cdcdf4df..36793d4c62 100644
--- a/pype/plugins/standalonepublisher/publish/extract_review.py
+++ b/pype/plugins/standalonepublisher/publish/extract_review.py
@@ -170,8 +170,8 @@ class ExtractReviewSP(pyblish.api.InstancePlugin):
                     "stagingDir": out_stagigng_dir,
                     "tags": new_tags,
                     "outputName": name,
-                    "startFrameReview": 1,
-                    "endFrameReview": video_len
+                    "frameStartFtrack": 1,
+                    "frameEndFtrack": video_len
                 })
                 # cleanup thumbnail from new repre
                 if repre_new.get("thumbnail"):