commit 74e8f546bbb1aefb42d427fbdc09de5a08b4943e
author Shawn Pearce <sop@google.com> Tue May 12 17:42:59 2015 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Tue May 12 17:43:00 2015 +0000
tree 68b75a935a1145403729b8c8cc31e3bffc806524
parent 680ab1276c13adf2aa3a77d331efbbb97a39b025
parent 1868fce64cce07304b4b031edf1531c8295c8463

Merge "Clip very long branch names to avoid overflow"

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

diff --git a/gitiles-servlet/src/main/java/com/google/gitiles/CommitData.java b/gitiles-servlet/src/main/java/com/google/gitiles/CommitData.java
index b968d89..107f1e1 100644
--- a/gitiles-servlet/src/main/java/com/google/gitiles/CommitData.java
+++ b/gitiles-servlet/src/main/java/com/google/gitiles/CommitData.java
@@ -152,9 +152,6 @@
       if (fs.contains(Field.PARENTS)) {
         result.parents = Arrays.asList(c.getParents());
       }
-      if (fs.contains(Field.SHORT_MESSAGE)) {
-        result.shortMessage = c.getShortMessage();
-      }
       if (fs.contains(Field.BRANCHES)) {
         result.branches = getRefsById(repo, c, Constants.R_HEADS);
       }
@@ -164,6 +161,20 @@
       if (fs.contains(Field.MESSAGE)) {
         result.message = c.getFullMessage();
       }
+      if (fs.contains(Field.SHORT_MESSAGE)) {
+        String msg = c.getShortMessage();
+        if (msg.length() > 80) {
+          String ft = result.message;
+          if (ft == null) {
+            ft = c.getFullMessage();
+          }
+          int lf = ft.indexOf('\n');
+          if (lf > 0) {
+            msg = ft.substring(0, lf);
+          }
+        }
+        result.shortMessage = msg;
+      }
       if (fs.contains(Field.DIFF_TREE)) {
         result.diffEntries = computeDiffEntries(repo, view, c);
       }