Index: lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibLearnerServlet.java =================================================================== RCS file: /usr/local/cvsroot/lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/Attic/ShibLearnerServlet.java,v diff -u -r1.1.2.1 -r1.1.2.2 --- lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibLearnerServlet.java 27 Aug 2007 05:46:08 -0000 1.1.2.1 +++ lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibLearnerServlet.java 7 Nov 2007 06:05:56 -0000 1.1.2.2 @@ -73,7 +73,7 @@ // add this shib user to lesson if they have learner role and are not already a member // TODO check this shib user's fed server id and org id against this server's shared lessons in lams_fed_lesson User user = null; - if (roles != null && roles.indexOf(Role.LEARNER)>0) { + if (roles != null && roles.indexOf(Role.LEARNER)>=0) { try { String localUsername = fedService.getLocalUsername(username, originServerURL); user = service.getUserByLogin(localUsername); Index: lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibMonitorServlet.java =================================================================== RCS file: /usr/local/cvsroot/lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/Attic/ShibMonitorServlet.java,v diff -u -r1.1.2.1 -r1.1.2.2 --- lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibMonitorServlet.java 27 Aug 2007 05:46:08 -0000 1.1.2.1 +++ lams_central/src/java/org/lamsfoundation/lams/web/shibboleth/ShibMonitorServlet.java 7 Nov 2007 06:05:56 -0000 1.1.2.2 @@ -75,7 +75,7 @@ // add this shib user to lesson as staff if they have monitor role and are not already staff // TODO check this shib user's fed server id and org id against this server's shared lessons in lams_fed_lesson User user = null; - if (roles != null && roles.indexOf(Role.MONITOR)>0) { + if (roles != null && roles.indexOf(Role.MONITOR)>=0) { try { String localUsername = fedService.getLocalUsername(username, originServerURL); user = service.getUserByLogin(localUsername);