You are here: Foswiki>Tasks Web>Item5937 (09 Nov 2010, MichaelDaum)Edit Attach

Item5937: move initialization of SwitchBoard to TWiki.spec

pencil
Priority: Normal
Current State: Closed
Released In: 1.0.0
Target Release: patch
Applies To: Engine
Component:
Branches:
Reported By: MichaelDaum
Waiting For:
Last Change By: MichaelDaum
... so that other plugins can register their handlers.

-- MichaelDaum - 18 Aug 2008

Yes! it's not a good design to have it within TWiki::UI. I placed it there in first place to make things work and then I'd think about it better, like many other issues (TWiki::generateHTTPHeaders, TWiki::writeCompletePage, etc). I'll file refactoring proposals after I have Item4662 completely solved.

-- GilmarSantosJr - 18 Aug 2008

I needed this for PublishPlugin so I did it. Old-style run scripts are still supported, but TWiki 5.0 requires a Config.spec entry as described in TWikiPlugins

-- CrawfordCurrie - 22 Oct 2008

I missed the part where SwitchBoard stuff was moved from TWiki::UI to TWiki.spec....

-- GilmarSantosJr - 23 Oct 2008

That's because I started to move it, then moved it back. TWiki.spec is for options, and as far as I can see the entries in the switchboard table are not optional; you need them to make TWiki work, and there is no value in making them configurable. You'll note that I broke up the table in UI.pm so that it isn't redefining the switchboard, just adding to it. The LocalSite is loaded after the UI module has already been BEGINned, then you can override entries added in UI.pm, but this way you are always sure to have - for example - an entry for view.

If you can think of a justification for having those entries in TWiki.spec, then it's easy enough to move them.

-- CrawfordCurrie - 24 Oct 2008

ItemTemplate edit

Summary move initialization of SwitchBoard to TWiki.spec
ReportedBy MichaelDaum
Codebase
SVN Range TWiki-5.0.0, Mon, 11 Aug 2008, build 17408
AppliesTo Engine
Component
Priority Normal
CurrentState Closed
WaitingFor
Checkins TWikirev:17677
TargetRelease patch
ReleasedIn 1.0.0
Topic revision: r12 - 09 Nov 2010, MichaelDaum
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