From 1a1e73649866a77e932075b901fe8edccf2e29ca Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Mon, 17 Feb 2020 17:36:31 +0100
Subject: [PATCH] moved identation by changing validation condition logic

---
 pype/plugins/global/publish/extract_jpeg.py | 95 +++++++++++----------
 1 file changed, 49 insertions(+), 46 deletions(-)

diff --git a/pype/plugins/global/publish/extract_jpeg.py b/pype/plugins/global/publish/extract_jpeg.py
index 28d16198cd..abd20bb9ea 100644
--- a/pype/plugins/global/publish/extract_jpeg.py
+++ b/pype/plugins/global/publish/extract_jpeg.py
@@ -41,63 +41,66 @@ class ExtractJpegEXR(pyblish.api.InstancePlugin):
 
         for repre in representations:
             self.log.debug(repre)
-            if 'review' in repre['tags'] or "thumb-nuke" in repre['tags']:
-                if not isinstance(repre['files'], list):
-                    continue
+            valid = 'review' in repre['tags'] or "thumb-nuke" in repre['tags']
+            if not valid:
+                continue
 
-                input_file = repre['files'][0]
+            if not isinstance(repre['files'], list):
+                continue
 
-                # input_file = (
-                #     collections[0].format('{head}{padding}{tail}') % start
-                # )
-                full_input_path = os.path.join(stagingdir, input_file)
-                self.log.info("input {}".format(full_input_path))
+            input_file = repre['files'][0]
 
-                filename = os.path.splitext(input_file)[0]
-                if not filename.endswith('.'):
-                    filename += "."
-                jpeg_file = filename + "jpg"
-                full_output_path = os.path.join(stagingdir, jpeg_file)
+            # input_file = (
+            #     collections[0].format('{head}{padding}{tail}') % start
+            # )
+            full_input_path = os.path.join(stagingdir, input_file)
+            self.log.info("input {}".format(full_input_path))
 
-                self.log.info("output {}".format(full_output_path))
+            filename = os.path.splitext(input_file)[0]
+            if not filename.endswith('.'):
+                filename += "."
+            jpeg_file = filename + "jpg"
+            full_output_path = os.path.join(stagingdir, jpeg_file)
 
-                config_data = instance.context.data['output_repre_config']
+            self.log.info("output {}".format(full_output_path))
 
-                proj_name = os.environ.get('AVALON_PROJECT', '__default__')
-                profile = config_data.get(proj_name, config_data['__default__'])
+            config_data = instance.context.data['output_repre_config']
 
-                jpeg_items = []
-                jpeg_items.append(
-                    os.path.join(os.environ.get("FFMPEG_PATH"), "ffmpeg"))
-                # override file if already exists
-                jpeg_items.append("-y")
-                # use same input args like with mov
-                jpeg_items.extend(profile.get('input', []))
-                # input file
-                jpeg_items.append("-i {}".format(full_input_path))
-                # output file
-                jpeg_items.append(full_output_path)
+            proj_name = os.environ.get('AVALON_PROJECT', '__default__')
+            profile = config_data.get(proj_name, config_data['__default__'])
 
-                subprocess_jpeg = " ".join(jpeg_items)
+            jpeg_items = []
+            jpeg_items.append(
+                os.path.join(os.environ.get("FFMPEG_PATH"), "ffmpeg"))
+            # override file if already exists
+            jpeg_items.append("-y")
+            # use same input args like with mov
+            jpeg_items.extend(profile.get('input', []))
+            # input file
+            jpeg_items.append("-i {}".format(full_input_path))
+            # output file
+            jpeg_items.append(full_output_path)
 
-                # run subprocess
-                self.log.debug("{}".format(subprocess_jpeg))
-                pype.api.subprocess(subprocess_jpeg)
+            subprocess_jpeg = " ".join(jpeg_items)
 
-                if "representations" not in instance.data:
-                    instance.data["representations"] = []
+            # run subprocess
+            self.log.debug("{}".format(subprocess_jpeg))
+            pype.api.subprocess(subprocess_jpeg)
 
-                representation = {
-                    'name': 'thumbnail',
-                    'ext': 'jpg',
-                    'files': jpeg_file,
-                    "stagingDir": stagingdir,
-                    "thumbnail": True,
-                    "tags": ['thumbnail']
-                }
+            if "representations" not in instance.data:
+                instance.data["representations"] = []
 
-                # adding representation
-                self.log.debug("Adding: {}".format(representation))
-                representations_new.append(representation)
+            representation = {
+                'name': 'thumbnail',
+                'ext': 'jpg',
+                'files': jpeg_file,
+                "stagingDir": stagingdir,
+                "thumbnail": True,
+                "tags": ['thumbnail']
+            }
+
+            # adding representation
+            self.log.debug("Adding: {}".format(representation))
+            representations_new.append(representation)
 
         instance.data["representations"] = representations_new