--- loncom/interface/lonmanagekeys.pm	2003/05/01 02:12:43	1.6
+++ loncom/interface/lonmanagekeys.pm	2003/05/02 16:58:56	1.10
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to manage course access keys 
 #
-# $Id: lonmanagekeys.pm,v 1.6 2003/05/01 02:12:43 www Exp $
+# $Id: lonmanagekeys.pm,v 1.10 2003/05/02 16:58:56 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -54,13 +54,48 @@ ENDHEAD
 
 # =================================================== Show student list to drop
 sub show_key_list {
-    my ($r,$comment,%cenv)=@_;
-
+    my ($r,$comment,$newonly,$checkonly,%cenv)=@_;
+    $comment=~s/\W/\./g;
     my %accesskeys=&Apache::lonnet::dump
 	('accesskeys',$cenv{'domain'},$cenv{'num'});
+    $r->print(<<ENDTABLEHEADER);
+<script>
+    function copyallcom(tf) {
+	for (i=0; i<tf.elements.length; i++) {
+            if  (tf.elements[i].name.indexOf('com_')==0) {
+	      tf.elements[i].value+=tf.copyall.value;
+            }
+        }
+
+    }
+</script>
+<h3>List of Keys/Enter New Comments</h3>
+<table border="2"><tr><th>Key</th><th>Checked Out</th>
+<th>Comments/Remarks/Notes</th>
+<th>Enter Additional Comments/Remarks/Notes<br />
+<input type="text" size="40" name="copyall" />
+<input type="button" value="Copy to All" onClick="copyallcom(this.form);" />
+</th></tr>
+ENDTABLEHEADER
     foreach (keys %accesskeys) {
-	$r->print($_.' '.$accesskeys{$_}.'<br />');
+        if ($_=~/^error\:/) {
+	    $r->print('<tr><td>No keys have been generated yet.</td></tr>');
+        } elsif ($accesskeys{$_}=~/$comment/) {
+	    my ($checkout,$com)=split(/\s*\#\s*/,$accesskeys{$_});
+            unless ($checkout) {
+		if ($checkonly) { next; }
+            } else {
+		if ($newonly) { next; }
+            }
+	   $r->print("\n<tr><td><tt>".$_.'</tt></td><td>'.($checkout?
+                     $checkout:'-').'</td><td>'.
+                     join('<br />',split(/\s*\;\s*/,$com)).
+		     '</td><td><input type="text" size="40" name="com_'.$_.
+		     '" value="" /></td></tr>');
+       }
     }
+    $r->print('</table>');
+    $r->print('<input type="submit" name="addcom" value="Add Above Comments to Keys" /><hr />');
     return '';
 #
 # Junk below
@@ -160,6 +195,25 @@ sub genkeys {
     $num.' access keys (Batch Number: '.$batchnumber.')',$batchnumber;
 }
 
+# ---------------------------------------------------------------- Add comments
+
+sub addcom {
+    my %cenv=@_;
+    my %newcomment=();
+    undef %newcomment;
+    foreach (keys %ENV) {
+	if ($_=~/^form\.com\_(.+)$/) {
+            my $key=$1;
+	    my $comment=$ENV{$_};
+            $comment=~s/^\s+//gs;
+            if ($comment) {
+               &Apache::lonnet::comment_access_key
+		   ($key,$cenv{'domain'},$cenv{'num'},$comment); 
+	   }
+	}
+    }
+    return '';
+}
 ###################################################################
 ###################################################################
 sub handler {
@@ -212,7 +266,11 @@ sub handler {
                 $r->print($msg.'<br />');
 	    }
             if ($ENV{'form.listkeys'}) {
-		&show_key_list($r,$ENV{'form.listcom'},%cenv);
+		&show_key_list($r,$ENV{'form.listcom'},
+                          $ENV{'form.newonly'},$ENV{'form.checkonly'},%cenv);
+            }
+            if ($ENV{'form.addcom'}) {
+		&addcom(%cenv);
             }
 # --- Menu
 	    $r->print('<h3>Key Access</h3>');
@@ -227,8 +285,10 @@ Number of keys to be generated: <input t
 Comments/Remarks/Notes: <input type="text" name="comments" size="30" /><br />
 <input type="submit" name="genkeys" value="Generate Keys" />
 <hr /><h3>List Keys</h3>
-Comments/Remarks/Notes/Batch Number:
+Comments/Remarks/Notes/User/Batch Number:
 <input type="text" name="listcom" size="30" value="$batchnumber" /><br />
+<input type="checkbox" name="newonly" /> Unused keys only<br />
+<input type="checkbox" name="checkonly" /> Used keys only<br />
 <input type="submit" name="listkeys" value="List Keys" />
 ENDKEYMENU
 	    $r->print('</form></body></html>');