You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by pe...@apache.org on 2012/02/03 14:50:42 UTC
git commit: WICKET-4384 improve wicket's handling of empty / null
page parameters:
Updated Branches:
refs/heads/master d3355535f -> baffa1d80
WICKET-4384 improve wicket's handling of empty / null page parameters:
- DefaultPageFactory#newPage() should be sure to not pass 'null' to a page constructor with page parameters
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/baffa1d8
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/baffa1d8
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/baffa1d8
Branch: refs/heads/master
Commit: baffa1d80eb579ab9bf0f6fd133b1c8b8c21e389
Parents: d335553
Author: Peter Ertl <pe...@apache.org>
Authored: Fri Feb 3 14:49:44 2012 +0100
Committer: Peter Ertl <pe...@apache.org>
Committed: Fri Feb 3 14:49:44 2012 +0100
----------------------------------------------------------------------
.../apache/wicket/session/DefaultPageFactory.java | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/baffa1d8/wicket-core/src/main/java/org/apache/wicket/session/DefaultPageFactory.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/session/DefaultPageFactory.java b/wicket-core/src/main/java/org/apache/wicket/session/DefaultPageFactory.java
index d2f7500..c6ed430 100644
--- a/wicket-core/src/main/java/org/apache/wicket/session/DefaultPageFactory.java
+++ b/wicket-core/src/main/java/org/apache/wicket/session/DefaultPageFactory.java
@@ -95,8 +95,10 @@ public final class DefaultPageFactory implements IPageFactory
// If we got a PageParameters constructor
if (constructor != null)
{
+ final PageParameters nullSafeParams = parameters == null ? new PageParameters() : parameters;
+
// return new Page(parameters)
- return processPage(newPage(constructor, parameters), parameters);
+ return processPage(newPage(constructor, nullSafeParams), nullSafeParams);
}
// Always try default constructor if one exists