[LON-CAPA-cvs] cvs: loncom /interface lonnavmaps.pm

www lon-capa-cvs@mail.lon-capa.org
Sat, 19 Apr 2003 20:47:41 -0000


www		Sat Apr 19 16:47:41 2003 EDT

  Modified files:              
    /loncom/interface	lonnavmaps.pm 
  Log:
  Uses new environment variable for role-dependent determination if user is
  "advanced"
  
  
Index: loncom/interface/lonnavmaps.pm
diff -u loncom/interface/lonnavmaps.pm:1.176 loncom/interface/lonnavmaps.pm:1.177
--- loncom/interface/lonnavmaps.pm:1.176	Fri Apr 18 09:51:46 2003
+++ loncom/interface/lonnavmaps.pm	Sat Apr 19 16:47:41 2003
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Navigate Maps Handler
 #
-# $Id: lonnavmaps.pm,v 1.176 2003/04/18 13:51:46 www Exp $
+# $Id: lonnavmaps.pm,v 1.177 2003/04/19 20:47:41 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -343,14 +343,9 @@
 }
 
 # This puts a human-readable name on the ENV variable.
-# FIXME: This needs better logic: Who gets the advanced view of navmaps?
-#   As of 3-13-03, it's an open question. Guy doesn't want to check
-#   roles directly because it should be a check of capabilities for future
-#   role compatibity. There is no capability that matches this one for
-#   now, so this is done. (A hack for 1.0 might be to simply check roles
-#   anyhow.)
+
 sub advancedUser {
-    return $ENV{'user.adv'};
+    return $ENV{'request.role.adv'};
 }