You are here: Foswiki>Tasks Web>Item13287 (01 Mar 2015, CrawfordCurrie)Edit Attach

Item13287: /o is poison

pencil
Priority: Urgent
Current State: Closed
Released In: n/a
Target Release: n/a
Applies To: Engine
Component:
Branches: master
Reported By: CrawfordCurrie
Waiting For:
Last Change By: CrawfordCurrie
After a discussion with George about /o and some background reading and thinking, I have concluded that /o is poisonous, and at high risk of creating bugs.

The main scenario I foresee is where /o has been used on a RE that interpolates a cfg var. cfg vars are supposed to be constant, but there are cases where we need to change them dynamically. Since /o applies the first time the RE is evaluated, it may be a lottery as to a before-change or after-change value is picked up for the interpolated var.

The safest course is to remove all /o's from core code.

-- CrawfordCurrie - 01 Mar 2015

 

ItemTemplate edit

Summary /o is poison
ReportedBy CrawfordCurrie
Codebase
SVN Range
AppliesTo Engine
Component
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:060fcbc48603
TargetRelease n/a
ReleasedIn n/a
CheckinsOnBranches master
trunkCheckins
masterCheckins distro:060fcbc48603
ItemBranchCheckins
Release01x01Checkins
Edit  | Attach | Print version | History: r2 < r1 | Backlinks | View wiki text | Edit wiki text | More topic actions
Topic revision: r2 - 01 Mar 2015, CrawfordCurrie - This page was cached on 30 Sep 2016 - 05:16.

The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License