Mercurial > dive4elements > framework
diff artifact-database/src/main/java/de/intevation/artifactdatabase/AbstractCallContext.java @ 362:5b6ba54d488a
Never return null from a function which is documented to do so.
artifacts/trunk@3443 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author | Felix Wolfsteller <felix.wolfsteller@intevation.de> |
---|---|
date | Fri, 16 Dec 2011 14:05:52 +0000 |
parents | 1d11a0531242 |
children | 1f51f3c46efe |
line wrap: on
line diff
--- a/artifact-database/src/main/java/de/intevation/artifactdatabase/AbstractCallContext.java Fri Dec 16 09:14:14 2011 +0000 +++ b/artifact-database/src/main/java/de/intevation/artifactdatabase/AbstractCallContext.java Fri Dec 16 14:05:52 2011 +0000 @@ -7,8 +7,11 @@ */ package de.intevation.artifactdatabase; +import org.apache.log4j.Logger; + import java.util.ArrayList; import java.util.List; +import java.util.Map; import java.util.HashMap; import de.intevation.artifacts.ArtifactDatabase; @@ -25,6 +28,8 @@ */ public abstract class AbstractCallContext implements CallContext { + Logger logger = Logger.getLogger(AbstractCallContext.class); + /** * The ArtifactDatabase instance. */ @@ -115,11 +120,16 @@ * @return list (empty list if none found, never null). */ public List<DataProvider> getDataProvider(Object key) { - return dataProviders != null - ? dataProviders.get(key) - : java.util.Collections.<DataProvider>emptyList(); + if (dataProviders != null) { + List<DataProvider> list = dataProviders.get(key); + return list != null + ? list + : java.util.Collections.<DataProvider>emptyList(); + } + return java.util.Collections.<DataProvider>emptyList(); } + /** * Let a DataProvider register itself with given key. * Multiple DataProvider can register under the same key.