Minor fix

Author: mayhem

metabrainz/admin/quickbooks/views.py

@@ -56,7 +56,7 @@ def create_invoices(client, invoices):
         '''
         Given a set of existing invoices, fetch the invoice from QuickBooks, make a copy, update it
         with new values and then have QuickBooks save the new invoice. Invoices are not sent,
-        and must be sent via the QuickBooks web interface.
+        and must be sent via the manage command from the metabrainz-prod container.
         '''
 
         for invoice in invoices:

metabrainz/invoices/send_invoices.py

@@ -165,6 +165,10 @@ def send_invoices(self):
                 current_app.logger.warn("Skip invoice %s with status %s" % (invoice.DocNumber, invoice.EmailStatus))
                 continue
 
+            if invoice.DocNumber is None:
+                current_app.logger.warn("Skip invoice %s. Bad invoice number." % invoice.DocNumber)
+                continue
+
             current_app.logger.warn("Invoice %s with status %s" % (invoice.DocNumber, invoice.EmailStatus))
             if float(invoice.TotalAmt) == 0.0:
                 current_app.logger.warn("  marking zero amount invoice %s as sent." % invoice.DocNumber)