You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ace.apache.org by pa...@apache.org on 2012/12/04 19:09:22 UTC

svn commit: r1417076 - /ace/trunk/org.apache.ace.webui.vaadin/src/org/apache/ace/webui/vaadin/AddArtifactWindow.java

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));