I was tracking down an issue I was having with Ubercart Optional Checkout Review and Ubercart Sage Pay in which the credit card info was lost on the actual transaction page.

I found out that because $_POST['sescrd'] was used, that was not there on the all the pre and post review form alterers that the optional checkout review tries to emulate. Although I can somehow workaround it on the optional checkout review, I believe this proposed change is the proper one.

Let me know what you think and what is the likehood for it to be accepted.

CommentFileSizeAuthor
#1 2123199-1-form-state-values-sescrd.patch1.84 KBhanoii
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hanoii’s picture

hanoii’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2123199-1-form-state-values-sescrd.patch, failed testing.

hanoii’s picture

hmmm, not's quite clear to me why it failed. I guess I'll have to go with the workaround on checkout review.

longwave’s picture

Category: bug » task

Not really a bug, but it would be good to figure out how 'sescrd' works at some point and use the proper form API for everything!