Order dependend ticket with NotIn ORM operator instead of itertools

helpdesk/apps.py

@@ -11,3 +11,7 @@ class HelpdeskConfig(AppConfig):
 
     def ready(self):
         from . import webhooks # noqa: F401
+
+        from django.db.models.fields import Field
+        from helpdesk.orm_operators import NotIn
+        Field.register_lookup(NotIn)

helpdesk/orm_operators.py

@@ -0,0 +1,7 @@
+from django.db.models.lookups import In
+
+class NotIn(In):
+    lookup_name = "not_in"
+
+    def get_rhs_op(self, connection, rhs):
+        return "NOT IN %s" % rhs

helpdesk/views/staff.py

@@ -424,11 +424,9 @@ def view_ticket(request, ticket_id):
 
         return redirect('helpdesk:edit_ticket_checklist', ticket.id, checklist.id)
 
-    open_dependencies = ticket.ticketdependency.filter(depends_on__status__in=Ticket.OPEN_STATUSES)
-    dependencies = [d for d in itertools.chain(
-        open_dependencies,
-        ticket.ticketdependency.all().difference(open_dependencies)
-    )]
+    # Open tickets on top
+    dependencies = list(ticket.ticketdependency.filter(depends_on__status__in=Ticket.OPEN_STATUSES)) \
+        + list(ticket.ticketdependency.filter(depends_on__status__not_in=Ticket.OPEN_STATUSES))
     
     return render(request, 'helpdesk/ticket.html', {
         'ticket': ticket,