Last modified: 2009-01-24 15:43:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 13535 - Allow a way to force numeric sort for sortable tables
Allow a way to force numeric sort for sortable tables
Status: RESOLVED DUPLICATE of bug 15406
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
Depends on:
  Show dependency treegraph
Reported: 2008-03-27 17:19 UTC by Frank Allard
Modified: 2009-01-24 15:43 UTC (History)
3 users (show)

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


Description Frank Allard 2008-03-27 17:19:08 UTC

These examples are in wikipedia on the page.  sm=n is suppose to make a column sort numeric when in the first sortable row.

{|class="wikitable sortable" <!-- | style="width:100%;" -->
! Summary{{smn}}
| 1.4 L{{smn}}
| 2 L{{smn}}
| 19 L{{smn}}
| 1 L{{smn}}

{|class="wikitable sortable"
! Summary
|1.4L sm=n
|2L sm=n
|19L sm=n
|1L  sm=n

When you press the sort button on the table, it orders the numbers: 1L, 1.4L, 19L, 2L. It may be new math, but it should be: 1L, 1.4L, 2L, 19L. (I used the sm=n flag, BTW) 

I'm using Firefox, but I don't think it matters...

It seems the sort functions are in this file:

The part that is missing should be in here:

function ts_resortTable(lnk) {
        // get the span
        var span = lnk.getElementsByTagName('span')[0];
        var td = lnk.parentNode;
        var tr = td.parentNode;
        var column = td.cellIndex;
        var table = tr.parentNode;
        while (table && !(table.tagName && table.tagName.toLowerCase() == 'table'))
                table = table.parentNode;
        if (!table) return;
        // Work out a type for the column
        if (table.rows.length <= 1) return;
        // Skip the first row if that's where the headings are
        var rowStart = (table.tHead && table.tHead.rows.length > 0 ? 0 : 1);
        var itm = "";
        for (var i = rowStart; i < table.rows.length; i++) {
                if (table.rows[i].cells.length > column) {
                        itm = ts_getInnerText(table.rows[i].cells[column]);
                        itm = itm.replace(/^[\s\xa0]+/, "").replace(/[\s\xa0]+$/, "");
                        if (itm != "") break;
        sortfn = ts_sort_caseinsensitive;
        if (itm.match(/^\d\d[\/. -][a-zA-Z]{3}[\/. -]\d\d\d\d$/))
                sortfn = ts_sort_date;
        if (itm.match(/^\d\d[\/.-]\d\d[\/.-]\d\d\d\d$/))
                sortfn = ts_sort_date;
        if (itm.match(/^\d\d[\/.-]\d\d[\/.-]\d\d$/))
                sortfn = ts_sort_date;
        if (itm.match(/^[\u00a3$\u20ac]/)) // pound dollar euro
                sortfn = ts_sort_currency;
        if (itm.match(/^[\d.,]+\%?$/))
                sortfn = ts_sort_numeric;
        var reverse = (span.getAttribute("sortdir") == 'down');
        var newRows = new Array();
        for (var j = rowStart; j < table.rows.length; j++) {
                var row = table.rows[j];
                var keyText = ts_getInnerText(row.cells[column]);
                var oldIndex = (reverse ? -j : j);
                newRows[newRows.length] = new Array(row, keyText, oldIndex);
        var arrowHTML;
        if (reverse) {
                        arrowHTML = '<img src="'+ ts_image_path + ts_image_down + '" alt="&darr;"/>';
        } else {
                        arrowHTML = '<img src="'+ ts_image_path + ts_image_up + '" alt="&uarr;"/>';
        // We appendChild rows that already exist to the tbody, so it moves them rather than creating new ones
        // don't do sortbottom rows
        for (var i = 0; i < newRows.length; i++) {
                if ((" "+newRows[i][0].className+" ").indexOf(" sortbottom ") == -1)
        // do sortbottom rows only
        for (var i = 0; i < newRows.length; i++) {
                if ((" "+newRows[i][0].className+" ").indexOf(" sortbottom ") != -1)
        // Delete any other arrows there may be showing
        var spans = getElementsByClassName(tr, "span", "sortarrow");
        for (var i = 0; i < spans.length; i++) {
                spans[i].innerHTML = '<img src="'+ ts_image_path + ts_image_none + '" alt="&darr;"/>';
        span.innerHTML = arrowHTML;
A line like:
        if (itm.match(/^.*sm=n.*$/))
                sortfn = ts_sort_numeric;

Needs to be inserted where at //INSERT HERE

Tested it and it works. Don't know if it breaks anything else (shouldn't). 
I think the other "sm=" switches need to be included too.

Comment 1 Brion Vibber 2008-03-28 19:39:54 UTC
What is "supposed" to have "sm=n"?

I find no such references in the code, so I'm assuming that this is some sort of custom JS extension that somebody, somewhere, on some site came up with, at some point in time. Where? Is it documented? Is it designed to work generally?
Comment 2 Jesse (Pathoschild) 2008-03-28 19:58:11 UTC
MetaWiki uses an extended version of the table sorting; see [[m:Help:Sorting]]. I wouldn't be surprised if some administrators copied it to other wikis.
Comment 3 Aryeh Gregor (not reading bugmail, please e-mail directly) 2008-04-24 15:21:18 UTC
This is not part of the software.  This is an enhancement request, not a bug.  What made anyone settle on this syntax?  It seems like you could come up with better syntax, like a class attribute.
Comment 4 dpotter 2009-01-24 15:43:21 UTC
Sortable is working as intended, even by metawiki standards. From [[m:Help:Sorting#Examples]] "if the number at the top has text after it, this makes the sorting mode alphabetic."  

I think we should close this enhancement request as a duplicate of 15406

*** This bug has been marked as a duplicate of bug 15406 ***

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