The test gateway checks for $order->payment_details['cc_cvv'] when running test_gateway_charge. CVV isn't stored by the gateway for future reference, so this produces a warning when charging a recurring transaction later.

Obviously not a biggie because it's just a test gateway, but figured I'd get rid of the notice ...

CommentFileSizeAuthor
#1 ubercart-1861940-test_gateway_recurring_warning.patch1.16 KBxurizaemon

Comments

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Assigned: xurizaemon » Unassigned
Status: Needs work » Needs review
StatusFileSize
new ubercart-1861940-test_gateway_recurring_warning.patch1.16 KB

Patch simply checks that the 'cc_cvv' index is set before testing its value.

tr’s picture

tr commented
Title: Warning when processing recurring transactions with test gateway » Support recurring transactions with test gateway
Category: bug » feature
Status: Needs review » Postponed (maintainer needs more info)

Recurring payments are handled by uc_recurring, not Ubercart core, so I'm changing this to a feature request. I think it's a reasonable thing to do, but is this the only thing that needs to be done for recurring payments? It would be great if there were test cases written for uc_recurring that took advantage of the test gateway, as the primary purpose of the test gateway is for testing ...

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Postponed (maintainer needs more info) » Fixed

Makes sense to me, all other values should be available from recurring payments. Committed.

Status: Fixed » Closed (fixed)

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