[Springframework-cvs] spring/src/org/springframework/web/servlet/handler AbstractUrlHandlerMapping.java, 1.44, 1.45

[Arjen J.W. P. <po...@us...>]

Update of /cvsroot/springframework/spring/src/org/springframework/web/servlet/handler
In directory fdv4jf1.ch3.sourceforge.com:/tmp/cvs-serv7622/src/org/springframework/web/servlet/handler

Modified Files:
	AbstractUrlHandlerMapping.java 
Log Message:
Improved logging of MVC:
- Added log messages for successful handling of requests
- Reduced some debug statements to trace

Index: AbstractUrlHandlerMapping.java
===================================================================
RCS file: /cvsroot/springframework/spring/src/org/springframework/web/servlet/handler/AbstractUrlHandlerMapping.java,v
retrieving revision 1.44
retrieving revision 1.45
diff -C2 -d -r1.44 -r1.45
*** AbstractUrlHandlerMapping.java	19 Feb 2008 21:59:47 -0000	1.44
--- AbstractUrlHandlerMapping.java	21 Oct 2008 10:15:42 -0000	1.45
***************
*** 21,25 ****
  import java.util.LinkedHashMap;
  import java.util.Map;
- 
  import javax.servlet.http.HttpServletRequest;
  import javax.servlet.http.HttpServletResponse;
--- 21,24 ----
***************
*** 158,164 ****
  	protected Object getHandlerInternal(HttpServletRequest request) throws Exception {
  		String lookupPath = this.urlPathHelper.getLookupPathForRequest(request);
- 		if (logger.isDebugEnabled()) {
- 			logger.debug("Looking up handler for [" + lookupPath + "]");
- 		}
  		Object handler = lookupHandler(lookupPath, request);
  		if (handler == null) {
--- 157,160 ----
***************
*** 177,180 ****
--- 173,182 ----
  			}
  		}
+ 		if (handler != null && logger.isDebugEnabled()) {
+ 			logger.debug("Mapping [" + lookupPath + "] to handler '" + handler + "'");
+ 		}
+ 		else if (handler == null && logger.isTraceEnabled()) {
+ 			logger.trace("No handler mapping found for [" + lookupPath + "]");
+ 		}
  		return handler;
  	}