SortableTable: parsing dates can be incorrect.

[guest]

The SortableTable? has a bug in the parsing of the "Date" columns. In fact when you pick the value of a column (for exemple "10/08/2006"), the string is converted in a Date without using the FORMAT attribute. But when you write the table, the Date object is converted into String using the FORMAT attribute. In the method 'parseDataFromTable' on row 310, you have the code:

if(type == String){
    o[field]=val;
} else {
   if (val!=null){
       o[field]=new type(val);
   } else {
       o[field]=new type();	
       //let it use the default.
   }
}

So if the type is 'Date' the string is converted into Date with a 'new Date(val)'. I think you must here use the dojo.date.parse with the format attribute to parse the string correctly.

[bill]

SortableTable? is deprecated so this won't be fixed, but assigning to Tom just in case the same problem exists in FilterTable? (sp?)

[Tom Trenka]

Will double check in Filtering but IIRC Adam set it up to use the localization stuff. The original (Sortable) uses the native JS date parsing to figure out what the value of the column is.

[Tom Trenka]

Closing this with the removal of Sortable from the codebase, and with the filing of the same basic bug on Filtering.