From 795ce2765773156da868720303a6b2638d14820b Mon Sep 17 00:00:00 2001
From: Paul Martin <paul@paulsputer.com>
Date: Sun, 27 Mar 2016 12:09:16 -0400
Subject: [PATCH] Friendly URL for EditFile Page #974
---
src/main/java/com/gitblit/wicket/pages/EditFilePage.java | 20 ++++++++++----------
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/main/java/com/gitblit/wicket/pages/EditFilePage.java b/src/main/java/com/gitblit/wicket/pages/EditFilePage.java
index 378f2a3..0037537 100644
--- a/src/main/java/com/gitblit/wicket/pages/EditFilePage.java
+++ b/src/main/java/com/gitblit/wicket/pages/EditFilePage.java
@@ -17,6 +17,7 @@
import java.io.IOException;
+import java.text.MessageFormat;
import java.util.HashSet;
import java.util.List;
import java.util.Set;
@@ -51,7 +52,7 @@
import com.gitblit.wicket.MarkupProcessor.MarkupDocument;
import com.gitblit.wicket.WicketUtils;
-@CacheControl(LastModified.BOOT)
+@CacheControl(LastModified.REPOSITORY)
public class EditFilePage extends RepositoryPage {
public EditFilePage(final PageParameters params) {
@@ -96,9 +97,9 @@
Fragment fragment;
String displayedCommitId = commit.getId().getName();
MarkupDocument markupDoc = processor.parse(repositoryName, displayedCommitId, documentPath, markupText);
- logger.info("Loading Edit File page: " + displayedCommitId);
+ logger.trace("Loading Edit File page: " + displayedCommitId);
- if (currentUser.canEdit(getRepositoryModel())) {
+ if (currentUser.canEdit(getRepositoryModel()) && JGitUtils.isTip(getRepository(), objectId.toString())) {
final Model<String> documentContent = new Model<String>(markupDoc.markup);
final Model<String> commitMessage = new Model<String>("Document update");
@@ -112,7 +113,6 @@
@Override
protected void onSubmit() {
-
final Repository repository = getRepository();
final String document = documentContent.getObject();
final String message = commitMessage.getObject();
@@ -125,7 +125,7 @@
try {
ObjectId docAtLoad = getRepository().resolve(commitIdAtLoad.getObject());
- logger.info("Commiting Edit File page: " + commitIdAtLoad.getObject());
+ logger.trace("Commiting Edit File page: " + commitIdAtLoad.getObject());
DirCache index = DirCache.newInCore();
DirCacheBuilder builder = index.builder();
@@ -156,12 +156,12 @@
}
if (success == false) {
- EditFilePage.this.error("Unable to commit document " + path, false);
+ getSession().error(MessageFormat.format(getString("gb.fileNotMergeable"),path));
return;
}
+ getSession().info(MessageFormat.format(getString("gb.fileCommitted"),path));
setResponsePage(EditFilePage.class, params);
- return;
}
};
@@ -197,12 +197,12 @@
fragment.add(new ExternalLink("rawLink", rawUrl));
add(fragment);
-
+
}
@Override
protected String getPageName() {
- return getString("gb.docs");
+ return getString("gb.editFile");
}
@Override
@@ -212,7 +212,7 @@
@Override
protected Class<? extends BasePage> getRepoNavPageClass() {
- return DocsPage.class;
+ return EditFilePage.class;
}
--
Gitblit v1.9.1