Item9107: Rationalise attach
, oops
, and manage
Priority: Enhancement
Current State: Proposal Required
Released In: n/a
Target Release: n/a
The
attach
script is a hangover from the days when commands each had their own script. All it does is to generate the attach topic screen.
The
manage
script is a crude dispatcher that handles various commands related to web management.
The
oops
script was originally intended to handle error reporting, but has been bastardised to handle functions such as
more topic actions
None of these scripts is intended to be linked to directly by users, or even from the
view
screen; they are all internal functions generated in other internally-generated screens.
The proposal is to:
- Replace
attach
, oops
and manage
with REST handlers.
- Change the existing
attach
, oops
and manage
scripts to invoke these REST handlers, for compatibility (is it worth bothering to do this?)
--
CrawfordCurrie - 06 Jun 2010
Shouldn't this be in development?
I would love to see a clean break, let's see how we could enable some ajax-heavy skins.
--
PaulHarvey - 06 Jun 2010
Yes this need to be a feature proposal. And for 2.0 naturally.
That said - yes less binary scripts to handle is a good goal.
--
KennethLavrsen - 06 Jun 2010
yes, its worth bothering with rearwards compatibility - if only to allow older internal skins to keep working
--
SvenDowideit - 06 Jun 2010
Sure it's for 2.0. However it's not a feature - user should not be affected unless we remove scripts.
Note that the
resetpasswd
script went this way some time ago.
--
CrawfordCurrie - 06 Jun 2010
Sorry Crawford, I always seem to read more into tasks than is really there.
So I assume this is just a simple re-factoring and not something more ambitious (the likes of which could be done in
RestPlugin).
--
PaulHarvey - 07 Jun 2010
A simple refactoring of internals would be step 1. Step 2 would be realigning the user scripts.
Confirmed.
--
CrawfordCurrie - 21 Jun 2010