--- loncom/interface/lonsimplepage.pm 2006/05/09 14:38:10 1.39 +++ loncom/interface/lonsimplepage.pm 2006/06/30 18:06:47 1.42 @@ -1,7 +1,7 @@ # The LearningOnline Network # Simple Page Editor # -# $Id: lonsimplepage.pm,v 1.39 2006/05/09 14:38:10 albertel Exp $ +# $Id: lonsimplepage.pm,v 1.42 2006/06/30 18:06:47 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -37,6 +37,7 @@ use Apache::lonfeedback; use Apache::lonlocal; use Apache::lonprintout; use Apache::lonxml; +use Apache::longroup; sub handler { my $r = shift; @@ -70,13 +71,13 @@ sub handler { if ($caller eq 'grppg') { $marker =~ s/\W//g; $namespace = 'grppage_'.$marker; - my %curr_groups = &Apache::loncommon::coursegroups($dom,$crs,$marker); + my %curr_groups = &Apache::longroup::coursegroups($dom,$crs,$marker); if (!%curr_groups) { &Apache::loncommon::simple_error_page($r,'','Invalid group name'); return OK; } %groupinfo = - &Apache::loncommon::get_group_settings($curr_groups{$marker}); + &Apache::longroup::get_group_settings($curr_groups{$marker}); } else { $marker=~s/\D//g; $namespace = 'smppage_'.$marker; @@ -127,9 +128,11 @@ sub handler { if ($caller eq 'grppg') { if (($grp_view_permission) || - (&Apache::loncommon::check_group_access($caller))) { - unless(&Apache::lonnet::allowed('vgh', - $env{'request.course.id'}.'/'.$marker)) { + (&Apache::longroup::check_group_access($caller))) { + if ((!&Apache::lonnet::allowed('vgh', + $env{'request.course.id'}.'/'.$marker)) + && (!&Apache::lonnet::allowed('mdg', + $env{'request.course.id'}.'/'.$marker))) { &display_group_links($r,$target,$marker,'view',%groupinfo); return OK; } @@ -148,8 +151,8 @@ sub handler { my $allowed; if ($caller eq 'grppg') { - $allowed = $grp_view_permission; - unless ($allowed) { + $allowed = &Apache::lonnet::allowed('mdg',$env{'request.course.id'}); + if (!$allowed) { $allowed = &Apache::lonnet::allowed('mgh',$env{'request.course.id'}. '/'.$marker); }