From b94965e87929c6b4e42f9a2078dc0d910cc2637d Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Wed, 09 May 2012 07:59:00 -0400 Subject: [PATCH] Merge pull request #17 from peterloron/master --- src/com/gitblit/wicket/pages/RepositoriesPage.java | 51 ++++++++++++++++----------------------------------- 1 files changed, 16 insertions(+), 35 deletions(-) diff --git a/src/com/gitblit/wicket/pages/RepositoriesPage.java b/src/com/gitblit/wicket/pages/RepositoriesPage.java index bc33b01..40d1a65 100644 --- a/src/com/gitblit/wicket/pages/RepositoriesPage.java +++ b/src/com/gitblit/wicket/pages/RepositoriesPage.java @@ -16,17 +16,17 @@ package com.gitblit.wicket.pages; import java.io.File; -import java.io.FileReader; +import java.io.FileInputStream; import java.io.InputStream; import java.io.InputStreamReader; import java.text.MessageFormat; import java.util.List; -import org.apache.wicket.Application; import org.apache.wicket.Component; import org.apache.wicket.PageParameters; import org.apache.wicket.markup.html.basic.Label; import org.apache.wicket.resource.ContextRelativeResource; +import org.eclipse.jgit.lib.Constants; import com.gitblit.GitBlit; import com.gitblit.Keys; @@ -91,22 +91,18 @@ @Override protected void addDropDownMenus(List<PageRegistration> pages) { - PageParameters currentParameters = getPageParameters(); - int daysBack = GitBlit.getInteger(Keys.web.activityDuration, 14); - if (currentParameters != null && !currentParameters.containsKey("db")) { - currentParameters.put("db", daysBack); - } + PageParameters params = getPageParameters(); DropDownMenuRegistration menu = new DropDownMenuRegistration("gb.filters", RepositoriesPage.class); // preserve time filter option on repository choices - menu.menuItems.addAll(getRepositoryFilterItems(currentParameters)); - + menu.menuItems.addAll(getRepositoryFilterItems(params)); + // preserve repository filter option on time choices - menu.menuItems.addAll(getTimeFilterItems(currentParameters)); - + menu.menuItems.addAll(getTimeFilterItems(params)); + if (menu.menuItems.size() > 0) { - // Reset Filter + // Reset Filter menu.menuItems.add(new DropDownMenuItem(getString("gb.reset"), null, null)); } @@ -124,14 +120,17 @@ File file = new File(messageSource); if (file.exists()) { try { - FileReader reader = new FileReader(file); + FileInputStream fis = new FileInputStream(file); + InputStreamReader reader = new InputStreamReader(fis, + Constants.CHARACTER_ENCODING); message = MarkdownUtils.transformMarkdown(reader); + reader.close(); } catch (Throwable t) { - message = "Failed to read " + file; + message = getString("gb.failedToRead") + " " + file; warn(message, t); } } else { - message = messageSource + " is not a valid file."; + message = messageSource + " " + getString("gb.isNotValidFile"); } } } @@ -143,31 +142,13 @@ try { ContextRelativeResource res = WicketUtils.getResource(file); InputStream is = res.getResourceStream().getInputStream(); - InputStreamReader reader = new InputStreamReader(is); + InputStreamReader reader = new InputStreamReader(is, Constants.CHARACTER_ENCODING); message = MarkdownUtils.transformMarkdown(reader); reader.close(); } catch (Throwable t) { - message = MessageFormat.format("Failed to read default message from {0}!", file); + message = MessageFormat.format(getString("gb.failedToReadMessage"), file); error(message, t, false); } return message; - } - - @Override - protected void onBeforeRender() { - if (GitBlit.isDebugMode()) { - // strip Wicket tags in debug mode for jQuery DOM traversal - Application.get().getMarkupSettings().setStripWicketTags(true); - } - super.onBeforeRender(); - } - - @Override - protected void onAfterRender() { - if (GitBlit.isDebugMode()) { - // restore Wicket debug tags - Application.get().getMarkupSettings().setStripWicketTags(false); - } - super.onAfterRender(); } } -- Gitblit v1.9.1