create new tag
, view all tags
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.

-- 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

Topic attachments
I Attachment History Action Size Date Who Comment
Perl source code filepm Save.pm r4 r3 r2 r1 manage 6.7 K 2004-07-14 - 06:17 MattWilkie Save.pm with integrated savemulti
Unknown file formatpatch add-savemulti-save-pm.patch r2 r1 manage 11.6 K 2004-07-14 - 06:22 MattWilkie patch to intgrate savemulti into SaveDotPm
Unknown file formatpatch savemulticgi-to-pm.patch r1 manage 11.0 K 2004-07-14 - 09:05 MattWilkie patch bin/savemulti to use the new SaveDotPm
Edit | Attach | Watch | Print version | History: r15 < r14 < r13 < r12 < r11 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r15 - 2004-08-20 - CrawfordCurrie
  • Learn about TWiki  
  • Download TWiki
This site is powered by the TWiki collaboration platform Powered by Perl Hosted by OICcam.com Ideas, requests, problems regarding TWiki? Send feedback. Ask community in the support forum.
Copyright © 1999-2018 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.