From 5d1fa90fccc1bd261c2af61eb639d677f372c363 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ond=C5=99ej=20Samohel?= <ondrej.samohel@pype.club>
Date: Thu, 27 Oct 2022 12:40:44 +0200
Subject: [PATCH] :recycle: soft fail when applying preset

---
 .../maya/plugins/publish/extract_playblast.py     |  2 +-
 openpype/vendor/python/common/capture.py          | 15 ++++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/openpype/hosts/maya/plugins/publish/extract_playblast.py b/openpype/hosts/maya/plugins/publish/extract_playblast.py
index 1b5b8d34e4..b19d24fad7 100644
--- a/openpype/hosts/maya/plugins/publish/extract_playblast.py
+++ b/openpype/hosts/maya/plugins/publish/extract_playblast.py
@@ -133,7 +133,7 @@ class ExtractPlayblast(publish.Extractor):
                 preset.update(panel_preset)
                 cmds.setFocus(panel)
 
-            path = capture.capture(**preset)
+            path = capture.capture(log=self.log, **preset)
 
         self.log.debug("playblast path  {}".format(path))
 
diff --git a/openpype/vendor/python/common/capture.py b/openpype/vendor/python/common/capture.py
index 86c1c60e56..09a42d84d1 100644
--- a/openpype/vendor/python/common/capture.py
+++ b/openpype/vendor/python/common/capture.py
@@ -7,6 +7,7 @@ Playblasting with independent viewport, camera and display options
 import re
 import sys
 import contextlib
+import logging
 
 from maya import cmds
 from maya import mel
@@ -21,6 +22,7 @@ version_info = (2, 3, 0)
 
 __version__ = "%s.%s.%s" % version_info
 __license__ = "MIT"
+logger = logging.getLogger("capture")
 
 
 def capture(camera=None,
@@ -46,7 +48,8 @@ def capture(camera=None,
             display_options=None,
             viewport_options=None,
             viewport2_options=None,
-            complete_filename=None):
+            complete_filename=None,
+            log=None):
     """Playblast in an independent panel
 
     Arguments:
@@ -91,6 +94,7 @@ def capture(camera=None,
             options, using `Viewport2Options`
         complete_filename (str, optional): Exact name of output file. Use this
             to override the output of `filename` so it excludes frame padding.
+        log (logger, optional): pass logger for logging messages.
 
     Example:
         >>> # Launch default capture
@@ -109,7 +113,9 @@ def capture(camera=None,
 
 
     """
-
+    global logger
+    if log:
+        logger = log
     camera = camera or "persp"
 
     # Ensure camera exists
@@ -736,7 +742,10 @@ def _applied_viewport_options(options, panel):
             plugin_options[plugin] = options.pop(plugin)
 
     # default options
-    cmds.modelEditor(panel, edit=True, **options)
+    try:
+        cmds.modelEditor(panel, edit=True, **options)
+    except TypeError as e:
+        logger.error("Cannot apply options {}".format(e))
 
     # plugin display filter options
     for plugin, state in plugin_options.items():