In fixing #553668: Attributes missing from 'view orders' form, I encountered the following error message:

Notice: Undefined variable: output in uc_payment_method_check() (line 400 of /var/www/dropdown6/sites/all/modules/ubercart/payment/uc_payment_pack/uc_payment_pack.module).

This occurred in a test site set up to test the referenced issue. I enabled the Payment method pack in order to debug the above issue and no other payment modules (eg. Test gateway) were enabled.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR’s picture

Version: 6.x-2.11 » 6.x-2.x-dev
Priority: Normal » Minor

Looks like there could be a problem at line 204 as well.

PHP notices in D6 are considered minor because D6 was not designed to be E_ALL compliant, and the default reporting level for Drupal in D6 is to suppress these notices. They only show up if you have changed your notification settings or if you're using a distribution like Pressflow which has non-standard notification settings.

A patch would be appreciated ...

trobey’s picture

Status: Active » Needs review
FileSize
1.48 KB

Here is a patch that fixes three error messages.

Status: Needs review » Needs work

The last submitted patch, Undefined variable output-1867632-2.patch, failed testing.

longwave’s picture

Status: Needs work » Fixed

Thanks for the patch. Fixed in http://drupalcode.org/project/ubercart.git/commitdiff/41a350f - based on the patch above, but closer to the changes already made in 7.x-3.x.

Status: Fixed » Closed (fixed)

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