You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jc...@apache.org on 2007/06/05 23:33:13 UTC

svn commit: r544643 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java

Author: jcompagner
Date: Tue Jun  5 14:33:12 2007
New Revision: 544643

URL: http://svn.apache.org/viewvc?view=rev&rev=544643
Log:
try catch around before render

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java?view=diff&rev=544643&r1=544642&r2=544643
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/Page.java Tue Jun  5 14:33:12 2007
@@ -877,7 +877,24 @@
 			((IFeedback)this).updateFeedback();
 		}
 
-		beforeRender();
+		try
+		{
+			beforeRender();
+		} 
+		catch(RuntimeException e)
+		{
+			// if an exception is thrown then we have to call after render
+			// else the components could be in a wrong state (rendering)
+			try
+			{
+				afterRender();
+			} 
+			catch(RuntimeException e2) 
+			{
+				// ignore this one could be a result off.
+			}
+			throw e;
+		}
 
 		// Visit all this page's children to reset markup streams and check
 		// rendering authorization, as appropriate. We set any result; positive