From e1908933fa9968b11e9ea03d88730aee3140534b Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Tue, 15 Oct 2013 09:43:09 -0400 Subject: [PATCH] Fix error on generating activity page when there is no activity --- src/main/java/com/gitblit/wicket/pages/BlobPage.java | 30 ++++++++++++++++++------------ 1 files changed, 18 insertions(+), 12 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/pages/BlobPage.java b/src/main/java/com/gitblit/wicket/pages/BlobPage.java index f6e9998..0c93d48 100644 --- a/src/main/java/com/gitblit/wicket/pages/BlobPage.java +++ b/src/main/java/com/gitblit/wicket/pages/BlobPage.java @@ -32,11 +32,14 @@ import com.gitblit.Keys; import com.gitblit.utils.JGitUtils; import com.gitblit.utils.StringUtils; +import com.gitblit.wicket.CacheControl; +import com.gitblit.wicket.CacheControl.LastModified; import com.gitblit.wicket.ExternalImage; import com.gitblit.wicket.WicketUtils; import com.gitblit.wicket.panels.CommitHeaderPanel; import com.gitblit.wicket.panels.PathBreadcrumbsPanel; +@CacheControl(LastModified.BOOT) public class BlobPage extends RepositoryPage { protected String fileExtension; @@ -47,7 +50,7 @@ Repository r = getRepository(); final String blobPath = WicketUtils.getPath(params); String [] encodings = GitBlit.getEncodings(); - + if (StringUtils.isEmpty(blobPath)) { // blob by objectid @@ -150,7 +153,7 @@ } } } - + protected String missingBlob(String blobPath, RevCommit commit) { StringBuilder sb = new StringBuilder(); sb.append("<div class=\"alert alert-error\">"); @@ -162,23 +165,23 @@ protected String generateSourceView(String source, String extension, boolean prettyPrint) { String [] lines = source.split("\n"); - + StringBuilder sb = new StringBuilder(); sb.append("<!-- start blob table -->"); sb.append("<table width=\"100%\"><tbody><tr>"); - + // nums column sb.append("<!-- start nums column -->"); sb.append("<td id=\"nums\">"); sb.append("<pre>"); - String numPattern = "<span id=\"L{0}\" class=\"num\">{0}</span>\n"; + String numPattern = "<span id=\"L{0}\" class=\"jump\"></span><a href=\"#L{0}\">{0}</a>\n"; for (int i = 0; i < lines.length; i++) { sb.append(MessageFormat.format(numPattern, "" + (i + 1))); } sb.append("</pre>"); sb.append("<!-- end nums column -->"); sb.append("</td>"); - + sb.append("<!-- start lines column -->"); sb.append("<td id=\"lines\">"); sb.append("<div class=\"sourceview\">"); @@ -188,13 +191,16 @@ sb.append("<pre class=\"plainprint\">"); } lines = StringUtils.escapeForHtml(source, true).split("\n"); - + sb.append("<table width=\"100%\"><tbody>"); - - String linePattern = "<tr class=\"{0}\"><td><a href=\"#L{2}\">{1}</a>\r</tr>"; + + String linePattern = "<tr class=\"{0}\"><td><div><span class=\"line\">{1}</span></div>\r</tr>"; for (int i = 0; i < lines.length; i++) { String line = lines[i].replace('\r', ' '); String cssClass = (i % 2 == 0) ? "even" : "odd"; + if (StringUtils.isEmpty(line.trim())) { + line = " "; + } sb.append(MessageFormat.format(linePattern, cssClass, line, "" + (i + 1))); } sb.append("</tbody></table></pre>"); @@ -202,10 +208,10 @@ sb.append("</div>"); sb.append("</td>"); sb.append("<!-- end lines column -->"); - + sb.append("</tr></tbody></table>"); sb.append("<!-- end blob table -->"); - + return sb.toString(); } @@ -213,7 +219,7 @@ protected String getPageName() { return getString("gb.view"); } - + @Override protected Class<? extends BasePage> getRepoNavPageClass() { return TreePage.class; -- Gitblit v1.9.1