Refs #131 -- Allow manual reissuing of invoices

src/pretix/control/logdisplay.py

@@ -16,6 +16,9 @@ def pretixcontrol_logentry_display(sender, logentry, **kwargs):
         'pretix.event.order.refunded': _('The order has been refunded.'),
         'pretix.event.order.cancelled': _('The order has been cancelled.'),
         'pretix.event.order.placed': _('The order has been created.'),
+        'pretix.event.order.invoice.generated': _('The invoice has been generated.'),
+        'pretix.event.order.invoice.regenerated': _('The invoice has been regenerated.'),
+        'pretix.event.order.invoice.reissued': _('The invoice has been reissued.'),
     }
     if logentry.action_type in plains:
         return plains[logentry.action_type]

src/pretix/control/templates/pretixcontrol/order/index.html

@@ -110,9 +110,18 @@
                                                 action="{% url "control:event.order.regeninvoice" event=request.event.slug organizer=request.event.organizer.slug code=order.code id=i.pk %}">
                                             {% csrf_token %}
                                             <button class="btn btn-default btn-xs">
-                                                {% trans "Regenerate invoice" %}
+                                                {% trans "Regenerate" %}
                                             </button>
                                         </form>
+                                        {% if not i.is_cancellation %}
+                                            <form class="form-inline helper-display-inline" method="post"
+                                                    action="{% url "control:event.order.reissueinvoice" event=request.event.slug organizer=request.event.organizer.slug code=order.code id=i.pk %}">
+                                                {% csrf_token %}
+                                                <button class="btn btn-default btn-xs">
+                                                    {% trans "Cancel and reissue" %}
+                                                </button>
+                                            </form>
+                                        {% endif %}
                                     {% endif %}
                                     {% if forloop.revcounter0 > 0 %}
                                         <br/>

src/pretix/control/urls.py

@@ -73,6 +73,8 @@ urlpatterns = [
             name='event.order.geninvoice'),
         url(r'^orders/(?P<code>[0-9A-Z]+)/invoices/(?P<id>\d+)/regenerate$', orders.OrderInvoiceRegenerate.as_view(),
             name='event.order.regeninvoice'),
+        url(r'^orders/(?P<code>[0-9A-Z]+)/invoices/(?P<id>\d+)/reissue$', orders.OrderInvoiceReissue.as_view(),
+            name='event.order.reissueinvoice'),
         url(r'^orders/(?P<code>[0-9A-Z]+)/extend$', orders.OrderExtend.as_view(),
             name='event.order.extend'),
         url(r'^orders/(?P<code>[0-9A-Z]+)/$', orders.OrderDetail.as_view(), name='event.order'),

src/pretix/control/views/orders.py

@@ -18,7 +18,8 @@ from pretix.base.models import (
 from pretix.base.services import tickets
 from pretix.base.services.export import export
 from pretix.base.services.invoices import (
-    generate_invoice, invoice_pdf, invoice_qualified, regenerate_invoice,
+    generate_cancellation, generate_invoice, invoice_pdf, invoice_qualified,
+    regenerate_invoice,
 )
 from pretix.base.services.mail import mail
 from pretix.base.services.orders import cancel_order, mark_order_paid
@@ -229,7 +230,7 @@ class OrderInvoiceCreate(OrderView):
             messages.error(self.request, _('An invoice for this order already exists.'))
         else:
             inv = generate_invoice(self.order)
-            self.order.log_action('pretix.event.order.invoice.generate', user=self.request.user, data={
+            self.order.log_action('pretix.event.order.invoice.generated', user=self.request.user, data={
                 'invoice': inv.pk
             })
             messages.success(self.request, _('The invoice has been generated.'))
@@ -247,12 +248,39 @@ class OrderInvoiceRegenerate(OrderView):
             inv = self.order.invoices.get(pk=kwargs.get('id'))
         except Order.DoesNotExist:
             messages.error(self.request, _('Unknown invoice.'))
+        else:
+            if inv.canceled:
+                messages.error(self.request, _('The invoice has already been canceled.'))
+            else:
+                inv = regenerate_invoice(inv)
+                self.order.log_action('pretix.event.order.invoice.regenerated', user=self.request.user, data={
+                    'invoice': inv.pk
+                })
+                messages.success(self.request, _('The invoice has been regenerated.'))
+        return redirect(self.get_order_url())
 
-        inv = regenerate_invoice(inv)
+    def get(self, *args, **kwargs):
-        self.order.log_action('pretix.event.order.invoice.regenerate', user=self.request.user, data={
+        return HttpResponseNotAllowed(['POST'])
-            'invoice': inv.pk
+
-        })
+
-        messages.success(self.request, _('The invoice has been regenerated.'))
+class OrderInvoiceReissue(OrderView):
+    permission = 'can_change_orders'
+
+    def post(self, *args, **kwargs):
+        try:
+            inv = self.order.invoices.get(pk=kwargs.get('id'))
+        except Order.DoesNotExist:
+            messages.error(self.request, _('Unknown invoice.'))
+        else:
+            if inv.canceled:
+                messages.error(self.request, _('The invoice has already been canceled.'))
+            else:
+                generate_cancellation(inv)
+                inv = generate_invoice(self.order)
+                self.order.log_action('pretix.event.order.invoice.reissued', user=self.request.user, data={
+                    'invoice': inv.pk
+                })
+                messages.success(self.request, _('The invoice has been reissued.'))
         return redirect(self.get_order_url())
 
     def get(self, *args, **kwargs):

src/pretix/presale/views/order.py

@@ -166,8 +166,10 @@ class OrderInvoiceCreate(EventViewMixin, OrderDetailMixin, View):
         elif self.order.invoices.exists():
             messages.error(self.request, _('An invoice for this order already exists.'))
         else:
-            generate_invoice(self.order)
+            i = generate_invoice(self.order)
-            self.order.log_action('pretix.event.order.invoice.generate')
+            self.order.log_action('pretix.event.order.invoice.generated', data={
+                'invoice': i.pk
+            })
             messages.success(self.request, _('The invoice has been generated.'))
         return redirect(self.get_order_url())