X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Ftemplate%2FParserFilter.java;h=7cf8c706f36448f528b1eca283bd5c688cdbed08;hp=3494c2186d44e261f8255c37edd0e94dd4e5f128;hb=7b55e0be6a3283e43a9bbab98f82aebdd948eb33;hpb=0df5e91852f737d760c5a9f54c5667309fbadcc2

diff --git a/src/main/java/net/pterodactylus/sone/template/ParserFilter.java b/src/main/java/net/pterodactylus/sone/template/ParserFilter.java
index 3494c21..7cf8c70 100644
--- a/src/main/java/net/pterodactylus/sone/template/ParserFilter.java
+++ b/src/main/java/net/pterodactylus/sone/template/ParserFilter.java
@@ -1,5 +1,5 @@
 /*
- * Sone - ParserFilter.java - Copyright © 2011 David Roden
+ * Sone - ParserFilter.java - Copyright © 2011–2016 David Roden
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,10 +17,14 @@
 
 package net.pterodactylus.sone.template;
 
-import java.io.IOException;
+import static java.lang.String.valueOf;
+import static net.pterodactylus.sone.utils.NumberParsers.parseInt;
+
 import java.io.StringReader;
 import java.io.StringWriter;
+import java.io.UnsupportedEncodingException;
 import java.io.Writer;
+import java.net.URLEncoder;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -35,7 +39,6 @@ import net.pterodactylus.sone.text.PostPart;
 import net.pterodactylus.sone.text.SonePart;
 import net.pterodactylus.sone.text.SoneTextParser;
 import net.pterodactylus.sone.text.SoneTextParserContext;
-import net.pterodactylus.sone.web.page.FreenetRequest;
 import net.pterodactylus.util.template.Filter;
 import net.pterodactylus.util.template.Template;
 import net.pterodactylus.util.template.TemplateContext;
@@ -59,10 +62,10 @@ public class ParserFilter implements Filter {
 	private final TemplateContextFactory templateContextFactory;
 
 	/** The template for {@link PlainTextPart}s. */
-	private final Template plainTextTemplate = TemplateParser.parse(new StringReader("<%text|html>"));
+	private static final Template plainTextTemplate = TemplateParser.parse(new StringReader("<%text|html>"));
 
 	/** The template for {@link FreenetLinkPart}s. */
-	private final Template linkTemplate = TemplateParser.parse(new StringReader("<a class=\"<%cssClass|html>\" href=\"<%link|html>\" title=\"<%title|html>\"><%text|html></a>"));
+	private static final Template linkTemplate = TemplateParser.parse(new StringReader("<a class=\"<%cssClass|html>\" href=\"<%link|html>\" title=\"<%title|html>\"><%text|html></a>"));
 
 	/**
 	 * Creates a new filter that runs its input through a {@link SoneTextParser}
@@ -85,61 +88,47 @@ public class ParserFilter implements Filter {
 	 * {@inheritDoc}
 	 */
 	@Override
-	public Object format(TemplateContext templateContext, Object data, Map<String, String> parameters) {
-		String text = String.valueOf(data);
-		int length = -1;
-		try {
-			length = Integer.parseInt(parameters.get("length"));
-		} catch (NumberFormatException nfe1) {
-			/* ignore. */
-		}
-		if ((length == -1) && (parameters.get("length") != null)) {
-			try {
-				length = Integer.parseInt(String.valueOf(templateContext.get(parameters.get("length"))));
-			} catch (NumberFormatException nfe1) {
-				/* ignore. */
-			}
+	public Object format(TemplateContext templateContext, Object data, Map<String, Object> parameters) {
+		String text = valueOf(data);
+		int length = parseInt(valueOf(parameters.get("length")), -1);
+		int cutOffLength = parseInt(valueOf(parameters.get("cut-off-length")), length);
+		Object sone = parameters.get("sone");
+		if (sone instanceof String) {
+			sone = core.getSone((String) sone).orNull();
 		}
-		String soneKey = parameters.get("sone");
-		if (soneKey == null) {
-			soneKey = "sone";
-		}
-		Sone sone = (Sone) templateContext.get(soneKey);
-		if (sone == null) {
-			sone = core.getSone(soneKey, false);
-		}
-		FreenetRequest request = (FreenetRequest) templateContext.get("request");
-		SoneTextParserContext context = new SoneTextParserContext(request, sone);
+		SoneTextParserContext context = new SoneTextParserContext((Sone) sone);
 		StringWriter parsedTextWriter = new StringWriter();
-		try {
-			Iterable<Part> parts = soneTextParser.parse(context, new StringReader(text));
-			if (length > -1) {
-				List<Part> shortenedParts = new ArrayList<Part>();
-				for (Part part : parts) {
-					if (part instanceof PlainTextPart) {
-						String longText = ((PlainTextPart) part).getText();
-						if (length >= longText.length()) {
-							shortenedParts.add(part);
+		Iterable<Part> parts = soneTextParser.parse(text, context);
+		if (length > -1) {
+			int allPartsLength = 0;
+			List<Part> shortenedParts = new ArrayList<Part>();
+			for (Part part : parts) {
+				if (part instanceof PlainTextPart) {
+					String longText = part.getText();
+					if (allPartsLength < cutOffLength) {
+						if ((allPartsLength + longText.length()) > cutOffLength) {
+							shortenedParts.add(new PlainTextPart(longText.substring(0, cutOffLength - allPartsLength) + "…"));
 						} else {
-							shortenedParts.add(new PlainTextPart(longText.substring(0, length) + "…"));
+							shortenedParts.add(part);
 						}
-						length -= longText.length();
-					} else if (part instanceof LinkPart) {
-						shortenedParts.add(part);
-						length -= ((LinkPart) part).getText().length();
-					} else {
+					}
+					allPartsLength += longText.length();
+				} else if (part instanceof LinkPart) {
+					if (allPartsLength < cutOffLength) {
 						shortenedParts.add(part);
 					}
-					if (length <= 0) {
-						break;
+					allPartsLength += part.getText().length();
+				} else {
+					if (allPartsLength < cutOffLength) {
+						shortenedParts.add(part);
 					}
 				}
+			}
+			if (allPartsLength >= length) {
 				parts = shortenedParts;
 			}
-			render(parsedTextWriter, parts);
-		} catch (IOException ioe1) {
-			/* no exceptions in a StringReader or StringWriter, ignore. */
 		}
+		render(parsedTextWriter, parts);
 		return parsedTextWriter.toString();
 	}
 
@@ -221,7 +210,12 @@ public class ParserFilter implements Filter {
 	 *            The part to render
 	 */
 	private void render(Writer writer, LinkPart linkPart) {
-		renderLink(writer, "/?_CHECKED_HTTP_=" + linkPart.getLink(), linkPart.getText(), linkPart.getTitle(), "internet");
+		try {
+			renderLink(writer, "/external-link/?_CHECKED_HTTP_=" + URLEncoder.encode(linkPart.getLink(), "UTF-8"), linkPart.getText(), linkPart.getTitle(), "internet");
+		} catch (UnsupportedEncodingException uee1) {
+			/* not possible for UTF-8. */
+			throw new RuntimeException("The JVM does not support UTF-8 encoding!", uee1);
+		}
 	}
 
 	/**
@@ -233,7 +227,11 @@ public class ParserFilter implements Filter {
 	 *            The part to render
 	 */
 	private void render(Writer writer, SonePart sonePart) {
-		renderLink(writer, "viewSone.html?sone=" + sonePart.getSone().getId(), SoneAccessor.getNiceName(sonePart.getSone()), SoneAccessor.getNiceName(sonePart.getSone()), "in-sone");
+		if ((sonePart.getSone() != null) && (sonePart.getSone().getName() != null)) {
+			renderLink(writer, "viewSone.html?sone=" + sonePart.getSone().getId(), SoneAccessor.getNiceName(sonePart.getSone()), SoneAccessor.getNiceName(sonePart.getSone()), "in-sone");
+		} else {
+			renderLink(writer, "/WebOfTrust/ShowIdentity?id=" + sonePart.getSone().getId(), sonePart.getSone().getId(), sonePart.getSone().getId(), "in-sone");
+		}
 	}
 
 	/**
@@ -245,7 +243,24 @@ public class ParserFilter implements Filter {
 	 *            The part to render
 	 */
 	private void render(Writer writer, PostPart postPart) {
-		renderLink(writer, "viewPost.html?post=" + postPart.getPost().getId(), getExcerpt(postPart.getPost().getText(), 20), SoneAccessor.getNiceName(postPart.getPost().getSone()), "in-sone");
+		SoneTextParser parser = new SoneTextParser(core, core);
+		SoneTextParserContext parserContext = new SoneTextParserContext(postPart.getPost().getSone());
+		Iterable<Part> parts = parser.parse(postPart.getPost().getText(), parserContext);
+		StringBuilder excerpt = new StringBuilder();
+		for (Part part : parts) {
+			excerpt.append(part.getText());
+			if (excerpt.length() > 20) {
+				int lastSpace = excerpt.lastIndexOf(" ", 20);
+				if (lastSpace > -1) {
+					excerpt.setLength(lastSpace);
+				} else {
+					excerpt.setLength(20);
+				}
+				excerpt.append("…");
+				break;
+			}
+		}
+		renderLink(writer, "viewPost.html?post=" + postPart.getPost().getId(), excerpt.toString(), SoneAccessor.getNiceName(postPart.getPost().getSone()), "in-sone");
 	}
 
 	/**
@@ -271,26 +286,4 @@ public class ParserFilter implements Filter {
 		linkTemplate.render(templateContext, writer);
 	}
 
-	//
-	// STATIC METHODS
-	//
-
-	/**
-	 * Returns up to {@code length} characters from the given text, appending
-	 * “…” if the text is longer.
-	 *
-	 * @param text
-	 *            The text to get an excerpt from
-	 * @param length
-	 *            The maximum length of the excerpt (without the ellipsis)
-	 * @return The excerpt of the text
-	 */
-	private static String getExcerpt(String text, int length) {
-		String filteredText = text.replaceAll("(\r\n)+", "\r\n").replaceAll("\n+", "\n").replace("\r\n", " ").replace('\n', ' ');
-		if (filteredText.length() > length) {
-			return filteredText.substring(0, length) + "…";
-		}
-		return filteredText;
-	}
-
 }