On the order complete page, the following errors were reported:

Warning: Missing argument 3 for uc_cart_checkout_review_form() in uc_cart_checkout_review_form() (line 413 of /Volumes/Webserver/sites/demo.com/sites/all/modules/contrib/ubercart/uc_cart/uc_cart.pages.inc).

Notice: Undefined variable: order in uc_cart_checkout_review_form() (line 414 of /Volumes/Webserver/sites/demo.com/sites/all/modules/contrib/ubercart/uc_cart/uc_cart.pages.inc).

Notice: Trying to get property of non-object in uc_payment_form_uc_cart_checkout_review_form_alter() (line 145 of /Volumes/Webserver/sites/demo.com/sites/all/modules/contrib/ubercart/payment/uc_payment/uc_payment.module).

When I echoed out the $order variable in uc_cart_checkout_review_form(), nothing was being passed over. Changing the following lines passed the order over correctly and resolved the errors for me:

-    $form_state['uc_order'] = $order;
+    $form_state['build_info']['args'] = array($order);

The last issue was resolved by Ben Clark's patch.

CommentFileSizeAuthor
#1 uc_optional_checkout_review-missing-arguments.patch848 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new uc_optional_checkout_review-missing-arguments.patch848 bytes

Patch with these changes attached. Patch also includes Ben Clark's tweaks.

bgm’s picture

bgm commented
Status: Active » Needs review

Ben Clark's patch has been applied: #1519042: Notice: Undefined index: sescrd in uc_credit_cart_review_pre_form_submit().

Which payment gateway are you testing with?

bgm’s picture

bgm commented

Issue #1762664: Authorize.net doesn't charge card unless... marked as a duplicate of this one.

bgm’s picture

bgm commented
Issue summary: View changes
Status: Needs review » Closed (cannot reproduce)

Closing old issue. No recent feedback, assuming fixed.