Do not call deprecated method on stripe.Customer

The invoices method was removed on the stripe.Customer object in stripe/stripe-python@d416e9e

This replaces that same logic to fetch the invoices and sync them and
does it within the method itself. This should be backwards compatible
with stripe-python < 2.0 as well as the 2.0 and up release.

Fixes #582

pinax/stripe/actions/invoices.py

@@ -131,7 +131,9 @@ def sync_invoices_for_customer(customer):
     Args:
         customer: the customer for whom to synchronize all invoices
     """
-    for invoice in customer.stripe_customer.invoices().data:
+    stripe_customer = customer.stripe_customer
+    stripe_invoices = stripe.Invoice.list(customer=stripe_customer.id)
+    for invoice in stripe_invoices.data:
         sync_invoice_from_stripe_data(invoice, send_receipt=False)
 
 

pinax/stripe/tests/test_actions.py

@@ -1667,9 +1667,11 @@ def test_sync_customer_purged_remotely_not_locally(self, RetrieveMock, SyncPayme
         self.assertTrue(PurgeLocalMock.called)
 
     @patch("pinax.stripe.actions.invoices.sync_invoice_from_stripe_data")
+    @patch("stripe.Invoice.list")
     @patch("stripe.Customer.retrieve")
-    def test_sync_invoices_for_customer(self, RetreiveMock, SyncMock):
+    def test_sync_invoices_for_customer(self, RetreiveMock, InvoicesMock, SyncMock):
         RetreiveMock().invoices().data = [Mock()]
+        InvoicesMock().data = [Mock()]
         invoices.sync_invoices_for_customer(self.customer)
         self.assertTrue(SyncMock.called)