Closed
Bug 947675
Opened 10 years ago
Closed 10 years ago
Bug id are sorted in wrong order
Categories
(bugzilla.mozilla.org :: MyDashboard, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: xidorn, Assigned: dkl)
References
()
Details
Attachments
(2 files)
43.19 KB,
image/png
|
Details | |
1.16 KB,
patch
|
glob
:
review+
|
Details | Diff | Splinter Review |
Bug id in tables are now sorted in alphabetical order, but it should be sorted in integer order. The attachment shows the problem.
Reporter | ||
Updated•10 years ago
|
Attachment #8344267 -
Attachment description: 屏幕快照 2013-12-08 下午2.24.13.png → screenshot
Reporter | ||
Updated•10 years ago
|
Attachment #8344267 -
Attachment filename: 屏幕快照 2013-12-08 下午2.24.13.png → screenshot.png
Assignee | ||
Comment 1•10 years ago
|
||
Strange I am unable to make this fail running nightly. The bug id column should already be sorting based on the integer value and not based on the HTML content of the table column. Are you using a special encoding for the dashboard page instead of UTF8? dkl
Flags: needinfo?(quanxunzhen)
Reporter | ||
Comment 2•10 years ago
|
||
No, I'm using UTF-8. And I meet the same problem either in Firefox and in Chrome.
It seems that bug ids are presented in string instead of number in the JSON data, which should be where the problem is. See attachment 8344022 [details].
Flags: needinfo?(quanxunzhen)
Assignee | ||
Comment 3•10 years ago
|
||
Comment on attachment 8345134 [details] [diff] [review] 947675_1.patch r=glob
Attachment #8345134 -
Flags: review?(glob) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Thanks for the report. Fix will be in the next code push to production soon. Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bmo/4.2 modified extensions/MyDashboard/lib/WebService.pm Committed revision 9176. dkl
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•