short the old/new links

Author: vladak

opengrok-web/src/main/java/org/opengrok/web/DiffData.java

@@ -19,7 +19,7 @@
 
 /*
  * Copyright (c) 2009, 2011, Jens Elkner.
- * Copyright (c) 2009, 2021, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2009, 2023, Oracle and/or its affiliates. All rights reserved.
  * Portions Copyright (c) 2020, Chris Fraire <[email protected]>.
  */
 package org.opengrok.web;
@@ -91,6 +91,14 @@ public String getRev(int index) {
         return rev[index];
     }
 
+    /**
+     * @param index index of the revision
+     * @return shortened revision string (maximum 8 characters) with ".." appended
+     */
+    public String getShortRev(int index) {
+        return rev[index].substring(0, Integer.min(rev[index].length(), 8)) + "..";
+    }
+
     public String[] getFile(int index) {
         return file[index];
     }

opengrok-web/src/main/webapp/diff.jsp

@@ -16,7 +16,7 @@ information: Portions Copyright [yyyy] [name of copyright owner]
 
 CDDL HEADER END
 
-Copyright (c) 2006, 2022, Oracle and/or its affiliates. All rights reserved.
+Copyright (c) 2006, 2023, Oracle and/or its affiliates. All rights reserved.
 Portions Copyright 2011 Jens Elkner.
 Portions Copyright (c) 2020, Chris Fraire <[email protected]>.
 --%>
@@ -163,9 +163,9 @@ include file="/mast.jsp"
             if (type == t) {
         %> <span class="active"><%= t.toString() %><%
                 if (t == DiffType.OLD) {
-            %>  ( <%= data.getRev(0) %> )<%
+            %>  (<%= data.getRev(0) %>)<%
                 } else if (t == DiffType.NEW) {
-            %>  ( <%= data.getRev(1) %> )<%
+            %>  (<%= data.getRev(1) %>)<%
                 }
             %></span><%
             } else {
@@ -175,9 +175,9 @@ include file="/mast.jsp"
 <%= QueryParameters.DIFF_LEVEL_PARAM_EQ %><%= full ? '1' : '0'%>"><%= t.toString() %>
             <%
                 if (t == DiffType.OLD) {
-            %>  ( <%= data.getRev(0) %> )<%
+            %>  (<%= data.getShortRev(0) %>)<%
                 } else if (t == DiffType.NEW) {
-            %>  ( <%= data.getRev(1) %> )<%
+            %>  (<%= data.getShortRev(1) %>)<%
                 }
             %></a></span><%
             }