From 1738a177d2adaaaa8fa3cc95806c327673d0a866 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Tue, 1 Nov 2022 14:43:34 +0100
Subject: [PATCH] tvpaint loaders are using attribute definitions instead of
 qargparse

---
 .../hosts/tvpaint/plugins/load/load_image.py  | 44 +++++++++---------
 .../plugins/load/load_reference_image.py      | 45 ++++++++++---------
 2 files changed, 46 insertions(+), 43 deletions(-)

diff --git a/openpype/hosts/tvpaint/plugins/load/load_image.py b/openpype/hosts/tvpaint/plugins/load/load_image.py
index 151db94135..5283d04355 100644
--- a/openpype/hosts/tvpaint/plugins/load/load_image.py
+++ b/openpype/hosts/tvpaint/plugins/load/load_image.py
@@ -1,4 +1,4 @@
-import qargparse
+from openpype.lib.attribute_definitions import BoolDef
 from openpype.hosts.tvpaint.api import plugin
 from openpype.hosts.tvpaint.api.lib import execute_george_through_file
 
@@ -27,26 +27,28 @@ class ImportImage(plugin.Loader):
         "preload": True
     }
 
-    options = [
-        qargparse.Boolean(
-            "stretch",
-            label="Stretch to project size",
-            default=True,
-            help="Stretch loaded image/s to project resolution?"
-        ),
-        qargparse.Boolean(
-            "timestretch",
-            label="Stretch to timeline length",
-            default=True,
-            help="Clip loaded image/s to timeline length?"
-        ),
-        qargparse.Boolean(
-            "preload",
-            label="Preload loaded image/s",
-            default=True,
-            help="Preload image/s?"
-        )
-    ]
+    @classmethod
+    def get_options(cls, contexts):
+        return [
+            BoolDef(
+                "stretch",
+                label="Stretch to project size",
+                default=cls.defaults["stretch"],
+                tooltip="Stretch loaded image/s to project resolution?"
+            ),
+            BoolDef(
+                "timestretch",
+                label="Stretch to timeline length",
+                default=cls.defaults["timestretch"],
+                tooltip="Clip loaded image/s to timeline length?"
+            ),
+            BoolDef(
+                "preload",
+                label="Preload loaded image/s",
+                default=cls.defaults["preload"],
+                tooltip="Preload image/s?"
+            )
+        ]
 
     def load(self, context, name, namespace, options):
         stretch = options.get("stretch", self.defaults["stretch"])
diff --git a/openpype/hosts/tvpaint/plugins/load/load_reference_image.py b/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
index 393236fba6..7f7a68cc41 100644
--- a/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
+++ b/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
@@ -1,7 +1,6 @@
 import collections
 
-import qargparse
-
+from openpype.lib.attribute_definitions import BoolDef
 from openpype.pipeline import (
     get_representation_context,
     register_host,
@@ -42,26 +41,28 @@ class LoadImage(plugin.Loader):
         "preload": True
     }
 
-    options = [
-        qargparse.Boolean(
-            "stretch",
-            label="Stretch to project size",
-            default=True,
-            help="Stretch loaded image/s to project resolution?"
-        ),
-        qargparse.Boolean(
-            "timestretch",
-            label="Stretch to timeline length",
-            default=True,
-            help="Clip loaded image/s to timeline length?"
-        ),
-        qargparse.Boolean(
-            "preload",
-            label="Preload loaded image/s",
-            default=True,
-            help="Preload image/s?"
-        )
-    ]
+    @classmethod
+    def get_options(cls, contexts):
+        return [
+            BoolDef(
+                "stretch",
+                label="Stretch to project size",
+                default=cls.defaults["stretch"],
+                tooltip="Stretch loaded image/s to project resolution?"
+            ),
+            BoolDef(
+                "timestretch",
+                label="Stretch to timeline length",
+                default=cls.defaults["timestretch"],
+                tooltip="Clip loaded image/s to timeline length?"
+            ),
+            BoolDef(
+                "preload",
+                label="Preload loaded image/s",
+                default=cls.defaults["preload"],
+                tooltip="Preload image/s?"
+            )
+        ]
 
     def load(self, context, name, namespace, options):
         stretch = options.get("stretch", self.defaults["stretch"])