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

Item750: INCLUDINGWEB and INCLUDINGTOPIC will cause confusion when used in an INCLUDE chain

Priority: CurrentState: AppliesTo: Component: WaitingFor:
Urgent Closed Engine System web topics  

what i should have written :/

topics that use INCLUDING* will trip up users - especially when they include something that is already INCLUDED - consider someone making a rss feed, so they INCLUDE Main's rss topic - they get a topic that show's Main's rss feed (until i fixed WebRssBase to use BASEWEB)

original bug report

because System . WebChanges uses INCLUDINGWEB, which is the next WEB up the INCLUDE chain, a topic that includes a topic that includes WebChanges will not do what it should.

WebChanges, and all the System web topics need to be converted from INCLUDINGWEB to BASEWEB for 1.0.1 .

this bug means anyone using topics that point to WebChanges will see the TWiki web's changes, not what it used to - so urgent !

-- SvenDowideit

Good catch. To assist and get it closed I did this one

-- KennethLavrsen - 13 Jan 2009

there are alot more topics with the same problem,

basically, all Web* topics are likely to be included in a chain. for eg, WebRss may be made by the user INCLUDING{Main.WebRss} which then INCLUDES{System.WebRss} - resulting in their rss feed saying that they are getting the changes of the 'Main' web.

same issue wrt INCLUDINGTOPIC, but even more subtle

but i've found a further oddity if you INCLUDE{System.Macros} - see Item771

-- SvenDowideit - 15 Jan 2009

ok, so the Web* topics are all done, and Item771 deals with the related issues for the Var* topics.

-- SvenDowideit - 05 Feb 2009

ItemTemplate edit

Summary INCLUDINGWEB and INCLUDINGTOPIC will cause confusion when used in an INCLUDE chain
ReportedBy SvenDowideit
Codebase 1.0.0, trunk
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Engine
Component System web topics
Priority Urgent
CurrentState Closed
WaitingFor
Checkins distro:852f7b57502b distro:be2067a2ce59 distro:326446cd2f7c distro:728c715abc83 distro:b2f4c0024a82
TargetRelease patch
ReleasedIn 1.0.1
Topic revision: r14 - 23 Feb 2009, KennethLavrsen
 
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. see CopyrightStatement. Creative Commons License