--- loncom/interface/lonpickstudent.pm	2011/06/07 20:18:59	1.30
+++ loncom/interface/lonpickstudent.pm	2023/09/28 15:56:48	1.34
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Pick a student from the classlist
 #
-# $Id: lonpickstudent.pm,v 1.30 2011/06/07 20:18:59 www Exp $
+# $Id: lonpickstudent.pm,v 1.34 2023/09/28 15:56:48 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -50,18 +50,20 @@ sub handler {
 
     &Apache::loncommon::get_unprocessed_cgi
         ($ENV{'QUERY_STRING'},['filter','form','unameelement','udomelement',
-			       'roles','courseadvonly','clicker']);
+			       'roles','courseadv','clicker','identelement']);
 # Allowed?
     my $allowed;
-    my $scope = $env{'request.course.id'};
-    if (!($allowed = &Apache::lonnet::allowed('srm',$scope))) {
-	$scope .= '/'.$env{'request.course.sec'};
-	$allowed = &Apache::lonnet::allowed('srm',$scope);
-	if ($allowed) { $allowed = 'section';	}
+    if ($env{'request.course.id'}) {
+        if (!($allowed = &Apache::lonnet::allowed('srm',$env{'request.course.id'}))) {
+            if ($env{'request.course.sec'}) {
+                $allowed = &Apache::lonnet::allowed('srm',
+                               "$env{'request.course.id'}/$env{'request.course.sec'}");
+                if ($allowed) { $allowed = 'section'; }
+            }
+        }
     }
 
-    unless (($env{'form.roles'}) ||
-            (($env{'request.course.id'}) && ($allowed))) {
+    unless (($env{'form.roles'}) || ($allowed)) {
 	$r->print(&mt('No context.').
 		  &Apache::loncommon::end_page());
         return OK;
@@ -77,16 +79,23 @@ sub handler {
   <input type="text" name="filter" value="$encoded_filter" />
   <input type="submit" name="Change" value="$change" />
 FILTER
-    foreach my $name ('form','unameelement','udomelement','roles') {
+    foreach my $name ('form','unameelement','udomelement','roles','courseadv','clicker','identelement') {
 	my $value = &HTML::Entities::encode($env{"form.$name"},'<>&"');
 	$filterbutton .= <<HIDDEN;
    <input type="hidden" name="$name" value="$value" />
 HIDDEN
     }
-    unless ($env{'form.courseadvonly'}) {
-        $filtermsg = '<br />'.
-	    &mt('Showing users with a name starting with [_1]',
-	        $filterbutton).'<br />';
+    unless ($env{'form.courseadv'} eq 'only') {
+        if ($env{'request.course.id'}) {
+            my $crstype = &Apache::loncommon::course_type();
+            my $showrole = &Apache::lonnet::plaintext('st',$crstype);
+            $filtermsg = &mt("Showing users with role of '[_1]' with name starting: [_2]",
+                             $showrole,$filterbutton);
+        } else {
+            $filtermsg =
+	        &mt('Showing users with a name starting with [_1]',
+	            $filterbutton);
+        }
     }
     if ($filter eq '') {
 	$filter = '.'
@@ -97,45 +106,74 @@ HIDDEN
     $r->print(<<ENDSCRIPT);
 <script type="text/javascript">
 function gochoose(uname,udom) {
-    opener.document.$env{'form.form'}.elements['$env{'form.unameelement'}'].value=uname;
-    var slct=opener.document.$env{'form.form'}.elements['$env{'form.udomelement'}'];
-    var i;
-    for (i=0;i<slct.length;i++) {
-        if (slct.options[i].value==udom) { slct.selectedIndex=i; }
+    var form = opener.document.$env{'form.form'};
+    form.elements["$env{'form.unameelement'}"].value=uname;
+    var slct = form.elements['$env{"form.udomelement"}'];
+    if ((slct.tagName === 'INPUT') && ((slct.type === 'text') || (slct.type === 'hidden'))) {
+        slct.value=udom;
+    } else if (slct.tagName === 'SELECT') {
+        var i;
+        for (i=0;i<slct.length;i++) {
+            if (slct.options[i].value==udom) {
+                slct.selectedIndex=i;
+                break;
+            }
+        }
+    }
+    var identname = "$env{'form.identelement'}";
+    if ((identname != undefined) && (identname != '') && (identname != 'null')) {
+        ident = opener.document.getElementsByName("$env{'form.identelement'}");
+        if (ident.length > 0) {
+            var j;
+            for (j=0; j<ident.length; j++) {
+                if (ident[j].value == 'uname') {
+                    if (ident[j].checked) {
+                        break;
+                    } else {
+                        ident[j].checked = true;
+                        window.opener.toggleIdentifier(form);
+                    }
+                } else {
+                    ident[j].checked = false;
+                }
+            }
+        }
     }
     self.close();
 }
 </script>
 ENDSCRIPT
 
-    $r->print('<form>');
+    $r->print('<form action="">');
     if ($allowed && (!$env{'form.roles'})) {
+        unless ($env{'form.courseadv'} eq 'none') {
 # -------------------------------------------------------- Get course personnel
-	$r->print('<h3>'.$env{'course.'.$env{'request.course.id'}.'.description'}.
-		  '</h3>');
-	my %coursepersonnel=
-	    &Apache::lonnet::get_course_adv_roles();
-	$r->print(&Apache::loncommon::start_data_table());
-	foreach my $role (sort(keys(%coursepersonnel))) {
-	    foreach my $user (split(/\,/,$coursepersonnel{$role})) {
-		my ($puname,$pudom)=split(/\:/,$user);
-		$r->print(&Apache::loncommon::start_data_table_row().
-			  '<td>'.
-			  '<input type="button" value="'.&mt('Select').'" onClick="gochoose('.
-			  "'".$puname."','".$pudom."')".'" /></td><td>'.$role.'</td><td>'.
-			  &Apache::loncommon::aboutmewrapper(&Apache::loncommon::plainname($puname,$pudom),$puname,$pudom).'</td>'.
-			  &Apache::loncommon::end_data_table_row());
-	    }
-	}
-	$r->print(&Apache::loncommon::end_data_table().'<p>&nbsp;');
-
-	$r->print($filtermsg);
+	    $r->print('<h3>'.$env{'course.'.$env{'request.course.id'}.'.description'}.
+		      '</h3>');
+	    my %coursepersonnel=
+	        &Apache::lonnet::get_course_adv_roles();
+	    $r->print(&Apache::loncommon::start_data_table());
+	    foreach my $role (sort(keys(%coursepersonnel))) {
+	        foreach my $user (split(/\,/,$coursepersonnel{$role})) {
+		    my ($puname,$pudom)=split(/\:/,$user);
+		    $r->print(&Apache::loncommon::start_data_table_row().
+			      '<td>'.
+			      '<input type="button" value="'.&mt('Select').'" onclick="gochoose('.
+			      "'".$puname."','".$pudom."')".'" /></td><td>'.$role.'</td><td>'.
+			      &Apache::loncommon::aboutmewrapper(&Apache::loncommon::plainname($puname,$pudom),$puname,$pudom).'</td>'.
+			      &Apache::loncommon::end_data_table_row());
+                }
+            }
+            $r->print(&Apache::loncommon::end_data_table().'<br /><hr />');
+        }
 
-	$r->print('</p>');
+        if ($filtermsg ne '') {
+            $r->print('<p>'.$filtermsg.'</p>');
+        }
 
         $r->rflush();
 
-        unless ($env{'form.courseadvonly'}) {
+        unless ($env{'form.courseadv'} eq 'only') {
 
 # ------------------------------------------------------------------ Students
 
@@ -202,7 +240,7 @@ ENDSCRIPT
                         }
 		        $result .=&Apache::loncommon::start_data_table_row().
 		    	          '<td>'.
-			          '<input type="button" value="'.&mt('Select').'" onClick="gochoose('.
+			          '<input type="button" value="'.&mt('Select').'" onclick="gochoose('.
 			          "'".$uname."','".$udom."')".'" /></td>'.
 			          '<td><tt>'.$uname.'</tt></td><td> <tt>'.$udom.
 			          '</tt></td><td>'.
@@ -260,7 +298,7 @@ ENDSCRIPT
 	    my ($uname,$udom) = split(':',$user);
 	    $r->print(&Apache::loncommon::start_data_table_row().
 		      '<td>'.
-		      '<input type="button" value="'.&mt('Select').'" onClick="gochoose('.
+		      '<input type="button" value="'.&mt('Select').'" onclick="gochoose('.
 		      "'".$uname."','".$udom."')".'" /></td>'.
 		      '<td><tt>'.$uname.'</tt></td>'.
 		      '<td><tt>'.$udom.'</tt></td><td>'.
@@ -276,7 +314,7 @@ ENDSCRIPT
     }
     $r->print('</form>'.&Apache::loncommon::end_page());
     return OK;
-} 
+}
 
 1;
 __END__