I would suggest, that we delay this extension until we have unified the stream and event pipeline objects and have them configurable. I think that we will then see, how this can fit into the big picture.
Carsten Anatoly Sennov wrote: > > Hello Gerhard! > You wrote to "Cocoon Developer List" <[EMAIL PROTECTED]> > on Sat, 20 > Apr 2002 11:23:47 +0200: > > GF> Hi Team, this bug + patch seems to me very valid and needful. > GF> <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=7952> > Thanks :-) > > GF> one problem...the patch changes the Enviroment interface and my > GF> experience is that you don't like this very much. > > GF> is there any problem when we add this two method to the > GF> Enviroment interface: > > Maybe this patch requires more explanation. > > First, it's a sort of quick hack to add the Last-Modified > handling to Cocoon > fast as I needed this for our site. > Second, since that it's working well for us, but... now I'm > thinking that we > need more integrated solution to handle Last-Modified and > Is-Modified-Since. > Currently Cocoon tries to handle Is-Modified-Since for the StreamPipeline > but not in the EventPipeline. The Environment already has the > provisions for > that (isResponseModified, setResponseModified). > Third, after this patch the Cocoon behavior is not configurable - it > generates Last-Modified is each and every response, this should > be changed: > we need to have means to turn this off for some buggy user agents > (IE 6) in > some situations to force them to update their local cache. > > So, I think that the below methods should be added, but some further > development is needed to make all things right :-) > > When I'll finish the first round of the Slide integration I'll return to > this problem. > > GF> <code> > GF> /** > GF> * Sets the timestamp for the responce. This may be used by * > GF> subclasses to perform some smart activity: for example, * to return > GF> the Last-Modified HTTP header or to set the * file attributes. > GF> */ > GF> void setTimestamp(long ts); > > GF> /** > GF> * Get current timestamp. > GF> */ > GF> long getTimestamp(); > GF> </code> > > --Anatoly Sennov > k2s ltd > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, email: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, email: [EMAIL PROTECTED]
