Item8709: REVINFO formating code did case insensitive format token replacement.

Priority: Urgent
Current State: Closed
Released In: 1.0.10, 1.1.0
Target Release: patch
Applies To: Engine
Reported By: KennethLavrsen
Waiting For:
Last Change By: KennethLavrsen
(this issue was found during development of 1.1 due to code reuse, but the fix has been applied to 1.0.10 as well)

up to foswiki 1.0.9, %!REVINFO{format="$TIME}% is evaluated as though the =$TIME+ is lower case. This is inconsistent, undocumented, and has been found to clash with SpreadSheetPlugin's $TIME() (and $DATE and presumably other) functions.

-- SvenDowideit - 16 Mar 2010

(Kenneth's original finding)

Someone has very recently changes SEARCH so the format is encoded so CALC fails.


%SEARCH{"%TOPIC%" scope="topic" nonoise="on" format="Hello $percntCALC{$ROUND($TIMEDIFF($TIME($createdate), $TODAY(), days),0)}$percnt"}%

Hello 2032

works in 1.0.9 and fails in trunk

-- KennethLavrsen - 15 Mar 2010

Tasks.Item8708 also reported this, and annoyingly, I comited a unit test to it.

-- SvenDowideit - 15 Mar 2010

reopening to remove a debugging unit test which fails 7 days later...


ItemTemplate edit

Summary REVINFO formating code did case insensitive format token replacement.
ReportedBy KennethLavrsen
Codebase 1.0.9, trunk
SVN Range
AppliesTo Engine
Priority Urgent
CurrentState Closed
Checkins distro:36bc5572868e distro:e5a2052bda6b distro:86374ea188ad
TargetRelease patch
ReleasedIn 1.0.10, 1.1.0
Topic revision: r7 - 08 Sep 2010, KennethLavrsen - This page was cached on 07 Oct 2015 - 02:36. Get a fresh version here.
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License