From 748dcf1ad207edd3dbf3bc98120d9e46bf9b39e0 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 11 Aug 2022 15:51:27 +0200
Subject: [PATCH] fix filter and sort

---
 .../workfile/abstract_template_loader.py      | 29 ++++++-------------
 1 file changed, 9 insertions(+), 20 deletions(-)

diff --git a/openpype/pipeline/workfile/abstract_template_loader.py b/openpype/pipeline/workfile/abstract_template_loader.py
index 5afec56d71..1c8ede25e6 100644
--- a/openpype/pipeline/workfile/abstract_template_loader.py
+++ b/openpype/pipeline/workfile/abstract_template_loader.py
@@ -351,11 +351,15 @@ class AbstractTemplateLoader:
         self.populate_template(ignored_ids=loaded_containers_ids)
 
     def get_placeholders(self):
-        placeholder_class = self.placeholder_class
-        placeholders = map(placeholder_class, self.get_template_nodes())
-        valid_placeholders = filter(placeholder_class.is_valid, placeholders)
-        sorted_placeholders = sorted(valid_placeholders,
-                                     key=placeholder_class.get_order)
+        placeholders = map(self.placeholder_class, self.get_template_nodes())
+        valid_placeholders = filter(
+            lambda i: i.is_valid,
+            placeholders
+        )
+        sorted_placeholders = list(sorted(
+            valid_placeholders,
+            key=lambda i: i.order
+        ))
         return sorted_placeholders
 
     @abstractmethod
@@ -450,21 +454,6 @@ class AbstractPlaceholder:
     def order(self):
         return self.data["order"]
 
-    def get_order(self):
-        """Placeholder order.
-
-        Order is used to sort them by priority
-        Priority is lowset first, highest last
-        (ex:
-            1: First to load
-            100: Last to load)
-
-        Returns:
-            int: Order priority
-        """
-
-        return self.order
-
     @property
     def loader_name(self):
         """Return placeholder loader type.