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

[raeburn]

raeburn		Sun Jan 12 23:34:21 2025 EDT

  Modified files:              
    /loncom/interface	lonuserutils.pm 
  Log:
  - Faster page load for listing users when clicker-ID unselected for display.
  
  
Index: loncom/interface/lonuserutils.pm
diff -u loncom/interface/lonuserutils.pm:1.222 loncom/interface/lonuserutils.pm:1.223
--- loncom/interface/lonuserutils.pm:1.222	Sat Aug 31 20:02:12 2024
+++ loncom/interface/lonuserutils.pm	Sun Jan 12 23:34:20 2025
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Utility functions for managing LON-CAPA user accounts
 #
-# $Id: lonuserutils.pm,v 1.222 2024/08/31 20:02:12 raeburn Exp $
+# $Id: lonuserutils.pm,v 1.223 2025/01/12 23:34:20 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -3714,9 +3714,11 @@
         foreach my $item (@{$keylist}) {
             $in{$item} = $sdata->[$index{$item}];
         }
-        my $clickers = (&Apache::lonnet::userenvironment($in{'domain'},$in{'username'},'clickers'))[1];
-        if ($clickers!~/\w/) { $clickers='-'; }
-        $in{'clicker'} = $clickers;
+        if (grep(/^clicker$/, at cols)) { 
+            my $clickers = (&Apache::lonnet::userenvironment($in{'domain'},$in{'username'},'clickers'))[1];
+            if ($clickers!~/\w/) { $clickers='-'; }
+            $in{'clicker'} = $clickers;
+        }
 	my $role = $in{'role'};
         $in{'role'}=&Apache::lonnet::plaintext($sdata->[$index{'role'}],$crstype);
         unless ($mode eq 'excel') {
@@ -3838,10 +3840,7 @@
                 } elsif ($item eq 'clicker') {
                     if (($context eq 'course') && ($mode ne 'autoenroll')) {
                         if ($env{'form.showrole'} eq 'st' || $env{'form.showrole'} eq 'Any') {
-                            my $clickers =
-                   (&Apache::lonnet::userenvironment($in{'domain'},$in{'username'},'clickers'))[1];
-                            if ($clickers!~/\w/) { $clickers='-'; }
-                            $r->print('<td>'.$clickers.'</td>');
+                            $r->print('<td>'.$in{'clicker'}.'</td>');
                         } else {
                              $r->print('<td> </td>'."\n");
                         }