d6_ubercart_order.yml

Change
payment_received: fulfillment

to

payment_received: completed

fulfillment is not part of the default order workflow

CommentFileSizeAuthor
#2 2911810-1.patch507 byteswilleaton

Comments

willeaton created an issue. See original summary.

willeaton’s picture

willeaton commented
Status: Active » Needs review
StatusFileSize
new 2911810-1.patch507 bytes

Created a patch, its a simple 2 liner patch, the fix was suggested and confirmed to be correct by @mglaman

Status: Needs review » Needs work

The last submitted patch, 2: 2911810-1.patch, failed testing. View results

willeaton’s picture

willeaton commented

I dont understand these test failures, I presume thats an issue with the test environment and not my patch...

willeaton’s picture

willeaton commented
Status: Needs work » Needs review
heddn’s picture

heddn
Primary language English
Location Illinois
commented

re #4, it was an intermittent test failure. Re-queuing the test seemed to fix it.

  • quietone committed 73fe578 on 8.x-2.x authored by willeaton 
    Issue #2911810 by willeaton: [Ubercart D6] wrong order state payment...
quietone’s picture

quietone commented
Status: Needs review » Fixed

@willeaton, thanks for finding and fixing this.

Status: Fixed » Closed (fixed)

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