Bugs item #1568053, was opened at 2006-09-30 00:06
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=1568053&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: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Martijn Dashorst (dashorst)
Assigned to: Nobody/Anonymous (nobody)
Summary: Exception in detach only appears in log
Initial Comment:
An exception in the detach phase is only logged into the wicket logs.
This can potentially be disastruous. We should keep track of the
exception and rethrow it (wrapped in a WicketRuntimeException?)
*after* detach has finished. I suggest only to track the first exception
that occurs.
Taken from Page.java:
public void detachModels()
{
// visit all this page's children to detach the models
visitChildren(new IVisitor()
{
public Object component(Component component)
{
try
{
// detach any models of the component
component.detachModels();
}
catch (Exception e) // catch anything; we MUST
detach all models
{
log.error("detaching models of
component "
+ component + " failed:", e);
}
return IVisitor.CONTINUE_TRAVERSAL;
}
});
detachModel();
}
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1568053&group_id=119783
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop
