[LON-CAPA-cvs] cvs: loncom /interface multidownload.pl

[banghart]

banghart		Mon May 21 18:37:52 2007 EDT

  Modified files:              
    /loncom/interface	multidownload.pl 
  Log:
  	Make style better.
  	Build time/date stamp better.
  
  
Index: loncom/interface/multidownload.pl
diff -u loncom/interface/multidownload.pl:1.28 loncom/interface/multidownload.pl:1.29
--- loncom/interface/multidownload.pl:1.28	Mon May 21 18:26:41 2007
+++ loncom/interface/multidownload.pl	Mon May 21 18:37:49 2007
@@ -2,7 +2,7 @@
 # CGI-script to allow download of all essay submissions of 
 # multiple students.
 #
-# $Id: multidownload.pl,v 1.28 2007/05/21 22:26:41 banghart Exp $
+# $Id: multidownload.pl,v 1.29 2007/05/21 22:37:49 banghart Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -118,8 +118,8 @@
     if (!open(MANIFEST, ">$doc_zip_root/zipdir/$unique_user/$unique_path/manifest.txt")) {
         &Apache::lonnet::logthis("Problem making manifest");
     }
-    print(MANIFEST &mt("Zip file generated ").&Apache::lonlocal::locallocaltime(time())."\n");
-    print(MANIFEST &mt("Files contained in this zip:\n"));
+    print MANIFEST (&mt("Zip file generated on [_1]",&Apache::lonlocal::locallocaltime(time()))."\n");
+    print MANIFEST (&mt("Files contained in this zip:\n"));
     my $file_problem = 0;
     my $current_student = 0;
     foreach my $stu (@stuchecked) {
@@ -129,7 +129,7 @@
         my ($stuname,$studom,$fullname) = split(/:/,$stu);
         my %record = &Apache::lonnet::restore($symb,$courseid,$studom,$stuname);
         my $file_url = '/uploaded/'.$studom.'/'.$stuname.'/portfolio';
-        print(MANIFEST $fullname."\n");
+        print MANIFEST ($fullname."\n");
         foreach my $partid (@$partlist) {
             my @ids = $res->responseIds($partid);
             foreach my $respid(@ids) {
@@ -138,7 +138,7 @@
                 foreach my $file (split(',',$record{"resource.$partid.$respid.portfiles"})) {
                     $file = $file_url.$file;
                     my ($file_name_only) = ($file =~ /^.*\/(.+$)/);
-                    print(MANIFEST "\t$file_name_only \n");
+                    print MANIFEST ("\t$file_name_only \n");
                     &Apache::lonnet::repcopy($file);
                     my $source = &Apache::lonnet::filelocation("",$file);
                     
@@ -177,7 +177,7 @@
     &Apache::lonhtmlcommon::Close_PrgWin('',\%prog_state);
     print('<p><a href="/zipspool/zipout/'.$unique_user.'/'.$zipout.'">'.
             &mt("Click to download").'</a></p><br />');
-    close MANIFEST;
+    close(MANIFEST);
 } else {
     print(&mt('You are not authorized to download student submissions.'));
 }