Modify payment_methods_pool about django-salesman HOT 4 CLOSED

@dinoperovic I think is_enabled is actually a much better idea and would solve all our questions much easier.

from django-salesman.

Hi @madisvain, would adding something like is_enabled(self, request) method on PaymentMethod class solve your case? That way you could disable payments for a specific request.

This requires adding an optional request param to the pool methods such as get_choices and get_payments and the drawback would be that all URL's for payments are still included (even though they are disabled).

Are there any other advantages to making PaymentMethodPool overridable instead?

from django-salesman.

@madisvain I will try to implement this soon then.

from django-salesman.

@madisvain added in v1.1.6

from django-salesman.