One of the major changes in
CairoRelease was to move the code from most of the cgi scripts into perl modules.
SavemultiCgiScript has been brought in line with this new policy and merged into
SaveDotPm.
Users of existing skins which use savemulti should not have to do anything other than upgrade to a twiki version dated 07-May-2004 or newer and then apply both patches. If all goes well, it will be included in the forthcoming
CairoRelease and the patches won't be necessary.
Skins which don't use savemulti are not affected.
Test results on TWiki 07-May-2004 on cygwin, apache 2.0.48, perl 5.8.2 using
SeeSkin:
procedure |
passed |
failed |
untested |
savemulti |
save, quietsave, checkpoint, cancel, preview, repRev, delRev, delRev, add/change forms |
|
|
save (_save) |
cancel, preview, repRev, delRev, add/change forms |
|
|
That normal save passing preview is a bit of a red herring, since in the twiki classic skin Preview calls Save, while in savemulti skins Save calls Preview.
Contributors:
--
CrawfordCurrie - 12 Jul 2004
--
MattWilkie - 12 Jul 2004
The heavy lifting and scracthing was done
CrawfordCurrie while
MattWilkie refactored, wrote docs and was responsible for making CC itchy in the first place. If it wasn't for
ColasNahaboo's original
SavemultiCgiScript there wouldn't have been any itching or scratching going on. Thanks to
TWikiIRC the turn around time on this has only been a week.
Stuff to be folded into other topics
repRev
allows editing og the metadata. I don't remember that being the case before. This behaviour predates this patch.
It sure doesn't help debugging that save
turns anything it doesn't understand into a normal save command. :(
Gawd, I hate that. I tried to rationalise it but just didn't go far enough, I guess. One day I'll do a proper cleanup of the return values and error handling.
--
CrawfordCurrie - 12 Jul 2004
--
MattWilkie - 12 Jul 2004
I am confused, what is the status of this patch? Marked as ready, but 100% for Cairo.
--
PeterThoeny - 16 Aug 2004
For the Cairo release I need to know if this is in/not in Cairo.
--
PeterThoeny - 16 Aug 2004
It's in, and fully functional.
--
CrawfordCurrie - 17 Aug 2004