commit 5e3b996bdda00d21742617ed1a99ac7c675c0c4e
author David Pursehouse <david.pursehouse@sonymobile.com> Thu Mar 10 08:29:54 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Thu Mar 10 08:29:55 2016 +0000
tree 373c615fcf690ff55c64ec6cca163486ac00a0de
parent 5f3e73bc8c8ac7df60d28941e2217289248dc7e2
parent 2809c48552f8a9893299e263497931f0bfde8cf3

Merge "Added not null check when collecting RefsSoyData"

gitiles-servlet/src/main/java/com/google/gitiles/RefServlet.java

diff --git a/gitiles-servlet/src/main/java/com/google/gitiles/RefServlet.java b/gitiles-servlet/src/main/java/com/google/gitiles/RefServlet.java
index b5a0636..ff48d4d 100644
--- a/gitiles-servlet/src/main/java/com/google/gitiles/RefServlet.java
+++ b/gitiles-servlet/src/main/java/com/google/gitiles/RefServlet.java
@@ -168,15 +168,18 @@
 
     for (Ref ref : refs) {
       String name = ref.getName().substring(prefix.length());
-      boolean needPrefix = !ref.getName().equals(refdb.getRef(name).getName());
-      Map<String, Object> value = Maps.newHashMapWithExpectedSize(3);
-      value.put("url", GitilesView.revision().copyFrom(view).setRevision(
-        Revision.unpeeled(needPrefix ? ref.getName() : name, ref.getObjectId())).toUrl());
-      value.put("name", name);
-      if (headLeaf != null) {
-        value.put("isHead", headLeaf.equals(ref));
+      Ref refForName = refdb.getRef(name);
+      if (refForName != null) {
+        boolean needPrefix = !ref.getName().equals(refForName.getName());
+        Map<String, Object> value = Maps.newHashMapWithExpectedSize(3);
+        value.put("url", GitilesView.revision().copyFrom(view).setRevision(
+          Revision.unpeeled(needPrefix ? ref.getName() : name, ref.getObjectId())).toUrl());
+        value.put("name", name);
+        if (headLeaf != null) {
+          value.put("isHead", headLeaf.equals(ref));
+        }
+        result.add(value);
       }
-      result.add(value);
     }
     return result;
   }