--- loncom/interface/lonindexer.pm	2001/05/19 14:31:45	1.3
+++ loncom/interface/lonindexer.pm	2001/06/21 20:57:12	1.11
@@ -6,7 +6,13 @@
 # 07/20-08/04 H.K. Ng
 #
 # 05/9-05/19/2001 H. K. Ng
-#
+# 05/21/2001 H. K. Ng
+# 05/23/2001 H. K. Ng
+# 05/31/2001 Scott Harrison
+# 06/01/2001 Scott Harrison
+# 06/02/2001 Scott Harrison
+# 06/15/2001 Scott Harrison
+
 package Apache::lonindexer;
 
 use strict;
@@ -15,9 +21,12 @@ use Apache::Constants qw(:common);
 use Apache::File;
 use GDBM_File;
 
+my %hash;
 my %dirs;
 my %language;
-
+my $hidden;
+my $extrafield;
+my $fnum;
 sub BEGIN {
     my $fh=Apache::File->new($Apache::lonnet::perlvar{'lonTabDir'}.'/language.tab');
     map {
@@ -31,8 +40,8 @@ sub handler {
     $r->content_type('text/html');
     $r->send_http_header;
     return OK if $r->header_only;
-
-    my $iconpath= $r->dir_config('lonIconsURL');
+    $fnum=0;
+    my $iconpath= $r->dir_config('lonIconsURL') . "/";
     my $domain  = $r->dir_config('lonDefDomain');
     my $role    = $r->dir_config('lonRole');
     my $loadlim = $r->dir_config('lonLoadLim');
@@ -41,6 +50,112 @@ sub handler {
     my $lonhost = $r->dir_config('lonHostID');
     my $tabdir  = $r->dir_config('lonTabDir');
 
+    my $fileclr='#ffffe6';
+# -------------------------------------- see if called from an interactive mode
+    map {
+       my ($name, $value) = split(/=/,$_);
+       $value =~ tr/+/ /;
+       $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C",hex($1))/eg;
+       if ($name eq 'catalogmode') {
+           $ENV{'form.'.$name}=$value;
+	   }
+    } (split(/&/,$ENV{'QUERY_STRING'}));
+
+    $hidden=''; my $closebutton='';
+    my $groupimportbutton='';
+    my $colspan=''; 
+    if ($ENV{'form.catalogmode'} eq 'interactive') {
+	$extrafield='<td bgcolor="'.$fileclr.'" valign="bottom">'.
+	    '<a name="$anchor"><img src="'.$iconpath.'whitespace1.gif"'.
+	    ' border="0"></td>';
+	$colspan=" colspan='2' ";
+	$hidden=<<END;
+<input type='hidden' name='catalogmode' value='interactive'>
+END
+        $closebutton=<<END;
+<input type="button" name="close" value='CLOSE' onClick="self.close()">
+END
+    }
+    elsif ($ENV{'form.catalogmode'} eq 'groupimport') {
+	$extrafield='<td bgcolor="'.$fileclr.'" valign="bottom">'.
+	    '<a name="$anchor"><img src="'.$iconpath.'whitespace1.gif"'.
+	    ' border="0"></td>';
+	$colspan=" colspan='2' ";
+	$hidden=<<END;
+<input type='hidden' name='catalogmode' value='groupimport'>
+END
+        $closebutton=<<END;
+<input type="button" name="close" value='CLOSE' onClick="self.close()">
+END
+        $groupimportbutton=<<END;
+<input type="button" name="groupimport" value='GROUP IMPORT' onClick="javascript:select_group()">
+END
+    }
+
+    my $catalogmodefunctions='';
+    if ($ENV{'form.catalogmode'} eq 'interactive' or
+	$ENV{'form.catalogmode'} eq 'groupimport') {
+	$catalogmodefunctions=<<END;
+function select_data(title,url) {
+    changeTitle(title);
+    changeURL(url);
+    self.close();
+}
+function save_group() {
+    for (var num=0; num<document.forms.fnum.fnum.value; num++) {
+	if (eval("document.forms.form"+num+".filelink.checked")) {
+	    alert(eval("document.forms.form"+num+".title.value")+
+		       eval("document.forms.form"+num+".filelink.value"));
+	}
+    }
+}
+function select_group() {
+    var linkflag=false;
+    for (var num=0; num<document.forms.fnum.fnum.value; num++) {
+	if (eval("document.forms.form"+num+".filelink.checked")) {
+//	    alert(eval("document.forms.form"+num+".title.value")+
+//		       eval("document.forms.form"+num+".filelink.value"));
+	    insertRowInLastRow();
+	    placeResourceInLastRow(
+		   eval("document.forms.form"+num+".title.value"),
+		   eval("document.forms.form"+num+".filelink.value"),
+		   linkflag
+				   );
+	    linkflag=true;
+	}
+    }
+    opener.editmode=0;
+    opener.notclear=0;
+    opener.linkmode=0;
+    opener.infoclear();
+    opener.draw();
+}
+function insertRowInLastRow() {
+    opener.insertrow(opener.maxrow);
+    opener.addobj(opener.maxrow,'e&2');
+}
+function placeResourceInLastRow (title,url,linkflag) {
+    opener.newresource(opener.maxrow,2,opener.escape(title),opener.escape(url),'false','normal');
+    opener.save();
+    opener.mostrecent=opener.obj.length-1;
+    if (linkflag) {
+	opener.joinres(opener.linkmode,opener.mostrecent,0);
+    }
+    opener.linkmode=opener.mostrecent;
+}
+function changeTitle(val) {
+    if (opener.inf.document.forms.resinfo.elements.t) {
+        opener.inf.document.forms.resinfo.elements.t.value=val;
+    }
+}
+function changeURL(val) {
+    if (opener.inf.document.forms.resinfo.elements.u) {
+	opener.inf.document.forms.resinfo.elements.u.value=val;
+    }
+}
+END
+    }
+
 # ---------------------------------------------------------------- Print Header
     $r->print(<<ENDHEADER);
 <html>
@@ -48,6 +163,7 @@ sub handler {
 <title>The LearningOnline Network With CAPA Directory Browser</title>
 
 <SCRIPT language="javascript">
+$catalogmodefunctions
 function openWindow(url, wdwName, w, h, toolbar,scrollbar) {
     var options = "width=" + w + ",height=" + h + ",";
     options += "resizable=yes,scrollbars="+scrollbar+",status=no,";
@@ -64,13 +180,26 @@ ENDHEADER
     my $line;
     my (@attrchk,@openpath);
     my $uri=$r->uri;
-    my $iconpath="/res/adm/pages/indexericons/";
 
     $r->print("<h2><font color=\"\#888888\">The LearningOnline With CAPA Network Directory Browser</font></h2>\n");
 
-    for (my $i=0; $i<=5; $i++) {
-	$attrchk[$i] = "checked" if $ENV{'form.attr'.$i} == 1;
-    }
+    my $diropendb = "/home/httpd/perl/tmp/$domain\_$ENV{'user.name'}_indexer.db";
+
+    if (tie(%hash,'GDBM_File',$diropendb,&GDBM_WRCREAT,0640)) {
+
+	if ($ENV{'form.attrs'} ne "") {
+	    for (my $i=0; $i<=5; $i++) {
+		delete $hash{'display_attrs_'.$i};
+		if ($ENV{'form.attr'.$i} == 1) {
+		    $attrchk[$i] = "checked";
+		    $hash{'display_attrs_'.$i} = 1;
+		}
+	    }
+	} else {
+	    for (my $i=0; $i<=5; $i++) {
+		$attrchk[$i] = "checked" if $hash{'display_attrs_'.$i} == 1;
+	    }
+	}
     $r->print(<<END);
 <b><font color="#666666">Display file attributes</font></b><br>
 <form method="post" name="fileattr" action="$uri" enctype="application/x-www-form-urlencoded">
@@ -83,52 +212,68 @@ ENDHEADER
 <td><input type=checkbox name=attr4 value="1" $attrchk[4]> Keywords</td>
 <td><input type=checkbox name=attr5 value="1" $attrchk[5]> Language</td>
 </tr></table>
-<input type="submit" name="dirlistattr" value="Review">&nbsp;
-<input type="submit" name="dirlistattr" value="Refresh">
+<input type="submit" name="attrs" value="Review">&nbsp;
+<input type="submit" name="attrs" value="Refresh">
+$hidden
+$closebutton
+$groupimportbutton
 </form>
 END
 
-    my $diropen = "/home/httpd/perl/tmp/$domain$ENV{'user.name'}_diropen.db";
-
-    if (tie(%dirs,'GDBM_File',$diropen,&GDBM_WRCREAT,0640)) {
-	my $titleclr="#ddffff";
-	$r->print("<table border=0><tr><td bgcolor=#eeeeee>\n");
-	$r->print("<table border=0><tr>\n");
-	$r->print("<td bgcolor=$titleclr><b>Name</b></td>\n");
-	$r->print("<td bgcolor=$titleclr align=right><b>Size (bytes) </b></td>\n") if ($ENV{'form.attr0'} == 1);
-	$r->print("<td bgcolor=$titleclr><b>Last accessed</b></td>\n") if ($ENV{'form.attr1'} == 1);
-	$r->print("<td bgcolor=$titleclr><b>Last modified</b></td>\n") if ($ENV{'form.attr2'} == 1);
-	$r->print("<td bgcolor=$titleclr><b>Author(s)</b></td>\n") if ($ENV{'form.attr3'} == 1);
-	$r->print("<td bgcolor=$titleclr><b>Keywords</b></td>\n") if ($ENV{'form.attr4'} == 1);
-	$r->print("<td bgcolor=$titleclr><b>Language</b></td>\n") if ($ENV{'form.attr5'} == 1);
-	$r->print("</tr>");
+    my $titleclr="#ddffff";
+    $r->print("<table border=0><tr><td bgcolor=#eeeeee>\n");
+    $r->print("<table border=0><tr>\n");
+    $r->print("<td $colspan bgcolor=$titleclr><b>Name</b></td>\n");
+    $r->print("<td bgcolor=$titleclr align=right><b>Size (bytes) </b></td>\n") if ($hash{'display_attrs_0'} == 1);
+    $r->print("<td bgcolor=$titleclr><b>Last accessed</b></td>\n") if ($hash{'display_attrs_1'} == 1);
+    $r->print("<td bgcolor=$titleclr><b>Last modified</b></td>\n") if ($hash{'display_attrs_2'} == 1);
+    $r->print("<td bgcolor=$titleclr><b>Author(s)</b></td>\n") if ($hash{'display_attrs_3'} == 1);
+    $r->print("<td bgcolor=$titleclr><b>Keywords</b></td>\n") if ($hash{'display_attrs_4'} == 1);
+    $r->print("<td bgcolor=$titleclr><b>Language</b></td>\n") if ($hash{'display_attrs_5'} == 1);
+    $r->print("</tr>");
+
+	map {
+	    if ($_ =~ /^diropen_status_/) {
+		my $key = $_;
+		$key =~ s/^diropen_status_//;
+		$dirs{$key} = $hash{$_};
+	    }
+	} keys %hash;
 
 	if ($ENV{'form.openuri'}) {  # take care of review and refresh options
 	    my $uri=$ENV{'form.openuri'};
-	    if (exists($dirs{$uri})) {
-		my $cursta = $dirs{$uri};
+	    if (exists($hash{'diropen_status_'.$uri})) {
+		my $cursta = $hash{'diropen_status_'.$uri};
 		$dirs{$uri} = 'open';
-		$dirs{$uri} = 'closed' if $cursta eq 'open';
+		$hash{'diropen_status_'.$uri} = 'open';
+		if ($cursta eq 'open') {
+		    $dirs{$uri} = 'closed';
+		    $hash{'diropen_status_'.$uri} = 'closed';
+		}
 	    } else {
+		$hash{'diropen_status_'.$uri} = 'open';
 		$dirs{$uri} = 'open';
 	    }
 	}
-	sort keys %dirs;
 
-	my $toplevel = "/res/";
-	my $indent = -1;
+#	my $toplevel = "/res/";
+	my $toplevel = $uri;
+	my $indent = 0;
 	&scanDir ($r,$toplevel,$indent);
 
+	$r->print("<form name='fnum'>");
+	$r->print("<input type='hidden' name='fnum' value='$fnum'></form>");
 	$r->print("</table>");
 	$r->print("</td></tr></table>");
 	$r->print("</body></html>\n");
-	untie(%dirs);
+	untie(%hash);
     } else {
 	$r->print("Unable to tie hash to db file");
     }
     return OK;
 }
 
+
 # --------------------recursive scan of a directory
 sub scanDir {
     my ($r,$startdir,$indent)=@_;
@@ -139,22 +284,26 @@ sub scanDir {
     my %dupdirs = %dirs;
     my @list=&get_list($r,$startdir);
     foreach my $line (@list) {
-	my ($strip,$domusr,$foo,$testdir,$foo)=split(/\&/,$line,5); 
-	if ($domusr eq "domain") {
+	my ($strip,$dom,$foo,$testdir,$foo)=split(/\&/,$line,5); 
+	next if $strip =~ /.*\.meta$/;
+	if ($dom eq "domain") {
 	    $compuri = join('',$strip,"/");  # domain list has /res/<domain name>
 	    $curdir = $compuri;
 	} else {
 	    $compuri = join('',$startdir,$strip,"/"); # user, dir & file having name only, i.e., w/o path
 	    $curdir = $startdir;
 	}
-	my $diropen = 0;
-	if (($dirptr&$testdir) or ($domusr =~ /^(domain|user)$/)) {
+	my $diropen = "closed";
+	if (($dirptr&$testdir) or ($dom =~ /^(domain|user)$/)) {
 	    while (my ($key,$val)= each %dupdirs) {
-		$diropen = 1 if ($key eq $compuri and $val eq "open");
+		if ($key eq $compuri and $val eq "open") {
+		    $diropen = "opened";
+		    delete $dupdirs{key},$dirs{$key};
+		}
 	    }
 	}
-	&display_line($r,$diropen,$line,$indent,$curdir);
-	&scanDir ($r,$compuri,$indent) if $diropen == 1;
+	&display_line($r,$diropen,$line,$indent,$curdir,@list);
+	&scanDir ($r,$compuri,$indent) if $diropen eq "opened";
     }
     $indent--;
 }
@@ -164,31 +313,20 @@ sub get_list {
     my ($r,$uri)=@_;
     my @list;
     my $luri = $uri;
-    my $domain  = $r->dir_config('lonDefDomain');
     $luri =~ s/\//_/g;
 
-    if ($ENV{'form.dirlistattr'} eq "Refresh") {
-	my $tmpdir="/home/httpd/perl/tmp";
-	my $filename;
-	opendir(DIR,$tmpdir);
-	while ($filename=readdir(DIR)) {
-	    if ($filename=~/^$domain$ENV{'user.name'}_dirlist.*\.tmp$/) {
-		unlink($tmpdir.'/'.$filename);
-	    }
-	}
-	closedir(DIR);
+    if ($ENV{'form.attrs'} eq "Refresh") {
+	map {
+	    delete $hash{$_} if ($_ =~ /^dirlist_files_/);
+	    } keys %hash;
     }
 
-    my $dirlist = "/home/httpd/perl/tmp/$domain$ENV{'user.name'}_dirlist$luri.tmp";
-    if (-e $dirlist) {
-	my $FH = Apache::File->new($dirlist);
-	@list=<$FH>;
+    if ($hash{'dirlist_files'.$luri}) {
+	@list = split(/\n/,$hash{'dirlist_files_'.$luri});
     } else {
-	@list=&Apache::lonnet::dirlist($uri);
-	my $FH = Apache::File->new(">$dirlist");
-	print $FH join("\n",@list);
+	@list = &Apache::lonnet::dirlist($uri);
+	$hash{'dirlist_files_'.$luri} = join('\n',@list);
     }
-    @list = sort(@list);
     return @list=&match_ext($r,@list);
 }
 
@@ -218,7 +356,6 @@ sub match_ext {
 	next if ($unpackline[0] eq "..");
 	my @filecom = split (/\./,$unpackline[0]);
 	my $fext = pop(@filecom);
-	next if $fext eq "meta";
 	my $fnptr = $unpackline[3]&$dirptr;
  	if ($fnptr == 0 and $unpackline[3] ne "") {
 	    foreach my $nextline (@fileext) {
@@ -228,53 +365,53 @@ sub match_ext {
 	    push @trimlist,$line;
 	}
     }
+    @trimlist = sort (@trimlist);
     return @trimlist;
 }
 
 #------------------- displays one line in appropriate table format
 sub display_line{
-    my ($r,$diropen,$line,$indent,$startdir)=@_;
+    my ($r,$diropen,$line,$indent,$startdir,@list)=@_;
     my (@pathfn, $fndir, $fnptr);
     my $dirptr=16384;
     my $fileclr="#ffffe6";
-    my $iconpath="/res/adm/pages/indexericons/";
+    my $iconpath= $r->dir_config('lonIconsURL') . "/";
 
     my @filecom = split (/\&/,$line);
     my @pathcom = split (/\//,$filecom[0]);
     my $listname = $pathcom[scalar(@pathcom)-1];
     my $fnptr = $filecom[3]&$dirptr;
+    my $msg = 'View '.$filecom[0].' resources';
+    $msg = 'Close '.$filecom[0].' directory' if $diropen eq "opened";
 
     my $tabtag="</td>";
     my $i=0;
 
     while ($i<=5) {
-	my $key="form.attr".$i;
-	$tabtag=join('',$tabtag,"<td bgcolor=",$fileclr,">&nbsp;</td>") if $ENV{$key} == 1;
+	$tabtag=join('',$tabtag,"<td bgcolor=",$fileclr,">&nbsp;</td>") if $hash{'display_attrs_'.$i} == 1;
 	$i++;
     }
     if ($filecom[1] eq "domain") {
-	$r->print("<tr>");
+	$r->print("<tr>$extrafield");
 	$r->print("<td bgcolor=$fileclr valign=bottom>");
 	&begin_form ($r,$filecom[0].'/');
 	my $anchor = $filecom[0].'/';
 	$anchor =~ s/\///g;
-	$r->print ("<a name=\"".$anchor."\">\n<input src=\"".$iconpath."comp.blue.gif\"");
-	$r->print (" name=\"View $filecom[0]/ resources\" height=\"22\" type=\"image\" border=\"0\">\n");
+	$r->print ('<a name="'.$anchor.'"><input src="'.$iconpath.'server.gif"');
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'."\n");
 	$r->print("Domain - $listname $tabtag</tr></form>\n");
 	return OK;
     }
     if ($filecom[1] eq "user") {
-	$r->print("<tr>");
+	$r->print("<tr>$extrafield");
 	$r->print("<td bgcolor=$fileclr valign=bottom>\n");
 	my $curdir = $startdir.$filecom[0].'/';
 	&begin_form ($r,$curdir);
 	my $anchor = $curdir;
 	$anchor =~ s/\///g;
-	$r->print ("<a name=\"$anchor\">\n<img src=",$iconpath,"white_space_20_22.gif border=0>\n");
-	$r->print ("<input src=\"$iconpath");
-	$r->print ("folder_pointer_closed.gif\"") if $diropen == 0;
-	$r->print ("folder_pointer_opened.gif\"") if $diropen == 1;
-	$r->print (" name=\"View $curdir resources\" height=\"22\" type=\"image\" border=\"0\">\n");
+	$r->print ('<a name="'.$anchor.'"><img src="'.$iconpath.'whitespace1.gif" border=0>'."\n");
+	$r->print ('<input src="'.$iconpath.'folder_pointer_'.$diropen.'.gif"'); 
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'."\n");
 	$r->print ("<img src=",$iconpath,"quill.gif border=0>\n");
 	$r->print ("$listname $tabtag</tr></form>\n");
 	return OK;
@@ -284,37 +421,72 @@ sub display_line{
 	my @file_ext = split (/\./,$listname);
 	my $curfext = $file_ext[scalar(@file_ext)-1];
 	my $filelink = $startdir.$filecom[0];
-	my $count = 0;
-	$r->print("<tr><td bgcolor=$fileclr>");
-	while ($count < $indent) {
-	    $r->print("<img src=",$iconpath,"white_space_20_22.gif border=0>\n");
+	$r->print("<tr><td nowrap valign='bottom' bgcolor=$fileclr>");
+	my $metafile = grep /^$filecom[0]\.meta\&/, @list;
+	my $title;
+        if ($ENV{'form.catalogmode'} eq 'interactive') {
+	    $title=$listname;
+	    $title = &Apache::lonnet::metadata($filelink,'title')
+		if ($metafile == 1);
+	    $title=$listname unless $title;
+	    $r->print("<a href='javascript:select_data(\"",
+	              $title,'","',$filelink,"\")'>");
+	    $r->print("<img src='",$iconpath,"select.gif' border=0></a>\n");
+	    $r->print("</td><td valign='bottom' nowrap bgcolor=$fileclr>");
+	}
+        elsif ($ENV{'form.catalogmode'} eq 'groupimport') {
+	    $title=$listname;
+	    $title = &Apache::lonnet::metadata($filelink,'title')
+		if ($metafile == 1);
+	    $title=$listname unless $title;
+	    $r->print("<form name='form$fnum'>\n");
+	    $r->print("<input type='checkbox' name='filelink"."' ".
+		      "value='$filelink'>\n");
+	    $r->print("<input type='hidden' name='title"."' ".
+		      "value='$title'>\n");
+	    $r->print("</form>\n");
+	    $r->print("</td><td valign='bottom' nowrap bgcolor=$fileclr>");
+  	    $fnum++;
+	}
+
+	if ($indent < 11) {
+	    $r->print("<img src=",$iconpath,"whitespace",$indent,".gif border=0>\n");
+	} elsif ($indent >0) {
+	    my $ten = int($indent/10.);
+	    my $rem = $indent%10.0;
+	    my $count = 0;
+	    while ($count < $ten) {
+		$r->print("<img src=",$iconpath,"whitespace10.gif border=0>\n");
 	    $count++;
+	    }
+	    $r->print("<img src=",$iconpath,"whitespace",$rem,".gif border=0>\n") if $rem > 0;
 	}
-	$r->print("<img src=",$iconpath,"white_space_20_22.gif border=0>\n");
+
 	$r->print("<img src=$iconpath$curfext.gif border=0>\n");
-	$r->print(" <a href=$filelink>",$listname,"</a>");
-	my $metafile = '/home/httpd/html'.$filelink.'.meta';
+	$r->print(" <a href=$filelink target='_blank'>",$listname,"</a>\n");
 
-	$r->print (" (<a href=\"javascript:openWindow('".$filelink.".meta', 'metadatafile', '400', '450', 'no', 'yes')\"; TARGET=_self>metadata</a>) ") if (-e $metafile);
+	$r->print (" (<a href=\"javascript:openWindow('".$filelink.".meta', 'metadatafile', '400', '450', 'no', 'yes')\"; TARGET=_self>metadata</a>) ") if ($metafile == 1);
 
-#	$r->print(" (<a href=$filelink.meta target=cat>metadata</a>)") if (-e $metafile);
-	$r=>print("</td>\n");
-	$r->print("<td bgcolor=$fileclr align=right valign=bottom> ",$filecom[8]," </td>\n") if $ENV{'form.attr0'} == 1;
-	$r->print("<td bgcolor=$fileclr valign=bottom> ".(localtime($filecom[9]))." </td>\n") if $ENV{'form.attr1'} == 1;
-	$r->print("<td bgcolor=$fileclr valign=bottom> ".(localtime($filecom[10]))." </td>\n") if $ENV{'form.attr2'} == 1;
+	$r->print("</td>\n");
+	$r->print("<td bgcolor=$fileclr align=right valign=bottom> ",$filecom[8]," </td>\n") 
+	    if $hash{'display_attrs_0'} == 1;
+	$r->print("<td bgcolor=$fileclr valign=bottom> ".(localtime($filecom[9]))." </td>\n") 
+	    if $hash{'display_attrs_1'} == 1;
+	$r->print("<td bgcolor=$fileclr valign=bottom> ".(localtime($filecom[10]))." </td>\n") 
+	    if $hash{'display_attrs_2'} == 1;
 
-	if ($ENV{'form.attr3'} == 1) {
-	    my $author = &Apache::lonnet::metadata($filelink,'author');
+	if ($hash{'display_attrs_3'} == 1) {
+	    my $author = &Apache::lonnet::metadata($filelink,'author') if ($metafile == 1);
 	    $author = '&nbsp;' if (!$author);
 	    $r->print("<td bgcolor=$fileclr valign=bottom> ".$author." </td>\n");
 	}
-	if ($ENV{'form.attr4'} == 1) {
-	    my $keywords = &Apache::lonnet::metadata($filelink,'keywords');
+	if ($hash{'display_attrs_4'} == 1) {
+	    my $keywords = &Apache::lonnet::metadata($filelink,'keywords') if ($metafile == 1);
 	    $keywords = '&nbsp;' if (!$keywords);
 	    $r->print("<td bgcolor=$fileclr valign=bottom> ".$keywords." </td>\n");
 	}
-	if ($ENV{'form.attr5'} == 1) {
-	    my $lang = &Apache::lonnet::metadata($filelink,'language');
+	if ($hash{'display_attrs_5'} == 1) {
+	    my $lang = &Apache::lonnet::metadata($filelink,'language') if ($metafile == 1);
 	    $lang = $language{$lang};
 	    $lang = '&nbsp;' if (!$lang);
 	    $r->print("<td bgcolor=$fileclr valign=bottom> ".$lang." </td>\n");
@@ -328,21 +500,24 @@ sub display_line{
 	my $curdir = $startdir.$filecom[0].'/';
 	my $anchor = $curdir;
 	$anchor =~ s/\///g;
-	$r->print("<tr><td bgcolor=$fileclr valign=bottom>");
+	$r->print("<tr>$extrafield<td bgcolor=$fileclr valign=bottom>");
 	&begin_form ($r,$curdir);
-
-	my $count = 0;
-	while ($count < $indent) {
-	    $r->print("<img src=",$iconpath,"white_space_20_22.gif border=0>\n");
+	my $indentm1 = $indent-1;
+	if ($indentm1 < 11 and $indentm1 > 0) {
+	    $r->print("<img src=",$iconpath,"whitespace",$indentm1,".gif border=0>\n");
+	} else {
+	    my $ten = int($indentm1/10.);
+	    my $rem = $indentm1%10.0;
+	    my $count = 0;
+	    while ($count < $ten) {
+		$r->print("<img src=",$iconpath,"whitespace10.gif border=0>\n");
 	    $count++;
+	    }
+	    $r->print("<img src=",$iconpath,"whitespace",$rem,".gif border=0>\n") if $rem > 0;
 	}
-
-	$r->print ("<a name=\"$anchor\">\n<input src=\"$iconpath");
-	$r->print ("folder_pointer_closed.gif\"") if $diropen == 0;
-	$r->print ("folder_pointer_opened.gif\"") if $diropen == 1;
-	$r->print (" name=\"View $curdir resources\" height=\"22\" type=\"image\" border=\"0\">\n");
-	$r->print("<img src=",$iconpath,"folder_closed.gif border=0>\n") if $diropen == 0;
-	$r->print("<img src=",$iconpath,"folder_opened.gif border=0>\n") if $diropen == 1;
+	$r->print ('<a name="'.$anchor.'"><input src="'.$iconpath.'folder_pointer_'.$diropen.'.gif"');
+	$r->print (' name="'.$msg.'" height="22" type="image" border="0">'."\n");
+	$r->print("<img src=",$iconpath,"folder_$diropen.gif border=0>\n");
 	$r->print("$listname $tabtag</tr></form>\n");
     }
 
@@ -353,13 +528,13 @@ sub begin_form {
     my ($r,$uri) = @_;
     my $anchor = $uri;
     $anchor =~ s/\///g;
-    $r->print ("<form method=\"post\" name=\"dirpath\" action=\"/res/\#$anchor\" enctype=\"application/x-www-form-urlencoded\">\n");
-    $r->print ("<input type=hidden name=openuri value=\"$uri\">\n");
+    $r->print ('<form method="post" name="dirpath" action="/res/#'.$anchor.'" enctype="application/x-www-form-urlencoded">'."\n");
+    $r->print ("$hidden<input type=hidden name=openuri value=\"$uri\">\n");
 
-    for (my $i=0; $i<=5; $i++) {
-	$r->print ("<input type=hidden name=attr$i value=\"1\">\n") if $ENV{'form.attr'.$i} == 1;
-    }
 }
 
 1;
 __END__
+
+
+