--- loncom/interface/portfolio.pm	2006/06/29 18:02:58	1.121.2.4
+++ loncom/interface/portfolio.pm	2006/07/03 04:25:08	1.121.2.5
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # portfolio browser
 #
-# $Id: portfolio.pm,v 1.121.2.4 2006/06/29 18:02:58 albertel Exp $
+# $Id: portfolio.pm,v 1.121.2.5 2006/07/03 04:25:08 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -297,17 +297,17 @@ sub display_directory {
 			      &mt($curr_access).'   ');
 		    $r->print('<a href="'.$url.'?access='.$filename.
 			      '&amp;currentpath='.$current_path.$groupecho.
-			      '">'.$access_admin_text.'</a></span>');
+			      '">'.$access_admin_text.'</a></span></td>');
 		}
-		$r->print('</td></tr>'.$/);
+		$r->print('</tr>'.$/);
             }
         }
     }
     if ($select_mode eq 'true') {
         $r->print('</table>
-            <input type="hidden" name="continue" value="true">
-            <input type="hidden" name="fieldname" value="'.$env{'form.fieldname'}.'">
-            <input type="hidden" name="mode" value="selectfile">
+            <input type="hidden" name="continue" value="true" />
+            <input type="hidden" name="fieldname" value="'.$env{'form.fieldname'}.'" />
+            <input type="hidden" name="mode" value="selectfile" />
             <input type="submit" name="submit" value="Select checked files, and continue selecting." /><br />
             <input type="button" name="doit" onClick= "finishSelect();" value="Select checked files, and close window" />
             <input type="hidden" name="currentpath" value="'.$current_path.'" />
@@ -1422,11 +1422,7 @@ ENDSMP
                     fileList = fileList + document.forms.checkselect.currentpath.value + document.forms.checkselect[i].value + "," ;
                 }
             }
-            opener.document.forms.lonhomework.
-ENDSMP
-    $javascript .= $env{'form.fieldname'};
-    $javascript .= (<<ENDSMP);
-        .value=fileList;
+            opener.document.forms.lonhomework.$env{'form.fieldname'}.value=fileList;
             self.close();
         }
         </script>