From 0f1ed036231d91ba6f8105d85b0c6e341e5b6487 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Fri, 23 Sep 2022 17:47:19 +0200
Subject: [PATCH] Add Select Invalid action to more validators

---
 .../fusion/plugins/publish/validate_background_depth.py     | 4 ++++
 .../plugins/publish/validate_create_folder_checked.py       | 3 +++
 .../plugins/publish/validate_filename_has_extension.py      | 3 +++
 .../fusion/plugins/publish/validate_saver_passthrough.py    | 3 +++
 .../hosts/fusion/plugins/publish/validate_unique_subsets.py | 6 ++++++
 5 files changed, 19 insertions(+)

diff --git a/openpype/hosts/fusion/plugins/publish/validate_background_depth.py b/openpype/hosts/fusion/plugins/publish/validate_background_depth.py
index f057989535..261533de01 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_background_depth.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_background_depth.py
@@ -3,6 +3,8 @@ import pyblish.api
 from openpype.pipeline.publish import RepairAction
 from openpype.pipeline import PublishValidationError
 
+from openpype.hosts.fusion.api.action import SelectInvalidAction
+
 
 class ValidateBackgroundDepth(pyblish.api.InstancePlugin):
     """Validate if all Background tool are set to float32 bit"""
@@ -14,6 +16,8 @@ class ValidateBackgroundDepth(pyblish.api.InstancePlugin):
     families = ["render"]
     optional = True
 
+    actions = [SelectInvalidAction]
+
     @classmethod
     def get_invalid(cls, instance):
 
diff --git a/openpype/hosts/fusion/plugins/publish/validate_create_folder_checked.py b/openpype/hosts/fusion/plugins/publish/validate_create_folder_checked.py
index 3674b33644..ba943abacb 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_create_folder_checked.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_create_folder_checked.py
@@ -3,6 +3,8 @@ import pyblish.api
 from openpype.pipeline.publish import RepairAction
 from openpype.pipeline import PublishValidationError
 
+from openpype.hosts.fusion.api.action import SelectInvalidAction
+
 
 class ValidateCreateFolderChecked(pyblish.api.InstancePlugin):
     """Valid if all savers have the input attribute CreateDir checked on
@@ -16,6 +18,7 @@ class ValidateCreateFolderChecked(pyblish.api.InstancePlugin):
     label = "Validate Create Folder Checked"
     families = ["render"]
     hosts = ["fusion"]
+    actions = [SelectInvalidAction]
 
     @classmethod
     def get_invalid(cls, instance):
diff --git a/openpype/hosts/fusion/plugins/publish/validate_filename_has_extension.py b/openpype/hosts/fusion/plugins/publish/validate_filename_has_extension.py
index 22f1db809c..bbba2dde6e 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_filename_has_extension.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_filename_has_extension.py
@@ -3,6 +3,8 @@ import os
 import pyblish.api
 from openpype.pipeline import PublishValidationError
 
+from openpype.hosts.fusion.api.action import SelectInvalidAction
+
 
 class ValidateFilenameHasExtension(pyblish.api.InstancePlugin):
     """Ensure the Saver has an extension in the filename path
@@ -17,6 +19,7 @@ class ValidateFilenameHasExtension(pyblish.api.InstancePlugin):
     label = "Validate Filename Has Extension"
     families = ["render"]
     hosts = ["fusion"]
+    actions = [SelectInvalidAction]
 
     def process(self, instance):
         invalid = self.get_invalid(instance)
diff --git a/openpype/hosts/fusion/plugins/publish/validate_saver_passthrough.py b/openpype/hosts/fusion/plugins/publish/validate_saver_passthrough.py
index bbafd8949e..56f2e7e6b8 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_saver_passthrough.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_saver_passthrough.py
@@ -1,6 +1,8 @@
 import pyblish.api
 from openpype.pipeline import PublishValidationError
 
+from openpype.hosts.fusion.api.action import SelectInvalidAction
+
 
 class ValidateSaverPassthrough(pyblish.api.ContextPlugin):
     """Validate saver passthrough is similar to Pyblish publish state"""
@@ -9,6 +11,7 @@ class ValidateSaverPassthrough(pyblish.api.ContextPlugin):
     label = "Validate Saver Passthrough"
     families = ["render"]
     hosts = ["fusion"]
+    actions = [SelectInvalidAction]
 
     def process(self, context):
 
diff --git a/openpype/hosts/fusion/plugins/publish/validate_unique_subsets.py b/openpype/hosts/fusion/plugins/publish/validate_unique_subsets.py
index 5f0f93f764..28bab59949 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_unique_subsets.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_unique_subsets.py
@@ -3,6 +3,8 @@ from collections import defaultdict
 import pyblish.api
 from openpype.pipeline import PublishValidationError
 
+from openpype.hosts.fusion.api.action import SelectInvalidAction
+
 
 class ValidateUniqueSubsets(pyblish.api.ContextPlugin):
     """Ensure all instances have a unique subset name"""
@@ -11,6 +13,7 @@ class ValidateUniqueSubsets(pyblish.api.ContextPlugin):
     label = "Validate Unique Subsets"
     families = ["render"]
     hosts = ["fusion"]
+    actions = [SelectInvalidAction]
 
     @classmethod
     def get_invalid(cls, context):
@@ -38,6 +41,9 @@ class ValidateUniqueSubsets(pyblish.api.ContextPlugin):
                     ))
                     invalid.extend(instances)
 
+        # Return tools for the invalid instances so they can be selected
+        invalid = [instance.data["tool"] for instance in invalid]
+
         return invalid
 
     def process(self, context):