[Treebase-guts] SF.net SVN: treebase:[437] trunk/treebase-web/src/main/java/org/cipres/ treebase/web/controllers/BaseFormController.java

[<rv...@us...>]

Revision: 437
          http://treebase.svn.sourceforge.net/treebase/?rev=437&view=rev
Author:   rvos
Date:     2010-01-14 14:56:33 +0000 (Thu, 14 Jan 2010)

Log Message:
-----------
Added more logging messages.

Modified Paths:
--------------
    trunk/treebase-web/src/main/java/org/cipres/treebase/web/controllers/BaseFormController.java

Modified: trunk/treebase-web/src/main/java/org/cipres/treebase/web/controllers/BaseFormController.java
===================================================================
--- trunk/treebase-web/src/main/java/org/cipres/treebase/web/controllers/BaseFormController.java	2010-01-14 14:53:30 UTC (rev 436)
+++ trunk/treebase-web/src/main/java/org/cipres/treebase/web/controllers/BaseFormController.java	2010-01-14 14:56:33 UTC (rev 437)
@@ -165,9 +165,15 @@
 		HttpServletResponse pResponse,
 		BindException pBindException,
 		Map pMap) throws Exception {
-		if (isAuthorizationChecked() || isReviewerAccessGranted(pRequest)) {
+		if ( isAuthorizationChecked() ) {
+			LOGGER.info("returning supplied ModelAndView because access was granted explicitly");
 			return super.showForm(pRequest, pResponse, pBindException, pMap);
+		}
+		if (isReviewerAccessGranted(pRequest)) {
+			LOGGER.info("returning supplied ModelAndView because reviewer access was granted");
+			return super.showForm(pRequest, pResponse, pBindException, pMap);
 		} else {
+			LOGGER.info("returning AUTHORIZATION_VIOLATION_VIEW");
 			return new ModelAndView(AUTHORIZATION_VIOLATION_VIEW);
 		}
 	}


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.