Priority: Urgent
Current State: Closed
Released In: 1.0.6
Target Release: patch
... because it tries to reimplement a date-time parser. We've got one already in Foswiki which does a better job:
Foswiki::Time::parseTime
.
Found another error where the date type of a column could not be derived reliably if cells have spaces while the rest of the column is of type date.
TablePlugin fills empty cells with nbsp and then does not strip it off again before trying to detect date type. wtf.
As a consequence sorting the table along a date-time column then fails.
--
MichaelDaum - 03 Jun 2009
Argh,
Foswiki::Time::parseTime
errors out if you throw an integer < 60 at it. Seems as if its regexes are too permissive.
Will wrap the call in
TablePlugin into a
try
.
The fix unfortunately also touches core code in
Foswiki::Time::parseTime
. Been checked in to stabel & branch. Changing
the classification of this bug accordingly.
--
MichaelDaum - 04 Jun 2009