Closed (fixed)
Project:
Ubercart
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
4 May 2010 at 15:17 UTC
Updated:
13 Nov 2011 at 01:12 UTC
Jump to comment: Most recent file
This is because of this part of the code, line 635:
<?php
foreach (uc_order_status_list('general') as $status) {
$options[$status['id']] = $status['title'];
}
foreach (uc_order_status_list('specific') as $status) {
$options[$status['id']] = $status['title'];
}
?>A quick way to fix it is to add the following after it:
<?php
$sorted_options = array();
foreach (uc_order_status_list() as $status) {
if (isset($options[$status['id']])) {
$sorted_options[$status['id']] = $status['title'];
}
}
$options = $sorted_options;
?>| Comment | File | Size | Author |
|---|---|---|---|
| #2 | uc_order.order_pane.inc_.patch | 799 bytes | anrikun |
Comments
Comment #1
anrikun commentedComment #2
anrikun commentedHere's the patch.
Comment #3
tr commentedLet's see if we can get this tested.
Comment #4
tr commentedComment #6
longwaveFixed in both branches. This may have originally been "by design" but it makes more sense to display the statuses in the same order everywhere.