--- loncom/homework/lonhomework.pm 2001/12/04 14:51:19 1.64 +++ loncom/homework/lonhomework.pm 2002/02/27 20:23:26 1.71 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Homework handler # -# $Id: lonhomework.pm,v 1.64 2001/12/04 14:51:19 albertel Exp $ +# $Id: lonhomework.pm,v 1.71 2002/02/27 20:23:26 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -34,6 +34,7 @@ use strict; use Apache::style; use Apache::lonxml; use Apache::lonnet; +use Apache::lonplot; use Apache::inputtags; use Apache::structuretags; use Apache::randomlabel; @@ -43,12 +44,16 @@ use Apache::outputtags; use Apache::Constants qw(:common); #use Time::HiRes qw( gettimeofday tv_interval ); -sub BEGIN { +BEGIN { &Apache::lonxml::register_insert(); } sub get_target { if ( $ENV{'request.state'} eq "published") { + if ( defined($ENV{'form.grade_target'}) + && ($Apache::lonhomework::viewgrades == 'F' )) { + return ($ENV{'form.grade_target'}); + } if ( defined($ENV{'form.submitted'})) { return ('grade', 'web'); } else { @@ -203,7 +208,9 @@ sub setupheader { } else { $request->content_type('text/html'); } - if (!$Apache::lonxml::debug) { &Apache::loncommon::no_cache($request); } + if (!$Apache::lonxml::debug && ($ENV{'REQUEST_METHOD'} eq 'GET')) { + &Apache::loncommon::no_cache($request); + } $request->send_http_header; return OK if $request->header_only; return '' @@ -211,32 +218,37 @@ sub setupheader { sub handle_save_or_undo { my ($request,$problem,$result) = @_; - my $fileout = &Apache::lonnet::filelocation("",$request->uri); - my $filesave=$fileout.".bak"; + my $file = &Apache::lonnet::filelocation("",$request->uri); + my $filebak =$file.".bak"; + my $filetmp =$file.".tmp"; my $error=0; if ($ENV{'form.Undo'} eq 'undo') { - if (copy($filesave,$fileout)) { - $request->print("Undid changes, Copied $filesave to $fileout
"); + my $error=0; + if (!copy($file,$filetmp)) { $error=1; } + if ((!$error) && (!copy($filebak,$file))) { $error=1; } + if ((!$error) && (!move($filetmp,$filebak))) { $error=1; } + if (!$error) { + $request->print("

Undid changes, Switched $filebak and $file

"); } else { - $request->print("Unable to undo, unable to copy $filesave to $fileout
"); + $request->print("

Unable to undo, unable to switch $filebak and $file

"); $error=1; } } else { - my $fs=Apache::File->new(">$filesave"); + my $fs=Apache::File->new(">$filebak"); if (defined($fs)) { print $fs $$problem; - $request->print("Making Backup to $filesave
"); + $request->print("Making Backup to $filebak
"); } else { - $request->print("Unable to make backup $filesave"); + $request->print("Unable to make backup $filebak"); $error=2; } - my $fh=Apache::File->new(">$fileout"); + my $fh=Apache::File->new(">$file"); if (defined($fh)) { print $fh $$result; - $request->print("Saving Modifications to $fileout
"); + $request->print("Saving Modifications to $file
"); } else { - $request->print("Unable to write to $fileout"); + $request->print("Unable to write to $file"); $error|=4; } } @@ -279,6 +291,7 @@ sub renderpage { my ($request,$file) = @_; my (@targets) = &get_target(); + &Apache::lonxml::debug("Running targets ".join(':',@targets)); foreach my $target (@targets) { #my $t0 = [&gettimeofday()]; my $problem=&Apache::lonnet::getfile($file); @@ -307,6 +320,7 @@ sub renderpage { &Apache::lonxml::error("Unable to find default_homework.lcpm"); $default=''; } + &Apache::lonxml::debug("Should be parsing now"); $result = &Apache::lonxml::xmlparse($target, $problem, $default.&setup_vars($target),%mystyle); @@ -345,24 +359,44 @@ sub get_template_list { } sub newproblem { - my ($request) = @_; - - my $extension=$request->uri; - $extension=~s:^.*\.([\w]+)$:$1:; - &Apache::lonxml::debug("Looking for :$extension:"); - if ($ENV{'form.template'}) { - use File::Copy; - my $file = &get_template_list($ENV{'form.template'},$extension); - my $dest = &Apache::lonnet::filelocation("",$request->uri); - copy($file,$dest); - &renderpage($request,$file); - } else { - my $templatelist=&get_template_list('',$extension); - my $url=$request->uri; - my $dest = &Apache::lonnet::filelocation("",$request->uri); - $request->print(<uri; + $extension=~s:^.*\.([\w]+)$:$1:; + &Apache::lonxml::debug("Looking for :$extension:"); + if ($ENV{'form.template'}) { + use File::Copy; + my $file = &get_template_list($ENV{'form.template'},$extension); + my $dest = &Apache::lonnet::filelocation("",$request->uri); + copy($file,$dest); + &renderpage($request,$dest); + } elsif($ENV{'form.newfile'}) { + # I don't like hard-coded filenames but for now, this will work. + use File::Copy; + my $templatefilename = + $request->dir_config('lonIncludes').'/templates/blank.problem'; + &Apache::lonxml::debug("$templatefilename"); + my $dest = &Apache::lonnet::filelocation("",$request->uri); + copy($templatefilename,$dest); + &renderpage($request,$dest); + }else { + my $templatelist=&get_template_list('',$extension); + my $url=$request->uri; + my $dest = &Apache::lonnet::filelocation("",$request->uri); + if (!defined($templatelist)) { + # We didn't find a template, so just create a blank problem. + $request->print(< +The requested file $url doesn\'t exist. You can create a new $extension
+
+
+
+ +ENDNEWPROBLEM + return ''; + } + $request->print(< -The request file $url doesn\'t exist. You can create a new $extension
+The requested file $url doesn\'t exist. You can create a new $extension