From: James Moger Date: Fri, 29 Mar 2013 12:38:20 +0000 (-0400) Subject: Disable view and blame links for deleted blobs (issue 216) X-Git-Tag: v1.3.0~217 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=59f09bdd4536dc3584c779c3985e4ff49d4a7e69;p=gitblit.git Disable view and blame links for deleted blobs (issue 216) --- diff --git a/releases.moxie b/releases.moxie index 535c7ecf..6a61d28e 100644 --- a/releases.moxie +++ b/releases.moxie @@ -11,6 +11,7 @@ r17: { - Fix internal error on folder history links (issue 192) - Fixed incorrect icon file name for .doc files (issue 200) - Do not queue emails with no recipients (issue 201) + - Disable view and blame links for deleted blobs (issue 216) - Fixed extracting Groovy scripts on Express installs (issue 220) - Use standard ServletRequestWrapper instead of custom wrapper (issue 224) diff --git a/src/main/java/com/gitblit/wicket/pages/CommitPage.java b/src/main/java/com/gitblit/wicket/pages/CommitPage.java index c5a24c8d..540d2a21 100644 --- a/src/main/java/com/gitblit/wicket/pages/CommitPage.java +++ b/src/main/java/com/gitblit/wicket/pages/CommitPage.java @@ -200,10 +200,12 @@ public class CommitPage extends RepositoryPage { .setEnabled(!entry.changeType.equals(ChangeType.ADD) && !entry.changeType.equals(ChangeType.DELETE))); item.add(new BookmarkablePageLink("view", BlobPage.class, WicketUtils - .newPathParameter(repositoryName, entry.commitId, entry.path))); + .newPathParameter(repositoryName, entry.commitId, entry.path)) + .setEnabled(!entry.changeType.equals(ChangeType.DELETE))); item.add(new BookmarkablePageLink("blame", BlamePage.class, WicketUtils .newPathParameter(repositoryName, entry.commitId, entry.path)) - .setEnabled(!entry.changeType.equals(ChangeType.ADD))); + .setEnabled(!entry.changeType.equals(ChangeType.ADD) + && !entry.changeType.equals(ChangeType.DELETE))); item.add(new BookmarkablePageLink("history", HistoryPage.class, WicketUtils .newPathParameter(repositoryName, entry.commitId, entry.path)) .setEnabled(!entry.changeType.equals(ChangeType.ADD)));