Author: snoopdave
Date: Sat Aug 3 14:35:46 2013
New Revision: 1510000
URL: http://svn.apache.org/r1510000
Log:
Apply proper escaping.
Modified:
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsFeedModel.java
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsModel.java
roller/trunk/app/src/main/webapp/WEB-INF/velocity/feeds.vm
Modified:
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsFeedModel.java
URL:
http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsFeedModel.java?rev=1510000&r1=1509999&r2=1510000&view=diff
==============================================================================
---
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsFeedModel.java
(original)
+++
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsFeedModel.java
Sat Aug 3 14:35:46 2013
@@ -26,6 +26,7 @@ import java.util.List;
import java.util.Map;
import java.util.Set;
import java.util.TreeSet;
+import org.apache.commons.lang.StringEscapeUtils;
import org.apache.commons.lang.StringUtils;
import org.apache.commons.logging.Log;
@@ -51,6 +52,7 @@ import org.apache.roller.weblogger.ui.re
import org.apache.roller.weblogger.ui.rendering.pagers.SearchResultsFeedPager;
import org.apache.roller.weblogger.ui.rendering.util.WeblogFeedRequest;
import org.apache.roller.weblogger.ui.rendering.util.WeblogRequest;
+import org.apache.roller.weblogger.util.Utilities;
/**
* Extends normal page renderer model to represent search results for Atom
@@ -254,7 +256,9 @@ public class SearchResultsFeedModel impl
}
public String getTerm() {
- return (feedRequest.getTerm() == null) ? "" :
feedRequest.getTerm();
+ String query =feedRequest.getTerm() ;
+ return (query == null)
+ ? "" :
StringEscapeUtils.escapeXml(Utilities.escapeHTML(query));
}
public int getHits() {
Modified:
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsModel.java
URL:
http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsModel.java?rev=1510000&r1=1509999&r2=1510000&view=diff
==============================================================================
---
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsModel.java
(original)
+++
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/model/SearchResultsModel.java
Sat Aug 3 14:35:46 2013
@@ -259,8 +259,8 @@ public class SearchResultsModel extends
public String getTerm() {
String query = searchRequest.getQuery();
- return (query == null) ? "" :
StringEscapeUtils.escapeXml(Utilities
- .escapeHTML(query));
+ return (query == null)
+ ? "" :
StringEscapeUtils.escapeXml(Utilities.escapeHTML(query));
}
public String getRawTerm() {
Modified: roller/trunk/app/src/main/webapp/WEB-INF/velocity/feeds.vm
URL:
http://svn.apache.org/viewvc/roller/trunk/app/src/main/webapp/WEB-INF/velocity/feeds.vm?rev=1510000&r1=1509999&r2=1510000&view=diff
==============================================================================
--- roller/trunk/app/src/main/webapp/WEB-INF/velocity/feeds.vm (original)
+++ roller/trunk/app/src/main/webapp/WEB-INF/velocity/feeds.vm Sat Aug 3
14:35:46 2013
@@ -118,7 +118,7 @@
<entry>
<id>$file.permalink</id>
<title type="html">$utils.escapeXML($file.name)</title>
- <author><name>$file.creator.screenName</name></author>
+
<author><name>$utils.escapeXML($utils.removeHTML($file.creator.screenName))</name></author>
<link rel="alternate" type="text/html" href="$file.permalink"/>
<published>$utils.formatIso8601Date($file.dateUploaded)</published>
<updated>$utils.formatIso8601Date($file.lastUpdated)</updated>