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 2012/01/09 16:01:45 UTC
git commit: Do not chain the constructors because this causes
NullPointerException when the parameter to Arrays.asList is null
Updated Branches:
refs/heads/master ba2a4177d -> d7c28049e
Do not chain the constructors because this causes NullPointerException when the parameter to Arrays.asList is null
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/d7c28049
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/d7c28049
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/d7c28049
Branch: refs/heads/master
Commit: d7c28049e464cc7a12952913c15095158b97a7f2
Parents: ba2a417
Author: martin-g <mg...@apache.org>
Authored: Mon Jan 9 17:01:39 2012 +0200
Committer: martin-g <mg...@apache.org>
Committed: Mon Jan 9 17:01:39 2012 +0200
----------------------------------------------------------------------
.../injection/CompoundFieldValueFactory.java | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/d7c28049/wicket-ioc/src/main/java/org/apache/wicket/injection/CompoundFieldValueFactory.java
----------------------------------------------------------------------
diff --git a/wicket-ioc/src/main/java/org/apache/wicket/injection/CompoundFieldValueFactory.java b/wicket-ioc/src/main/java/org/apache/wicket/injection/CompoundFieldValueFactory.java
index f1cdce8..54f96de 100644
--- a/wicket-ioc/src/main/java/org/apache/wicket/injection/CompoundFieldValueFactory.java
+++ b/wicket-ioc/src/main/java/org/apache/wicket/injection/CompoundFieldValueFactory.java
@@ -44,7 +44,9 @@ public class CompoundFieldValueFactory implements IFieldValueFactory
*/
public CompoundFieldValueFactory(final IFieldValueFactory[] factories)
{
- this(Arrays.asList(factories));
+ Args.notNull(factories, "factories");
+
+ delegates.addAll(Arrays.asList(factories));
}
/**