Last modified: 2010-07-04 17:19:32 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T17890, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 15890 - descending tablesort broken for negative numbers
descending tablesort broken for negative numbers
Status: CLOSED WORKSFORME
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.10.x
All All
: Normal minor (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-10-08 06:50 UTC by OSRC Sysint Team
Modified: 2010-07-04 17:19 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
correct descending table sorting for negative numbers (401 bytes, patch)
2008-10-08 08:54 UTC, OSRC Sysint Team
Details

Description OSRC Sysint Team 2008-10-08 06:50:36 UTC
A table of type wikitable sortable that contains negative numbers is not sorted correctly when descending sorting is requested. Ascending sorting works as expected. The result looks like descending search does not recognise negative numbers as numbers, but I don't get how this is possible since ts_resortTable() in wikiprefs.js decides sorting direction after the object type is recognised.

Example:
1,-1, 0, -4, 3, -11, -7, 7, -5, 11
is sorted as 
7, 3, 11, 1, 0, -7, -5, -4, -11, -1
Comment 1 OSRC Sysint Team 2008-10-08 08:54:29 UTC
Created attachment 5400 [details]
correct descending table sorting for negative numbers
Comment 2 Siebrand Mazeland 2008-11-02 23:21:56 UTC
MediaWiki 1.10 is no longer supported. WORKSFORME in 1.14 alpha r42593. See http://meta.wikimedia.org/wiki/Help:Sorting#Examples . I suggest you make a local patch.
Comment 3 OSRC Sysint Team 2008-11-03 08:08:04 UTC
Siebrand wrote:
> I suggest you make a local patch.

I did. Thanks for your help.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links