Offers are consumed twice on basket lines

I am wondering about how to apply offer for consume offer for django oscar 1.6.4,

I was digging in to the code base, and found out that we call consume 2 times when applying a benefit. For example

• In apps.basket.abstract_models.py

 def discount(self, discount_value, affected_quantity, incl_tax=True,
                 offer=None):
        """
        Apply a discount to this line
        """
        if incl_tax:
            if self._discount_excl_tax > 0:
                raise RuntimeError(
                    "Attempting to discount the tax-inclusive price of a line "
                    "when tax-exclusive discounts are already applied")
            self._discount_incl_tax += discount_value
        else:
            if self._discount_incl_tax > 0:
                raise RuntimeError(
                    "Attempting to discount the tax-exclusive price of a line "
                    "when tax-inclusive discounts are already applied")
            self._discount_excl_tax += discount_value
        self.consume(affected_quantity, offer=offer)

And then again in apps.offer.conditions.py in each condition, they have def consume_items(self, offer, basket, affected_lines): and call line.consume(quantity_to_consume, offer=offer) method again.

I don’t think it’s supposed to call consume twice like this?

Technical details

• Oscar version: django oscar 1.6.4

[pjstevns]

I’ll prepare a fix for this asap.

[pjstevns]

Fix delivered in PR https://github.com/django-oscar/django-oscar/pull/2970