From 47d874250304063db92918cdf627cfbe1655842a Mon Sep 17 00:00:00 2001 From: briancaruso Date: Mon, 25 Jul 2011 14:54:03 +0000 Subject: [PATCH] checking for null RequestDispatcher in PageRoutingFitler --- .../mannlib/vitro/webapp/filters/PageRoutingFilter.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/filters/PageRoutingFilter.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/filters/PageRoutingFilter.java index 537e45b38..d4b9be61c 100644 --- a/webapp/src/edu/cornell/mannlib/vitro/webapp/filters/PageRoutingFilter.java +++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/filters/PageRoutingFilter.java @@ -81,7 +81,13 @@ public class PageRoutingFilter implements Filter{ String controllerName = getControllerToForwardTo(req, pageUri, pageDao); log.debug(path + " is being forwarded to controller " + controllerName); - RequestDispatcher rd = filterConfig.getServletContext().getNamedDispatcher( controllerName ); + RequestDispatcher rd = filterConfig.getServletContext().getNamedDispatcher( controllerName ); + if( rd == null ){ + log.error(path + " should be forwarded to controller " + controllerName + " but there " + + "is no servlet named that defined for the web application in web.xml"); + //TODO: what should be done in this case? + } + rd.forward(req, response); }else if( "/".equals( path ) || path.isEmpty() ){ log.debug("url '" +path + "' is being forward to home controller" );