--- loncom/interface/lonindexer.pm	2005/02/17 08:50:20	1.130
+++ loncom/interface/lonindexer.pm	2005/03/01 03:20:41	1.131
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Directory Indexer
 #
-# $Id: lonindexer.pm,v 1.130 2005/02/17 08:50:20 albertel Exp $
+# $Id: lonindexer.pm,v 1.131 2005/03/01 03:20:41 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -178,7 +178,7 @@ ENDSUBM
 	    $colspan=" colspan='2' ";
             my $cl=&mt('Close');
             $closebutton=<<END;
-<input type="button" name="close" value='$cl' onClick="self.close()">
+<input type="button" name="close" value='$cl' onClick="self.close()" />
 END
         }
 	elsif ($ENV{'form.catalogmode'} eq 'groupimport') {
@@ -189,11 +189,11 @@ END
 	    my $cl=&mt('Close');
             my $gi=&mt('Import');
             $closebutton=<<END;
-<input type="button" name="close" value='$cl' onClick="self.close()">
+<input type="button" name="close" value='$cl' onClick="self.close()" />
 END
             $groupimportbutton=<<END;
 <input type="button" name="groupimport" value='$gi'
-onClick="javascript:select_group()">
+onClick="javascript:select_group()" />
 END
         }
 	# Additions made by Matthew to make the browser a little easier to deal
@@ -591,7 +591,7 @@ END
 	
 # ---------------------------- embed hidden information useful for group import
 	$r->print("<form name='fnum'>");
-	$r->print("<input type='hidden' name='fnum' value='$fnum'></form>");
+	$r->print("<input type='hidden' name='fnum' value='$fnum' /></form>");
 
 # -------------------------------------------------------------- end the tables
 	$r->print('</table>');
@@ -824,12 +824,12 @@ sub display_line {
 		   'enctype="application/x-www-form-urlencoded"'.
                    '>'."\n");
 	$r->print ('<input type=hidden name=openuri value="'.
-		   $startdir.'">'."\n");
-	$r->print ('<input type="hidden" name="acts" value="">'."\n");
+		   $startdir.'" />'."\n");
+	$r->print ('<input type="hidden" name="acts" value="" />'."\n");
 	$r->print ('<input src="'.$iconpath.'arrow_up.gif"');
-	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'.
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0" />'.
 		   "\n");
-	$r->print(&mt("Up")." $tabtag</tr></form>\n");
+	$r->print(&mt("Up")."</form> $tabtag</tr>\n");
 	return OK;
     }
 # Do we have permission to look at this?
@@ -841,7 +841,7 @@ sub display_line {
 
 # display domain
     if ($filecom[1] eq 'domain') {
- 	$r->print ('<input type="hidden" name="dirPointer" value="on">'."\n")
+ 	$r->print ('<input type="hidden" name="dirPointer" value="on" />'."\n")
  	    if ($ENV{'form.dirPointer'} eq "on");
 	$r->print("<tr valign='$valign' bgcolor=$fileclr>$extrafield");
 	$r->print("<td>");
@@ -849,10 +849,10 @@ sub display_line {
 	my $anchor = $filecom[0];
 	$anchor =~ s/\///g;
 	$r->print ('<a name="'.$anchor.'">');
-	$r->print ('<input type="hidden" name="acts" value="">');
+	$r->print ('<input type="hidden" name="acts" value="" />');
 	$r->print ('<input src="'.$iconpath.'folder_pointer_'.
 		   $diropen.'.gif"'); 
-	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'.
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0" />'.
 		   "\n");
 	my $quotable_filecom = &Apache::loncommon::escape_single($filecom[0]);
 	$r->print ('<a href="javascript:gothere(\''.$quotable_filecom.
@@ -863,7 +863,7 @@ sub display_line {
 	    $r->print("(".$Apache::lonnet::domaindescription{$listname}.
 		      ")");
 	}
-	$r->print (" $tabtag</tr></form>\n");
+	$r->print ("</form> $tabtag</tr>\n");
 	return OK;
 
 # display user directory
@@ -877,10 +877,10 @@ sub display_line {
 	&begin_form ($r,$curdir);
 	$r->print ('<a name="'.$anchor.'"><img src="'.$iconpath.
 		   'whitespace1.gif" border="0" />'."\n");
-	$r->print ('<input type="hidden" name="acts" value="">');
+	$r->print ('<input type="hidden" name="acts" value="" />');
 	$r->print ('<input src="'.$iconpath.'folder_pointer_'.$diropen.
 		   '.gif"'); 
-	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'.
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0" />'.
 		   "\n");
 	my $quotable_curdir = &Apache::loncommon::escape_single($curdir);
 	$r->print ('<a href="javascript:gothere(\''.$quotable_curdir
@@ -891,7 +891,7 @@ sub display_line {
 	my $plainname=&Apache::loncommon::plainname($listname,$domain);
 	$r->print ($listname);
 	if (defined($plainname) && $plainname) { $r->print(" ($plainname) "); }
-	$r->print ($tabtag.'</tr></form>'."\n");
+	$r->print ('</form>'.$tabtag.'</tr>'."\n");
 	return OK;
     }
 
@@ -910,7 +910,7 @@ sub display_line {
         }
 	# Set the icon for the file
 	my $iconname = &Apache::loncommon::icon($listname);
-	$r->print("<tr valign='$valign' bgcolor=$fileclr><td nowrap='1' align='top'>");
+	$r->print("<tr valign='$valign' bgcolor=$fileclr><td nowrap='nowrap' valign='top'>");
 	
         if ($ENV{'form.catalogmode'} eq 'interactive') {
 	    my $quotable_filelink = &Apache::loncommon::escape_single($filelink);
@@ -927,7 +927,7 @@ sub display_line {
 	    if ($hash{'store_'.$filelink}) {
 		$r->print("checked");
 	    }
-	    $r->print(">\n");
+	    $r->print(" />\n");
 	    $r->print("</form>\n");
 	    $r->print("</td><td nowrap>");
 	    $hash{"pre_${fnum}_link"}=$filelink;
@@ -962,10 +962,10 @@ sub display_line {
 	    my $curdir = $startdir.$filecom[0].'/';
 	    my $anchor = $curdir;
 	    $anchor =~ s/\///g;
-	    $r->print ('<input type="hidden" name="acts" value="">');
+	    $r->print ('<input type="hidden" name="acts" value="" />');
 	    $r->print ('<a name="'.$anchor.'"><input src="'.$iconpath.
 		       'folder_pointer_'.$diropen.'.gif"');
-	    $r->print (' name="'.$msg.'" height="22" type="image" border="0">'.
+	    $r->print (' name="'.$msg.'" height="22" type="image" border="0" />'.
 		       "\n");
 	}
 # Filetype icons
@@ -1115,10 +1115,10 @@ sub display_line {
 	    $r->print ("<img src=",$iconpath,"whitespace",$rem,
 		       ".gif border='0' />\n") if $rem > 0;
 	}
-	$r->print ('<input type="hidden" name="acts" value="">');
+	$r->print ('<input type="hidden" name="acts" value="" />');
 	$r->print ('<a name="'.$anchor.'"><input src="'.$iconpath.
 		   'folder_pointer_'.$diropen.'.gif"');
-	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'.
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0" />'.
 		   "\n");
 	my $quotable_curdir = &Apache::loncommon::escape_single($curdir);
 	$r->print ('<a href="javascript:gothere(\''
@@ -1200,9 +1200,9 @@ sub begin_form {
 	       '" onSubmit="return rep_dirpath(\''.$dnum.'\''.
 	       ',document.forms.fileattr.acts.value)" '.
 	       'enctype="application/x-www-form-urlencoded">'."\n");
-    $r->print ('<input type="hidden" name="openuri" value="'.$uri.'">'.
+    $r->print ('<input type="hidden" name="openuri" value="'.$uri.'" />'.
 	       "\n");
-    $r->print ('<input type="hidden" name="dirPointer" value="on">'."\n");
+    $r->print ('<input type="hidden" name="dirPointer" value="on" />'."\n");
     $dnum++;
 }