Sitemesh整合struts2使用多个装饰器(非/*)的BUG修正

最新推荐文章于 2014-05-21 10:37:05 发布

原创最新推荐文章于 2014-05-21 10:37:05 发布 · 2.8k 阅读

0 ·

CC 4.0 BY-SA版权

文章标签：

#struts #decorator #null #string #servlet #浏览器

本文描述了Sitemesh在与Struts框架整合时出现的问题：由于路径获取不一致导致装饰器配置失效。文章提供了修改建议，通过调整路径获取逻辑确保装饰器配置正确加载。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

Bug报告地址: http://jira.opensymphony.com/browse/SIM-127

Bug描述: 在和sitemesh和struts整合使用时, 由于request.getRequestURI()得到的是请求的路径(浏览器地址栏输入的路径), 而通过request.getPathInfo()得到的确实配置的实际页面的路径, 因此, 根据sitemesh的com.opensymphony.module.sitemesh.mapper.ConfigDecoratorMapper.getDecorator(HttpServletRequest, Page)这个方法的处理不能得到有效的装饰器配置.

涉及源代码:

public Decorator getDecorator(HttpServletRequest request, Page page) { String thisPath = request.getServletPath(); // getServletPath() returns null unless the mapping corresponds to a servlet if (thisPath == null) { String requestURI = request.getRequestURI(); if (request.getPathInfo() != null) { // strip the pathInfo from the requestURI thisPath = requestURI.substring(0, requestURI.indexOf(request.getPathInfo())); } else { thisPath = requestURI; } } String name = null; try { name = configLoader.getMappedName(thisPath); } catch (ServletException e) { e.printStackTrace(); } Decorator result = getNamedDecorator(request, name); return result == null ? super.getDecorator(request, page) : result; }

处理方案(此种解决方案经过测试在使用一级目录配置装饰器的时候工作良好.):

public Decorator getDecorator(HttpServletRequest request, Page page) { String thisPath = request.getServletPath(); // getServletPath() returns null unless the mapping corresponds to a // servlet if (thisPath == null) { String requestURI = request.getRequestURI(); if (request.getPathInfo() != null) { // strip the pathInfo from the requestURI thisPath = requestURI.substring(0, requestURI.indexOf(request .getPathInfo())); } else { thisPath = requestURI; } } else { thisPath = request.getRequestURI(); } String name = null; try { name = configLoader.getMappedName(thisPath); } catch (ServletException e) { e.printStackTrace(); } Decorator result = getNamedDecorator(request, name); return result == null ? super.getDecorator(request, page) : result; }