From 3e087ada55f8b0e83b146578b695552db9dc8d97 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Wed, 27 Apr 2011 20:55:38 -0400
Subject: [PATCH] Integrated pagelinks into RepositoryPage. Fixed string encoding.

---
 src/com/gitblit/utils/StringUtils.java           |   26 +++
 src/com/gitblit/wicket/pages/SummaryPage.html    |    3 
 src/com/gitblit/wicket/pages/HistoryPage.html    |    3 
 src/com/gitblit/wicket/RepositoryPage.java       |  149 +++++++++++++++++-
 src/com/gitblit/wicket/pages/MarkdownPage.html   |    3 
 src/com/gitblit/wicket/pages/TagPage.html        |    3 
 src/com/gitblit/wicket/pages/SearchPage.html     |    3 
 src/com/gitblit/wicket/pages/TreePage.html       |    3 
 src/com/gitblit/wicket/pages/CommitPage.html     |    3 
 src/com/gitblit/wicket/pages/LogPage.html        |    3 
 src/com/gitblit/wicket/RepositoryPage.html       |   30 +++
 src/com/gitblit/wicket/pages/MarkdownPage.java   |   25 +--
 src/com/gitblit/wicket/resources/gitblit.css     |   18 +-
 /dev/null                                        |  150 ------------------
 src/com/gitblit/wicket/pages/CommitDiffPage.html |    3 
 src/com/gitblit/wicket/pages/DocsPage.html       |    3 
 src/com/gitblit/wicket/pages/BlobPage.html       |    3 
 src/com/gitblit/wicket/pages/TagsPage.html       |    3 
 src/com/gitblit/wicket/pages/TicketsPage.html    |    3 
 src/com/gitblit/wicket/pages/BlobDiffPage.html   |    3 
 src/com/gitblit/wicket/pages/TicketPage.html     |    3 
 src/com/gitblit/wicket/pages/BranchesPage.html   |    3 
 src/com/gitblit/utils/JGitUtils.java             |    7 
 23 files changed, 215 insertions(+), 238 deletions(-)

diff --git a/src/com/gitblit/utils/JGitUtils.java b/src/com/gitblit/utils/JGitUtils.java
index e5dbfd2..8814c1f 100644
--- a/src/com/gitblit/utils/JGitUtils.java
+++ b/src/com/gitblit/utils/JGitUtils.java
@@ -4,6 +4,7 @@
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
+import java.nio.charset.Charset;
 import java.text.DateFormat;
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
@@ -264,12 +265,14 @@
 	}
 
 	public static String getRawContentAsString(Repository r, RevBlob blob) {
-		return new String(getRawContent(r, blob));
+		byte [] content = getRawContent(r, blob);
+		return new String(content, Charset.forName("UTF-8"));
 	}
 
 	public static String getRawContentAsString(Repository r, RevCommit commit, String blobPath) {
 		RevObject obj = getRevObject(r, commit.getTree(), blobPath);
-		return new String(getRawContent(r, (RevBlob) obj));
+		byte [] content = getRawContent(r, (RevBlob) obj);
+		return new String(content, Charset.forName("UTF-8"));
 	}
 
 	public static List<PathModel> getFilesInPath(Repository r, String basePath, String objectId) {
diff --git a/src/com/gitblit/utils/StringUtils.java b/src/com/gitblit/utils/StringUtils.java
index 6d646df..ff9fac4 100644
--- a/src/com/gitblit/utils/StringUtils.java
+++ b/src/com/gitblit/utils/StringUtils.java
@@ -1,12 +1,18 @@
 package com.gitblit.utils;
 
+import java.io.IOException;
+import java.io.StringReader;
+import java.io.StringWriter;
 import java.io.UnsupportedEncodingException;
 import java.security.MessageDigest;
 import java.security.NoSuchAlgorithmException;
 import java.util.List;
 
+import org.tautua.markdownpapers.Markdown;
+import org.tautua.markdownpapers.parser.ParseException;
+
 public class StringUtils {
-	
+
 	public static boolean isEmpty(String value) {
 		return value == null || value.trim().length() == 0;
 	}
@@ -107,4 +113,22 @@
 		}
 	}
 
+	public static String transformMarkdown(String markdown) throws java.text.ParseException {
+		// Read raw markdown content and transform it to html		
+		StringReader reader = new StringReader(markdown);
+		StringWriter writer = new StringWriter();
+		try {
+			Markdown md = new Markdown();
+			md.transform(reader, writer);
+			return writer.toString();
+		} catch (ParseException p) {			
+			throw new java.text.ParseException(p.getMessage(), 0);
+		} finally {
+			reader.close();
+			try {
+				writer.close();
+			} catch (IOException e) {
+			}
+		}
+	}
 }
diff --git a/src/com/gitblit/wicket/RepositoryPage.html b/src/com/gitblit/wicket/RepositoryPage.html
new file mode 100644
index 0000000..4b3f2e7
--- /dev/null
+++ b/src/com/gitblit/wicket/RepositoryPage.html
@@ -0,0 +1,30 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml"  
+      xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-xhtml1.3-strict.dtd"  
+      xml:lang="en"  
+      lang="en"> 
+
+<body>
+	<wicket:extend>
+		<!-- page header bar -->	
+		<div>
+			<!-- floating search form on right -->
+			<form wicket:id="searchForm">
+				<div class="search">
+					<select wicket:id="searchType"/>			
+					<input type="text" id="searchBox" wicket:id="searchBox" size="25" value=""/>
+				</div>
+			</form>
+		
+			<!-- page nav links -->
+			<div class="page_nav">		
+				<a wicket:id="summary"><wicket:message key="gb.summary"></wicket:message></a> | <a wicket:id="log"><wicket:message key="gb.log"></wicket:message></a> | <a wicket:id="branches"><wicket:message key="gb.branches"></wicket:message></a> | <a wicket:id="tags"><wicket:message key="gb.tags"></wicket:message></a> | <a wicket:id="tree"><wicket:message key="gb.tree"></wicket:message></a> <span wicket:id="extra"><span wicket:id="extraSeparator"></span><span wicket:id="extraLink"></span></span>
+			</div>
+		</div>
+	
+		<!-- page content -->
+		<wicket:child />
+		
+	</wicket:extend>
+</body>
+</html>
\ No newline at end of file
diff --git a/src/com/gitblit/wicket/RepositoryPage.java b/src/com/gitblit/wicket/RepositoryPage.java
index 30e41a5..28cdcd0 100644
--- a/src/com/gitblit/wicket/RepositoryPage.java
+++ b/src/com/gitblit/wicket/RepositoryPage.java
@@ -1,5 +1,7 @@
 package com.gitblit.wicket;
 
+import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -7,7 +9,16 @@
 import org.apache.wicket.Component;
 import org.apache.wicket.PageParameters;
 import org.apache.wicket.markup.html.basic.Label;
+import org.apache.wicket.markup.html.form.DropDownChoice;
+import org.apache.wicket.markup.html.form.StatelessForm;
+import org.apache.wicket.markup.html.form.TextField;
+import org.apache.wicket.markup.html.link.BookmarkablePageLink;
 import org.apache.wicket.markup.html.panel.Fragment;
+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.IModel;
+import org.apache.wicket.model.Model;
 import org.eclipse.jgit.diff.DiffEntry.ChangeType;
 import org.eclipse.jgit.lib.PersonIdent;
 import org.eclipse.jgit.lib.Repository;
@@ -20,9 +31,15 @@
 import com.gitblit.utils.JGitUtils;
 import com.gitblit.utils.JGitUtils.SearchType;
 import com.gitblit.utils.StringUtils;
+import com.gitblit.wicket.pages.BranchesPage;
+import com.gitblit.wicket.pages.DocsPage;
+import com.gitblit.wicket.pages.LogPage;
 import com.gitblit.wicket.pages.RepositoriesPage;
 import com.gitblit.wicket.pages.SearchPage;
-import com.gitblit.wicket.panels.PageLinksPanel;
+import com.gitblit.wicket.pages.SummaryPage;
+import com.gitblit.wicket.pages.TagsPage;
+import com.gitblit.wicket.pages.TicketsPage;
+import com.gitblit.wicket.pages.TreePage;
 import com.gitblit.wicket.panels.RefsPanel;
 
 public abstract class RepositoryPage extends BasePage {
@@ -34,7 +51,21 @@
 	private transient Repository r = null;
 
 	private final Logger logger = LoggerFactory.getLogger(RepositoryPage.class);
-	
+
+	private final Map<String, String> knownPages = new HashMap<String, String>() {
+
+		private static final long serialVersionUID = 1L;
+
+		{
+			put("summary", "gb.summary");
+			put("log", "gb.log");
+			put("branches", "gb.branches");
+			put("tags", "gb.tags");
+			put("tree", "gb.tree");
+			put("tickets", "gb.tickets");
+		}
+	};
+
 	public RepositoryPage(PageParameters params) {
 		super(params);
 		if (!params.containsKey("r")) {
@@ -46,12 +77,69 @@
 
 		Repository r = getRepository();
 
-		// setup the page links and disable this page's link
-		PageLinksPanel pageLinks = new PageLinksPanel("pageLinks", r, repositoryName, getPageName());
-		add(pageLinks);
-		pageLinks.disablePageLink(getPageName());
+		// standard page links
+		add(new BookmarkablePageLink<Void>("summary", SummaryPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+		add(new BookmarkablePageLink<Void>("log", LogPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+		add(new BookmarkablePageLink<Void>("branches", BranchesPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+		add(new BookmarkablePageLink<Void>("tags", TagsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+		add(new BookmarkablePageLink<Void>("tree", TreePage.class, WicketUtils.newRepositoryParameter(repositoryName)));
 
+		// per-repository extra page links 
+		List<String> extraPageLinks = new ArrayList<String>();
+
+		// Conditionally add tickets page
+		boolean checkTickets = JGitUtils.getRepositoryUseTickets(r);
+		if (checkTickets && JGitUtils.getTicketsBranch(r) != null) {
+			extraPageLinks.add("tickets");
+		}
+
+		// Conditionally add docs page
+		boolean checkDocs = JGitUtils.getRepositoryUseDocs(r);
+		if (checkDocs) {
+			extraPageLinks.add("docs");
+		}
+
+		ListDataProvider<String> extrasDp = new ListDataProvider<String>(extraPageLinks);
+		DataView<String> extrasView = new DataView<String>("extra", extrasDp) {
+			private static final long serialVersionUID = 1L;
+
+			public void populateItem(final Item<String> item) {
+				String extra = item.getModelObject();
+				if (extra.equals("tickets")) {
+					item.add(new Label("extraSeparator", " | "));
+					item.add(new LinkPanel("extraLink", null, getString("gb.tickets"), TicketsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+				} else if (extra.equals("docs")) {
+					item.add(new Label("extraSeparator", " | "));
+					item.add(new LinkPanel("extraLink", null, getString("gb.docs"), DocsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
+				}
+			}
+		};
+		add(extrasView);
+		
+		// disable current page
+		disablePageLink(getPageName());
+
+		// add floating search form
+		SearchForm searchForm = new SearchForm("searchForm", repositoryName);
+		add(searchForm);
+		searchForm.setTranslatedAttributes();
+		
+		// set stateless page preference
 		setStatelessHint(true);
+	}
+
+	public void disablePageLink(String pageName) {
+		for (String wicketId : knownPages.keySet()) {
+			String key = knownPages.get(wicketId);
+			String linkName = getString(key);
+			if (linkName.equals(pageName)) {
+				Component c = get(wicketId);
+				if (c != null) {
+					c.setEnabled(false);
+				}
+				break;
+			}
+		}
 	}
 
 	protected Repository getRepository() {
@@ -108,7 +196,6 @@
 
 	protected abstract String getPageName();
 
-	
 	protected Component createPersonPanel(String wicketId, PersonIdent identity, SearchType searchType) {
 		boolean showEmail = GitBlit.self().settings().getBoolean(Keys.web.showEmailAddresses, false);
 		if (!showEmail || StringUtils.isEmpty(identity.getName()) || StringUtils.isEmpty(identity.getEmailAddress())) {
@@ -130,14 +217,14 @@
 			LinkPanel nameLink = new LinkPanel("personName", "list", identity.getName(), SearchPage.class, WicketUtils.newSearchParameter(repositoryName, objectId, identity.getName(), searchType));
 			setPersonSearchTooltip(nameLink, identity.getName(), searchType);
 			fullPerson.add(nameLink);
-			
+
 			LinkPanel addressLink = new LinkPanel("personAddress", "list", "<" + identity.getEmailAddress() + ">", SearchPage.class, WicketUtils.newSearchParameter(repositoryName, objectId, identity.getEmailAddress(), searchType));
 			setPersonSearchTooltip(addressLink, identity.getEmailAddress(), searchType);
 			fullPerson.add(addressLink);
 			return fullPerson;
 		}
 	}
-	
+
 	protected void setPersonSearchTooltip(Component component, String value, SearchType searchType) {
 		if (searchType.equals(SearchType.AUTHOR)) {
 			WicketUtils.setHtmlTooltip(component, getString("gb.searchForAuthor") + " " + value);
@@ -145,7 +232,7 @@
 			WicketUtils.setHtmlTooltip(component, getString("gb.searchForCommitter") + " " + value);
 		}
 	}
-	
+
 	protected void setChangeTypeTooltip(Component container, ChangeType type) {
 		switch (type) {
 		case ADD:
@@ -163,7 +250,7 @@
 			break;
 		}
 	}
-	
+
 	@Override
 	protected void onBeforeRender() {
 		// dispose of repository object
@@ -191,4 +278,44 @@
 	protected PageParameters newPathParameter(String path) {
 		return WicketUtils.newPathParameter(repositoryName, objectId, path);
 	}
+
+	class SearchForm extends StatelessForm<Void> {
+		private static final long serialVersionUID = 1L;
+
+		private final String repositoryName;
+
+		private final IModel<String> searchBoxModel = new Model<String>("");
+
+		private final IModel<SearchType> searchTypeModel = new Model<SearchType>(SearchType.COMMIT);
+
+		public SearchForm(String id, String repositoryName) {
+			super(id);
+			this.repositoryName = repositoryName;
+			DropDownChoice<SearchType> searchType = new DropDownChoice<SearchType>("searchType", Arrays.asList(SearchType.values()));
+			searchType.setModel(searchTypeModel);
+			add(searchType.setVisible(GitBlit.self().settings().getBoolean(Keys.web.showSearchTypeSelection, false)));
+			TextField<String> searchBox = new TextField<String>("searchBox", searchBoxModel);
+			add(searchBox);
+		}
+		
+		void setTranslatedAttributes() {			
+			WicketUtils.setHtmlTooltip(get("searchType"), getString("gb.searchTypeTooltip"));
+			WicketUtils.setHtmlTooltip(get("searchBox"), getString("gb.searchTooltip"));
+			WicketUtils.setInputPlaceholder(get("searchBox"), getString("gb.search"));			
+		}
+
+		@Override
+		public void onSubmit() {
+			SearchType searchType = searchTypeModel.getObject();
+			String searchString = searchBoxModel.getObject();
+			for (SearchType type : SearchType.values()) {
+				if (searchString.toLowerCase().startsWith(type.name().toLowerCase() + ":")) {
+					searchType = type;
+					searchString = searchString.substring(type.name().toLowerCase().length() + 1).trim();
+					break;
+				}
+			}
+			setResponsePage(SearchPage.class, WicketUtils.newSearchParameter(repositoryName, null, searchString, searchType));
+		}
+	}
 }
diff --git a/src/com/gitblit/wicket/pages/BlobDiffPage.html b/src/com/gitblit/wicket/pages/BlobDiffPage.html
index 1227169..4818aa8 100644
--- a/src/com/gitblit/wicket/pages/BlobDiffPage.html
+++ b/src/com/gitblit/wicket/pages/BlobDiffPage.html
@@ -6,9 +6,6 @@
 
 <body>
 <wicket:extend>
-
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
 	
 	<!-- blob nav links -->	
 	<div class="page_nav2">
diff --git a/src/com/gitblit/wicket/pages/BlobPage.html b/src/com/gitblit/wicket/pages/BlobPage.html
index 80ff059..5237443 100644
--- a/src/com/gitblit/wicket/pages/BlobPage.html
+++ b/src/com/gitblit/wicket/pages/BlobPage.html
@@ -16,9 +16,6 @@
 <!-- need to specify body.onload -->
 <body onload="prettyPrint()">
 
-		<!-- page nav links -->	
-		<div wicket:id="pageLinks">[page links]</div>
-	
 		<!-- blob nav links -->	
 		<div class="page_nav2">
 			<span wicket:id="blameLink">[blame link]</span> | <a wicket:id="historyLink"><wicket:message key="gb.history"></wicket:message></a> | <a wicket:id="rawLink"><wicket:message key="gb.raw"></wicket:message></a> | <a wicket:id="headLink"><wicket:message key="gb.head"></wicket:message></a>
diff --git a/src/com/gitblit/wicket/pages/BranchesPage.html b/src/com/gitblit/wicket/pages/BranchesPage.html
index 606cbf4..65fd9b9 100644
--- a/src/com/gitblit/wicket/pages/BranchesPage.html
+++ b/src/com/gitblit/wicket/pages/BranchesPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- branches -->	
 	<div style="margin-top:5px;" wicket:id="branchesPanel">[branches panel]</div>
 	
diff --git a/src/com/gitblit/wicket/pages/CommitDiffPage.html b/src/com/gitblit/wicket/pages/CommitDiffPage.html
index 4e3a268..50a8877 100644
--- a/src/com/gitblit/wicket/pages/CommitDiffPage.html
+++ b/src/com/gitblit/wicket/pages/CommitDiffPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-	
 	<!-- commitdiff nav links -->	
 	<div class="page_nav2">
 		<wicket:message key="gb.parent"></wicket:message>: <span wicket:id="parentLink">[parent link]</span> | <a wicket:id="patchLink"><wicket:message key="gb.patch"></wicket:message></a> | <a wicket:id="commitLink"><wicket:message key="gb.commit"></wicket:message></a>
diff --git a/src/com/gitblit/wicket/pages/CommitPage.html b/src/com/gitblit/wicket/pages/CommitPage.html
index 2601303..059d77e 100644
--- a/src/com/gitblit/wicket/pages/CommitPage.html
+++ b/src/com/gitblit/wicket/pages/CommitPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-	
 	<!-- commit nav links -->	
 	<div class="page_nav2">
 		<wicket:message key="gb.parent"></wicket:message>: <span wicket:id="parentLink">[parent link]</span> | <a wicket:id="patchLink"><wicket:message key="gb.patch"></wicket:message></a> | <span wicket:id="commitdiffLink">[commitdiff link]</span>
diff --git a/src/com/gitblit/wicket/pages/DocsPage.html b/src/com/gitblit/wicket/pages/DocsPage.html
index c098bcf..0da4bde 100644
--- a/src/com/gitblit/wicket/pages/DocsPage.html
+++ b/src/com/gitblit/wicket/pages/DocsPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 	
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-	
 	<!-- header -->
 	<div style="margin-top:5px;" class="header"><span wicket:id="header">[header]</span></div>
 	
diff --git a/src/com/gitblit/wicket/pages/HistoryPage.html b/src/com/gitblit/wicket/pages/HistoryPage.html
index f132e59..f9bd2f6 100644
--- a/src/com/gitblit/wicket/pages/HistoryPage.html
+++ b/src/com/gitblit/wicket/pages/HistoryPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- pager links -->
 	<div style="padding-top:5px;">
 		<a wicket:id="firstPageTop"><wicket:message key="gb.pageFirst"></wicket:message></a> | <a wicket:id="prevPageTop"><wicket:message key="gb.pagePrevious"></wicket:message></a> | <a wicket:id="nextPageTop"><wicket:message key="gb.pageNext"></wicket:message></a> 
diff --git a/src/com/gitblit/wicket/pages/LogPage.html b/src/com/gitblit/wicket/pages/LogPage.html
index ad39b77..8e5cb96 100644
--- a/src/com/gitblit/wicket/pages/LogPage.html
+++ b/src/com/gitblit/wicket/pages/LogPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- pager links -->
 	<div style="padding-top:5px;">
 		<a wicket:id="firstPageTop"><wicket:message key="gb.pageFirst"></wicket:message></a> | <a wicket:id="prevPageTop"><wicket:message key="gb.pagePrevious"></wicket:message></a> | <a wicket:id="nextPageTop"><wicket:message key="gb.pageNext"></wicket:message></a> 
diff --git a/src/com/gitblit/wicket/pages/MarkdownPage.html b/src/com/gitblit/wicket/pages/MarkdownPage.html
index 9b00934..b8c9cfb 100644
--- a/src/com/gitblit/wicket/pages/MarkdownPage.html
+++ b/src/com/gitblit/wicket/pages/MarkdownPage.html
@@ -13,9 +13,6 @@
 
 <body>
 <wicket:extend>
-		<!-- page nav links -->	
-		<div wicket:id="pageLinks">[page links]</div>
-	
 		<!-- markdown nav links -->	
 		<div class="page_nav2">
 			<span wicket:id="blameLink">[blame link]</span> | <a wicket:id="historyLink"><wicket:message key="gb.history"></wicket:message></a> | <a wicket:id="rawLink"><wicket:message key="gb.raw"></wicket:message></a> | <a wicket:id="headLink"><wicket:message key="gb.head"></wicket:message></a>
diff --git a/src/com/gitblit/wicket/pages/MarkdownPage.java b/src/com/gitblit/wicket/pages/MarkdownPage.java
index 83083ab..64d3ce3 100644
--- a/src/com/gitblit/wicket/pages/MarkdownPage.java
+++ b/src/com/gitblit/wicket/pages/MarkdownPage.java
@@ -1,7 +1,6 @@
 package com.gitblit.wicket.pages;
 
-import java.io.StringReader;
-import java.io.StringWriter;
+import java.text.ParseException;
 
 import org.apache.wicket.PageParameters;
 import org.apache.wicket.markup.html.basic.Label;
@@ -9,19 +8,14 @@
 import org.eclipse.jgit.lib.Constants;
 import org.eclipse.jgit.lib.Repository;
 import org.eclipse.jgit.revwalk.RevCommit;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.tautua.markdownpapers.Markdown;
-import org.tautua.markdownpapers.parser.ParseException;
 
 import com.gitblit.utils.JGitUtils;
+import com.gitblit.utils.StringUtils;
 import com.gitblit.wicket.RepositoryPage;
 import com.gitblit.wicket.WicketUtils;
 
 public class MarkdownPage extends RepositoryPage {
 	
-	private final Logger logger = LoggerFactory.getLogger(MarkdownPage.class);
-
 	public MarkdownPage(PageParameters params) {
 		super(params);
 
@@ -36,17 +30,14 @@
 		add(new BookmarkablePageLink<Void>("rawLink", RawPage.class, WicketUtils.newPathParameter(repositoryName, objectId, markdownPath)));
 		add(new BookmarkablePageLink<Void>("headLink", MarkdownPage.class, WicketUtils.newPathParameter(repositoryName, Constants.HEAD, markdownPath)));
 
-		// Read raw markdown content and transform it to html
-		String htmlText = "";
+		// Read raw markdown content and transform it to html		
+		String markdownText = JGitUtils.getRawContentAsString(r, commit, markdownPath);
+		String htmlText;
 		try {
-			String rawText = JGitUtils.getRawContentAsString(r, commit, markdownPath);
-			StringReader reader = new StringReader(rawText);
-			StringWriter writer = new StringWriter();
-			Markdown md = new Markdown();
-			md.transform(reader, writer);
-			htmlText = writer.toString();
+			htmlText = StringUtils.transformMarkdown(markdownText);
 		} catch (ParseException p) {
-			logger.error("Failed to parse markdown text from " + markdownPath, p);
+			error(p.getMessage());
+			htmlText = markdownText;
 		}
 		
 		// Add the html to the page
diff --git a/src/com/gitblit/wicket/pages/SearchPage.html b/src/com/gitblit/wicket/pages/SearchPage.html
index 246601a4..9bb1f41 100644
--- a/src/com/gitblit/wicket/pages/SearchPage.html
+++ b/src/com/gitblit/wicket/pages/SearchPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- pager links -->
 	<div style="padding-top:5px;">
 		<a wicket:id="firstPageTop"><wicket:message key="gb.pageFirst"></wicket:message></a> | <a wicket:id="prevPageTop"><wicket:message key="gb.pagePrevious"></wicket:message></a> | <a wicket:id="nextPageTop"><wicket:message key="gb.pageNext"></wicket:message></a> 
diff --git a/src/com/gitblit/wicket/pages/SummaryPage.html b/src/com/gitblit/wicket/pages/SummaryPage.html
index 6abef54..6dbcfa2 100644
--- a/src/com/gitblit/wicket/pages/SummaryPage.html
+++ b/src/com/gitblit/wicket/pages/SummaryPage.html
@@ -6,9 +6,6 @@
 
 <body>
 <wicket:extend>
-
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
 	
 	<div style="clear:both;">
 		<!-- Repository Activity Chart -->	
diff --git a/src/com/gitblit/wicket/pages/TagPage.html b/src/com/gitblit/wicket/pages/TagPage.html
index e410462..839375b 100644
--- a/src/com/gitblit/wicket/pages/TagPage.html
+++ b/src/com/gitblit/wicket/pages/TagPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 	
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- summary header -->
 	<div style="margin-top: 5px;" class="header" wicket:id="commit">[shortlog header]</div>
 	
diff --git a/src/com/gitblit/wicket/pages/TagsPage.html b/src/com/gitblit/wicket/pages/TagsPage.html
index 3a49a74..03f1a0d 100644
--- a/src/com/gitblit/wicket/pages/TagsPage.html
+++ b/src/com/gitblit/wicket/pages/TagsPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- tags panel -->
 	<div style="margin-top:5px;" wicket:id="tagsPanel">[tags panel]</div>
 	
diff --git a/src/com/gitblit/wicket/pages/TicketPage.html b/src/com/gitblit/wicket/pages/TicketPage.html
index b61f7f9..160bc0c 100644
--- a/src/com/gitblit/wicket/pages/TicketPage.html
+++ b/src/com/gitblit/wicket/pages/TicketPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-	
 	<!-- ticket title -->
 	<div style="font-size:150%;padding-top:5px;" wicket:id="ticketTitle">[ticket title]</div>
 		
diff --git a/src/com/gitblit/wicket/pages/TicketsPage.html b/src/com/gitblit/wicket/pages/TicketsPage.html
index 0c865f3..0913dc2 100644
--- a/src/com/gitblit/wicket/pages/TicketsPage.html
+++ b/src/com/gitblit/wicket/pages/TicketsPage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-
 	<!-- header -->	
 	<div style="margin-top:5px;" class="header" wicket:id="header">[header]</div>
 	
diff --git a/src/com/gitblit/wicket/pages/TreePage.html b/src/com/gitblit/wicket/pages/TreePage.html
index 90ca62b..35eac29 100644
--- a/src/com/gitblit/wicket/pages/TreePage.html
+++ b/src/com/gitblit/wicket/pages/TreePage.html
@@ -7,9 +7,6 @@
 <body>
 <wicket:extend>
 
-	<!-- page nav links -->	
-	<div wicket:id="pageLinks">[page links]</div>
-	
 	<!-- blob nav links -->	
 	<div class="page_nav2">
 		<a wicket:id="historyLink"><wicket:message key="gb.history"></wicket:message></a> | <a wicket:id="headLink"><wicket:message key="gb.head"></wicket:message></a>
diff --git a/src/com/gitblit/wicket/panels/PageLinksPanel.html b/src/com/gitblit/wicket/panels/PageLinksPanel.html
deleted file mode 100644
index 7cad29f..0000000
--- a/src/com/gitblit/wicket/panels/PageLinksPanel.html
+++ /dev/null
@@ -1,21 +0,0 @@
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
-<html xmlns="http://www.w3.org/1999/xhtml"  
-      xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-xhtml1.3-strict.dtd"  
-      xml:lang="en"  
-      lang="en"> 
-
-<wicket:panel>
-	<!-- floating search form on right -->
-	<form wicket:id="searchForm">
-		<div class="search">
-			<select wicket:id="searchType"/>			
-			<input type="text" id="searchBox" wicket:id="searchBox" size="25" value=""/>
-		</div>
-	</form>
-		
-	<!-- page nav links -->
-	<div class="page_nav">		
-		<a wicket:id="summary"><wicket:message key="gb.summary"></wicket:message></a> | <a wicket:id="log"><wicket:message key="gb.log"></wicket:message></a> | <a wicket:id="branches"><wicket:message key="gb.branches"></wicket:message></a> | <a wicket:id="tags"><wicket:message key="gb.tags"></wicket:message></a> | <a wicket:id="tree"><wicket:message key="gb.tree"></wicket:message></a> <span wicket:id="extra"><span wicket:id="extraSeparator"></span><span wicket:id="extraLink"></span></span>
-	</div>	
-</wicket:panel>
-</html>
\ No newline at end of file
diff --git a/src/com/gitblit/wicket/panels/PageLinksPanel.java b/src/com/gitblit/wicket/panels/PageLinksPanel.java
deleted file mode 100644
index 0f33501..0000000
--- a/src/com/gitblit/wicket/panels/PageLinksPanel.java
+++ /dev/null
@@ -1,150 +0,0 @@
-package com.gitblit.wicket.panels;
-
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-
-import org.apache.wicket.Component;
-import org.apache.wicket.markup.html.basic.Label;
-import org.apache.wicket.markup.html.form.DropDownChoice;
-import org.apache.wicket.markup.html.form.StatelessForm;
-import org.apache.wicket.markup.html.form.TextField;
-import org.apache.wicket.markup.html.link.BookmarkablePageLink;
-import org.apache.wicket.markup.html.panel.Panel;
-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.IModel;
-import org.apache.wicket.model.Model;
-import org.eclipse.jgit.lib.Repository;
-
-import com.gitblit.GitBlit;
-import com.gitblit.Keys;
-import com.gitblit.utils.JGitUtils;
-import com.gitblit.utils.JGitUtils.SearchType;
-import com.gitblit.wicket.LinkPanel;
-import com.gitblit.wicket.WicketUtils;
-import com.gitblit.wicket.pages.BranchesPage;
-import com.gitblit.wicket.pages.DocsPage;
-import com.gitblit.wicket.pages.LogPage;
-import com.gitblit.wicket.pages.SearchPage;
-import com.gitblit.wicket.pages.SummaryPage;
-import com.gitblit.wicket.pages.TagsPage;
-import com.gitblit.wicket.pages.TicketsPage;
-import com.gitblit.wicket.pages.TreePage;
-
-public class PageLinksPanel extends Panel {
-
-	private static final long serialVersionUID = 1L;
-
-	private final Map<String, String> knownPages = new HashMap<String, String>() {
-
-		private static final long serialVersionUID = 1L;
-
-		{
-			put("summary", "gb.summary");
-			put("log", "gb.log");
-			put("branches", "gb.branches");
-			put("tags", "gb.tags");
-			put("tree", "gb.tree");
-			put("tickets", "gb.tickets");
-		}
-	};
-
-	public PageLinksPanel(String id, Repository r, final String repositoryName, String pageName) {
-		super(id);
-
-		// summary
-		add(new BookmarkablePageLink<Void>("summary", SummaryPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-		add(new BookmarkablePageLink<Void>("log", LogPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-		add(new BookmarkablePageLink<Void>("branches", BranchesPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-		add(new BookmarkablePageLink<Void>("tags", TagsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-		add(new BookmarkablePageLink<Void>("tree", TreePage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-
-		List<String> extras = new ArrayList<String>();
-
-		// Get the repository tickets setting
-		boolean checkTickets = JGitUtils.getRepositoryUseTickets(r);
-		if (checkTickets && JGitUtils.getTicketsBranch(r) != null) {
-			extras.add("tickets");
-		}
-
-		// Get the repository docs setting
-		boolean checkDocs = JGitUtils.getRepositoryUseDocs(r);
-		if (checkDocs) {
-			extras.add("docs");
-		}
-
-		ListDataProvider<String> extrasDp = new ListDataProvider<String>(extras);
-		DataView<String> extrasView = new DataView<String>("extra", extrasDp) {
-			private static final long serialVersionUID = 1L;
-
-			public void populateItem(final Item<String> item) {
-				String extra = item.getModelObject();
-				if (extra.equals("tickets")) {
-					item.add(new Label("extraSeparator", " | "));
-					item.add(new LinkPanel("extraLink", null, getString("gb.tickets"), TicketsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-				} else if (extra.equals("docs")) {
-					item.add(new Label("extraSeparator", " | "));
-					item.add(new LinkPanel("extraLink", null, getString("gb.docs"), DocsPage.class, WicketUtils.newRepositoryParameter(repositoryName)));
-				}
-			}
-		};
-		add(extrasView);
-
-		add(new SearchForm("searchForm", repositoryName));
-	}
-
-	public void disablePageLink(String pageName) {
-		for (String wicketId : knownPages.keySet()) {
-			String key = knownPages.get(wicketId);
-			String linkName = getString(key);
-			if (linkName.equals(pageName)) {
-				Component c = get(wicketId);
-				if (c != null) {
-					c.setEnabled(false);
-				}
-				break;
-			}
-		}
-	}
-
-	class SearchForm extends StatelessForm<Void> {
-		private static final long serialVersionUID = 1L;
-
-		private final String repositoryName;
-
-		private final IModel<String> searchBoxModel = new Model<String>("");
-
-		private final IModel<SearchType> searchTypeModel = new Model<SearchType>(SearchType.COMMIT);
-
-		public SearchForm(String id, String repositoryName) {
-			super(id);
-			this.repositoryName = repositoryName;
-			DropDownChoice<SearchType> searchType = new DropDownChoice<SearchType>("searchType", Arrays.asList(SearchType.values()));
-			searchType.setModel(searchTypeModel);
-			WicketUtils.setHtmlTooltip(searchType, getString("gb.searchTypeTooltip"));
-			add(searchType.setVisible(GitBlit.self().settings().getBoolean(Keys.web.showSearchTypeSelection, false)));
-			TextField<String> searchBox = new TextField<String>("searchBox", searchBoxModel);
-			add(searchBox);
-			WicketUtils.setHtmlTooltip(searchBox, getString("gb.searchTooltip"));
-			WicketUtils.setInputPlaceholder(searchBox, getString("gb.search"));
-		}
-
-		@Override
-		public void onSubmit() {
-			SearchType searchType = searchTypeModel.getObject();
-			String searchString = searchBoxModel.getObject();
-			for (SearchType type : SearchType.values()) {
-				if (searchString.toLowerCase().startsWith(type.name().toLowerCase() + ":")) {
-					searchType = type;
-					searchString = searchString.substring(type.name().toLowerCase().length() + 1).trim();
-					break;
-				}
-			}
-			setResponsePage(SearchPage.class, WicketUtils.newSearchParameter(repositoryName, null, searchString, searchType));
-		}
-	}
-}
\ No newline at end of file
diff --git a/src/com/gitblit/wicket/resources/gitblit.css b/src/com/gitblit/wicket/resources/gitblit.css
index ce3be67..5384546 100644
--- a/src/com/gitblit/wicket/resources/gitblit.css
+++ b/src/com/gitblit/wicket/resources/gitblit.css
@@ -32,14 +32,14 @@
 
 /* age0: age < 60*60*2 */
 .age0 {
-	color: #009900;
+	color: #008000;
 	font-style: italic;
 	font-weight: bold;
 }
 
 /* age1: 60*60*2 <= age < 60*60*24*2 */
 .age1 {
-	color: #009900;
+	color: #008000;
 	font-style: italic;
 }
 
@@ -272,7 +272,7 @@
 }
 
 span.diff.remove {
-	color: #cc0000;
+	color: #FFDDDD;
 	font-family: inherit;
 }
 
@@ -373,7 +373,7 @@
 }
 
 span.addition {
-	background-color: #bbffbb;
+	background-color: #ccffcc;
 }
 
 span.modification {
@@ -381,11 +381,11 @@
 }
 
 span.deletion {
-	background-color: #ff8888;
+	background-color: #f8bbbb;
 }
 
 span.rename {
-	background-color: #8888ff;
+	background-color: #cAc2f5;
 }
 
 div.commitLegend {
@@ -616,16 +616,16 @@
 }
 
 span .remoteRef {
-	background-color: #aaaaff;
+	background-color: #cAc2f5;
 	border-color: #ccccff #0033cc #0033cc #ccccff;
 }
 
 span .tagRef {
 	background-color: #ffffaa;
-	border-color: #ffffcc #ffee00 #ffee00 #ffffcc;
+	border-color: #ffcc00 #ffcc00 #ffcc00 #ffcc00;
 }
 
 span .headRef {
-	background-color: #aaffaa;
+	background-color: #ccffcc;
 	border-color: #ccffcc #00cc33 #00cc33 #ccffcc;
 }
\ No newline at end of file

--
Gitblit v1.9.1