Item5937: move initialization of SwitchBoard to TWiki.spec
Priority: Normal
Current State: Closed
Released In: 1.0.0
Target Release: patch
Applies To: Engine
Component:
Branches:
... 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