--- loncom/interface/printout.pl 2005/05/03 21:15:28 1.79 +++ loncom/interface/printout.pl 2006/04/24 21:30:06 1.98 @@ -1,7 +1,6 @@ #!/usr/bin/perl # CGI-script to run LaTeX, dvips, ps2ps, ps2pdf etc. # -# $Id: printout.pl,v 1.79 2005/05/03 21:15:28 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -29,79 +28,38 @@ use lib '/home/httpd/lib/perl'; use LONCAPA::loncgi; use File::Path; +use File::Basename; use IO::File; use Image::Magick; -use Apache::lonhtmlcommon; -use Apache::lonnet(); -use Apache::loncommon; +use Apache::lonhtmlcommon(); +use Apache::lonnet; +use Apache::loncommon(); use Apache::lonlocal; -use Apache::lonmsg; +use Apache::lonmsg(); use LONCAPA::Enrollment; use strict; -# Get the list of course coordinators authorized on this course: -#Parameters: -# id - Course id. -# dom - Domain of course. -#Returns: -# A (possibly empty) list of course coordinators. Each course coordinator -# is given as user:domain -# -sub get_cc_list { - my ($id, $dom) = @_; - my @personel = &Apache::lonnet::getkeys('nohist_userroles', $dom, $id); - my @coordinators; - foreach my $person (@personel) { - my @info = split(/:/, $person); - my $role = $info[0]; - my $user = $info[1]; - my $udom = $info[2]; - if ($role eq 'cc') { - my $isactive = &LONCAPA::Enrollment::check_user_status($udom, $user, $dom, $id, 'cc'); - if ($isactive eq 'ok') { - push @coordinators, $user.':'.$udom; - } - } - } - return (@coordinators); -} -# + # Determine if a user is operating as a student for this course/domain. -# -# #Parameters: -# course - The course id. -# cdom - The course domain. -# +# none #Implicit: # $env{request.role} contains the role under which this user operated this # this request. sub is_student { - my ($course, $cdom) = @_; - my $active_role = $env{'request.role'}; - my @role_fields = split(/\//, $active_role); - - - if ( ($role_fields[0] eq 'st.') && ($role_fields[1] eq $cdom) && - ($role_fields[2] eq $course)) { - return 1; - } else { - return 0; - } - - + return ($env{'request.role'}=~/^st\./); } # # Debugging: Dump the environment for debugging. # sub dumpenv { - print "

-------------------

"; + print "
-------------------
"; foreach my $key (sort (keys %env)) { print "
$key -> $env{$key}"; } - print "

-------------------

"; + print "
-------------------
"; } # @@ -110,6 +68,7 @@ sub dumpenv { # a student printing inside a course results in messages to the course coordinator. #Parmaeters: # identifier - The unique identifier of this cgi request. +# badresource- Filepath to most likely failing # logfile - The contents of the log file (included in the message). # texfile - reference to an array containing the LaTeX input file # (included in the message). @@ -123,14 +82,18 @@ sub dumpenv { # was attempted. # sub send_error_mail { - my ($identifier,$logfile, $texfile) = @_; + my ($identifier, $badresource, $logfile, $texfile) = @_; my $user = $env{"cgi.$identifier.user"}; my $domain = $env{"cgi.$identifier.domain"}; my $courseid = $env{"cgi.$identifier.courseid"}; my $coursedom= $env{"cgi.$identifier.coursedom"}; my $resources= $env{"cgi.$identifier.resources"}; - # &dumpenv(); + # resource file->URL + # + my $badurl = &Apache::lonnet::declutter($badresource); + + # &dumpenv(); @@ -140,20 +103,22 @@ sub send_error_mail { if(defined($user) && defined($domain) && defined($courseid) && defined($coursedom) && defined($resources) ){ - my @cclist = &get_cc_list($courseid, $coursedom); - # Only mail if the conditions are ripe for it: # The user is a student in the course: # - if (&is_student( $courseid, $coursedom)) { + if (&is_student()) { # build the subject and message body: - print "sending message to course coordinators.
"; - - my $subject = "Print failed for $user".'@'.$domain; + # print "sending message to course coordinators.
"; + + # Todo: Convert badurl into a url from file path: + + my $subject = "Error [$badurl] Print failed for $user".'@'.$domain; my $message .= "Print failed to render LaTeX for $user".'@'."$domain\n"; $message .= " User was attempting to print: \n"; - $message .= " $resources\n"; + foreach my $resource (split(/:/,$resources)) { + $message .= " $resource\n"; + } $message .= "--------------------LaTeX logfile:------------ \n"; $message .= $logfile; $message .= "-----------------LaTeX source file: ------------\n"; @@ -161,7 +126,11 @@ sub send_error_mail { foreach my $line (@$texfile) { $message .= "$line\n"; } - foreach my $dest (@cclist) { + my (undef, %receivers) = &Apache::lonfeedback::decide_receiver(undef, 0, + 1,1,1); + # print "
sending...section: $env{'request.course.sec'}"; + foreach my $dest (keys %receivers) { + # print "
dest is $dest"; my @destinfo = split(/:/,$dest); my $user = $destinfo[0]; my $dom = $destinfo[1]; @@ -192,8 +161,7 @@ END } &Apache::lonlocal::get_language_handle(); &Apache::loncommon::content_type(undef,'text/html'); - my $bodytag=&Apache::loncommon::bodytag('Creating PDF','',''); - print $bodytag; + print(&Apache::loncommon::start_page('Creating PDF')); my $identifier = $ENV{'QUERY_STRING'}; my $texfile = $env{'cgi.'.$identifier.'.file'}; @@ -208,14 +176,14 @@ END my $student_names = $env{'cgi.'.$identifier.'.studentnames'}; my $backref = &Apache::lonnet::unescape($env{'cgi.'.$identifier.'.backref'}); - - my $adv = $env{'request.role.adv'}; my @names_pack=(); if ($student_names=~/_END_/) { @names_pack=split(/_ENDPERSON_/,$student_names); } +print "Return to last resource.

"; + my $figfile = $texfile; $figfile =~ s/^([^\.]+printout)[^t]+\.tex/$1\.dat/; my $duefile = $texfile; @@ -230,7 +198,8 @@ END close $temporary_file; my $noteps; my %prog_state; - if ($adv) { %prog_state=&Apache::lonhtmlcommon::Create_PrgWin('','Coverting Images to EPS','Picture Conversion Status',$#content_of_file,'inline','80'); } + if ($advanced_role) { %prog_state=&Apache::lonhtmlcommon::Create_PrgWin('','Coverting Images to EPS','Picture Conversion Status',$#content_of_file,'inline','80'); } + print('
'); foreach my $not_eps (@content_of_file) { chomp($not_eps); if ($not_eps ne '') { @@ -239,7 +208,8 @@ END # print "$status_statement\n"; $not_eps=~s|\/\.\/|\/|g; my $eps_f = $not_eps; - $eps_f =~ s/\.[^.]*$/\.eps/i; + # $eps_f =~ s/\.[^.]*$/\.eps/i; + $eps_f .= '.eps'; # Just append the eps ext. if ($eps_f=~/\/home\/([^\/]+)\/public_html\//) { $eps_f=~s/\/home\/([^\/]+)\/public_html/$1/; $eps_f = '/home/httpd/prtspool/'.$eps_f; @@ -259,7 +229,7 @@ END $not_eps =~ s/\s+$//; $not_eps =~ s/ /\\ /g; if ( exists($done_conversion{$not_eps})) { next; } - if ($adv) { + if ($advanced_role) { my $prettyname=$not_eps; $prettyname=~s|/home/([^/]+)/public_html|/priv/$1|; $prettyname=~s|$Apache::lonnet::perlvar{'lonDocRoot'}/|/|; @@ -268,18 +238,39 @@ END $done_conversion{$not_eps}=1; # print "Converting $not_eps -> $eps_f"; # Debugging system("convert $not_eps $eps_f"); - #check is eps exist in prtspool - if(not -e $eps_f) { + # check is eps exist in prtspool + if (not -e $eps_f) { + # converting an animated gif creates either: + # anim.gif.eps.0 + # or + # anim.gif-0.eps for (my $i=0;$i<10000;$i++) { if (-e $eps_f.'.'.$i) { - rename $eps_f.'.'.$i, $eps_f; + rename($eps_f.'.'.$i, $eps_f); last; } + my $anim_eps = $eps_f; + $anim_eps =~ s/(\.[^.]*)\.eps$/$1-$i\.eps/i; + if (-e $anim_eps) { + rename($anim_eps, $eps_f); + last; + } + } + } + # imagemagick 6.2.0-6.2.7 fails to properly handle + # convert anim.gif anim.gif.eps + # it creates anim.eps instead. + if (not -e $eps_f) { + my $eps_f2 = $eps_f; + $eps_f2 =~ s/\.[^.]*\.eps$/\.eps/i; + if(-e $eps_f2) { + rename($eps_f2,$eps_f); } - } + } + } } - if ($adv) { + if ($advanced_role) { &Apache::lonhtmlcommon::Close_PrgWin('',\%prog_state); } unlink($figfile); @@ -290,16 +281,17 @@ END @texfile=(); for (my $i=1;$i<=$number_of_files;$i++) { my $new_texfile=$texfile; - $new_texfile=~s/\.tex/_$i\.tex/; + $new_texfile=~s/\.tex//; + $new_texfile = sprintf("%s_%03d.tex", $new_texfile,$i); push @texfile,$new_texfile; } } my $ind=-1; my %prog_state; -print "Return to last resource.

"; -if ($adv) { %prog_state=&Apache::lonhtmlcommon::Create_PrgWin('','Print Status','Class Print Status',$number_of_files,'inline','80'); } +if ($advanced_role) { %prog_state=&Apache::lonhtmlcommon::Create_PrgWin('','Print Status','Class Print Status',$number_of_files,'inline','80'); } print "
"; +my $num_files = @texfile; foreach $texfile (@texfile) { my $status_statement=''; my $link_text='download PDF'; @@ -307,36 +299,65 @@ foreach $texfile (@texfile) { my @stud_info=split(/_END_/,$names_pack[$ind]); my @tempo_array=split(/:/,$stud_info[0]); my $name; + my $name_range=''; if ($tempo_array[3]) { $name=$tempo_array[3]; + ($name_range) = split(/,/,$name, 2); } else { $name=$tempo_array[0].'@'.$tempo_array[1]; + $name_range = $tempo_array[0]; + } + if (($name ne "") && ($name ne '@') ) { # Could be printing codes... + $link_text=''.$name.''; + $status_statement.=$name; } - $link_text=''.$name.''; - $status_statement.=$name; if ($#stud_info>0) { @tempo_array=split(/:/,$stud_info[-1]); if ($tempo_array[3]) { $name=$tempo_array[3]; + my ($lastname) = split(/,/, $name,2); + $name_range .= "-".$lastname; } else { $name=$tempo_array[0].'@'.$tempo_array[1]; + $name_range .= '-'.$tempo_array[0]; + } + if (($name ne "") && ($name ne '@')) { + $link_text.=' - '.$name.''; + $status_statement.=' - '.$name; + } - $link_text.=' - '.$name.''; - $status_statement.=' - '.$name; - } - if ($adv) { &Apache::lonhtmlcommon::Update_PrgWin('',\%prog_state,'Creating PDF for '.$status_statement); } + } + if(($num_files > 1) && ($link_text eq 'download PDF')) { # Printing codes + $link_text = ''.basename($texfile,'.tex').'.pdf'; + $status_statement .= basename($texfile); + } + $name_range =~ s/'//g; # O'Neil -> ONeil e.g. + print "
"; + if ($advanced_role) { &Apache::lonhtmlcommon::Update_PrgWin('',\%prog_state,'Creating PDF for '.$status_statement); } + # This little piece of dirt puts username ranges into the original tex + # Tex filename from which they'll propagate into the other filenames as well. + # if (-e $texfile) { + if (($name_range ne '') && ($num_files > 1)) { + my $newtexfile = $texfile; + $newtexfile =~ s/\.tex/$name_range\.tex/; + rename($texfile, $newtexfile); + $texfile = $newtexfile; + } $texfile =~ m/^(.*)\/([^\/]+)$/; my $name_file = $2; my $path_file = $1.'/'; chdir $path_file; - my $dvi_file= $name_file; $dvi_file =~ s/\.tex/\.dvi/; + my $dvi_file= $name_file; $dvi_file =~ s/\.tex/$name_range\.dvi/; &busy_wait_command("latex $name_file 1>/dev/null 2>/dev/null", "for $status_statement now LaTeXing file", \%prog_state,$dvi_file); if ($tableofcontents eq 'yes') { &busy_wait_command("latex $name_file 1>/dev/null 2>/dev/null", - "for $status_statement now LaTeXing file for table of contents", + "for $status_statement First LaTeX of file for table of contents", + \%prog_state,$dvi_file); + &busy_wait_command("latex $name_file 1>/dev/null 2>/dev/null", + "for $status_statement Second LaTeX of file for table of contents", \%prog_state,$dvi_file); } #to create table of contents my $idxname=$name_file; @@ -359,30 +380,42 @@ foreach $texfile (@texfile) { $temporary_file = IO::File->new('>'.$logfilename); print $temporary_file 'LOGFILE
'.$body_log_file.'
'."\n"; if ($body_log_file=~m/!\s+Emergency stop/) { - #LaTeX failed to parse tex file - print "

LaTeX could not successfully parse your tex file.

"; - print "It probably has errors in it.
"; my $whereitbegins = rindex $body_log_file,'STAMPOFPASSEDRESOURCESTART'; my $whereitends = rindex $body_log_file,'STAMPOFPASSEDRESOURCEEND'; + my $badresource; + my $badtext; if ($whereitbegins!=-1 and $whereitends!=-1) { - print "With very high probability this error occured in ".substr($body_log_file,$whereitbegins+26,$whereitends-$whereitbegins-26)."

"; - } - print "Here are the error messages in the LaTeX log file

";
-	  my $sygnal = 0;
-	  for (my $i=0;$i<=$#content_of_file;$i++) {
-	      if ($content_of_file[$i]=~m/^Runaway argument?/ or $content_of_file[$i]=~m/^!/) {
-		  $sygnal = 1;
-	      } 
-	      if ($content_of_file[$i]=~m/Here is how much of/) {
-		  $sygnal = 0;
-	      } 
-	      if ($sygnal) {
-		  print "$content_of_file[$i]";
-	      }  
+	      $badtext = substr($body_log_file,$whereitbegins+26, $whereitends-$whereitbegins-26);
+	      $whereitbegins  = rindex $badtext,'located in';
+	      if ($whereitbegins != -1) {
+		  
+		  $badresource = substr($badtext, $whereitbegins+27, 
+					length($badtext) - $whereitbegins - 48);
+		  # print "
failing resourcename: $badresource
"; + } } - print "
\n"; + if ($advanced_role) { - print "

Advanced role

"; + #LaTeX failed to parse tex file + print "

LaTeX could not successfully parse your tex file.

"; + print "It probably has errors in it.
"; + print "With very high probability this error occured in ".$badtext."

"; + print "Here are the error messages in the LaTeX log file
";
+
+	      my $sygnal = 0;
+	      for (my $i=0;$i<=$#content_of_file;$i++) {
+		  if ($content_of_file[$i]=~m/^Runaway argument?/ or $content_of_file[$i]=~m/^!/) {
+		      $sygnal = 1;
+		  } 
+		  if ($content_of_file[$i]=~m/Here is how much of/) {
+		      $sygnal = 0;
+		  } 
+		  if ($sygnal) {
+		      print "$content_of_file[$i]";
+		  }  
+	      }
+	      print "
\n"; + # print "
Advanced role
"; print "The link to "; $logfilename=~s/\/home\/httpd//; print "Your log file "; @@ -400,15 +433,24 @@ foreach $texfile (@texfile) { $texfile=~s/\/home\/httpd//; print "Your original LaTeX file "; print "\n"; + my $help_text = &Apache::loncommon::help_open_topic("Print_Resource", "Help on printing"); + print ("$help_text"); + } else { # Student role... # at this point: # $body_log_file - contains the log file. # $name_file - is the name of the LaTeX file. # $identifier - is the unique LaTeX identifier.l + print "
There are errors in $badtext"; + print "
These errors prevent this resource from printing correctly"; my $tex_handle = IO::File->new($name_file); my @tex_contents = <$tex_handle>; - &send_error_mail($identifier, $body_log_file, \@tex_contents); + &send_error_mail($identifier, $badresource, $body_log_file, \@tex_contents); + print "
A message has been sent to the instructor describing this failure
"; + my $help_text = &Apache::loncommon::help_open_topic("Print_Resource", "Help on printing"); + print ("$help_text"); + } } elsif ($body_log_file=~m//) { @@ -438,7 +480,10 @@ foreach $texfile (@texfile) { my $ps_file = my $tempo_file = $1.'temporar.ps'; my $pdf_file = $1.'.pdf'; if ($laystyle eq 'album' and $numberofcolumns eq '2') { - $comma = "psnup -2 -s1.0 $new_name_file"; + my $papera=$paper; + if ($papera eq 'letter') {$papera='';} + if ($papera ne '') {$papera='-p'.$papera;} + $comma = "psnup $papera -2 -s1.0 $new_name_file"; &busy_wait_command("$comma $tempo_file 1>/dev/null 2>/dev/null", "for $status_statement now Modifying PS layout", \%prog_state,$tempo_file); @@ -508,13 +553,13 @@ foreach $texfile (@texfile) { $new_name_file =~ m/^(.*)\./; my $ps_file = my $tempo_file = $1.'temporar.ps'; my $pdf_file = $1.'.pdf'; + $papera=~s/t/p/; if ($laystyle eq 'album' and $numberofcolumns eq '2') { - $comma = "psnup -2 -s1.0 $new_name_file"; + $comma = "psnup $papera -2 -s1.0 $new_name_file"; &busy_wait_command("$comma $tempo_file 1>/dev/null 2>/dev/null", "for $status_statement now Modifying PS layout", \%prog_state,$tempo_file); } elsif ($laystyle eq 'book' and $numberofcolumns eq '2') { - $papera=~s/t/p/; $comma = 'pstops '.$papera.' "2:0+1(0.48w,0)"'; &busy_wait_command("$comma $new_name_file $tempo_file 1>/dev/null 2>/dev/null", "for $status_statement now Modifying PS layout", @@ -581,8 +626,8 @@ if ($number_of_files>1) { $zipfile=~s/\/home\/httpd//; print "
A ZIP file of all the PDFs."; } -if ($adv) { &Apache::lonhtmlcommon::Close_PrgWin('',\%prog_state); } - +if ($advanced_role) { &Apache::lonhtmlcommon::Close_PrgWin('',\%prog_state); } +print(&Apache::loncommon::end_page()); my $done; sub REAPER { $done=1; @@ -594,7 +639,7 @@ sub busy_wait_command { $SIG{CHLD} = \&REAPER; $done=0; my $pid=open(CMD,"$command |"); - if ($adv) { + if ($advanced_role) { &Apache::lonhtmlcommon::Update_PrgWin('',$progress_win,$message); } while(!$done) { @@ -604,7 +649,7 @@ sub busy_wait_command { my $size=(stat($output_file))[7]; $extra_msg=", $size bytes generated"; } - if ($adv) { + if ($advanced_role) { &Apache::lonhtmlcommon::Update_PrgWin('',$progress_win, $message.$extra_msg); }