i think this will fix all these weird problems we are having. sure, components need to be 1:1 but i thought we were going to have that anyways.
with this approach we can also probably do some nifty things if we allow pragmatic assigned for the fragment to a component.
-Igor
On 11/18/05, Juergen Donnerstag <[EMAIL PROTECTED]> wrote:
Not sure, but I guess it was last week. Please check the revision date
for Component.java for details.
Juergen
On 11/18/05, Martijn Dashorst <[EMAIL PROTECTED] > wrote:
> Juergen,
>
> When did you try to fix it? We are not working on the /real/ head, here,
> but have created our own branch. Last update on our side was from last
> monday, 10:22am CET
>
> Martijn
>
>
>
> On 11/18/05, Juergen Donnerstag <[EMAIL PROTECTED]> wrote:
> >
> > At least I tried to fixed it. There should be a proper exception and a
> > test for it now.
> >
> > Though isFullRender can not replace markupStreamPos and though IMO it
> > introduces some redundancy (because though not completely true it
> > basically is markupStreamPos < 0), it offers additional flexibility
> > and the code is more readable. This change is not yet committed.
> >
> > Juergen
> >
> > On 11/18/05, Martijn Dashorst < [EMAIL PROTECTED] > wrote:
> > > The markupStreamPos was causing the panel to re-iterate the render
> process
> > > until the heapspace error occurred. The problem is, that when the second
> > > time the wicket:id="foo" is processed, the component points to earlier
> in
> > > the markupstream. Thus never ending the loop.
> > >
> > > Martijn
> > >
> > >
> > >
> > > On 11/18/05, Johan Compagner < [EMAIL PROTECTED]> wrote:
> > > > i thought juergen did fix this ?
> > > > There is a test now if it was rendered twice Or is that test to late?
> > > >
> > > > I still think that a isFullRender() from the request cycle instead of
> that
> > > markupstreamPos test is a better thing to do.
> > > >
> > > > johan
> > > >
> > > >
> > > >
> > > >
> > > > On 11/18/05, SourceForge.net < [EMAIL PROTECTED]> wrote:
> > > > > Bugs item #1359899, was opened at 2005-11-18 12:52
> > > > > Message generated for change (Tracker Item Submitted) made by Item
> > > Submitter
> > > > > You can respond by visiting:
> > > > >
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > > > >
> > > > > Please note that this message will contain a full copy of the
> comment
> > > thread,
> > > > > including the initial issue submission, for this request,
> > > > > not just the latest update.
> > > > > Category: core
> > > > > Group: 1.2
> > > > > Status: Open
> > > > > Resolution: None
> > > > > Priority: 9
> > > > > Submitted By: Martijn Dashorst (dashorst)
> > > > > Assigned to: Nobody/Anonymous (nobody)
> > > > > Summary: Out of heapspace error while reusing component instance
> > > > >
> > > > > Initial Comment:
> > > > > This code example should give a wicket error, but
> > > > > instead gives a HTTP ERROR: 500 Java+heap+space.
> > > > >
> > > > > <html>
> > > > > <body>
> > > > > <span wicket:id="foo">foo</span>
> > > > > <span wicket:id="foo">bar</span>
> > > > > </body>
> > > > > </html>
> > > > >
> > > > >
> > > > > import wicket.markup.html.basic.Label ;
> > > > >
> > > > > public class Index extends QuickStartPage
> > > > > {
> > > > > public Index()
> > > > > {
> > > > > add(new Label("foo", "die, Die, DIE!"));
> > > > > }
> > > > > }
> > > > >
> > > > >
> > > > >
> > >
> ----------------------------------------------------------------------
> > > > >
> > > > > You can respond by visiting:
> > > > >
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > > > >
> > > > >
> > > > >
> -------------------------------------------------------
> > > > > This SF.Net email is sponsored by the JBoss Inc. Get Certified
> Today
> > > > > Register for a JBoss Training Course. Free Certification Exam
> > > > > for All Training Attendees Through End of 2005. For more info visit:
> > > > >
> http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
> > > > > _______________________________________________
> > > > > Wicket-develop mailing list
> > > > > [email protected]
> > > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > > >
> > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Living a wicket life...
> > >
> > > Martijn Dashorst - http://www.jroller.com/page/dashorst
> > >
> > > Wicket 1.1 is out:
> http://wicket.sourceforge.net/wicket-1.1
> >
> >
> > -------------------------------------------------------
> > This SF.Net email is sponsored by the JBoss Inc. Get Certified Today
> > Register for a JBoss Training Course. Free Certification Exam
> > for All Training Attendees Through End of 2005. For more info visit:
> > http://ads.osdn.com/?ad_idv28&alloc_id845&opclick
> > _______________________________________________
> > Wicket-develop mailing list
> > [email protected]
> >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> >
>
>
>
> --
> Living a wicket life...
>
> Martijn Dashorst - http://www.jroller.com/page/dashorst
>
> Wicket 1.1 is out: http://wicket.sourceforge.net/wicket-1.1
-------------------------------------------------------
This SF.Net email is sponsored by the JBoss Inc. Get Certified Today
Register for a JBoss Training Course. Free Certification Exam
for All Training Attendees Through End of 2005. For more info visit:
http://ads.osdn.com/?ad_idv28&alloc_id845&opclick
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop
