--- loncom/interface/lonparmset.pm	2005/08/27 00:03:32	1.242
+++ loncom/interface/lonparmset.pm	2005/08/29 22:57:09	1.244
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to set parameters for assessments
 #
-# $Id: lonparmset.pm,v 1.242 2005/08/27 00:03:32 banghart Exp $
+# $Id: lonparmset.pm,v 1.244 2005/08/29 22:57:09 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -1950,7 +1950,7 @@ sub crsenv {
 			     'disablesigfigs'
                              );
 	foreach my $parameter (sort(keys(%values))) {
-            unless ($parameter =~ m/^internal\./) {
+            unless (($parameter =~ m/^internal\./)||($parameter =~ m/^metadata\./)) {
                 if (! $descriptions{$parameter}) {
                     $descriptions{$parameter}=$parameter;
                     push(@Display_Order,$parameter);
@@ -2728,8 +2728,9 @@ ENDMAINFORMHEAD
 sub setinstmeta {
     my ($r)=@_;
     my $next_meta;
-    if ($env{'form.nextmeta'}) {
-        $next_meta = $env{'form.nextmeta'};
+    my $output;
+    if ($env{'form.serial'}) {
+        $next_meta = $env{'form.serial'};
     } else {
         $next_meta = 1;
     }
@@ -2742,6 +2743,7 @@ sub setinstmeta {
 						    'Set Instructor Metadata'));
     my $dom = $env{'course.'.$env{'request.course.id'}.'.domain'};
     my $crs = $env{'course.'.$env{'request.course.id'}.'.num'};
+    my $key_base = $env{'course.'.$env{'request.course.id'}.'.metadata.'};
     if ($env{'form.state'}eq 'add') {
         $r->print('<strong>Compose a new metadata item:</strong><br />');
         $r->print('<form method="post" action="/adm/parmset">');
@@ -2756,22 +2758,56 @@ sub setinstmeta {
         $r->print('</form>');
     } elsif ($env{'form.state'}eq 'save') {
         $r->print('Here we save the metadata');
+        $key_base.=$env{'form.serial'}.'.';
+        $r->print('The title: '.$env{'form.title'}.'<br>');
+        $r->print('The values: '.$env{'form.values'}.'<br>');
+        $r->print('The type: '.$env{'form.type'}.'<br>');
+        $r->print('The number: '.$env{'form.serial'}.'<br>');
+        my $name = 'metadata.'.$env{'form.serial'}.'.title';
+        my $value = $env{'form.title'};
+        my $put_result = &Apache::lonnet::put('environment',
+                                                  {$name=>$value},$dom,$crs);
+        $r->print('the put result was: '.$put_result.'<br />');
+        $name = 'metadata.'.$env{'form.serial'}.'.type';
+        $value = $env{'form.type'};
+        $put_result = &Apache::lonnet::put('environment',
+                                                  {$name=>$value},$dom,$crs);
+        $name = 'metadata.'.$env{'form.serial'}.'.values';
+        $value = $env{'form.values'};
+        $put_result = &Apache::lonnet::put('environment',
+                                                  {$name=>$value},$dom,$crs);
+        &Apache::lonnet::coursedescription($env{'request.course.id'});        
     } else {
+        
         foreach my $key (sort keys %env) {
-            my $found_one = 0;
-            if ($key=~m/^course\.$env{'request.course.id'}\.metadata\.title/) {
-                unless ($found_one) {
-                    $found_one = 1;
-                    $r->print('<strong>Existing Metadata</strong><br >');
-                }
-                $r->print($env{$key}.'<br />');
-                $key=~~m/^course\.$env{'request.course.id'}\.metadata\.title\.(d*)\./;
+            if ($key=~m/^course\.$env{'request.course.id'}\.metadata\.\d+\.title$/) {
+                $key=~m/^course\.$env{'request.course.id'}\.metadata\.(\d+)\.title/;
+                
                 if ($1 ge $next_meta) {
-                    $next_meta = $1 ++;
+                    $next_meta = $1 + 1;
                 }
+                $output.='<tr><td>'.$env{$key}.'</td>';
+                $output.='<td>Select one: <input type="radio" name="type" value="one" /><br />';
+                $output.='Select many: <input type="radio" name="type" value="many" /></td>';
+                $output.='<td><input type="text" value="'.$env{$key.'.values'}.'" /></td></tr>';
             }
         }
-        $r->print('<a href="/adm/parmset?action=setinstmeta&state=add&nextmeta='.$next_meta.'">Add Metadata Item</a><br />');
+    $output.='<tr><td><i>Create new instructor metadata item</i><br />';
+    $output.='<input type="text" name="instmeta" /></td><td>Options here</td>';
+    $output.='<td><input type="hidden" name="serial" value="'.$next_meta.'" /></td></tr>';
+    $r->print(<<ENDenv);       
+        <form method="post" action="/adm/parmset?action=setinstmeta" name="form">
+        
+        <p>
+        <table border=2>
+        <tr><th>Title</th><th>Options</th><th>Choices</th></tr>
+        $output
+        </table>
+        <input type="submit" name="instmeta" value="Update Instructor Metadata">
+        </form>
+
+ENDenv
+        
     }
     $r->print('</body>
                 </html>');
@@ -2962,7 +2998,7 @@ sub handler {
                                              'pres_marker',
                                              'pres_value',
                                              'pres_type',
-                                             'udom','uname','symb']);
+                                             'udom','uname','symb','serial']);
 
 
     &Apache::lonhtmlcommon::clear_breadcrumbs();