Author: paulb
Date: Tue Dec  4 18:09:21 2012
New Revision: 1417076

URL: http://svn.apache.org/viewvc?rev=1417076&view=rev
Log:
ACE-309 fixed potential nullpointer exception because code assumes the URL 
property to always be set

Modified:
    
ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java

Modified: 
ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java
URL: 
http://svn.apache.org/viewvc/ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java?rev=1417076&r1=1417075&r2=1417076&view=diff
==============================================================================
--- 
ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java
 (original)
+++ 
ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java
 Tue Dec  4 18:09:21 2012
@@ -655,7 +655,7 @@ abstract class AddArtifactWindow extends
 
         for (ArtifactObject ao : artifactObjects) {
             String artifactURL = ao.getURL();
-            if (artifactURL.startsWith(baseURL)) {
+            if (artifactURL != null && artifactURL.startsWith(baseURL)) {
                 // we now know this artifact comes from the OBR we are 
querying,
                 // so we are interested.
                 fromRepository.add(convertToOBREntry(ao, artifactURL));


Reply via email to