The following error is recorded in dblog when loading the checkout page.

Notice: Undefined variable: user in uc_payment_get_details() (line 413 of ubercart/payment/uc_payment/uc_payment.module).

CommentFileSizeAuthor
#4 ubercart-n1854598-4.patch857 bytesDamienMcKenna
#1 ubercart-n1854598.patch498 bytesDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
ubercart-n1854598.patch498 bytes

The global $user variable wasn't loaded before using $user->uid.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Title: Undefined variable: user in uc_payment_get_details » Undefined variables in uc_payment_get_details
Status: Needs review » Needs work

Another error two lines later:

Notice: Undefined index: cart_order in uc_payment_get_details() (line 410 of ubercart/payment/uc_payment/uc_payment.module).

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

A third error, just a few lines further on:

Notice: Trying to get property of non-object in uc_payment_get_details() (line 415 of ubercart/payment/uc_payment/uc_payment.module).

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review
FileSize
ubercart-n1854598-4.patch857 bytes

This patch resolves all three errors.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Needs review » Fixed

Committed, thanks for the patches.

Status: Fixed » Closed (fixed)

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