You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by pb...@apache.org on 2009/12/26 01:48:33 UTC
svn commit: r893915 - in
/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html:
BaseHandlerTag.java BaseTag.java FormTag.java MultiboxTag.java
OptionTag.java OptionsCollectionTag.java OptionsTag.java
Author: pbenedict
Date: Sat Dec 26 00:48:32 2009
New Revision: 893915
URL: http://svn.apache.org/viewvc?rev=893915&view=rev
Log:
STR-3191: Undo r815660 against 1.3
Modified:
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseHandlerTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/MultiboxTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsCollectionTag.java
struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsTag.java
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseHandlerTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseHandlerTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseHandlerTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseHandlerTag.java Sat Dec 26 00:48:32 2009
@@ -913,7 +913,7 @@
protected void prepareIndex(StringBuffer handlers, String name)
throws JspException {
if (name != null) {
- handlers.append(TagUtils.getInstance().filter(name));
+ handlers.append(name);
}
handlers.append("[");
@@ -1156,9 +1156,9 @@
Object value) {
if (value != null) {
handlers.append(" ");
- handlers.append(TagUtils.getInstance().filter(name));
+ handlers.append(name);
handlers.append("=\"");
- handlers.append(TagUtils.getInstance().filter(value.toString()));
+ handlers.append(value);
handlers.append("\"");
}
}
@@ -1221,5 +1221,4 @@
property, beanName));
}
}
-
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/BaseTag.java Sat Dec 26 00:48:32 2009
@@ -177,20 +177,18 @@
.getContextPath());
contextBase.append("/");
- tag.append(TagUtils.getInstance().filter(
- RequestUtils.createServerUriStringBuffer(scheme,
- serverName, port, contextBase.toString()).toString()));
+ tag.append(RequestUtils.createServerUriStringBuffer(scheme,
+ serverName, port, contextBase.toString()).toString());
} else {
- tag.append(TagUtils.getInstance().filter(
- RequestUtils.createServerUriStringBuffer(scheme,
- serverName, port, uri).toString()));
+ tag.append(RequestUtils.createServerUriStringBuffer(scheme,
+ serverName, port, uri).toString());
}
tag.append("\"");
if (this.target != null) {
tag.append(" target=\"");
- tag.append(TagUtils.getInstance().filter(this.target));
+ tag.append(this.target);
tag.append("\"");
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/FormTag.java Sat Dec 26 00:48:32 2009
@@ -608,10 +608,9 @@
(HttpServletResponse) this.pageContext.getResponse();
results.append(" action=\"");
- results.append(TagUtils.getInstance().filter(
- response.encodeURL(
+ results.append(response.encodeURL(
TagUtils.getInstance().getActionMappingURL(calcAction,
- this.pageContext))));
+ this.pageContext)));
results.append("\"");
}
@@ -642,7 +641,7 @@
results.append("<div><input type=\"hidden\" name=\"");
results.append(Constants.TOKEN_KEY);
results.append("\" value=\"");
- results.append(TagUtils.getInstance().filter(token));
+ results.append(token);
if (this.isXhtml()) {
results.append("\" />");
@@ -664,9 +663,9 @@
String value) {
if (value != null) {
results.append(" ");
- results.append(TagUtils.getInstance().filter(attribute));
+ results.append(attribute);
results.append("=\"");
- results.append(TagUtils.getInstance().filter(value));
+ results.append(value);
results.append("\"");
}
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/MultiboxTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/MultiboxTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/MultiboxTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/MultiboxTag.java Sat Dec 26 00:48:32 2009
@@ -200,7 +200,7 @@
throw e;
}
- prepareAttribute(results, "value", value);
+ prepareAttribute(results, "value", TagUtils.getInstance().filter(value));
return value;
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionTag.java Sat Dec 26 00:48:32 2009
@@ -304,31 +304,31 @@
if (style != null) {
results.append(" style=\"");
- results.append(TagUtils.getInstance().filter(style));
+ results.append(style);
results.append("\"");
}
if (styleId != null) {
results.append(" id=\"");
- results.append(TagUtils.getInstance().filter(styleId));
+ results.append(styleId);
results.append("\"");
}
if (styleClass != null) {
results.append(" class=\"");
- results.append(TagUtils.getInstance().filter(styleClass));
+ results.append(styleClass);
results.append("\"");
}
if (dir != null) {
results.append(" dir=\"");
- results.append(TagUtils.getInstance().filter(dir));
+ results.append(dir);
results.append("\"");
}
if (lang != null) {
results.append(" lang=\"");
- results.append(TagUtils.getInstance().filter(lang));
+ results.append(lang);
results.append("\"");
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsCollectionTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsCollectionTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsCollectionTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsCollectionTag.java Sat Dec 26 00:48:32 2009
@@ -323,13 +323,13 @@
if (style != null) {
sb.append(" style=\"");
- sb.append(TagUtils.getInstance().filter(style));
+ sb.append(style);
sb.append("\"");
}
if (styleClass != null) {
sb.append(" class=\"");
- sb.append(TagUtils.getInstance().filter(styleClass));
+ sb.append(styleClass);
sb.append("\"");
}
Modified: struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsTag.java
URL: http://svn.apache.org/viewvc/struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsTag.java?rev=893915&r1=893914&r2=893915&view=diff
==============================================================================
--- struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsTag.java (original)
+++ struts/struts1/branches/STRUTS_1_3_BRANCH/taglib/src/main/java/org/apache/struts/taglib/html/OptionsTag.java Sat Dec 26 00:48:32 2009
@@ -332,13 +332,13 @@
if (style != null) {
sb.append(" style=\"");
- sb.append(TagUtils.getInstance().filter(style));
+ sb.append(style);
sb.append("\"");
}
if (styleClass != null) {
sb.append(" class=\"");
- sb.append(TagUtils.getInstance().filter(styleClass));
+ sb.append(styleClass);
sb.append("\"");
}