You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by iv...@apache.org on 2010/12/17 17:00:32 UTC

svn commit: r1050432 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java

Author: ivaynberg
Date: Fri Dec 17 16:00:32 2010
New Revision: 1050432

URL: http://svn.apache.org/viewvc?rev=1050432&view=rev
Log:
making page oninitialize() final
Issue: WICKET-3218

Modified:
    wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java

Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java?rev=1050432&r1=1050431&r2=1050432&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/Page.java Fri Dec 17 16:00:32 2010
@@ -673,6 +673,17 @@ public abstract class Page extends Marku
 		return stateless;
 	}
 
+	@Override
+	protected final void onInitialize()
+	{
+		/*
+		 * final in the page because: *** doesnt make sense for pages *** can be called from page's
+		 * constructor - which is bad *** if delayed can lead to page not being initialized after
+		 * its constructor call - which is bad
+		 */
+		super.onInitialize();
+	}
+
 	/**
 	 * Redirect to this page.
 	 *