From e8f9c130bbb7b805f6ea68a3bbc37d81168d5893 Mon Sep 17 00:00:00 2001
From: Petr Kalis <petr.kalis@gmail.com>
Date: Thu, 23 Mar 2023 18:11:28 +0100
Subject: [PATCH] OP-3951 - merged global validator and unreal one

Both should do same thing. Combination of host and family should be safe.
---
 .../publish/validate_sequence_frames.py       | 41 -------------------
 .../publish/validate_sequence_frames.py       |  4 +-
 2 files changed, 2 insertions(+), 43 deletions(-)
 delete mode 100644 openpype/hosts/unreal/plugins/publish/validate_sequence_frames.py

diff --git a/openpype/hosts/unreal/plugins/publish/validate_sequence_frames.py b/openpype/hosts/unreal/plugins/publish/validate_sequence_frames.py
deleted file mode 100644
index 87f1338ee8..0000000000
--- a/openpype/hosts/unreal/plugins/publish/validate_sequence_frames.py
+++ /dev/null
@@ -1,41 +0,0 @@
-import clique
-
-import pyblish.api
-
-
-class ValidateSequenceFrames(pyblish.api.InstancePlugin):
-    """Ensure the sequence of frames is complete
-
-    The files found in the folder are checked against the frameStart and
-    frameEnd of the instance. If the first or last file is not
-    corresponding with the first or last frame it is flagged as invalid.
-    """
-
-    order = pyblish.api.ValidatorOrder
-    label = "Validate Sequence Frames"
-    families = ["render"]
-    hosts = ["unreal"]
-    optional = True
-
-    def process(self, instance):
-        representations = instance.data.get("representations")
-        for repr in representations:
-            patterns = [clique.PATTERNS["frames"]]
-            collections, remainder = clique.assemble(
-                repr["files"], minimum_items=1, patterns=patterns)
-
-            assert not remainder, "Must not have remainder"
-            assert len(collections) == 1, "Must detect single collection"
-            collection = collections[0]
-            frames = list(collection.indexes)
-
-            current_range = (frames[0], frames[-1])
-            required_range = (instance.data["frameStart"],
-                              instance.data["frameEnd"])
-
-            if current_range != required_range:
-                raise ValueError(f"Invalid frame range: {current_range} - "
-                                 f"expected: {required_range}")
-
-            missing = collection.holes().indexes
-            assert not missing, "Missing frames: %s" % (missing,)
diff --git a/openpype/plugins/publish/validate_sequence_frames.py b/openpype/plugins/publish/validate_sequence_frames.py
index c932c0e779..56641fbf57 100644
--- a/openpype/plugins/publish/validate_sequence_frames.py
+++ b/openpype/plugins/publish/validate_sequence_frames.py
@@ -21,8 +21,8 @@ class ValidateSequenceFrames(pyblish.api.InstancePlugin):
 
     order = pyblish.api.ValidatorOrder
     label = "Validate Sequence Frames"
-    families = ["imagesequence"]
-    hosts = ["shell"]
+    families = ["imagesequence", "render"]
+    hosts = ["shell", "unreal"]
 
     def process(self, instance):
         representations = instance.data.get("representations")