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.

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

insparrow’s picture

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

bgm’s picture

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

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

bgm’s picture

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

Closing old issue. No recent feedback, assuming fixed.