You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2013/02/27 15:46:47 UTC

git commit: WICKET-5023 Page store process is broken if an exception occurs at commit request phase.

Updated Branches:
  refs/heads/master d42e91d6a -> 114068e9d


WICKET-5023 Page store process is broken if an exception occurs at commit request phase.


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/114068e9
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/114068e9
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/114068e9

Branch: refs/heads/master
Commit: 114068e9d2e01974d25d70df82f8958a4c1564be
Parents: d42e91d
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Wed Feb 27 16:43:01 2013 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Wed Feb 27 16:43:01 2013 +0200

----------------------------------------------------------------------
 .../org/apache/wicket/page/RequestAdapter.java     |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/114068e9/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java b/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
index 2f50ab6..23b1d30 100644
--- a/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
+++ b/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
@@ -160,7 +160,17 @@ public abstract class RequestAdapter
 					log.error("Error detaching page", e);
 				}
 
-				if (!page.isPageStateless())
+				boolean isPageStateless;
+				try
+				{
+					isPageStateless = page.isPageStateless();
+				}
+				catch (Exception x)
+				{
+					log.warn("An error occurred while checking whether a page is stateless. Assuming it is stateful.", x);
+					isPageStateless = false;
+				}
+				if (isPageStateless == false)
 				{
 					statefulPages.add(page);
 				}