--- loncom/interface/lonmenu.pm	2012/05/18 14:27:11	1.371
+++ loncom/interface/lonmenu.pm	2012/05/24 23:30:16	1.376
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Routines to control the menu
 #
-# $Id: lonmenu.pm,v 1.371 2012/05/18 14:27:11 raeburn Exp $
+# $Id: lonmenu.pm,v 1.376 2012/05/24 23:30:16 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -115,6 +115,18 @@ entries from mydesk.tab
 
 Same as primary_menu() but operates on @secondary_menu.
 
+=item create_submenu()
+
+Creates XHTML for unordered list of sub-menu items which belong to a 
+particular top-level menu item. Uses hover pseudo class in css to display
+dropdown list when mouse hovers over top-level item. Support for IE6 
+(no hover psuedo class) via LC_hoverable class for <li> tag for top-
+level item, which employs jQuery to handle behavior on mouseover.
+
+Inputs: 4 - (a) link and (b) target for anchor href in top level item,
+            (c) title for text wrapped by anchor tag in top level item.
+            (d) reference to array of arrays of sub-menu items.
+
 =item innerregister()
 
 This gets called in order to register a URL in the body of the document
@@ -225,50 +237,32 @@ sub primary_menu {
         
         my $title = $menuitem->[3];
         if (defined($primary_submenu{$title})) {
-            my ($link,$target,$numsub);
+            my ($link,$target);
             if ($menuitem->[0] ne '') {
                 $link = $menuitem->[0];
                 $target = '_top';
             } else {
                 $link = '#';
             }
+            my @primsub;
             if (ref($primary_submenu{$title}) eq 'ARRAY') {
-                $numsub = @{$primary_submenu{$title}};
-                if ($numsub) {
-                    $title =
-                        '<span class="LC_nobreak">'.$title.
-                        '<span class="LC_fontsize_small">'.
-                        '&#9660;</span></span>';
-                }
-            }
-            $menu .= '<li><a href="'.$link.'" target="'.$target.'">'.$title.'</a>';
-            if ($numsub) {
-                $menu .= '<ul>';
-                foreach my $item (@{$primary_submenu{$menuitem->[3]}}) {
-                    if (ref($item) eq 'ARRAY') {
-                        if ($item->[2] eq 'wishlist') {
-                            next unless ((&Apache::lonnet::allowed('bre',"/res/$env{'user.domain'}/")) ||
-                                         (&Apache::lonnet::allowed('bro',"/res/$env{'user.domain'}/")));
-                        } elsif ($item->[2] eq 'reqcrs') {
-                            next unless(&check_for_rcrs());
-                        } elsif (($item->[2] eq 'portfolio') ||
-                                 ($item->[2] eq 'blog')) {
-                            if (!&Apache::lonnet::usertools_access(
-                                    $env{'user.name'},
-                                    $env{'user.domain'},
-                                    $item->[2],undef,'tools')) {
-                                next;
-                            }
-                        }
-                        $menu .= '<li style="margin:0;padding:0">'.
-                                 '<a href="'.$item->[0].
-                                 '" style="padding:0 0 0 10px">'.
-                                 $item->[1].'</a></li>';
-                    }
+                foreach my $item (@{$primary_submenu{$title}}) {
+                    next if (($item->[2] eq 'wishlist') &&
+                             ((!&Apache::lonnet::allowed('bre',"/res/$env{'user.domain'}/")) &&
+                              (!&Apache::lonnet::allowed('bro',"/res/$env{'user.domain'}/"))));
+                    next if (($item->[2] eq 'reqcrs') && (!&check_for_rcrs()));
+                    next if ((($item->[2] eq 'portfolio') ||
+                             ($item->[2] eq 'blog')) &&
+                             (!&Apache::lonnet::usertools_access('','',$item->[2],
+                                                           undef,'tools')));
+                    push(@primsub,$item);
+                }
+                if (@primsub > 0) {
+                    $menu .= &create_submenu($link,$target,$title,\@primsub);
+                } elsif ($link) {
+                    $menu .= '<li><a href="'.$link.'" target="'.$target.'">'.$title.'</a></li>';
                 }
-                $menu .= '</ul>';
             }
-            $menu .= '</li>';
         } elsif ($$menuitem[3] eq 'Help') { # special treatment for helplink
             if ($public) {
                 my $origmail = $Apache::lonnet::perlvar{'lonSupportEMail'};
@@ -325,6 +319,7 @@ sub secondary_menu {
                                                ? "/$env{'request.course.sec'}"
                                                : '');
     my $canedit       = &Apache::lonnet::allowed('mdc', $env{'request.course.id'});
+    my $canviewroster = $env{'course.'.$env{'request.course.id'}.'.student_classlist_view'};
     my $canviewgrps   = &Apache::lonnet::allowed('vcg', $crs_sec); 
     my $canmodifyuser = &Apache::lonnet::allowed('cst', $crs_sec); 
     my $canviewwnew   = &Apache::lonnet::allowed('whn', $crs_sec); 
@@ -353,7 +348,7 @@ sub secondary_menu {
         next if    $$menuitem[4]   eq 'cst'
                 && !$canmodifyuser;
         next if    $$menuitem[4]   eq 'ncst'
-                && $canmodifyuser;
+                && ($canmodifyuser || !$canviewroster);
         next if    $$menuitem[4]   eq 'mgr'
                 && !$canmgr;
         next if    $$menuitem[4]   eq 'nmgr'
@@ -404,6 +399,33 @@ sub secondary_menu {
     return "<ul id=\"LC_secondary_menu\">$menu</ul>";
 }
 
+sub create_submenu {
+    my ($link,$target,$title,$submenu) = @_;
+    return unless (ref($submenu) eq 'ARRAY');
+    my $menu = '<li class="LC_hoverable">'.
+               '<a href="'.$link.'" target="'.$target.'">'.
+               '<span class="LC_nobreak">'.$title.
+               '<span class="LC_fontsize_small" style="font-weight:normal;">'.
+               ' &#9660;</span></span></a>'.
+               '<ul>';
+    my $count = 0;
+    my $numsub = scalar(@{$submenu});
+    foreach my $item (@{$submenu}) {
+        $count ++;
+        if (ref($item) eq 'ARRAY') {
+            my $borderbot;
+            if ($count == $numsub) {
+                $borderbot = 'border-bottom:1px solid black;';
+            }
+            $menu .= '<li style="margin:0;padding:0;'.
+                     $borderbot.'"><a href="'.$item->[0].'">'.
+                     $item->[1].'</a></li>';
+        }
+    }
+    $menu .= '</ul></li>';
+    return $menu;
+}
+
 sub innerregister {
     my ($forcereg,$bread_crumbs) = @_;
     my $const_space = ($env{'request.state'} eq 'construct');
@@ -724,7 +746,7 @@ ENDMENUITEMS
             if (&Apache::lonnet::allowed('bre', $env{'request.course.id'}) eq 'F') {
                 # wishlist is only available for users with access to resource-pool
                 $menuitems .= (<<ENDMENUITEMS);
-s&9&1&wishlist-link.png&Stored Links&wishlistlink[_2]&set_wishlistlink()&Save a link for this resource in in your personal Stored Links repository&&1
+s&9&1&wishlist-link.png&Stored Links&wishlistlink[_2]&set_wishlistlink()&Save a link for this resource in your personal Stored Links repository&&1
 ENDMENUITEMS
             }
 	}
@@ -1327,6 +1349,20 @@ function open_StoredLinks_Import(rat) {
    newWin.focus();
 }
 
+(function (\$) {
+  \$(document).ready(function () {
+    \$.single=function(a){return function(b){a[0]=b;return a}}(\$([1]));
+    /*\@cc_on
+      if (!window.XMLHttpRequest) {
+        \$('.LC_hoverable').each(function () {
+          this.attachEvent('onmouseenter', function (evt) { \$.single(evt.srcElement).addClass('hover'); });
+          this.attachEvent('onmouseleave', function (evt) { \$.single(evt.srcElement).removeClass('hover'); });
+        });
+      }
+    \@*/
+  });
+}(jQuery));
+
 ENDUTILITY
 }