Currently createPayment sends the commerce order id to authorize.net using the invoiceNumber attribute. For consistency it would be nice if it did the same thing when issuing a refund.

CommentFileSizeAuthor
#3 order_id_as_invoiceNumber_in_refund-2935995-3.patch883 bytesthechanceg
#2 order_id_as_invoiceNumber_in_refund-2935995-2.patch941 bytesthechanceg

Comments

thechanceg created an issue. See original summary.

thechanceg’s picture

thechanceg commented
Status: Active » Needs review
StatusFileSize
new order_id_as_invoiceNumber_in_refund-2935995-2.patch941 bytes

Simple patch that does the same thing that createPayment does.

I've tested this against the authorize.net api and it works.

thechanceg’s picture

thechanceg commented
StatusFileSize
new order_id_as_invoiceNumber_in_refund-2935995-3.patch883 bytes

I was just reviewing my patch and noticed that I accidentally let the CustomerIP line sneak in there. Fixed patch.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Fixed

Thanks!

  • mglaman committed 7dfff14 on 8.x-1.x authored by thechanceg 
    Issue #2935995 by thechanceg: Send Order ID as invoiceNumber during...

Status: Fixed » Closed (fixed)

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