--- loncom/interface/printout.pl 2005/05/03 21:15:28 1.79
+++ loncom/interface/printout.pl 2005/07/22 11:42:50 1.85
@@ -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
#
@@ -32,7 +31,7 @@ use File::Path;
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 +39,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 +52,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 "-------------------
";
+ print "
-------------------
";
}
#
@@ -110,6 +72,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 +86,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 +107,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 +128,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];
@@ -239,7 +210,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;
@@ -359,30 +331,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 +389,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 +424,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 +497,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",