Hi Martin, I explored the object hierarchy, and it seems *StringBufferResourceStream *may suffer from the same problem, if its constructed and no append or prepend method is called on it (which is a weird scenario of course).
Thanks! On Tue, Nov 23, 2010 at 4:27 PM, Martin Grigorov-4 [via Apache Wicket] < ml-node+3056080-692802461-65...@n4.nabble.com<ml-node%2b3056080-692802461-65...@n4.nabble.com> > wrote: > Done. > r1038292 > > On Tue, Nov 23, 2010 at 8:20 PM, Martin Grigorov <[hidden > email]<http://user/SendEmail.jtp?type=node&node=3056080&i=0>>wrote: > > > > Looks like a bug. > > I think StringResourceStream has to initialize its lastModifiedTime in > the > > constructor to Time.now(). > > > > I'll create a ticket and fix it. > > > > > > On Tue, Nov 23, 2010 at 8:05 PM, jcgarciam <[hidden > > email]<http://user/SendEmail.jtp?type=node&node=3056080&i=1>> > wrote: > > > >> > >> Hi Folks, > >> > >> Probably im doing it wrong, but please bear with me, currently i'm > trying > >> to > >> do a very simple download operation in wicket 1.5M3, but it fails with a > > >> NPE > >> in (ResourceStreamResource.java line 72), because it seems is always > >> expecting that any IResourceStream implementation set the > lastModifiedTime > >> in the underlying stream before calling dataNeedsToBeWritten which > >> StringResourceStream doesn't do it ( i need to explicitly set it to make > > >> it > >> work ) . > >> > >> @Override > >> protected ResourceResponse newResourceResponse(Attributes > >> attributes) > >> { > >> ResourceResponse data = new ResourceResponse(); > >> (72)-> data.setLastModified(stream.lastModifiedTime().toDate()); > >> > >> > >> My code using [StringResourceStream] > >> <code> > >> final StringBuilder content = new StringBuilder("Hello,world"); > >> add(new Link<Void>("downloadDoc") { > >> @Override > >> public void onClick() { > >> StringResourceStream stream = new > >> StringResourceStream(content.toString(),"html/csv"); > >> > >> getRequestCycle().scheduleRequestHandlerAfterCurrent(new > >> ResourceStreamRequestHandler(stream) > >> .setFileName("demo.csv") > >> > >> .setContentDisposition(ContentDisposition.ATTACHMENT)); > >> } > >> }); > >> </code> > >> > >> Calling [stream.setLastModified(Time.now());] right before scheduling > the > >> request handler make the download to works, but it seems that probably > im > >> doing something wrong (not supposed to a download on this way) or does > it > >> sounds like a bug in [ResourceStreamResource.java]? > >> > >> Thanks, > >> > >> Attached is a quickstart > >> > >> > http://apache-wicket.1842946.n4.nabble.com/file/n3056036/DownloadStreamBug.7z<http://apache-wicket.1842946.n4.nabble.com/file/n3056036/DownloadStreamBug.7z?by-user=t> > >> DownloadStreamBug.7z > >> > >> > >> -- > >> View this message in context: > >> > http://apache-wicket.1842946.n4.nabble.com/Simple-Download-from-StringResourceStream-cause-NPE-wicket-1-5M3-possible-bug-tp3056036p3056036.html<http://apache-wicket.1842946.n4.nabble.com/Simple-Download-from-StringResourceStream-cause-NPE-wicket-1-5M3-possible-bug-tp3056036p3056036.html?by-user=t> > >> Sent from the Users forum mailing list archive at Nabble.com. > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [hidden > >> email]<http://user/SendEmail.jtp?type=node&node=3056080&i=2> > >> For additional commands, e-mail: [hidden > >> email]<http://user/SendEmail.jtp?type=node&node=3056080&i=3> > >> > >> > > > > > ------------------------------ > View message @ > http://apache-wicket.1842946.n4.nabble.com/Simple-Download-from-StringResourceStream-cause-NPE-wicket-1-5M3-possible-bug-tp3056036p3056080.html > > To unsubscribe from Simple Download from StringResourceStream cause NPE > wicket 1.5M3 (possible bug??), click > here<http://apache-wicket.1842946.n4.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=3056036&code=amNnYXJjaWFtQGdtYWlsLmNvbXwzMDU2MDM2fDEyNTYxMzc3ODY=>. > > -- Sincerely, JC (http://www.linkedin.com/in/jcgarciam) Work smarter, not harder!. -- View this message in context: http://apache-wicket.1842946.n4.nabble.com/Simple-Download-from-StringResourceStream-cause-NPE-wicket-1-5M3-possible-bug-tp3056036p3056096.html Sent from the Users forum mailing list archive at Nabble.com. --------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org For additional commands, e-mail: users-h...@wicket.apache.org
