diff --git a/apps/showcase/src/test/java/it/org/apache/struts2/showcase/StaticContentTest.java b/apps/showcase/src/test/java/it/org/apache/struts2/showcase/StaticContentTest.java index 8e274dd08..06057f9f1 100644 --- a/apps/showcase/src/test/java/it/org/apache/struts2/showcase/StaticContentTest.java +++ b/apps/showcase/src/test/java/it/org/apache/struts2/showcase/StaticContentTest.java @@ -40,7 +40,7 @@ public class StaticContentTest { public void testInvalidRersources2() throws Exception { try (final WebClient webClient = new WebClient()) { try { - webClient.getPage(ParameterUtils.getBaseUrl() + "/struts/..%252f"); + webClient.getPage(ParameterUtils.getBaseUrl() + "/static/..%252f"); Assert.fail("Previous request should have failed"); } catch (FailingHttpStatusCodeException e) { } @@ -49,7 +49,7 @@ public class StaticContentTest { /*public void testInvalidRersources3() throws IOException { try { - beginAt("/struts/..%252f..%252f..%252fWEB-INF/classes/org/apache/struts2/showcase/action/EmployeeAction.class/"); + beginAt("/static/..%252f..%252f..%252fWEB-INF/classes/org/apache/struts2/showcase/action/EmployeeAction.class/"); fail("Previous request should have failed"); } catch (TestingEngineResponseException ex) { // ok diff --git a/bundles/admin/src/main/resources/osgi/admin/shell.ftl b/bundles/admin/src/main/resources/osgi/admin/shell.ftl index aa7d8a9f2..62d0cce6c 100644 --- a/bundles/admin/src/main/resources/osgi/admin/shell.ftl +++ b/bundles/admin/src/main/resources/osgi/admin/shell.ftl @@ -22,13 +22,13 @@ OSGi Console - " /> - " /> - " /> + " /> + " /> + " /> - - - + + +
diff --git a/bundles/admin/src/main/resources/osgi/admin/viewBundle.ftl b/bundles/admin/src/main/resources/osgi/admin/viewBundle.ftl index 64ebbb9ca..97f22a3d6 100644 --- a/bundles/admin/src/main/resources/osgi/admin/viewBundle.ftl +++ b/bundles/admin/src/main/resources/osgi/admin/viewBundle.ftl @@ -22,11 +22,11 @@ ${bundle.symbolicName!} - " /> - " /> + " /> + " /> - - + + - + + diff --git a/core/src/main/java/org/apache/struts2/StrutsConstants.java b/core/src/main/java/org/apache/struts2/StrutsConstants.java index e5ed0433d..8009eb45c 100644 --- a/core/src/main/java/org/apache/struts2/StrutsConstants.java +++ b/core/src/main/java/org/apache/struts2/StrutsConstants.java @@ -129,6 +129,9 @@ public final class StrutsConstants { /** Token to use to indicate start of theme to be expanded. */ public static final String STRUTS_UI_THEME_EXPANSION_TOKEN = "struts.ui.theme.expansion.token"; + /** A path to static content, by default and from historical point of view it's /static. */ + public static final String STRUTS_UI_STATIC_CONTENT_PATH = "struts.ui.staticContentPath"; + /** The maximize size of a multipart request (file upload) */ public static final String STRUTS_MULTIPART_MAXSIZE = "struts.multipart.maxSize"; diff --git a/core/src/main/java/org/apache/struts2/components/UIBean.java b/core/src/main/java/org/apache/struts2/components/UIBean.java index 68fff203a..2bafabc27 100644 --- a/core/src/main/java/org/apache/struts2/components/UIBean.java +++ b/core/src/main/java/org/apache/struts2/components/UIBean.java @@ -30,6 +30,7 @@ import org.apache.struts2.components.template.Template; import org.apache.struts2.components.template.TemplateEngine; import org.apache.struts2.components.template.TemplateEngineManager; import org.apache.struts2.components.template.TemplateRenderingContext; +import org.apache.struts2.dispatcher.StaticContentLoader; import org.apache.struts2.util.TextProviderHelper; import org.apache.struts2.views.annotations.StrutsTagAttribute; import org.apache.struts2.views.util.ContextUtil; @@ -317,7 +318,7 @@ import java.util.Map; * * tooltipIcon * String - * /struts/static/tooltip/tooltip.gif + * /static/tooltip/tooltip.gif * The url to the tooltip icon * * tooltipDelay @@ -506,6 +507,8 @@ public abstract class UIBean extends Component { protected String defaultTemplateDir; protected String defaultUITheme; protected String uiThemeExpansionToken; + protected String uiStaticContentPath; + protected TemplateEngineManager templateEngineManager; @Inject(StrutsConstants.STRUTS_UI_TEMPLATEDIR) @@ -523,6 +526,11 @@ public abstract class UIBean extends Component { this.uiThemeExpansionToken = uiThemeExpansionToken; } + @Inject(StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH) + public void setStaticContentPath(String uiStaticContentPath) { + this.uiStaticContentPath = StaticContentLoader.Validator.validateStaticContentPath(uiStaticContentPath); + } + @Inject public void setTemplateEngineManager(TemplateEngineManager mgr) { this.templateEngineManager = mgr; @@ -645,6 +653,8 @@ public abstract class UIBean extends Component { addParameter("themeExpansionToken", uiThemeExpansionToken); addParameter("expandTheme", uiThemeExpansionToken + theme); + addParameter("staticContentPath", findString(uiStaticContentPath)); + String name = null; String providedLabel = null; diff --git a/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java b/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java index 2842155cb..3c44fac2a 100644 --- a/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java +++ b/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java @@ -29,6 +29,7 @@ import java.util.regex.Pattern; import org.apache.commons.lang3.StringUtils; import org.apache.struts2.StrutsConstants; +import org.apache.struts2.dispatcher.StaticContentLoader; public class ConstantConfig { private Boolean devMode; @@ -143,6 +144,7 @@ public class ConstantConfig { private BeanConfig localizedTextProvider; private Boolean disallowProxyMemberAccess; private Integer ognlAutoGrowthCollectionLimit; + private String staticContentPath; protected String beanConfToString(BeanConfig beanConf) { return beanConf == null ? null : beanConf.getName(); @@ -273,6 +275,7 @@ public class ConstantConfig { map.put(StrutsConstants.STRUTS_LOCALIZED_TEXT_PROVIDER, beanConfToString(localizedTextProvider)); map.put(StrutsConstants.STRUTS_DISALLOW_PROXY_MEMBER_ACCESS, Objects.toString(disallowProxyMemberAccess, null)); map.put(StrutsConstants.STRUTS_OGNL_AUTO_GROWTH_COLLECTION_LIMIT, Objects.toString(ognlAutoGrowthCollectionLimit, null)); + map.put(StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, Objects.toString(staticContentPath, StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH)); return map; } @@ -1344,4 +1347,12 @@ public class ConstantConfig { public void setOgnlAutoGrowthCollectionLimit(Integer ognlAutoGrowthCollectionLimit) { this.ognlAutoGrowthCollectionLimit = ognlAutoGrowthCollectionLimit; } + + public String getStaticContentPath() { + return staticContentPath; + } + + public void setStaticContentPath(String staticContentPath) { + this.staticContentPath = StaticContentLoader.Validator.validateStaticContentPath(staticContentPath); + } } diff --git a/core/src/main/java/org/apache/struts2/dispatcher/DefaultStaticContentLoader.java b/core/src/main/java/org/apache/struts2/dispatcher/DefaultStaticContentLoader.java index 42f17160e..0ec528602 100644 --- a/core/src/main/java/org/apache/struts2/dispatcher/DefaultStaticContentLoader.java +++ b/core/src/main/java/org/apache/struts2/dispatcher/DefaultStaticContentLoader.java @@ -34,7 +34,12 @@ import java.io.OutputStream; import java.io.UnsupportedEncodingException; import java.net.URL; import java.net.URLDecoder; -import java.util.*; +import java.util.ArrayList; +import java.util.Calendar; +import java.util.Collections; +import java.util.LinkedList; +import java.util.List; +import java.util.StringTokenizer; /** *

@@ -43,8 +48,8 @@ import java.util.*; * *

* This class is used to serve common static content needed when using various parts of Struts, such as JavaScript - * files, CSS files, etc. It works by looking for requests to /struts/* (or /static/*), and then mapping the value after "/struts/" - * to common packages in Struts and, optionally, in your class path. By default, the following packages are + * files, CSS files, etc. It works by looking for requests to {@link #uiStaticContentPath}/* and then mapping the value + * after to common packages in Struts and, optionally, in your class path. By default, the following packages are * automatically searched: *

* @@ -55,7 +60,7 @@ import java.util.*; * * *

- * This means that you can simply request /struts/xhtml/styles.css and the XHTML UI theme's default stylesheet + * This means that you can simply request {@link #uiStaticContentPath}/xhtml/styles.css and the XHTML UI theme's default stylesheet * will be returned. Likewise, many of the AJAX UI components require various JavaScript files, which are found in the * org.apache.struts2.static package. If you wish to add additional packages to be searched, you can add a comma * separated (space, tab and new line will do as well) list in the filter init parameter named "packages". Be @@ -68,7 +73,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { /** * Provide a logging instance. */ - private Logger LOG = LogManager.getLogger(DefaultStaticContentLoader.class); + private final Logger LOG = LogManager.getLogger(DefaultStaticContentLoader.class); /** * Store set of path prefixes to use with static resources. @@ -80,6 +85,11 @@ public class DefaultStaticContentLoader implements StaticContentLoader { */ protected boolean serveStatic; + /** + * Store state of {@link StrutsConstants#STRUTS_UI_STATIC_CONTENT_PATH} setting. + */ + protected String uiStaticContentPath; + /** * Store state of StrutsConstants.STRUTS_SERVE_STATIC_BROWSER_CACHE setting. */ @@ -100,20 +110,23 @@ public class DefaultStaticContentLoader implements StaticContentLoader { /** * Modify state of StrutsConstants.STRUTS_SERVE_STATIC_CONTENT setting. * - * @param serveStaticContent - * New setting + * @param serveStaticContent New setting */ @Inject(StrutsConstants.STRUTS_SERVE_STATIC_CONTENT) public void setServeStaticContent(String serveStaticContent) { this.serveStatic = BooleanUtils.toBoolean(serveStaticContent); } + @Inject(StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH) + public void setStaticContentPath(String uiStaticContentPath) { + this.uiStaticContentPath = StaticContentLoader.Validator.validateStaticContentPath(uiStaticContentPath); + } + /** * Modify state of StrutsConstants.STRUTS_SERVE_STATIC_BROWSER_CACHE * setting. * - * @param serveStaticBrowserCache - * New setting + * @param serveStaticBrowserCache New setting */ @Inject(StrutsConstants.STRUTS_SERVE_STATIC_BROWSER_CACHE) public void setServeStaticBrowserCache(String serveStaticBrowserCache) { @@ -122,6 +135,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { /** * Modify state of StrutsConstants.STRUTS_I18N_ENCODING setting. + * * @param encoding New setting */ @Inject(StrutsConstants.STRUTS_I18N_ENCODING) @@ -164,8 +178,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { /** * Create a string array from a comma-delimited list of packages. * - * @param packages - * A comma-delimited String listing packages + * @param packages A comma-delimited String listing packages * @return A string array of packages */ protected List parse(String packages) { @@ -194,7 +207,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { * javax.servlet.http.HttpServletResponse) */ public void findStaticResource(String path, HttpServletRequest request, HttpServletResponse response) - throws IOException { + throws IOException { String name = cleanupPath(path); for (String pathPrefix : pathPrefixes) { URL resourceUrl = findResource(buildPath(name, pathPrefix)); @@ -225,7 +238,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { LOG.warn("Unable to send error response, code: {};", HttpServletResponse.SC_NOT_FOUND, e1); } catch (IllegalStateException ise) { // Log illegalstate instead of passing unrecoverable exception to calling thread - LOG.warn("Unable to send error response, code: {}; isCommited: {};", HttpServletResponse.SC_NOT_FOUND, response.isCommitted(), ise); + LOG.warn("Unable to send error response, code: {}; isCommitted: {};", HttpServletResponse.SC_NOT_FOUND, response.isCommitted(), ise); } } @@ -285,7 +298,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { * Look for a static resource in the classpath. * * @param path The resource path - * @return The inputstream of the resource + * @return The URL of the resource * @throws IOException If there is a problem locating the resource */ protected URL findResource(String path) throws IOException { @@ -293,7 +306,7 @@ public class DefaultStaticContentLoader implements StaticContentLoader { } /** - * @param name resource name + * @param name resource name * @param packagePrefix The package prefix to use to locate the resource * @return full path * @throws UnsupportedEncodingException If there is a encoding problem @@ -310,7 +323,6 @@ public class DefaultStaticContentLoader implements StaticContentLoader { } - /** * Determine the content type for the resource name. * @@ -342,12 +354,9 @@ public class DefaultStaticContentLoader implements StaticContentLoader { /** * Copy bytes from the input stream to the output stream. * - * @param input - * The input stream - * @param output - * The output stream - * @throws IOException - * If anything goes wrong + * @param input The input stream + * @param output The output stream + * @throws IOException If anything goes wrong */ protected void copy(InputStream input, OutputStream output) throws IOException { final byte[] buffer = new byte[4096]; @@ -359,15 +368,18 @@ public class DefaultStaticContentLoader implements StaticContentLoader { } public boolean canHandle(String resourcePath) { - return serveStatic && (resourcePath.startsWith("/struts/") || resourcePath.startsWith("/static/")); + return serveStatic && resourcePath.startsWith(uiStaticContentPath + "/"); } /** * @param path requested path - * @return path without leading "/struts" or "/static" + * @return path without leading {@link #uiStaticContentPath} */ protected String cleanupPath(String path) { - //path will start with "/struts" or "/static", remove them - return path.substring(7); + if (path.startsWith(uiStaticContentPath)) { + return path.substring(uiStaticContentPath.length()); + } else { + return path; + } } } diff --git a/core/src/main/java/org/apache/struts2/dispatcher/StaticContentLoader.java b/core/src/main/java/org/apache/struts2/dispatcher/StaticContentLoader.java index 7859702b1..f3a15e5ea 100644 --- a/core/src/main/java/org/apache/struts2/dispatcher/StaticContentLoader.java +++ b/core/src/main/java/org/apache/struts2/dispatcher/StaticContentLoader.java @@ -18,6 +18,10 @@ */ package org.apache.struts2.dispatcher; +import org.apache.commons.lang3.StringUtils; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; +import org.apache.struts2.StrutsConstants; import org.apache.struts2.config.StrutsBeanSelectionProvider; import javax.servlet.http.HttpServletRequest; @@ -35,16 +39,22 @@ import java.io.IOException; */ public interface StaticContentLoader { + /** + * Default path at which static content is served, can be changed + * by using {@link org.apache.struts2.StrutsConstants#STRUTS_UI_STATIC_CONTENT_PATH} + */ + String DEFAULT_STATIC_CONTENT_PATH = "/static"; + /** * @param path Requested resource path * @return true if this loader is able to load this type of resource, false otherwise */ - public boolean canHandle(String path); + boolean canHandle(String path); /** * @param filterConfig The filter configuration */ - public abstract void setHostConfig(HostConfig filterConfig); + void setHostConfig(HostConfig filterConfig); /** * Locate a static resource and copy directly to the response, setting the @@ -55,7 +65,39 @@ public interface StaticContentLoader { * @param response The response * @throws IOException If anything goes wrong */ - public abstract void findStaticResource(String path, HttpServletRequest request, HttpServletResponse response) - throws IOException; + void findStaticResource(String path, HttpServletRequest request, HttpServletResponse response) throws IOException; + class Validator { + + private static final Logger LOG = LogManager.getLogger(DefaultStaticContentLoader.class); + + public static String validateStaticContentPath(String uiStaticContentPath) { + if (StringUtils.isBlank(uiStaticContentPath)) { + LOG.warn("\"{}\" has been set to \"{}\", falling back into default value \"{}\"", + StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, + uiStaticContentPath, + StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH); + return StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH; + } else if ("/".equals(uiStaticContentPath)) { + LOG.warn("\"{}\" cannot be set to \"{}\", falling back into default value \"{}\"", + StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, + uiStaticContentPath, + StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH); + return StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH; + } else if (!uiStaticContentPath.startsWith("/")) { + LOG.warn("\"{}\" must start with \"/\", but has been set to \"{}\", prepending the missing \"/\"!", + StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, + uiStaticContentPath); + return "/" + uiStaticContentPath; + } else if (uiStaticContentPath.endsWith("/")) { + LOG.warn("\"{}\" must not end with \"/\", but has been set to \"{}\", removing all trailing \"/\"!", + StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, + uiStaticContentPath); + return StringUtils.stripEnd(uiStaticContentPath, "/"); + } else { + LOG.debug("\"{}\" has been set to \"{}\"", StrutsConstants.STRUTS_UI_STATIC_CONTENT_PATH, uiStaticContentPath); + return uiStaticContentPath; + } + } + } } diff --git a/core/src/main/resources/org/apache/struts2/default.properties b/core/src/main/resources/org/apache/struts2/default.properties index c61c5d4bc..df4069400 100644 --- a/core/src/main/resources/org/apache/struts2/default.properties +++ b/core/src/main/resources/org/apache/struts2/default.properties @@ -83,11 +83,15 @@ struts.multipart.maxSize=2097152 ### prior to a comma e.g. struts.action.extension=, or struts.action.extension=x,y,z,, struts.action.extension=action,, -### Used by FilterDispatcher +### Used by Dispatcher ### If true then Struts serves static content from inside its jar. -### If false then the static content must be available at /struts +### If false then the static content must be available at /static struts.serve.static=true +### A path from which a static content is served, it must start with "/" +### and it cannot end with "/" +struts.ui.staticContentPath=/static + ### Used by FilterDispatcher ### This is good for development where one wants changes to the static content be ### fetch on each request. diff --git a/core/src/main/resources/template/css_xhtml/form-validate.ftl b/core/src/main/resources/template/css_xhtml/form-validate.ftl index f1b205382..a45aa7189 100644 --- a/core/src/main/resources/template/css_xhtml/form-validate.ftl +++ b/core/src/main/resources/template/css_xhtml/form-validate.ftl @@ -19,7 +19,7 @@ */ --> <#if parameters.validate!false == true> - + <#if parameters.onsubmit??> ${tag.addParameter('onsubmit', "${parameters.onsubmit}; return validateForm_${parameters.id}();")} <#else> diff --git a/core/src/main/resources/template/css_xhtml/head.ftl b/core/src/main/resources/template/css_xhtml/head.ftl index d30a25aa8..7d493e8ee 100644 --- a/core/src/main/resources/template/css_xhtml/head.ftl +++ b/core/src/main/resources/template/css_xhtml/head.ftl @@ -18,5 +18,5 @@ * under the License. */ --> - rel="stylesheet" href="<@s.url value='/struts/css_xhtml/styles.css' includeParams='none' encode='false' />" type="text/css" /> + rel="stylesheet" href="<@s.url value='${parameters.staticContentPath}/css_xhtml/styles.css' includeParams='none' encode='false' />" type="text/css" /> <#include "/${parameters.templateDir}/simple/head.ftl" /> diff --git a/core/src/main/resources/template/simple/form-close-tooltips.ftl b/core/src/main/resources/template/simple/form-close-tooltips.ftl index ac3a6daa1..21e25a4fe 100644 --- a/core/src/main/resources/template/simple/form-close-tooltips.ftl +++ b/core/src/main/resources/template/simple/form-close-tooltips.ftl @@ -24,7 +24,7 @@ --><#t/> <#if (parameters.hasTooltip!false)><#t/> <#lt/> - <#lt/> - <#lt/>" <#include "/${parameters.templateDir}/simple/nonce.ftl" /> /> + <#lt/> + <#lt/>" <#include "/${parameters.templateDir}/simple/nonce.ftl" /> /> <#t/> \ No newline at end of file diff --git a/core/src/main/resources/template/simple/head.ftl b/core/src/main/resources/template/simple/head.ftl index 30047158c..359d7bf71 100644 --- a/core/src/main/resources/template/simple/head.ftl +++ b/core/src/main/resources/template/simple/head.ftl @@ -18,4 +18,4 @@ * under the License. */ --> - \ No newline at end of file + \ No newline at end of file diff --git a/core/src/main/resources/template/simple/inputtransferselect.ftl b/core/src/main/resources/template/simple/inputtransferselect.ftl index dae1a07c3..a9258dd6a 100644 --- a/core/src/main/resources/template/simple/inputtransferselect.ftl +++ b/core/src/main/resources/template/simple/inputtransferselect.ftl @@ -19,7 +19,7 @@ */ --> <#if !stack.findValue("#inputtransferselect_js_included")??><#t/> - + <#assign temporaryVariable = stack.setValue("#inputtransferselect_js_included", "true") /><#t/> <#t/> diff --git a/core/src/main/resources/template/simple/optiontransferselect.ftl b/core/src/main/resources/template/simple/optiontransferselect.ftl index 910dc6a95..e0d00eef0 100644 --- a/core/src/main/resources/template/simple/optiontransferselect.ftl +++ b/core/src/main/resources/template/simple/optiontransferselect.ftl @@ -19,7 +19,7 @@ */ --> <#if !stack.findValue("#optiontransferselect_js_included")??><#t/> - + <#assign temporaryVariable = stack.setValue("#optiontransferselect_js_included", "true") /><#t/> <#t/>
diff --git a/core/src/main/resources/template/simple/updownselect.ftl b/core/src/main/resources/template/simple/updownselect.ftl index 4f57b849a..f1185d3a5 100644 --- a/core/src/main/resources/template/simple/updownselect.ftl +++ b/core/src/main/resources/template/simple/updownselect.ftl @@ -19,7 +19,7 @@ */ --> <#if !stack.findValue("#optiontransferselect_js_included")??><#t/> - + <#assign temporaryVariable = stack.setValue("#optiontransferselect_js_included", "true") /><#t/> <#t/>
diff --git a/core/src/main/resources/template/xhtml/form-validate.ftl b/core/src/main/resources/template/xhtml/form-validate.ftl index 3a67f445a..33e8b5507 100644 --- a/core/src/main/resources/template/xhtml/form-validate.ftl +++ b/core/src/main/resources/template/xhtml/form-validate.ftl @@ -19,7 +19,7 @@ */ --> <#if parameters.validate!false == true> - + <#if parameters.onsubmit??> ${tag.addParameter('onsubmit', "${parameters.onsubmit}; return validateForm_${parameters.id?replace('[^a-zA-Z0-9_]', '_', 'r')}();")} <#else> diff --git a/core/src/main/resources/template/xhtml/head.ftl b/core/src/main/resources/template/xhtml/head.ftl index 48678f269..7b5bfe08d 100644 --- a/core/src/main/resources/template/xhtml/head.ftl +++ b/core/src/main/resources/template/xhtml/head.ftl @@ -18,5 +18,5 @@ * under the License. */ --> - /> + /> <#include "/${parameters.templateDir}/simple/head.ftl" /> diff --git a/core/src/main/resources/template/xhtml/tooltip.ftl b/core/src/main/resources/template/xhtml/tooltip.ftl index d0b35e726..faa69f69b 100644 --- a/core/src/main/resources/template/xhtml/tooltip.ftl +++ b/core/src/main/resources/template/xhtml/tooltip.ftl @@ -23,7 +23,7 @@ <#if parameters.tooltipIconPath??><#t/> src='<@s.url value="${parameters.tooltipIconPath}" includeParams="none" encode="false" />' <#else><#t/> - src='<@s.url value="/struts/tooltip.gif" includeParams="none" encode="false" />' + src='<@s.url value="${parameters.staticContentPath}/tooltip.gif" includeParams="none" encode="false" />' <#t/> <#if (parameters.jsTooltipEnabled!'false') == 'true'> onmouseover="domTT_activate(this, event, 'content', '<#outputformat 'JavaScript'>${parameters.tooltip}'<#t/> diff --git a/core/src/test/java/org/apache/struts2/components/UIBeanTest.java b/core/src/test/java/org/apache/struts2/components/UIBeanTest.java index 387d6b591..e4b0c6373 100644 --- a/core/src/test/java/org/apache/struts2/components/UIBeanTest.java +++ b/core/src/test/java/org/apache/struts2/components/UIBeanTest.java @@ -25,6 +25,8 @@ import org.apache.struts2.StrutsInternalTestCase; import org.apache.struts2.components.template.Template; import org.apache.struts2.components.template.TemplateEngine; import org.apache.struts2.components.template.TemplateEngineManager; +import org.apache.struts2.dispatcher.DefaultStaticContentLoader; +import org.apache.struts2.dispatcher.StaticContentLoader; import org.springframework.mock.web.MockHttpServletRequest; import org.springframework.mock.web.MockHttpServletResponse; @@ -343,4 +345,39 @@ public class UIBeanTest extends StrutsInternalTestCase { assertEquals(nonceVal, dblSelect.getParameters().get("nonce")); } + + public void testSetNullUiStaticContentPath() { + // given + ValueStack stack = ActionContext.getContext().getValueStack(); + MockHttpServletRequest req = new MockHttpServletRequest(); + MockHttpServletResponse res = new MockHttpServletResponse(); + + TextField field = new TextField(stack, req, res); + + // when + field.setStaticContentPath(null); + // then + assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, field.uiStaticContentPath); + + // when + field.setStaticContentPath(" "); + // then + assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, field.uiStaticContentPath); + + // when + field.setStaticContentPath("content"); + // then + assertEquals("/content", field.uiStaticContentPath); + + // when + field.setStaticContentPath("/content"); + // then + assertEquals("/content", field.uiStaticContentPath); + + // when + field.setStaticContentPath("/content/"); + // then + assertEquals("/content", field.uiStaticContentPath); + } + } diff --git a/core/src/test/java/org/apache/struts2/config/entities/ConstantConfigTest.java b/core/src/test/java/org/apache/struts2/config/entities/ConstantConfigTest.java index 01503dd42..a9efe5920 100644 --- a/core/src/test/java/org/apache/struts2/config/entities/ConstantConfigTest.java +++ b/core/src/test/java/org/apache/struts2/config/entities/ConstantConfigTest.java @@ -18,11 +18,17 @@ */ package org.apache.struts2.config.entities; +import com.opensymphony.xwork2.ActionContext; import com.opensymphony.xwork2.TestBean; import com.opensymphony.xwork2.inject.Container; +import com.opensymphony.xwork2.util.ValueStack; import org.apache.struts2.StrutsConstants; +import org.apache.struts2.components.TextField; +import org.apache.struts2.dispatcher.StaticContentLoader; import org.junit.Assert; import org.junit.Test; +import org.springframework.mock.web.MockHttpServletRequest; +import org.springframework.mock.web.MockHttpServletResponse; import java.util.ArrayList; import java.util.Arrays; @@ -109,4 +115,30 @@ public class ConstantConfigTest { map.get(StrutsConstants.STRUTS_DEV_MODE_EXCLUDED_CLASSES)); } + @Test + public void testSettingStaticContentPath() { + // given + ConstantConfig config = new ConstantConfig(); + + // when + config.setStaticContentPath(null); + // then + Assert.assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, config.getStaticContentPath()); + + // when + config.setStaticContentPath(" "); + // then + Assert.assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, config.getStaticContentPath()); + + // when + config.setStaticContentPath("content"); + // then + Assert.assertEquals("/content", config.getStaticContentPath()); + + // when + config.setStaticContentPath("/content"); + // then + Assert.assertEquals("/content", config.getStaticContentPath()); + } + } diff --git a/core/src/test/java/org/apache/struts2/dispatcher/DefaultStaticContentLoaderTest.java b/core/src/test/java/org/apache/struts2/dispatcher/DefaultStaticContentLoaderTest.java index 8f8eda1eb..b472ce5c9 100644 --- a/core/src/test/java/org/apache/struts2/dispatcher/DefaultStaticContentLoaderTest.java +++ b/core/src/test/java/org/apache/struts2/dispatcher/DefaultStaticContentLoaderTest.java @@ -30,13 +30,12 @@ import static org.easymock.EasyMock.expectLastCall; import static org.easymock.EasyMock.replay; public class DefaultStaticContentLoaderTest extends StrutsInternalTestCase { + private HttpServletRequest requestMock; private HttpServletResponse responseMock; - private HostConfig hostConfigMock; private DefaultStaticContentLoader defaultStaticContentLoader; - public void testParsePackages() throws Exception { - + public void testParsePackages() { DefaultStaticContentLoader filterDispatcher = new DefaultStaticContentLoader(); List result1 = filterDispatcher.parse("foo.bar.package1 foo.bar.package2 foo.bar.package3"); List result2 = filterDispatcher.parse("foo.bar.package1\tfoo.bar.package2\tfoo.bar.package3"); @@ -102,14 +101,46 @@ public class DefaultStaticContentLoaderTest extends StrutsInternalTestCase { } } - protected void setUp() { - requestMock = (HttpServletRequest) createMock(HttpServletRequest.class); - responseMock = (HttpServletResponse) createMock(HttpServletResponse.class); - hostConfigMock = (HostConfig) createMock(HostConfig.class); + public void testStaticContentPath() { + // given + DefaultStaticContentLoader loader = new DefaultStaticContentLoader(); + + // when + loader.setStaticContentPath(null); + // then + assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, loader.uiStaticContentPath); + + // when + loader.setStaticContentPath(" "); + // then + assertEquals(StaticContentLoader.DEFAULT_STATIC_CONTENT_PATH, loader.uiStaticContentPath); + + // when + loader.setStaticContentPath("content"); + // then + assertEquals("/content", loader.uiStaticContentPath); + + // when + loader.setStaticContentPath("/content"); + // then + assertEquals("/content", loader.uiStaticContentPath); + + // when + loader.setStaticContentPath("/content/"); + // then + assertEquals("/content", loader.uiStaticContentPath); + } + + protected void setUp() throws Exception { + super.setUp(); + requestMock = createMock(HttpServletRequest.class); + responseMock = createMock(HttpServletResponse.class); + HostConfig hostConfigMock = createMock(HostConfig.class); expect(hostConfigMock.getInitParameter("packages")).andStubReturn(null); expect(hostConfigMock.getInitParameter("loggerFactory")).andStubReturn(null); defaultStaticContentLoader = new DefaultStaticContentLoader(); defaultStaticContentLoader.setHostConfig(hostConfigMock); defaultStaticContentLoader.setEncoding("UTF-8"); + defaultStaticContentLoader.setStaticContentPath("/static"); } } diff --git a/core/src/test/java/org/apache/struts2/dispatcher/StaticContentLoaderTest.java b/core/src/test/java/org/apache/struts2/dispatcher/StaticContentLoaderTest.java index c99127519..fbcf4e62b 100644 --- a/core/src/test/java/org/apache/struts2/dispatcher/StaticContentLoaderTest.java +++ b/core/src/test/java/org/apache/struts2/dispatcher/StaticContentLoaderTest.java @@ -39,7 +39,7 @@ public class StaticContentLoaderTest extends TestCase { private MockHttpServletResponse res; public void testCantHandleWithoutServingStatic() { - StaticContentLoader contentLoader = new DefaultStaticContentLoader(); + DefaultStaticContentLoader contentLoader = new DefaultStaticContentLoader(); assertFalse(contentLoader.canHandle("/static/test1.css")); assertFalse(contentLoader.canHandle("/struts/test1.css")); @@ -49,14 +49,15 @@ public class StaticContentLoaderTest extends TestCase { public void testCanHandle() { DefaultStaticContentLoader contentLoader = new DefaultStaticContentLoader(); contentLoader.setServeStaticContent("true"); + contentLoader.setStaticContentPath("/static"); assertTrue(contentLoader.canHandle("/static/test1.css")); - assertTrue(contentLoader.canHandle("/struts/test1.css")); + assertFalse(contentLoader.canHandle("/struts/test1.css")); assertFalse(contentLoader.canHandle("test1.css")); } - public void testValidRersources() throws IOException { - contentLoader.findStaticResource("/struts/resource.css", req, res); + public void testValidResources() throws IOException { + contentLoader.findStaticResource("/static/resource.css", req, res); assertTrue(res.getContentAsString().contains("color: red;")); } @@ -67,25 +68,25 @@ public class StaticContentLoaderTest extends TestCase { } public void testInvalidRersources2() throws IOException { - contentLoader.findStaticResource("/struts/..", req, res); + contentLoader.findStaticResource("/static/..", req, res); assertEquals(HttpServletResponse.SC_NOT_FOUND, res.getStatus()); assertEquals(0, res.getContentLength()); } public void testInvalidRersources3() throws IOException { - contentLoader.findStaticResource("/struts/../othertest.properties", req, res); + contentLoader.findStaticResource("/static/../othertest.properties", req, res); assertEquals(HttpServletResponse.SC_NOT_FOUND, res.getStatus()); assertEquals(0, res.getContentLength()); } public void testInvalidRersources4() throws IOException { - contentLoader.findStaticResource("/struts/..%252f", req, res); + contentLoader.findStaticResource("/static/..%252f", req, res); assertEquals(HttpServletResponse.SC_NOT_FOUND, res.getStatus()); assertEquals(0, res.getContentLength()); } public void testInvalidRersources5() throws IOException { - contentLoader.findStaticResource("/struts/..%252fothertest.properties", req, res); + contentLoader.findStaticResource("/static/..%252fothertest.properties", req, res); assertEquals(HttpServletResponse.SC_NOT_FOUND, res.getStatus()); assertEquals(0, res.getContentLength()); } @@ -95,6 +96,7 @@ public class StaticContentLoaderTest extends TestCase { super.setUp(); this.contentLoader = new DefaultStaticContentLoader(); + this.contentLoader.setStaticContentPath("/static"); MockServletContext servletContext = new MockServletContext(); req = new MockHttpServletRequest(servletContext); res = new MockHttpServletResponse(); diff --git a/core/src/test/java/org/apache/struts2/dispatcher/StrutsPrepareAndExecuteFilterIntegrationTest.java b/core/src/test/java/org/apache/struts2/dispatcher/StrutsPrepareAndExecuteFilterIntegrationTest.java index 7bc46ed74..a5419cd8b 100644 --- a/core/src/test/java/org/apache/struts2/dispatcher/StrutsPrepareAndExecuteFilterIntegrationTest.java +++ b/core/src/test/java/org/apache/struts2/dispatcher/StrutsPrepareAndExecuteFilterIntegrationTest.java @@ -179,7 +179,7 @@ public class StrutsPrepareAndExecuteFilterIntegrationTest extends TestCase { } }; - request.setRequestURI("/struts/utils.js"); + request.setRequestURI("/static/utils.js"); StrutsPrepareAndExecuteFilter filter = new StrutsPrepareAndExecuteFilter(); filter.init(filterConfig); filter.doFilter(request, response, filterChain); diff --git a/core/src/test/java/org/apache/struts2/dispatcher/TwoFilterIntegrationTest.java b/core/src/test/java/org/apache/struts2/dispatcher/TwoFilterIntegrationTest.java index b432a8d28..edc88ba17 100644 --- a/core/src/test/java/org/apache/struts2/dispatcher/TwoFilterIntegrationTest.java +++ b/core/src/test/java/org/apache/struts2/dispatcher/TwoFilterIntegrationTest.java @@ -79,7 +79,7 @@ public class TwoFilterIntegrationTest extends TestCase { } public void testStaticExecute() throws ServletException, IOException { - MockHttpServletResponse response = run("/struts/utils.js", filterPrepare, filterExecute, failFilter); + MockHttpServletResponse response = run("/static/utils.js", filterPrepare, filterExecute, failFilter); assertEquals(200, response.getStatus()); assertTrue(response.getContentAsString().contains("StrutsUtils")); } diff --git a/core/src/test/java/org/apache/struts2/views/jsp/ui/TooltipTest.java b/core/src/test/java/org/apache/struts2/views/jsp/ui/TooltipTest.java index 50d2bc5f5..8c8d8e1c8 100644 --- a/core/src/test/java/org/apache/struts2/views/jsp/ui/TooltipTest.java +++ b/core/src/test/java/org/apache/struts2/views/jsp/ui/TooltipTest.java @@ -52,7 +52,7 @@ public class TooltipTest extends AbstractUITagTest { tag.setTooltip("myTooltip"); tag.setTooltipConfig( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip.gif', " + "'tooltipDelay':'500', " + "'jsTooltipEnabled':'true' "+ "}" @@ -86,7 +86,7 @@ public class TooltipTest extends AbstractUITagTest { tag.setTooltip("myTooltip"); tag.setTooltipConfig( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip.gif', " + "'tooltipDelay':'500', " + "'jsTooltipEnabled':'false' "+ "}" @@ -119,7 +119,7 @@ public class TooltipTest extends AbstractUITagTest { //same parameters as the OGNL map configuration, output must be the same tag.setTooltip("myTooltip"); - tag.setTooltipIconPath("/struts/tooltip/myTooltip.gif"); + tag.setTooltipIconPath("/static/tooltip/myTooltip.gif"); tag.setTooltipDelay("500"); tag.setJavascriptTooltip("true"); @@ -142,7 +142,7 @@ public class TooltipTest extends AbstractUITagTest { formTag.setTooltipConfig( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip.gif', " + "'tooltipDelay':'500', " + "'jsTooltipEnabled':'true' "+ "}" @@ -174,7 +174,7 @@ public class TooltipTest extends AbstractUITagTest { // same parameters as the OGNL map configuration, output must be the same formTag.setTooltip("myTooltip"); - formTag.setTooltipIconPath("/struts/tooltip/myTooltip.gif"); + formTag.setTooltipIconPath("/static/tooltip/myTooltip.gif"); formTag.setTooltipDelay("500"); formTag.setJavascriptTooltip("true"); @@ -204,7 +204,7 @@ public class TooltipTest extends AbstractUITagTest { formTag.setTooltipConfig( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip.gif', " + "'tooltipDelay':'500', " + "'jsTooltipEnabled':'true' "+ "}" @@ -219,7 +219,7 @@ public class TooltipTest extends AbstractUITagTest { tag.setTooltip("myTooltip"); tag.setTooltipConfig( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip2.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip2.gif', " + "'tooltipDelay':'5000' " + "}" ); @@ -242,7 +242,7 @@ public class TooltipTest extends AbstractUITagTest { // same parameters as the OGNL map configuration, output must be the same formTag.setTooltip("myTooltip"); - formTag.setTooltipIconPath("/struts/tooltip/myTooltip.gif"); + formTag.setTooltipIconPath("/static/tooltip/myTooltip.gif"); formTag.setTooltipDelay("500"); formTag.setJavascriptTooltip("true"); @@ -255,7 +255,7 @@ public class TooltipTest extends AbstractUITagTest { //same parameters as the OGNL map configuration, output must be the same tag.setTooltip("myTooltip"); - tag.setTooltipIconPath("/struts/tooltip/myTooltip2.gif"); + tag.setTooltipIconPath("/static/tooltip/myTooltip2.gif"); tag.setTooltipDelay("5000"); tag.setJavascriptTooltip("true"); @@ -280,7 +280,7 @@ public class TooltipTest extends AbstractUITagTest { formParamTag.setName("tooltipConfig"); formParamTag.setValue( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip.gif', " + "'tooltipDelay':'500', " + "'jsTooltipEnabled':'true' "+ "}" @@ -298,7 +298,7 @@ public class TooltipTest extends AbstractUITagTest { textFieldParamTag.setName("tooltipConfig"); textFieldParamTag.setValue( "#{" + - "'tooltipIcon':'/struts/tooltip/myTooltip2.gif', " + + "'tooltipIcon':'/static/tooltip/myTooltip2.gif', " + "'tooltipDelay':'5000' "+ "}" ); @@ -330,7 +330,7 @@ public class TooltipTest extends AbstractUITagTest { formParamTag.setName("tooltipConfig"); StrutsMockBodyContent bodyContent = new StrutsMockBodyContent(new MockJspWriter()); bodyContent.setString( - "tooltipIcon=/struts/tooltip/myTooltip.gif| " + + "tooltipIcon=/static/tooltip/myTooltip.gif| " + "tooltipDelay=500| " + "jsTooltipEnabled=true " ); @@ -348,7 +348,7 @@ public class TooltipTest extends AbstractUITagTest { textFieldParamTag.setName("tooltipConfig"); StrutsMockBodyContent bodyContent2 = new StrutsMockBodyContent(new MockJspWriter()); bodyContent2.setString( - "tooltipIcon=/struts/tooltip/myTooltip2.gif| " + + "tooltipIcon=/static/tooltip/myTooltip2.gif| " + "tooltipDelay=5000 " ); textFieldParamTag.setBodyContent(bodyContent2); diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Formtag-11.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Formtag-11.txt index ca4358a65..7a4889cab 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Formtag-11.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Formtag-11.txt @@ -1,8 +1,8 @@ - +
-
+
-
+
-
+
-
+
-
+
- +
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-1.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-1.txt index 2ee370596..f16886524 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-1.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-1.txt @@ -1,7 +1,7 @@
- +
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-2.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-2.txt index 2713ceca6..359475c64 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-2.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-2.txt @@ -1,7 +1,7 @@
- +
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-3.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-3.txt index fee900366..46724ab70 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-3.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-3.txt @@ -1,7 +1,7 @@
- +
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-4.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-4.txt index 61668b22a..39d5b1b79 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-4.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-4.txt @@ -1,7 +1,7 @@
- +
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-5.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-5.txt index dc0057fdd..fb0c97934 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-5.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-5.txt @@ -1,7 +1,7 @@
- + @@ -16,6 +16,6 @@ - - + + diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-4.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-4.txt index f8aad53bd..dcdd0387a 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-4.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-4.txt @@ -4,7 +4,7 @@ diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/updownselecttag-1.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/updownselecttag-1.txt index 2f0974e93..3d3542779 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/updownselecttag-1.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/updownselecttag-1.txt @@ -1,5 +1,5 @@
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-6.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-6.txt index ade6e348a..68b99d066 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-6.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-6.txt @@ -1,7 +1,7 @@
- + @@ -15,5 +15,5 @@
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-7.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-7.txt index 9becafca5..270464f26 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-7.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/optiontransferselect-7.txt @@ -1,7 +1,7 @@
- + @@ -15,7 +15,7 @@
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-1.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-1.txt index 8394cb502..80b16d008 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-1.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-1.txt @@ -4,7 +4,7 @@ MyLabel:
- - + + diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-2.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-2.txt index 596854e73..88a298fa9 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-2.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-2.txt @@ -4,7 +4,7 @@ 		MyLabel:
- - \ No newline at end of file + + \ No newline at end of file diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-3.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-3.txt index e4885645a..2c1e49b45 100644 --- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-3.txt +++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/tooltip-3.txt @@ -4,7 +4,7 @@ 		MyLabel: MyLabel:myTooltip
- +
- +
- +
- +
- +
- +
- +
- +
- +
- +
- +