Seems like there's a bug introduced while this fixing #2928286: Simplify order refresh logic.

Now in commerce_discount.rules.inc we have this method:

/**
 * Rules action: Remove discount components on product line items.
 */
function commerce_discount_remove_discount_components_on_products($line_item_wrapper) {
  commerce_discount_remove_discount_components($line_item_wrapper->commerce_unit_price, array('product_discount'));
  commerce_discount_remove_discount_components($line_item_wrapper->total, array('product_discount'));
}

It should probably be $line_item_wrapper->commerce_total as commerce line item entity doesn't usually have "total" property.

You can see error messages related to this issue if you have rules logs enabled.

Comments

potop created an issue. See original summary.

TynanFox’s picture

Yup.
#1 worked for me to fix the error.

  • jsacksick committed c908db1 on 7.x-1.x
    Issue #2931755 by potop, jsacksick: Fix...
jsacksick’s picture

Status: Active » Fixed

Thanks a lot for reporting that!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.