Item8173: TablePlugin does not understand standard date formats

pencil
Priority: Urgent
Current State: Closed
Released In: 1.0.6
Target Release: patch
Applies To: Engine
Component: TablePlugin
Branches:
Reported By: Foswiki:Main.MichaelDaum
Waiting For:
Last Change By: KennethLavrsen
... 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

 

ItemTemplate edit

Summary TablePlugin does not understand standard date formats
ReportedBy Foswiki:Main.MichaelDaum
Codebase
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Engine
Component TablePlugin
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:d7c9646cb367 distro:44952ab0d3bf distro:21c7dbd7cbc6 distro:85ed6f85717b distro:53a9dc0711a8
TargetRelease patch
ReleasedIn 1.0.6
Topic revision: r9 - 22 Jun 2009, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy