You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ha...@apache.org on 2010/12/20 08:32:14 UTC

svn commit: r1051007 - /geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java

Author: hanhongfang
Date: Mon Dec 20 07:32:13 2010
New Revision: 1051007

URL: http://svn.apache.org/viewvc?rev=1051007&view=rev
Log:
GERONIMODEVTOOLS-689 NullPointerException when publishing with Geronimo Eclipse Plugin v2.1.7

Modified:
    geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java

Modified: geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java
URL: http://svn.apache.org/viewvc/geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java?rev=1051007&r1=1051006&r2=1051007&view=diff
==============================================================================
--- geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java (original)
+++ geronimo/devtools/eclipse-plugin/branches/2.2/plugins/org.apache.geronimo.st.v21.core/src/main/java/org/apache/geronimo/st/v21/core/internal/DependencyHelper.java Mon Dec 20 07:32:13 2010
@@ -439,7 +439,9 @@ public class DependencyHelper {
         }
 
         //put module's environment into the cache for next retrieve
-        environmentCache.put(module, environment);
+        if (environment != null) {
+            environmentCache.put(module, environment);
+        }        
         
         Trace.tracePoint("Exit ", "DependencyHelper.getEnvironment", environment);
         return environment;