--- loncom/interface/printout.pl 2005/05/02 21:57:44 1.77
+++ loncom/interface/printout.pl 2005/08/04 21:26:23 1.89
@@ -1,7 +1,6 @@
#!/usr/bin/perl
# CGI-script to run LaTeX, dvips, ps2ps, ps2pdf etc.
#
-# $Id: printout.pl,v 1.77 2005/05/02 21:57:44 foxr Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -29,10 +28,11 @@
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::lonnet;
use Apache::loncommon;
use Apache::lonlocal;
use Apache::lonmsg;
@@ -40,32 +40,7 @@ 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.
#
@@ -78,30 +53,18 @@ sub get_cc_list {
# $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.adv'});
}
#
# Debugging: Dump the environment for debugging.
#
sub dumpenv {
- print "
-------------------
";
+ print "
-------------------
";
foreach my $key (sort (keys %env)) {
- print "
$key -> $env{$key}";
+ print "
$key -> $env{$key}";
}
- print "
-------------------
";
+ print "
-------------------
";
}
#
@@ -110,6 +73,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 +87,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,17 +108,17 @@ 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)) {
# 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";
@@ -161,7 +129,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];
@@ -201,12 +173,12 @@ END
my $numberofcolumns = $env{'cgi.'.$identifier.'.numcol'};
my $paper = $env{'cgi.'.$identifier.'.paper'};
my $selectionmade = $env{'cgi.'.$identifier.'.selection'};
- my $tableofcontents = $env{'cgi.'.$identifier.'tableofcontents'};
- my $tableofindex = $env{'cgi.'.$identifier.'tableofindex'};
- my $advanced_role = $env{'cgi.'.$identifier.'role'};
- my $number_of_files = $env{'cgi.'.$identifier.'numberoffiles'}+1;
- my $student_names = $env{'cgi.'.$identifier.'studentnames'};
- my $backref = &Apache::lonnet::unescape($env{'cgi.'.$identifier.'backref'});
+ my $tableofcontents = $env{'cgi.'.$identifier.'.tableofcontents'};
+ my $tableofindex = $env{'cgi.'.$identifier.'.tableofindex'};
+ my $advanced_role = $env{'cgi.'.$identifier.'.role'};
+ my $number_of_files = $env{'cgi.'.$identifier.'.numberoffiles'}+1;
+ my $student_names = $env{'cgi.'.$identifier.'.studentnames'};
+ my $backref = &Apache::lonnet::unescape($env{'cgi.'.$identifier.'.backref'});
my $adv = $env{'request.role.adv'};
@@ -239,7 +211,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;
@@ -290,7 +263,8 @@ 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;
}
}
@@ -300,6 +274,7 @@ 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'); }
print "
";
+my $num_files = @texfile;
foreach $texfile (@texfile) {
my $status_statement='';
my $link_text='download PDF';
@@ -307,30 +282,56 @@ 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(($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 ($adv) { &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);
@@ -359,30 +360,41 @@ 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 ";
@@ -406,9 +418,12 @@ foreach $texfile (@texfile) {
# $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
"
}
} elsif ($body_log_file=~m//) {
@@ -438,7 +453,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 +526,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",