django-oscar / django-oscar-paypal

PayPal integration for django-oscar. Can be used without Oscar too.

Home Page:https://django-oscar-paypal.readthedocs.io/en/latest/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Signature of Repository.get_shipping_methods

st8st8 opened this issue · comments

Line 447 of paypal/express/views.py calls repo.get_shipping_methods with the signature:

def get_shipping_methods(self, user, basket, shipping_address):
    repo = Repository()
    return repo.get_shipping_methods(
        user, basket, shipping_addr=shipping_address)

But in oscar/apps/shipping/repository.py it's given as:

# API
def get_shipping_methods(self, basket, shipping_addr=None, **kwargs):

ie with no "user" argument. I'm not sure which is correct?

Good catch. The signature changed with Oscar 1.1. I'll fix it and will issue a release no later than next week.

FYI, my work-in-progress branch is at #113.