Minor bug:

in the uc_order.admin.inc line 1260, Undefined variable 'order_id'.

if ($order === FALSE) {
drupal_set_message(t('Order @order_id does not exist.', array('@order_id' => $order_id)));
drupal_goto('admin/store/orders');
}

should be:

if ($order === FALSE) {
drupal_set_message(t('Order @order_id does not exist.', array('@order_id' => $form_state['values']['order_id'])));
drupal_goto('admin/store/orders');
}

Comments

lixiphp’s picture

Status: Active » Needs review
TR’s picture

Status: Needs review » Postponed (maintainer needs more info)

When/where did you see this notice? How can I reproduce it?

longwave’s picture

Status: Postponed (maintainer needs more info) » Fixed

I don't even see how that code is reachable, as the form cannot be generated if the order cannot be loaded, so I removed that block entirely.

Status: Fixed » Closed (fixed)

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