--- loncom/homework/lonhomework.pm 2002/01/17 12:23:31 1.69
+++ loncom/homework/lonhomework.pm 2002/01/30 17:39:13 1.70
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# The LON-CAPA Homework handler
#
-# $Id: lonhomework.pm,v 1.69 2002/01/17 12:23:31 harris41 Exp $
+# $Id: lonhomework.pm,v 1.70 2002/01/30 17:39:13 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -218,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 $fileoutUnable 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