Foswiki on GitHub is open for business! Next release meeting: Monday September 29, 1300Z

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

Priority: CurrentState: AppliesTo: Component: WaitingFor:
Urgent Closed Engine    
(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 1661

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 Foswikirev:6750 Foswikirev:6878
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 CopyrightStatement. Creative Commons License