From 280b46029bbb771136949f10f775123db012c291 Mon Sep 17 00:00:00 2001
From: "rogan.dawes" <rogan.dawes@4033779f-a91e-0410-96ef-6bf7bf53c507>
Date: Thu, 10 Jan 2008 20:43:48 +0000
Subject: [PATCH] Make stage completion messages appear in a logical order

git-svn-id: http://webgoat.googlecode.com/svn/trunk@270 4033779f-a91e-0410-96ef-6bf7bf53c507
---
 .../lessons/RoleBasedAccessControl/RoleBasedAccessControl.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ webgoat/main/project/JavaSource/org/owasp/webgoat/lessons/RoleBasedAccessControl/RoleBasedAccessControl.java b/ webgoat/main/project/JavaSource/org/owasp/webgoat/lessons/RoleBasedAccessControl/RoleBasedAccessControl.java
index ad3284814..cdc541c22 100644
--- a/ webgoat/main/project/JavaSource/org/owasp/webgoat/lessons/RoleBasedAccessControl/RoleBasedAccessControl.java	
+++ b/ webgoat/main/project/JavaSource/org/owasp/webgoat/lessons/RoleBasedAccessControl/RoleBasedAccessControl.java	
@@ -233,6 +233,8 @@ public class RoleBasedAccessControl extends GoatHillsFinancial
 	}
 	catch (UnauthorizedException ue2)
 	{
+	    s.setMessage("You are not authorized to perform this function");
+
 		// Update lesson status if necessary.
 		String stage = getStage(s);
 		if (STAGE2.equals(stage))
@@ -273,7 +275,6 @@ public class RoleBasedAccessControl extends GoatHillsFinancial
 			}
 		}
 		
-	    s.setMessage("You are not authorized to perform this function");
 	    System.out.println("Authorization failure");
 	    setCurrentAction(s, ERROR_ACTION);
 	    ue2.printStackTrace();