Describe your bug or feature request.

The order items automatically added by the BuyXGetY promotion offer are cleared after all processors run by the OrderRefresh service.

It feels more right to actually do that from the promotion order processor... So that any processor that runs after the promotion order processor don't actually get this order item that's about to be removed.

Patch to follow.

CommentFileSizeAuthor
#4 3307069-4.patch2.64 KBjsacksick
#3 3307069-2.patch2.61 KBjsacksick
#2 3306669-2.patch1.47 KBjsacksick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jsacksick created an issue. See original summary.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Active » Needs review
FileSize
3306669-2.patch1.47 KB
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
FileSize
3307069-2.patch2.61 KB

Shoot... Wrong patch.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
FileSize
3307069-4.patch2.64 KB

  • jsacksick committed 1874745 on 8.x-2.x 
    Issue #3307069 by jsacksick: [BuyXGetY] Remove automatically added order...
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs review » Fixed

  • jsacksick committed c412059 on 3.0.x 
    Issue #3307069 by jsacksick: [BuyXGetY] Remove automatically added order...

Status: Fixed » Closed (fixed)

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