Good point. In fact, it seems to me that the contract between transform steps is currently insufficient - each step should probably make an attempt to communicate to the subsequent step what the appropriate content type is. That way no matter what type of transform you have, halting should provide the appropriate content-type.
I'll try adding a "setOutputType()" method to the TransformStep interface. Calling it will be recommended but optional. At the moment I imagine that only the LastStep will do anything with it. Jeff Schnitzer [EMAIL PROTECTED] > -----Original Message----- > From: Mike Moulton [mailto:[EMAIL PROTECTED] > Sent: Friday, March 21, 2003 5:28 PM > To: [EMAIL PROTECTED] > Subject: Re: [Mav-user] [PROPOSAL] Release v2.2.0 of Maverick > > Before releasing 2.2 there is a bug I noticed a while ago that doesn't > seem to have been fixed. > > The problem is that the UNFINISHED_CONTENTTYPE isn't explicitly set > when maxTransforms=0. This caused problems on some browsers, primarily > IE 5.2 for Mac. > > To fix this I add the following lines to the getNextStep() method in > MaverickContext.java. > > if (this.transformCount == 0) > this.getRealResponse().setContentType(UNFINISHED_CONTENTTYPE); > > These lines were added within the 'if' statement. The new 'if' > statement looks as follows. > > if (this.nextTransform >= this.transformCount) > { > if (this.transformCount == 0) > this.getRealResponse().setContentType(UNFINISHED_CONTENTTYPE); > > this.log.debug("...which is the LastStep"); > return new LastStep(this); > } > > There is also a patch attached for the version contained in the 2.1.2 > release. > > Note: This may not be the best solution to the problem. > > -- Mike ------------------------------------------------------- This SF.net email is sponsored by:Crypto Challenge is now open! Get cracking and register here for some mind boggling fun and the chance of winning an Apple iPod: http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en [INVALID FOOTER]
