From 35270e7032005ba06ee4bdd38413d232986f8608 Mon Sep 17 00:00:00 2001
From: Raphael Michel <mail@raphaelmichel.de>
Date: Wed, 20 Jan 2021 18:10:40 +0100
Subject: [PATCH] Allow to change order of refund list

---
 src/pretix/control/forms/filter.py            |  7 ++++
 .../pretixcontrol/orders/refunds.html         | 36 +++++++++++++++----
 2 files changed, 37 insertions(+), 6 deletions(-)

diff --git a/src/pretix/control/forms/filter.py b/src/pretix/control/forms/filter.py
index 24177c484..84004cc60 100644
--- a/src/pretix/control/forms/filter.py
+++ b/src/pretix/control/forms/filter.py
@@ -1506,6 +1506,9 @@ class VoucherTagFilterForm(FilterForm):
 
 
 class RefundFilterForm(FilterForm):
+    orders = {'provider': 'provider', 'state': 'state', 'order': 'order__code',
+              'source': 'source', 'amount': 'amount', 'created': 'created'}
+
     provider = forms.ChoiceField(
         label=_('Payment provider'),
         choices=[
@@ -1542,6 +1545,10 @@ class RefundFilterForm(FilterForm):
             qs = qs.filter(state__in=[OrderRefund.REFUND_STATE_CREATED, OrderRefund.REFUND_STATE_TRANSIT,
                                       OrderRefund.REFUND_STATE_EXTERNAL])
 
+        if fdata.get('ordering'):
+            qs = qs.order_by(self.get_order_by())
+        else:
+            qs = qs.order_by('-created')
         return qs
 
 
diff --git a/src/pretix/control/templates/pretixcontrol/orders/refunds.html b/src/pretix/control/templates/pretixcontrol/orders/refunds.html
index 758dbe472..275ef8836 100644
--- a/src/pretix/control/templates/pretixcontrol/orders/refunds.html
+++ b/src/pretix/control/templates/pretixcontrol/orders/refunds.html
@@ -38,12 +38,36 @@
             <table class="table table-condensed table-hover">
                 <thead>
                 <tr>
-                    <th>#</th>
-                    <th>{% trans "Payment provider" %}</th>
-                    <th>{% trans "Start date" %}</th>
-                    <th>{% trans "Source" %}</th>
-                    <th>{% trans "Status" %}</th>
-                    <th class="text-right flip">{% trans "Amount" %}</th>
+                    <th>
+                        #
+                        <a href="?{% url_replace request 'ordering' '-order' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'order' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
+                    <th>
+                        {% trans "Payment provider" %}
+                        <a href="?{% url_replace request 'ordering' '-provider' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'provider' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
+                    <th>
+                        {% trans "Start date" %}
+                        <a href="?{% url_replace request 'ordering' '-created' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'created' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
+                    <th>
+                        {% trans "Source" %}
+                        <a href="?{% url_replace request 'ordering' '-source' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'source' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
+                    <th>
+                        {% trans "Status" %}
+                        <a href="?{% url_replace request 'ordering' '-state' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'state' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
+                    <th class="text-right flip">
+                        {% trans "Amount" %}
+                        <a href="?{% url_replace request 'ordering' '-amount' %}"><i class="fa fa-caret-down"></i></a>
+                        <a href="?{% url_replace request 'ordering' 'amount' %}"><i class="fa fa-caret-up"></i></a></th>
+                    </th>
                     <th class="text-right flip">{% trans "Actions" %}</th>
                 </tr>
                 </thead>