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

pencil
Priority: Urgent
Current State: Closed
Released In: 1.0.10, 1.1.0
Target Release: patch
Applies To: Engine
Component:
Branches:
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.

This

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

Hello 5118

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
Component
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:36bc5572868e distro:e5a2052bda6b distro:86374ea188ad
TargetRelease patch
ReleasedIn 1.0.10, 1.1.0
Topic revision: r7 - 08 Sep 2010, 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