Null or blank dates are not handled reasonably by SortableTable

[[email protected]…]

I consider it a defect, you may consider it an enhancement, but the SortableTable? should handle null or blank (missing) values in a Date field in some consistent way, and display these values as a blank. In the current implementation of parseData and parseTable, the Date constructor is called and the default (i.e. Today) is returned.

I suggest that the "null/missing Date represented as a blank" use case is much more common than the need to generate a "Today" (actually "Now") Date value from a null or blank.

SortableTable? methods parseData and parseTable will have to be modified to skip parsing for null or blank Date column values and dojo.date.strftime must be modified to return "" when passed a null or blank object. I haven't looked into it in detail, but sorting on Date fields seems to work fine for missing values with the above patches --- The missing values sort after (greater than) any Date value, although there could be two different preferences about what the default value should be for sorting --- whether a missing date is "before" all other dates, or "after".

I can provide a patch if you'd like, but the required changes are pretty straightforward.

[[email protected]…]

I meant parseDataFromTable, not parseTable --- in src/widget/html/SortableTable.js

[Tom Trenka]

Will deal with when I do the rewrite.

[Tom Trenka]

This should be handled fine within the replacement widget for Sortable called FilteringTable?.

[(none)]

Milestone 0.4 deleted