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
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop

Reply via email to