From 713d27789bc0b5d82b607fed20e5f16ad9216898 Mon Sep 17 00:00:00 2001
From: Paul Martin <paul@paulsputer.com>
Date: Sun, 27 Mar 2016 12:09:16 -0400
Subject: [PATCH] Disable edit links in read only mode #974

---
 src/main/java/com/gitblit/wicket/pages/DocsPage.java |   17 ++++++++++++++---
 1 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/main/java/com/gitblit/wicket/pages/DocsPage.java b/src/main/java/com/gitblit/wicket/pages/DocsPage.java
index 9301692..5244386 100644
--- a/src/main/java/com/gitblit/wicket/pages/DocsPage.java
+++ b/src/main/java/com/gitblit/wicket/pages/DocsPage.java
@@ -27,15 +27,19 @@
 import org.apache.wicket.markup.repeater.Item;
 import org.apache.wicket.markup.repeater.data.DataView;
 import org.apache.wicket.markup.repeater.data.ListDataProvider;
+import org.apache.wicket.model.StringResourceModel;
+import org.eclipse.jgit.diff.DiffEntry.ChangeType;
 import org.eclipse.jgit.lib.Repository;
 import org.eclipse.jgit.revwalk.RevCommit;
 
 import com.gitblit.models.PathModel;
+import com.gitblit.models.UserModel;
 import com.gitblit.servlet.RawServlet;
 import com.gitblit.utils.ByteFormat;
 import com.gitblit.utils.JGitUtils;
 import com.gitblit.utils.StringUtils;
 import com.gitblit.wicket.CacheControl;
+import com.gitblit.wicket.GitBlitWebSession;
 import com.gitblit.wicket.CacheControl.LastModified;
 import com.gitblit.wicket.MarkupProcessor;
 import com.gitblit.wicket.MarkupProcessor.MarkupDocument;
@@ -54,6 +58,9 @@
 		MarkupProcessor processor = new MarkupProcessor(app().settings(), app().xssFilter());
 
 		Repository r = getRepository();
+		UserModel currentUser = (GitBlitWebSession.get().getUser() != null) ? GitBlitWebSession.get().getUser() : UserModel.ANONYMOUS;
+		final boolean userCanEdit = currentUser.canEdit(getRepositoryModel());
+		
 		RevCommit head = JGitUtils.getCommit(r, objectId);
 		final String commitId = getBestCommitId(head);
 
@@ -102,9 +109,12 @@
 				@Override
 				public void populateItem(final Item<MarkupDocument> item) {
 					MarkupDocument doc = item.getModelObject();
-					// document page links
+					
 					item.add(new BookmarkablePageLink<Void>("editLink", EditFilePage.class,
-							WicketUtils.newPathParameter(repositoryName, commitId, doc.documentPath)));
+							WicketUtils.newPathParameter(repositoryName, commitId, doc.documentPath))
+							.setEnabled(userCanEdit));
+					
+					// document page links					
 					item.add(new BookmarkablePageLink<Void>("blameLink", BlamePage.class,
 							WicketUtils.newPathParameter(repositoryName, commitId, doc.documentPath)));
 					item.add(new BookmarkablePageLink<Void>("historyLink", HistoryPage.class,
@@ -151,7 +161,8 @@
 				item.add(new BookmarkablePageLink<Void>("view", DocPage.class, WicketUtils
 						.newPathParameter(repositoryName, commitId, entry.path)));
 				item.add(new BookmarkablePageLink<Void>("edit", EditFilePage.class, WicketUtils
-						.newPathParameter(repositoryName, commitId, entry.path)));
+						.newPathParameter(repositoryName, commitId, entry.path))
+						.setEnabled(userCanEdit));
 				String rawUrl = RawServlet.asLink(getContextUrl(), repositoryName, commitId, entry.path);
 				item.add(new ExternalLink("raw", rawUrl));
 				item.add(new BookmarkablePageLink<Void>("blame", BlamePage.class, WicketUtils

--
Gitblit v1.9.1