Fix Use string and not variables in %-notation fornat

cotisations/forms.py

@@ -355,7 +355,7 @@ class RechargeForm(FormRevMixin, Form):
             raise forms.ValidationError(
                 _("Requested amount is too small. Minimum amount possible : \
                 %(min_online_amount)s €.") % {
-                    min_online_amount: OptionalUser.get_cached_value(
+                    'min_online_amount': OptionalUser.get_cached_value(
                         'min_online_payment'
                     )
                 }
@@ -364,7 +364,7 @@ class RechargeForm(FormRevMixin, Form):
             raise forms.ValidationError(
                 _("Requested amount is too high. Your balance can't exceed \
                 %(max_online_balance)s €.") % {
-                    max_online_balance: OptionalUser.get_cached_value(
+                    'max_online_balance': OptionalUser.get_cached_value(
                         'max_solde'
                     )
                 }

cotisations/payment.py

@@ -27,7 +27,7 @@ def accept_payment(request, factureid):
     messages.success(
         request,
         _("The payment of %(amount)s € has been accepted.") % {
-            amount: facture.prix()
+            'amount': facture.prix()
         }
     )
     return redirect(reverse('users:profil', kwargs={'userid':request.user.id}))

cotisations/views.py

@@ -161,8 +161,8 @@ def new_facture(request, user, userid):
                     request,
                     _("The cotisation of %(member_name)s has been \
                     extended to %(end_date)s.") % {
-                        member_name: user.pseudo,
-                        end_date: user.end_adhesion()
+                        'member_name': user.pseudo,
+                        'end_date': user.end_adhesion()
                     }
                 )
             # Else, only tell the invoice was created
@@ -508,7 +508,7 @@ def del_paiement(request, instances):
                     request,
                     _("The payment method %(method_name)s has been \
                     successfully deleted.") % {
-                        method_name: payment_del
+                        'method_name': payment_del
                     }
                 )
             except ProtectedError:
@@ -516,7 +516,7 @@ def del_paiement(request, instances):
                     request,
                     _("The payment method %(method_name)s can't be deleted \
                     because there are invoices using it.") % {
-                        method_name: payment_del
+                        'method_name': payment_del
                     }
                 )
         return redirect(reverse('cotisations:index-paiement'))
@@ -586,7 +586,7 @@ def del_banque(request, instances):
                     request,
                     _("The bank %(bank_name)s has been successfully \
                     deleted.") % {
-                        bank_name: bank_del
+                        'bank_name': bank_del
                     }
                 )
             except ProtectedError:
@@ -594,7 +594,7 @@ def del_banque(request, instances):
                     request,
                     _("The bank %(bank_name)s can't be deleted \
                     because there are invoices using it.") % {
-                        bank_name: bank_del
+                        'bank_name': bank_del
                     }
                 )
         return redirect(reverse('cotisations:index-banque'))
@@ -779,8 +779,8 @@ def new_facture_solde(request, userid):
                     request,
                     _("The cotisation of %(member_name)s has been successfully \
                     extended to %(end_date)s.") % {
-                        member_name: user.pseudo,
-                        end_date: user.end_adhesion()
+                        'member_name': user.pseudo,
+                        'end_date': user.end_adhesion()
                     }
                 )
             # Else, only tell the invoice was created