From 2dd75ea2527dea650829ee5149de7ab02b007aa3 Mon Sep 17 00:00:00 2001
From: Raphael Michel <mail@raphaelmichel.de>
Date: Thu, 25 Jul 2019 11:47:23 +0200
Subject: [PATCH] Hide fees on changing payment method when no fees are taken

---
 .../templates/pretixpresale/event/order_pay_change.html       | 4 +++-
 src/pretix/presale/views/order.py                             | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/pretix/presale/templates/pretixpresale/event/order_pay_change.html b/src/pretix/presale/templates/pretixpresale/event/order_pay_change.html
index dfdc2d297b..ddb7de0d1e 100644
--- a/src/pretix/presale/templates/pretixpresale/event/order_pay_change.html
+++ b/src/pretix/presale/templates/pretixpresale/event/order_pay_change.html
@@ -23,7 +23,9 @@
                     <div class="panel-heading">
                         <h4 class="panel-title">
                             <label class="radio">
-                                <strong class="pull-right">{% if p.fee_diff >= 0 %}+{% else %}-{% endif %} {{ p.fee_diff_abs|money:event.currency }}</strong>
+                                {% if show_fees %}
+                                    <strong class="pull-right">{% if p.fee_diff >= 0 %}+{% else %}-{% endif %} {{ p.fee_diff_abs|money:event.currency }}</strong>
+                                {% endif %}
                                 <input type="radio" name="payment" value="{{ p.provider.identifier }}"
                                         data-parent="#payment_accordion"
                                         {% if selected == p.provider.identifier %}checked="checked"{% endif %}
diff --git a/src/pretix/presale/views/order.py b/src/pretix/presale/views/order.py
index 9aa1b693b2..83840ed7a8 100644
--- a/src/pretix/presale/views/order.py
+++ b/src/pretix/presale/views/order.py
@@ -581,6 +581,7 @@ class OrderPayChangeMethod(EventViewMixin, OrderDetailMixin, TemplateView):
         ctx = super().get_context_data(**kwargs)
         ctx['order'] = self.order
         ctx['providers'] = self.provider_forms
+        ctx['show_fees'] = any(p['fee_diff'] for p in self.provider_forms)
         return ctx
 
     def get_confirm_url(self, payment):