'."\n".
' '.&mt('Please select').' '."\n";
if ($context eq 'course') {
- $options .= &default_course_roles($context,%customroles);
+ $options .= &default_course_roles($context,$checkpriv,%customroles);
} elsif ($context eq 'construction_space') {
- my @roles = &construction_space_roles();
+ my @roles = &construction_space_roles($checkpriv);
foreach my $role (@roles) {
my $plrole=&Apache::lonnet::plaintext($role);
$options .= ' '.$plrole.' '."\n";
}
} elsif ($context eq 'domain') {
- my @roles = &domain_roles();
+ my @roles = &domain_roles($checkpriv);
foreach my $role (@roles) {
my $plrole=&Apache::lonnet::plaintext($role);
$options .= ' '.$plrole.' ';
@@ -888,7 +906,7 @@ sub default_role_selector {
&Apache::loncommon::start_data_table_row()."\n".
' '."\n".
''."\n".
- &default_course_roles($context,%customroles)."\n".
+ &default_course_roles($context,$checkpriv,%customroles)."\n".
' '.
''.
''.
@@ -910,14 +928,15 @@ sub default_role_selector {
}
sub default_course_roles {
- my ($context,%customroles) = @_;
+ my ($context,$checkpriv,%customroles) = @_;
my $output;
- my @roles = &course_roles($context);
+ my @roles = &course_roles($context,$checkpriv);
foreach my $role (@roles) {
my $plrole=&Apache::lonnet::plaintext($role);
$output .= ' '.$plrole.' ';
}
if (keys(%customroles) > 0) {
+ my %customroles = &my_custom_roles();
foreach my $cust (sort(keys(%customroles))) {
my $custrole='cr_cr_'.$env{'user.domain'}.
'_'.$env{'user.name'}.'_'.$cust;
@@ -928,47 +947,61 @@ sub default_course_roles {
}
sub construction_space_roles {
+ my ($checkpriv) = @_;
my @allroles = ('ca','aa');
my @roles;
- foreach my $role (@allroles) {
- if (&Apache::lonnet::allowed('c'.$role,$env{'user.domain'}.'/'.$env{'user.name'})) {
- push(@roles,$role);
+ if ($checkpriv) {
+ foreach my $role (@allroles) {
+ if (&Apache::lonnet::allowed('c'.$role,$env{'user.domain'}.'/'.$env{'user.name'})) {
+ push(@roles,$role);
+ }
}
+ return @roles;
+ } else {
+ return @allroles;
}
- return @roles;
}
sub domain_roles {
+ my ($checkpriv) = @_;
my @allroles = ('dc','li','dg','au','sc');
my @roles;
- foreach my $role (@allroles) {
- if (&Apache::lonnet::allowed('c'.$role,$env{'request.role.domain'})) {
- push(@roles,$role);
+ if ($checkpriv) {
+ foreach my $role (@allroles) {
+ if (&Apache::lonnet::allowed('c'.$role,$env{'request.role.domain'})) {
+ push(@roles,$role);
+ }
}
+ return @roles;
+ } else {
+ return @allroles;
}
- return @roles;
}
sub course_roles {
- my ($context) = @_;
+ my ($context,$checkpriv) = @_;
my @allroles = ('st','ta','ep','in','cc');
my @roles;
if ($context eq 'domain') {
@roles = @allroles;
} elsif ($context eq 'course') {
if ($env{'request.course.id'}) {
- foreach my $role (@allroles) {
- if (&Apache::lonnet::allowed('c'.$role,$env{'request.course.id'})) {
- push(@roles,$role);
- } else {
- if ($role ne 'cc' && $env{'request.course.section'} ne '') {
- if (!&Apache::lonnet::allowed('c'.$role,
- $env{'request.course.id'}.'/'.
- $env{'request.course.section'})) {
- push(@roles,$role);
+ if ($checkpriv) {
+ foreach my $role (@allroles) {
+ if (&Apache::lonnet::allowed('c'.$role,$env{'request.course.id'})) {
+ push(@roles,$role);
+ } else {
+ if ($role ne 'cc' && $env{'request.course.section'} ne '') {
+ if (!&Apache::lonnet::allowed('c'.$role,
+ $env{'request.course.id'}.'/'.
+ $env{'request.course.section'})) {
+ push(@roles,$role);
+ }
}
}
}
+ } else {
+ @roles = @allroles;
}
}
}
@@ -976,18 +1009,18 @@ sub course_roles {
}
sub curr_role_permissions {
- my ($context,$setting) = @_;
+ my ($context,$setting,$checkpriv) = @_;
my @roles;
if ($context eq 'construction_space') {
- @roles = &construction_space_roles();
+ @roles = &construction_space_roles($checkpriv);
} elsif ($context eq 'domain') {
if ($setting eq 'course') {
- @roles = &course_roles($context);
+ @roles = &course_roles($context,$checkpriv);
} else {
- @roles = &domain_roles();
+ @roles = &domain_roles($checkpriv);
}
} elsif ($context eq 'course') {
- @roles = &course_roles($context);
+ @roles = &course_roles($context,$checkpriv);
}
return @roles;
}
@@ -1005,49 +1038,45 @@ sub my_custom_roles {
return %returnhash;
}
-sub print_html_classlist {
- my ($r,$mode,$permission,$context) = @_;
+sub print_userlist {
+ my ($r,$mode,$permission,$context,$formname,$totcodes,$codetitles,
+ $idlist,$idlist_titles) = @_;
+ my $format = $env{'form.output'};
if (! exists($env{'form.sortby'})) {
$env{'form.sortby'} = 'username';
}
- if ($env{'form.status'} !~ /^(Any|Expired|Active|Future)$/) {
- $env{'form.status'} = 'Active';
+ if ($env{'form.Status'} !~ /^(Any|Expired|Active|Future)$/) {
+ $env{'form.Status'} = 'Active';
}
my $status_select = &Apache::lonhtmlcommon::StatusOptions
- ($env{'form.status'});
+ ($env{'form.Status'});
+ if ($env{'form.showrole'} eq '') {
+ $env{'form.showrole'} = 'Any';
+ }
if (! defined($env{'form.output'}) ||
$env{'form.output'} !~ /^(csv|excel|html)$/ ) {
$env{'form.output'} = 'html';
}
- if ($context eq 'course') {
- my $cid =$env{'request.course.id'};
- my $cdom=$env{'course.'.$cid.'.domain'};
- my $cnum=$env{'course.'.$cid.'.num'};
- #
- # List course personnel
- my %coursepersonnel=&Apache::lonnet::get_course_adv_roles($cdom.'/'.$cnum);
- #
-
- $r->print(' '.&Apache::loncommon::start_data_table());
- foreach my $role (sort keys %coursepersonnel) {
- next if ($role =~ /^\s*$/);
- $r->print(&Apache::loncommon::start_data_table_row().
- ' '.$role.' ');
- foreach my $user (split(',',$coursepersonnel{$role})) {
- my ($puname,$pudom)=split(':',$user);
- $r->print(' '.&Apache::loncommon::aboutmewrapper(
- &Apache::loncommon::plainname($puname,$pudom),
- $puname,$pudom));
- }
- $r->print(' '.&Apache::loncommon::end_data_table_row());
- }
- $r->print(&Apache::loncommon::end_data_table());
+ my @statuses;
+ if ($env{'form.Status'} eq 'Any') {
+ @statuses = ('previous','active','future');
+ } elsif ($env{'form.Status'} eq 'Expired') {
+ @statuses = ('previous');
+ } elsif ($env{'form.Status'} eq 'Active') {
+ @statuses = ('active');
+ } elsif ($env{'form.Status'} eq 'Future') {
+ @statuses = ('future');
}
+
+# if ($context eq 'course') {
+# $r->print(&display_adv_courseroles());
+# }
#
# Interface output
- $r->print(' '."\n".
+ ' ');
$r->print("\n");
if ($env{'form.action'} ne 'modifystudent') {
@@ -1066,71 +1095,506 @@ sub print_html_classlist {
$output_selector .= '';
$r->print(''.&mt('Output Format: [_1]',$output_selector).' '.(' 'x3));
}
- $r->print(''.&mt('Student Status: [_1]',$status_select)." \n");
- $r->print(' '.
- "\n
\n");
-
+ $r->print(''.&mt('User Status: [_1]',$status_select).' '.(' 'x3)."\n");
+ my $roleselected = '';
+ if ($env{'form.showrole'} eq 'Any') {
+ $roleselected = ' selected="selected" ';
+ }
+ my $role_select;
+ if ($context eq 'domain') {
+ $role_select = &domain_roles_select();
+ $r->print(''.&mt('Role Type: [_1]',$role_select).' ');
+ } else {
+ $role_select = ''."\n".
+ ''.
+ &mt('Any role').' ';
+ my @poss_roles = &curr_role_permissions($context);
+ foreach my $role (@poss_roles) {
+ $roleselected = '';
+ if ($role eq $env{'form.showrole'}) {
+ $roleselected = ' selected="selected" ';
+ }
+ my $plrole=&Apache::lonnet::plaintext($role);
+ $role_select .= ''.$plrole.' ';
+ }
+ $roleselected = '';
+ if ($env{'form.showrole'} eq 'cr') {
+ $roleselected = ' selected="selected" ';
+ }
+ $role_select .= ''.&mt('Custom role').' '.
+ ' ';
+ $r->print(''.&mt('Role: [_1]',$role_select).' ');
+ }
+ if (!(($context eq 'domain') && ($env{'form.roletype'} eq 'course'))) {
+ $r->print(&list_submit_button(&mt('Update Display'))."\n\n");
+ }
+ my ($indexhash,$keylist) = &make_keylist_array();
+ my (%userlist,%userinfo);
+ if ($context eq 'domain' && $env{'form.roletype'} eq 'course') {
+ my $courseform =
+ &Apache::lonhtmlcommon::course_selection($formname,$totcodes,
+ $codetitles,$idlist,$idlist_titles);
+ $r->print(''.&Apache::lonhtmlcommon::start_pick_box()."\n".
+ &Apache::lonhtmlcommon::start_pick_box()."\n".
+ &Apache::lonhtmlcommon::row_title(&mt('Select Course(s)'),
+ 'LC_oddrow_value')."\n".
+ $courseform."\n".
+ &Apache::lonhtmlcommon::row_closure(1).
+ &Apache::lonhtmlcommon::end_pick_box().'
'.
+ ''.&list_submit_button(&mt('Update Display')).
+ "\n
\n");
+ }
+ $r->print(' '.&mt('Searching').' ... ');
+ $r->rflush();
if ($context eq 'course') {
- #
- # Print the classlist
- $r->print(''.&mt('Current Class List').' ');
- my ($classlist,$keylist)=&Apache::loncoursedata::get_classlist();
-
- if (exists($permission->{'view_section'})) {
- my $sec = &Apache::loncoursedata::CL_SECTION();
- foreach my $student (keys(%{$classlist})) {
- if ($classlist->{$student}[$sec] ne $permission->{'view_section'}) {
- delete($classlist->{$student});
+ my $classlist = &Apache::loncoursedata::get_classlist();
+ my $secidx = &Apache::loncoursedata::CL_SECTION();
+ foreach my $student (keys(%{$classlist})) {
+ if (exists($permission->{'view_section'})) {
+ if ($classlist->{$student}[$secidx] ne $permission->{'view_section'}) {
+ next;
+ } else {
+ $userlist{$student} = $classlist->{$student};
}
+ } else {
+ $userlist{$student} = $classlist->{$student};
}
}
-
- if (! defined($classlist)) {
- $r->print(&mt('There are no students currently enrolled.')."\n");
+ my $cid =$env{'request.course.id'};
+ my $cdom=$env{'course.'.$cid.'.domain'};
+ my $cnum=$env{'course.'.$cid.'.num'};
+ my $showroles;
+ if ($env{'form.showrole'} ne 'Any') {
+ $showroles = [$env{'form.showrole'}];
} else {
- # Print out the available choices
- if ($env{'form.action'} eq 'modifystudent') {
- &show_users_list($r,$context,'view','modify',
- $env{'form.Status'},$classlist,$keylist);
- } else {
- &show_users_list($r,$context,$env{'form.output'},'aboutme',
- $env{'form.Status'},$classlist,$keylist);
+ $showroles = undef;
+ }
+ my %advrolehash = &Apache::lonnet::get_my_roles($cnum,$cdom,undef,
+ \@statuses,$showroles);
+ &gather_userinfo($context,$format,\%userlist,$indexhash,\%userinfo,
+ \%advrolehash);
+ } else {
+ my (%cstr_roles,%dom_roles);
+ if ($context eq 'construction_space') {
+ # List co-authors and assistant co-authors
+ my @possroles = ('ca','aa');
+ %cstr_roles = &Apache::lonnet::get_my_roles(undef,undef,undef,
+ \@statuses,\@possroles);
+ &gather_userinfo($context,$format,\%userlist,$indexhash,\%userinfo,
+ \%cstr_roles);
+ } elsif ($context eq 'domain') {
+ if ($env{'form.roletype'} eq 'domain') {
+ %dom_roles = &Apache::lonnet::get_domain_roles($env{'request.role.domain'});
+ foreach my $key (keys(%dom_roles)) {
+ if (ref($dom_roles{$key}) eq 'HASH') {
+ &gather_userinfo($context,$format,\%userlist,$indexhash,
+ \%userinfo,$dom_roles{$key});
+ }
+ }
+ } elsif ($env{'form.roletype'} eq 'construction_space') {
+ my %dom_roles = &Apache::lonnet::get_domain_roles($env{'request.role.domain'},['au']);
+ my %coauthors;
+ foreach my $key (keys(%dom_roles)) {
+ if (ref($dom_roles{$key}) eq 'HASH') {
+ if ($env{'form.showrole'} eq 'au') {
+ &gather_userinfo($context,$format,\%userlist,$indexhash,
+ \%userinfo,$dom_roles{$key});
+ } else {
+ my @possroles;
+ if ($env{'form.showrole'} eq 'Any') {
+ @possroles = ('ca','aa');
+ } else {
+ @possroles = ($env{'form.showrole'});
+ }
+ foreach my $author (sort(keys(%{$dom_roles{$key}}))) {
+ my ($role,$authorname,$authordom) = split(/:/,$author);
+ my $extent = '/'.$authordom.'/'.$authorname;
+ %{$coauthors{$extent}} =
+ &Apache::lonnet::get_my_roles($authorname,
+ $authordom,undef,\@statuses,\@possroles);
+ }
+ &gather_userinfo($context,$format,\%userlist,
+ $indexhash,\%userinfo,\%coauthors);
+ }
+ }
+ }
+ } elsif ($env{'form.roletype'} eq 'course') {
+ if ($env{'form.coursepick'}) {
+ my %courses = &process_coursepick();
+ my %allusers;
+ foreach my $cid (keys(%courses)) {
+ my %coursehash =
+ &Apache::lonnet::coursedescription($cid,{'one_time' => 1});
+ my $cdom = $coursehash{'domain'};
+ my $cnum = $coursehash{'num'};
+ my $cdesc = $coursehash{'description'};
+ my (@roles,@sections,%access,%users,%userdata,
+ %users,%statushash);
+ if ($env{'form.showrole'} eq 'Any') {
+ @roles = &course_roles($context);
+ } else {
+ @roles = ($env{'form.showrole'});
+ }
+ foreach my $role (@roles) {
+ %{$users{$role}} = ();
+ }
+ foreach my $type (@statuses) {
+ $access{$type} = $type;
+ }
+ &Apache::loncommon::get_course_users($cdom,$cnum,\%access,\@roles,\@sections,\%users,\%userdata,\%statushash);
+ foreach my $user (keys(%userdata)) {
+ next if (ref($userinfo{$user}) eq 'HASH');
+ foreach my $item ('fullname','id') {
+ $userinfo{$user}{$item} = $userdata{$user}[$indexhash->{$item}];
+ }
+ }
+ foreach my $role (keys(%users)) {
+ foreach my $user (keys(%{$users{$role}})) {
+ my $uniqid = $user.':'.$role;
+ $allusers{$uniqid}{$cid} = { desc => $cdesc,
+ secs => $statushash{$user}{$role},
+ };
+ }
+ }
+ }
+ &gather_userinfo($context,$format,\%userlist,$indexhash,
+ \%userinfo,\%allusers);
+ } else {
+ return;
+ }
}
}
- } elsif ($context eq 'construction_space') {
- # List co-authors and assistant co-authors
- my @statuses;
- if ($env{'form.status'} eq 'Any') {
- @statuses = ('previous','active','future');
- } elsif ($env{'form.status'} eq 'Expired') {
- @statuses = ('previous');
- } elsif ($env{'form.status'} eq 'Active') {
- @statuses = ('active');
- } elsif ($env{'form.status'} eq 'Future') {
- @statuses = ('future');
- }
- my @possroles = ('ca','aa');
- my %cstr_roles = &Apache::lonnet::get_my_roles(undef,undef,undef,
- \@statuses,\@possroles);
- if (keys(%cstr_roles) == 0) {
- $r->print(&mt('There are no authors or co-authors.')."\n");
+ }
+ if (keys(%userlist) == 0) {
+ if ($context eq 'construction_space') {
+ $r->print(&mt('There are no co-authors to display.')."\n");
+ } elsif ($context eq 'domain') {
+ if ($env{'form.roletype'} eq 'domain') {
+ $r->print(&mt('There are no users with domain roles to display.')."\n");
+ } elsif ($env{'form.roletype'} eq 'construction_space') {
+ $r->print(&mt('There are no authors or co-authors to display.')."\n");
+ } elsif ($env{'form.roletype'} eq 'course') {
+ $r->print(&mt('There are no course users to display')."\n");
+ }
+ } elsif ($context eq 'course') {
+ $r->print(&mt('There are no course users to display.')."\n");
+ }
+ } else {
+ # Print out the available choices
+ if ($env{'form.action'} eq 'modifystudent') {
+ &show_users_list($r,$context,'view','modify',
+ $env{'form.Status'},\%userlist,$keylist);
} else {
- # Print out the available choices
- if ($env{'form.action'} eq 'modifystudent') {
- &show_users_list($r,$context,'view','modify',
- $env{'form.Status'},\%cstr_roles);
- } else {
- &show_users_list($r,$context,$env{'form.output'},'aboutme',
- $env{'form.Status'},\%cstr_roles);
+ &show_users_list($r,$context,$env{'form.output'},'aboutme',
+ $env{'form.Status'},\%userlist,$keylist);
+ }
+ }
+ $r->print('');
+}
+
+sub list_submit_button {
+ my ($text) = @_;
+ return ' ';
+}
+
+sub gather_userinfo {
+ my ($context,$format,$userlist,$indexhash,$userinfo,$rolehash) = @_;
+ foreach my $item (keys(%{$rolehash})) {
+ @{$userlist->{$item}} = ();
+ my %userdata;
+ if ($context eq 'construction_space' || $context eq 'course') {
+ ($userdata{'username'},$userdata{'domain'},$userdata{'role'}) =
+ split(/:/,$item);
+ ($userdata{'start'},$userdata{'end'})=split(/:/,$rolehash->{$item});
+ &build_user_record(\%userdata,$userinfo,$indexhash,$item,$userlist);
+ } elsif ($context eq 'domain') {
+ if ($env{'form.roletype'} eq 'domain') {
+ ($userdata{'role'},$userdata{'username'},$userdata{'domain'}) =
+ split(/:/,$item);
+ ($userdata{'end'},$userdata{'start'})=split(/:/,$rolehash->{$item});
+ &build_user_record(\%userdata,$userinfo,$indexhash,$item,$userlist);
+ } elsif ($env{'form.roletype'} eq 'construction_space') {
+ if (ref($rolehash->{$item}) eq 'HASH') {
+ $userdata{'extent'} = $item;
+ foreach my $key (keys(%{$rolehash->{$item}})) {
+ ($userdata{'username'},$userdata{'domain'},$userdata{'role'}) = split(/:/,$key);
+ ($userdata{'start'},$userdata{'end'}) =
+ split(/:/,$rolehash->{$item}{$key});
+ my $uniqid = $key.':'.$item;
+ &build_user_record(\%userdata,$userinfo,$indexhash,$uniqid,$userlist);
+ }
+ }
+ } elsif ($env{'form.roletype'} eq 'course') {
+ ($userdata{'username'},$userdata{'domain'},$userdata{'role'}) =
+ split(/:/,$item);
+ if (ref($rolehash->{$item}) eq 'HASH') {
+ foreach my $cid (sort(keys(%{$rolehash->{$item}}))) {
+ if (ref($rolehash->{$item}{$cid}) eq 'HASH') {
+ my $spanstart = '';
+ my $spanend = '; ';
+ my $space = ', ';
+ if ($format eq 'html' || $format eq 'view') {
+ $spanstart = '';
+ $spanend = ' ';
+ $space = ', ';
+ }
+ $userdata{'extent'} .= $spanstart.
+ $rolehash->{$item}{$cid}{'desc'}.$space;
+ if (ref($rolehash->{$item}{$cid}{'secs'}) eq 'HASH') {
+ foreach my $sec (sort(keys(%{$rolehash->{$item}{$cid}{'secs'}}))) {
+ $userdata{'extent'} .= $sec.$space.$rolehash->{$item}{$cid}{'secs'}{$sec}.$spanend;
+ }
+ }
+ }
+ }
+ }
+ &build_user_record(\%userdata,$userinfo,$indexhash,$item,$userlist);
+ }
+ }
+ }
+ return;
+}
+
+sub build_user_record {
+ my ($userdata,$userinfo,$indexhash,$record_key,$userlist) = @_;
+ &process_date_info($userdata);
+ my $username = $userdata->{'username'};
+ my $domain = $userdata->{'domain'};
+ if (ref($userinfo->{$username.':'.$domain}) eq 'HASH') {
+ $userdata->{'fullname'} =
+ $userinfo->{$username.':'.$domain}{'fullname'};
+ $userdata->{'id'} = $userinfo->{$username.':'.$domain}{'id'};
+ } else {
+ &aggregate_user_info($domain,$username,$userinfo);
+ $userdata->{'fullname'} = $userinfo->{$username.':'.$domain}{'fullname'};
+ $userdata->{'id'} = $userinfo->{$username.':'.$domain}{'id'};
+ }
+ foreach my $key (keys(%{$indexhash})) {
+ if (defined($userdata->{$key})) {
+ $userlist->{$record_key}[$indexhash->{$key}] = $userdata->{$key};
+ }
+ }
+ return;
+}
+
+sub courses_selector {
+ my ($cdom,$formname) = @_;
+ my %coursecodes = ();
+ my %codes = ();
+ my @codetitles = ();
+ my %cat_titles = ();
+ my %cat_order = ();
+ my %idlist = ();
+ my %idnums = ();
+ my %idlist_titles = ();
+ my $caller = 'global';
+ my $totcodes = 0;
+ my $format_reply;
+ my $jscript = '';
+
+ my $totcodes =
+ &Apache::courseclassifier::retrieve_instcodes(\%coursecodes,
+ $cdom,$totcodes);
+ if ($totcodes > 0) {
+ $format_reply =
+ &Apache::lonnet::auto_instcode_format($caller,$cdom,\%coursecodes,
+ \%codes,\@codetitles,\%cat_titles,\%cat_order);
+ if ($format_reply eq 'ok') {
+ my $numtypes = @codetitles;
+ &Apache::courseclassifier::build_code_selections(\%codes,\@codetitles,\%cat_titles,\%cat_order,\%idlist,\%idnums,\%idlist_titles);
+ my ($scripttext,$longtitles) = &Apache::courseclassifier::javascript_definitions(\@codetitles,\%idlist,\%idlist_titles,\%idnums,\%cat_titles);
+ my $longtitles_str = join('","',@{$longtitles});
+ my $allidlist = $idlist{$codetitles[0]};
+ $jscript .= &Apache::courseclassifier::courseset_js_start($formname,$longtitles_str,$allidlist);
+ $jscript .= $scripttext;
+ $jscript .= &Apache::courseclassifier::javascript_code_selections($formname,@codetitles);
+ }
+ }
+ my $cb_jscript = &Apache::loncommon::coursebrowser_javascript($cdom);
+
+ my %elements = (
+ Year => 'selectbox',
+ coursepick => 'radio',
+ coursetotal => 'text',
+ courselist => 'text',
+ );
+ $jscript .= &Apache::lonhtmlcommon::set_form_elements(\%elements);
+ if ($env{'form.coursepick'} eq 'category') {
+ $jscript .= qq|
+function setCourseCat(formname) {
+ if (formname.Year.options[formname.Year.selectedIndex].value == -1) {
+ return;
+ }
+ courseSet('Year');
+ for (var j=0; j 1) {
+ my @course_ids = split(/&&/,$env{'form.courselist'});
+ foreach my $cid (@course_ids) {
+ $courses{$cid} = '';
}
+ } else {
+ $courses{$env{'form.courselist'}} = '';
}
+ }
+ return %courses;
+}
+
+sub instcode_from_coursefilter {
+ my $instcode = '';
+ my @cats = ('Semester','Year','Department','Number');
+ foreach my $category (@cats) {
+ if (defined($env{'form.'.$category})) {
+ unless ($env{'form.'.$category} eq '-1') {
+ $instcode .= $env{'form.'.$category};
+ }
+ }
+ }
+ if ($instcode eq '') {
+ $instcode = '.';
+ }
+ return $instcode;
+}
+
+sub display_adv_courseroles {
+ my $output;
+ #
+ # List course personnel
+ my %coursepersonnel =
+ &Apache::lonnet::get_course_adv_roles($env{'request.course.id'});
+ #
+ $output = ' '.&Apache::loncommon::start_data_table();
+ foreach my $role (sort(keys(%coursepersonnel))) {
+ next if ($role =~ /^\s*$/);
+ $output .= &Apache::loncommon::start_data_table_row().
+ ''.$role.' ';
+ foreach my $user (split(',',$coursepersonnel{$role})) {
+ my ($puname,$pudom)=split(':',$user);
+ $output .= ' '.&Apache::loncommon::aboutmewrapper(
+ &Apache::loncommon::plainname($puname,$pudom),
+ $puname,$pudom);
+ }
+ $output .= ' '.&Apache::loncommon::end_data_table_row();
+ }
+ $output .= &Apache::loncommon::end_data_table();
+}
+
+sub make_keylist_array {
+ my ($index,$keylist);
+ $index->{'domain'} = &Apache::loncoursedata::CL_SDOM();
+ $index->{'username'} = &Apache::loncoursedata::CL_SNAME();
+ $index->{'end'} = &Apache::loncoursedata::CL_END();
+ $index->{'start'} = &Apache::loncoursedata::CL_START();
+ $index->{'id'} = &Apache::loncoursedata::CL_ID();
+ $index->{'section'} = &Apache::loncoursedata::CL_SECTION();
+ $index->{'fullname'} = &Apache::loncoursedata::CL_FULLNAME();
+ $index->{'status'} = &Apache::loncoursedata::CL_STATUS();
+ $index->{'type'} = &Apache::loncoursedata::CL_TYPE();
+ $index->{'lockedtype'} = &Apache::loncoursedata::CL_LOCKEDTYPE();
+ $index->{'groups'} = &Apache::loncoursedata::CL_GROUP();
+ $index->{'email'} = &Apache::loncoursedata::CL_PERMANENTEMAIL();
+ $index->{'role'} = &Apache::loncoursedata::CL_ROLE();
+ $index->{'extent'} = &Apache::loncoursedata::CL_EXTENT();
+ foreach my $key (keys(%{$index})) {
+ $keylist->[$index->{$key}] = $key;
+ }
+ return ($index,$keylist);
+}
+
+sub aggregate_user_info {
+ my ($udom,$uname,$userinfo) = @_;
+ my %info=&Apache::lonnet::get('environment',
+ ['firstname','middlename',
+ 'lastname','generation','id'],
+ $udom,$uname);
+ my ($tmp) = keys(%info);
+ my ($fullname,$id);
+ if ($tmp =~/^(con_lost|error|no_such_host)/i) {
+ $fullname = 'not available';
+ $id = 'not available';
+ &Apache::lonnet::logthis('unable to retrieve environment '.
+ 'for '.$uname.':'.$udom);
} else {
+ $fullname = &Apache::lonnet::format_name(@info{qw/firstname middlename lastname generation/},'lastname');
+ $id = $info{'id'};
+ }
+ $userinfo->{$uname.':'.$udom} = {
+ fullname => $fullname,
+ id => $id,
+ };
+ return;
+}
+sub process_date_info {
+ my ($userdata) = @_;
+ my $now = time;
+ $userdata->{'status'} = 'Active';
+ if ($userdata->{'start'} > 0) {
+ if ($now < $userdata->{'start'}) {
+ $userdata->{'status'} = 'Future';
+ }
+ }
+ if ($userdata->{'end'} > 0) {
+ if ($now > $userdata->{'end'}) {
+ $userdata->{'status'} = 'Expired';
+ }
}
+ return;
}
sub show_users_list {
- my ($r,$context,$mode,$linkto,$statusmode,$classlist,$keylist)=@_;
+ my ($r,$context,$mode,$linkto,$statusmode,$userlist,$keylist)=@_;
#
# Variables for excel output
my ($excel_workbook, $excel_sheet, $excel_filename,$row,$format);
@@ -1139,16 +1603,22 @@ sub show_users_list {
my ($CSVfile,$CSVfilename);
#
my $sortby = $env{'form.sortby'};
- if ($sortby !~ /^(username|domain|section|groups|fullname|id|start|end|type)$/) {
+ my @sortable = ('username','domain','id','fullname','start','end','email','role');
+ if ($context eq 'course') {
+ push(@sortable,('section','groups','type'));
+ } else {
+ push(@sortable,'extent');
+ }
+ if (!grep(/^\Q$sortby\E$/,@sortable)) {
$sortby = 'username';
}
- my ($cid,$cdom,$cnum,$classgroups,$displayphotos,$displayclickers)=@_;
+ my ($cid,$cdom,$cnum,$classgroups,$displayphotos,$displayclickers);
if ($context eq 'course') {
$cid=$env{'request.course.id'};
$cdom = $env{'course.'.$cid.'.domain'};
$cnum = $env{'course.'.$cid.'.num'};
- $classgroups = &Apache::loncoursedata::get_group_memberships(
- $classlist,$keylist,$cdom,$cnum);
+ ($classgroups) = &Apache::loncoursedata::get_group_memberships(
+ $userlist,$keylist,$cdom,$cnum);
if (! exists($env{'form.displayphotos'})) {
$env{'form.displayphotos'} = 'off';
}
@@ -1194,17 +1664,34 @@ END
'start' => "start date",
'end' => "end date",
'status' => "status",
+ 'role' => "role",
'type' => "enroll type/action",
'email' => "email address",
'clicker' => "clicker id",
'photo' => "photo",
+ 'extent' => "extent",
);
+ if ($context eq 'domain' && $env{'form.roletype'} eq 'course') {
+ $lt{'extent'} = &mt('Course(s): description, section(s), status');
+ } elsif ($context eq 'construction_space') {
+ $lt{'extent'} = &mt('Author');
+ }
my @cols = ('username','domain','id','fullname');
if ($context eq 'course') {
push(@cols,'section');
}
- push(@cols,('start','end'));
- if ($statusmode eq 'Any') {
+ if (!($context eq 'domain' && $env{'form.roletype'} eq 'course')) {
+ push(@cols,('start','end'));
+ }
+ if ($env{'form.showrole'} eq 'Any' || $env{'form.showrole'} eq 'cr') {
+ push(@cols,'role');
+ }
+ if ($context eq 'domain' && ($env{'form.roletype'} eq 'construction_space' ||
+ $env{'form.roletype'} eq 'course')) {
+ push (@cols,'extent');
+ }
+ if (($statusmode eq 'Any') &&
+ (!($context eq 'domain' && $env{'form.roletype'} eq 'course'))) {
push(@cols,'status');
}
if ($context eq 'course') {
@@ -1212,6 +1699,13 @@ END
}
push(@cols,'email');
+ my $rolefilter;
+ if ($env{'form.showrole'} ne 'Any') {
+ $rolefilter = &Apache::lonnet::plaintext($env{'form.showrole'});
+ }
+ my $results_description = &results_header_row($rolefilter,$statusmode,
+ $context);
+ $r->print(''.$results_description.' ');
if ($mode eq 'html' || $mode eq 'view') {
if ($linkto eq 'aboutme') {
$r->print(&mt("Select a user name to view the user's personal page."));
@@ -1222,7 +1716,6 @@ END
END
-
$r->print("\n\n".
&Apache::loncommon::start_data_table().
&Apache::loncommon::start_data_table_header_row());
@@ -1236,8 +1729,9 @@ END
");
}
foreach my $item (@cols) {
- $r->print('
'.$lt{$item}.' '."\n");
+ $r->print("$lt{$item} \n");
}
+ my %role_types = &role_type_names();
if ($context eq 'course') {
# Clicker display on or off?
my %clicker_options = &Apache::lonlocal::texthash(
@@ -1270,16 +1764,15 @@ END
$photo_options{$photochg}.' '.$lt{'photo'}."\n".
' '."\n");
}
- $r->print(&Apache::end_data_table_header_row());
+ $r->print(&Apache::loncommon::end_data_table_header_row());
}
# Done with the HTML header line
-
} elsif ($mode eq 'csv') {
#
# Open a file
$CSVfilename = '/prtspool/'.
- $env{'user.name'}.'_'.$env{'user.domain'}.'_'.
- time.'_'.rand(1000000000).'.csv';
+ $env{'user.name'}.'_'.$env{'user.domain'}.'_'.
+ time.'_'.rand(1000000000).'.csv';
unless ($CSVfile = Apache::File->new('>/home/httpd'.$CSVfilename)) {
$r->log_error("Couldn't open $CSVfilename for output $!");
$r->print("Problems occured in writing the csv file. ".
@@ -1289,11 +1782,7 @@ END
}
#
# Write headers and data to file
- if($statusmode eq 'Expired') {
- print $CSVfile '"'.&mt('Users with expired roles').'"'."\n"; }
- if($statusmode eq 'Future') {
- print $CSVfile '"'.&mt('Users with future roles').'"'."\n";
- }
+ print $CSVfile '"'.$results_description.'"'."\n";
print $CSVfile '"'.join('","',map {
&Apache::loncommon::csv_translate($lt{$_})
} (@cols)).'"'."\n";
@@ -1303,18 +1792,7 @@ END
&Apache::loncommon::create_workbook($r);
return if (! defined($excel_workbook));
$excel_sheet = $excel_workbook->addworksheet('userlist');
- #
- my $description;
- if ($context eq 'course') {
- $description = &mt('Class List for ').
- $env{'course.'.$env{'request.course.id'}.'.description'};
- } elsif ($context eq 'construction_space') {
- $description = &mt('List of co-authors for construction space for [_1]',
- &Apache::loncommon::plainname($env{'user.name'},$env{'user.domain'}));
- } else {
- $description = &mt('List of users for domain: [_1]',&Apache::lonnet::domain($env{'request.role.domain'},'description'));
- }
- $excel_sheet->write($row++,0,$description,$format->{'h1'});
+ $excel_sheet->write($row++,0,$results_description,$format->{'h2'});
#
my @colnames = map {$lt{$_}} (@cols);
$excel_sheet->write($row++,0,\@colnames,$format->{'bold'});
@@ -1322,84 +1800,284 @@ END
# Done with header lines in all formats
- #
- # Sort the users
my %index;
my $i;
- foreach (@$keylist) {
- $index{$_} = $i++;
+ foreach my $idx (@$keylist) {
+ $index{$idx} = $i++;
}
- $index{'groups'} = scalar(@$keylist);
+ # Get groups, role, permanent e-mail so we can sort on them if
+ # necessary.
+ foreach my $user (keys(%{$userlist})) {
+ my ($uname,$udom,$role,$groups,$email);
+ if ($context eq 'domain') {
+ if ($env{'form.roletype'} eq 'domain') {
+ ($role,$uname,$udom) = split(/:/,$user);
+
+ } elsif ($env{'form.roletype'} eq 'construction_space') {
+ ($uname,$udom,$role) = split(/:/,$user,-1);
+ } elsif ($env{'form.roletype'} eq 'course') {
+ ($uname,$udom,$role) = split(/:/,$user);
+ }
+ } else {
+ ($uname,$udom,$role) = split(/:/,$user,-1);
+ if (($context eq 'course') && $role eq '') {
+ $role = 'st';
+ }
+ }
+ $userlist->{$user}->[$index{'role'}] = $role;
+ if (($env{'form.showrole'} ne 'Any') && (!($env{'form.showrole'} eq 'cr' && $role =~ /^cr\//)) && ($role ne $env{'form.showrole'})) {
+ delete($userlist->{$user});
+ next;
+ }
+ if (ref($classgroups) eq 'HASH') {
+ $groups = $classgroups->{$user};
+ }
+ if (ref($groups->{active}) eq 'HASH') {
+ $userlist->{$user}->[$index{'groups'}] = join(', ',keys(%{$groups->{'active'}}));
+ }
+ my %emails = &Apache::loncommon::getemails($uname,$udom);
+ if ($emails{'permanentemail'} =~ /\S/) {
+ $userlist->{$user}->[$index{'email'}] = $emails{'permanentemail'};
+ }
+ }
+
+ #
+ # Sort the users
my $index = $index{$sortby};
my $second = $index{'username'};
my $third = $index{'domain'};
- my @Sorted_Students = sort {
- lc($classlist->{$a}->[$index]) cmp lc($classlist->{$b}->[$index])
+ my @sorted_users = sort {
+ lc($userlist->{$a}->[$index]) cmp lc($userlist->{$b}->[$index])
||
- lc($classlist->{$a}->[$second]) cmp lc($classlist->{$b}->[$second]) ||
- lc($classlist->{$a}->[$third]) cmp lc($classlist->{$b}->[$third])
- } (keys(%$classlist));
- my $studentcount = 0;
+ lc($userlist->{$a}->[$second]) cmp lc($userlist->{$b}->[$second]) ||
+ lc($userlist->{$a}->[$third]) cmp lc($userlist->{$b}->[$third])
+ } (keys(%$userlist));
+ my $usercount = 0;
my $autocount = 0;
my $manualcount = 0;
- my $unlockcount = 0;
my $lockcount = 0;
- foreach my $student (@Sorted_Students) {
- my $sdata = $classlist->{$student};
- my $groups = $classgroups->{$student};
- my $username = $sdata->[$index{'username'}];
- my $domain = $sdata->[$index{'domain'}];
- my $section = $sdata->[$index{'section'}];
- my $active_groups;
- if (ref($groups->{active}) eq 'HASH') {
- $active_groups = join(', ',keys(%{$groups->{'active'}}));
- }
- my $name = $sdata->[$index{'fullname'}];
- my $id = $sdata->[$index{'id'}];
- my $status = $sdata->[$index{'status'}];
- next if (($statusmode ne 'Any') && ($status ne $statusmode));
- my $start = $sdata->[$index{'start'}];
- my $end = $sdata->[$index{'end'}];
- if (! defined($start) || $start == 0) {
- $start = &mt('none');
+ my $unlockcount = 0;
+ foreach my $user (@sorted_users) {
+ my $sdata = $userlist->{$user};
+ my %in;
+ foreach my $item (@{$keylist}) {
+ $in{$item} = $sdata->[$index{$item}];
+ }
+ next if (($statusmode ne 'Any') && ($in{'status'} ne $statusmode));
+ $in{'role'}=&Apache::lonnet::plaintext($sdata->[$index{'role'}]);
+ if (! defined($in{'start'}) || $in{'start'} == 0) {
+ $in{'start'} = &mt('none');
} else {
- $start = &Apache::lonlocal::locallocaltime($start);
+ $in{'start'} = &Apache::lonlocal::locallocaltime($in{'start'});
}
- if (! defined($end) || $end == 0) {
- $end = &mt('none');
+ if (! defined($in{'end'}) || $in{'end'} == 0) {
+ $in{'end'} = &mt('none');
} else {
- $end = &Apache::lonlocal::locallocaltime($end);
+ $in{'end'} = &Apache::lonlocal::locallocaltime($in{'end'});
+ }
+ $usercount ++;
+ if ($mode eq 'view' || $mode eq 'html' || $mode eq 'autoenroll') {
+ $r->print(&Apache::loncommon::start_data_table_row());
+ $r->print("$usercount \n");
+ if ($linkto eq 'aboutme') {
+ $in{'username'} =
+ &Apache::loncommon::aboutmewrapper($in{'username'},
+ $in{'username'},
+ $in{'domain'});
+ } elsif ($linkto eq 'modify') {
+ $in{'username'} = ''.
+ $in{'username'}." \n";
+ }
+ foreach my $item (@cols) {
+ $r->print(''.$in{$item}.' '."\n");
+ }
+ if ($context eq 'course') {
+ if ($displayclickers eq 'on') {
+ my $clickers =
+ (&Apache::lonnet::userenvironment($in{'domain'},$in{'username'},'clickers'))[1];
+ if ($clickers!~/\w/) { $clickers='-'; }
+ $r->print(''.$clickers.' ');
+ } else {
+ $r->print(' ');
+ }
+ if ($displayphotos eq 'on') {
+ if ($env{'course.'.$env{'request.course.id'}.
+ '.internal.showphoto'}) {
+ my $imgurl =
+ &Apache::lonnet::retrievestudentphoto($in{'domain'},$in{'username'},'gif','thumbnail');
+ $r->print(' ');
+ } else {
+ $r->print(' ');
+ }
+ }
+ }
+ $r->print(&Apache::loncommon::end_data_table_row());
+ } elsif ($mode eq 'csv') {
+ next if (! defined($CSVfile));
+ # no need to bother with $linkto
+ if (! defined($in{'start'}) || $in{'start'} == 0) {
+ $in{'start'} = &mt('none');
+ } else {
+ $in{'start'} = &Apache::lonlocal::locallocaltime($in{'start'});
+ }
+ if (! defined($in{'end'}) || $in{'end'} == 0) {
+ $in{'end'} = &mt('none');
+ } else {
+ $in{'end'} = &Apache::lonlocal::locallocaltime($in{'end'});
+ }
+ my @line = ();
+ foreach my $item (@cols) {
+ push @line,&Apache::loncommon::csv_translate($in{$item});
+ }
+ print $CSVfile '"'.join('","',@line).'"'."\n";
+ } elsif ($mode eq 'excel') {
+ my $col = 0;
+ foreach my $item (@cols) {
+ if ($item eq 'start' || $item eq 'end') {
+ if (defined($item) && $item != 0) {
+ $excel_sheet->write($row,$col++,
+ &Apache::lonstathelpers::calc_serial($in{item}),
+ $format->{'date'});
+ } else {
+ $excel_sheet->write($row,$col++,'none');
+ }
+ } else {
+ $excel_sheet->write($row,$col++,$in{$item});
+ }
+ }
+ $row++;
}
- my $status = $sdata->[$index{'status'}];
- next if ($status ne 'Active');
- #
- $r->print(&Apache::loncommon::start_data_table_row());
- $r->print(<<"END");
-
- $username
- $domain
- $id
- $name
- $start
- $end
-END
- $r->print(&Apache::loncommon::end_data_table_row());
}
- $r->print(&Apache::loncommon::end_data_table().' ');
- %lt=&Apache::lonlocal::texthash(
- 'dp' => "Expire User Roles",
- 'ca' => "check all",
- 'ua' => "uncheck all",
- );
- $r->print(<<"END");
-
-
-
-
-END
+ if ($mode eq 'view' || $mode eq 'html' || $mode eq 'autoenroll') {
+ $r->print(&Apache::loncommon::end_data_table().' ');
+ } elsif ($mode eq 'excel') {
+ $excel_workbook->close();
+ $r->print(''.
+ &mt('Your Excel spreadsheet').' '.&mt('is ready for download').'.
'."\n");
+ } elsif ($mode eq 'csv') {
+ close($CSVfile);
+ $r->print(''.
+ &mt('Your CSV file').' is ready for download.'.
+ "\n");
+ $r->rflush();
+ }
+ if ($mode eq 'autoenroll') {
+ return ($usercount,$autocount,$manualcount,$lockcount,$unlockcount);
+ }
return;
}
+sub role_type_names {
+ my %lt = &Apache::lonlocal::texthash (
+ 'domain' => 'Domain Roles',
+ 'construction_space' => 'Co-Author Roles',
+ 'course' => 'Course Roles',
+ );
+ return %lt;
+}
+
+sub results_header_row {
+ my ($rolefilter,$statusmode,$context) = @_;
+ my $description;
+ if ($context eq 'course') {
+ $description = &mt('Course - ').$env{'course.'.$env{'request.course.id'}.'.description'}.': ';
+ if ($statusmode eq 'Expired') {
+ $description .= &mt('Users in course with expired [_1] roles',$rolefilter);
+ }
+ if ($statusmode eq 'Future') {
+ $description .= &mt('Users in course with future [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Active') {
+ $description .= &mt('Users in course with active [_1] roles',$rolefilter);
+ } else {
+ if ($rolefilter eq 'Any') {
+ $description .= &mt('All users in course');
+ } else {
+ $description .= &mt('All users in course with [_1] roles',$rolefilter);
+ }
+ }
+ } elsif ($context eq 'construction_space') {
+ $description = &mt('Author space for [_1].').' ';
+ if ($statusmode eq 'Expired') {
+ $description .= &mt('Co-authors with expired [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Future') {
+ $description .= &mt('Co-authors with future [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Active') {
+ $description .= &mt('Co-authors with active [_1] roles',$rolefilter);
+ } else {
+ if ($rolefilter eq 'Any') {
+ $description .= &mt('All co-authors',$rolefilter);
+ } else {
+ $description .= &mt('All co-authors with [_1] roles',$rolefilter);
+ }
+ }
+ } elsif ($context eq 'domain') {
+ my $domdesc = &Apache::lonnet::domain($env{'request.role.domain'},'description');
+ $description = &mt('Domain - ').$domdesc.': ';
+ if ($env{'form.roletype'} eq 'domain') {
+ if ($statusmode eq 'Expired') {
+ $description .= &mt('Users in domain with expired [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Future') {
+ $description .= &mt('Users in domain with future [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Active') {
+ $description .= &mt('Users in domain with active [_1] roles',$rolefilter);
+ } else {
+ if ($rolefilter eq 'Any') {
+ $description .= &mt('All users in domain',$rolefilter);
+ } else {
+ $description .= &mt('All users in domain with [_1] roles',$rolefilter);
+ }
+ }
+ } elsif ($env{'form.roletype'} eq 'construction_space') {
+ if ($statusmode eq 'Expired') {
+ $description .= &mt('Co-authors in domain with expired [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Future') {
+ $description .= &mt('Co-authors in domain with future [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Active') {
+ $description .= &mt('Co-authors in domain with active [_1] roles',$rolefilter);
+ } else {
+ if ($rolefilter eq 'Any') {
+ $description .= &mt('All users with co-author roles in domain',$rolefilter);
+ } else {
+ $description .= &mt('All co-authors in domain with [_1] roles',$rolefilter);
+ }
+ }
+ } elsif ($env{'form.roletype'} eq 'course') {
+ my $coursefilter = $env{'form.coursepick'};
+ if ($coursefilter eq 'category') {
+ my $instcode = &instcode_from_coursefilter();
+ if ($instcode eq '.') {
+ $description .= &mt('All courses in domain').' - ';
+ } else {
+ $description .= &mt('Courses in domain with institutional code: [_1]',$instcode).' - ';
+ }
+ } elsif ($coursefilter eq 'selected') {
+ $description .= &mt('Selected courses in domain').' - ';
+ } elsif ($coursefilter eq 'all') {
+ $description .= &mt('All courses in domain').' - ';
+ }
+ if ($statusmode eq 'Expired') {
+ $description .= &mt('users with expired [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Future') {
+ $description .= &mt('users with future [_1] roles',$rolefilter);
+ } elsif ($statusmode eq 'Active') {
+ $description .= &mt('users with active [_1] roles',$rolefilter);
+ } else {
+ if ($rolefilter eq 'Any') {
+ $description .= &mt('all users');
+ } else {
+ $description .= &mt('users with [_1] roles',$rolefilter);
+ }
+ }
+ }
+ }
+ return $description;
+}
+
#################################################
#################################################
sub show_drop_list {
@@ -1561,8 +2239,6 @@ END
return;
}
-
-
#
# Print out the initial form to get the file containing a list of users
#
@@ -1572,11 +2248,11 @@ sub print_first_users_upload_form {
$str = ' ';
$str .= ' ';
$str .= ' ';
- $str .= "".&mt('Upload a list of users')." \n";
+ $str .= "".&mt('Upload a file containing information about users')." \n";
$str .= &Apache::loncommon::upfile_select_html();
$str .= "\n";
$str .= ' '."\n";
+ &mt('Upload file of users').'">'."\n";
$str .= ' '.
&mt('Ignore First Line')."
\n";
$str .= &Apache::loncommon::help_open_topic("Course_Create_Class_List",
@@ -1805,9 +2481,9 @@ sub upfile_drop_add {
my $role = '';
if (defined($fields{'role'})) {
if ($entries{$fields{'role'}}) {
- my @poss_roles =
+ my @poss_roles =
&curr_role_permissions($context,$setting);
- if (grep(/^\Q$entries{$fields{'role'}}\E/,@poss_roles)) {
+ if (grep(/^\Q$entries{$fields{'role'}}\E/,@poss_roles)) {
$role=$entries{$fields{'role'}};
} else {
my $rolestr = join(', ',@poss_roles);