You are here: Foswiki>Tasks Web>Item893 (23 Feb 2009, KennethLavrsen)Edit Attach

Item893: configure extension installer DOS's by doing a saveTopic and saveAttachment

Priority: Urgent
Current State: Closed
Released In: 1.0.1
Target Release: patch
Applies To: Engine
Component: configure
Reported By: SvenDowideit
Waiting For:
Last Change By: KennethLavrsen
which is a dangerous waste of server resources.

maxing out the server at install time is very very unwanted.

If the files need to be attached, the author of the plugin should do so manually before upload.

For contribs like YahooUI and FamFam, doing it at install time adds hours of processing, with potential timeouts and other issues.

let alone that you end up with massive topics for no reason.

on windows, with rcslite, you end up with FamFam being essentially un-installable, and there is zero need for the attach metadata to be in this case. (and IF or WHEN we use that for replacing DocGraphics, we'd do that at development time, not install).

-- SvenDowideit - 29 Jan 2009

This one goes way back. originally one of the main complaints about automated installation was it's propensity for overwriting what was already there. This was moderately important in the days when plugins were mostly configured through the installed topic.

Given that we've pushed people to move into Config.spec instead of using preferences, then we migh be able to get away with removing this - or at least let the author of the plugin state that it isn't needed (could be done in the MANIFEST I guess)

-- CrawfordCurrie - 29 Jan 2009

I coded the !noci directive (and per file (noci) ) for the extension author to suppress checkin of their files.

Just add noci as the first line in your MANIFEST (only works with latest

-- CrawfordCurrie - 05 Feb 2009

ItemTemplate edit

Summary configure extension installer DOS's by doing a saveTopic and saveAttachment
ReportedBy SvenDowideit
Codebase 1.0.0, trunk
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Engine
Component configure
Priority Urgent
CurrentState Closed
Checkins distro:62c444b44ce0 distro:f17043f49de9
TargetRelease patch
ReleasedIn 1.0.1
Topic revision: r6 - 23 Feb 2009, KennethLavrsen
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