If we turn off all the fields on the checkout page
/admin/store/settings/countries/fields
Then after editing any order
For example page:
/admin/store/orders/ORDER_ID/edit
There are errors:
Notice: Undefined variable: changes в функции uc_order_pane_ship_to() (строка 77 в файле /var/www/jeto/data/www/jeto.ru/sites/all/modules/ubercart/uc_order/uc_order.order_pane.inc).
Notice: Undefined variable: changes в функции uc_order_pane_bill_to() (строка 121 в файле /var/www/jeto/data/www/jeto.ru/sites/all/modules/ubercart/uc_order/uc_order.order_pane.inc).
Bug in the code:
case 'edit-process':
foreach ($form_state['values'] as $key => $value) {
if (substr($key, 0, 9) == 'delivery_') {
if (uc_address_field_enabled(substr($key, 9))) {
$changes[$key] = $value;
}
}
}
return $changes;
Not one field is not included, hence the data is not assembled into a variable, hence there is no variable.
Patch I enclose.
Comment | File | Size | Author |
---|---|---|---|
#7 | 2862008-7.patch | 1.98 KB | TR |
| |||
uc_order--fixed-notice-variable-changes.patch | 984 bytes | yuraosn | |
Comments
Comment #2
yuraosn CreditAttribution: yuraosn as a volunteer and commentedComment #3
TR CreditAttribution: TR commentedComment #5
yuraosn CreditAttribution: yuraosn as a volunteer and commentedComment #6
TR CreditAttribution: TR commentedIn Drupal, fixed point releases like 7.x-3.10 can't be changed - all patches must apply to the head of the development branch, in this case 7.x-3.x-dev.
Your patch failed the test because it is in the wrong format. It looks like you used -p4 instead of -p1. Please fix the format and upload the patch again. See https://www.drupal.org/patch for more details.
Comment #7
TR CreditAttribution: TR commentedHere's a real patch for the testbot to try...
Comment #9
TR CreditAttribution: TR commentedCommitted #7.