--- loncom/homework/lonhomework.pm	2003/09/18 19:59:15	1.145
+++ loncom/homework/lonhomework.pm	2004/02/16 20:30:34	1.168
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # The LON-CAPA Homework handler
 #
-# $Id: lonhomework.pm,v 1.145 2003/09/18 19:59:15 albertel Exp $
+# $Id: lonhomework.pm,v 1.168 2004/02/16 20:30:34 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -24,11 +24,7 @@
 # /home/httpd/html/adm/gpl.txt
 #
 # http://www.lon-capa.org/
-#
-# Guy Albertelli
-# 11/30 Gerd Kortemeyer
-# 6/1,8/17,8/18 Gerd Kortemeyer
-# 7/18 Jeremy Bowers
+
 
 package Apache::lonhomework;
 use strict;
@@ -54,6 +50,7 @@ use Apache::chemresponse();
 use Apache::Constants qw(:common);
 use HTML::Entities();
 use Apache::loncommon();
+use Apache::lonlocal;
 #use Time::HiRes qw( gettimeofday tv_interval );
 
 BEGIN {
@@ -89,8 +86,8 @@ sub get_target {
 		return ('web');
 	    }
 	} else {
-	    if ( $ENV{'form.problemmode'} eq 'View' ||
-		 $ENV{'form.problemmode'} eq 'Discard Edits and View') {
+	    if ( $ENV{'form.problemmode'} eq &mt('View') ||
+		 $ENV{'form.problemmode'} eq &mt('Discard Edits and View')) {
 		if ( defined($ENV{'form.submitted'}) &&
 		     (!defined($ENV{'form.resetdata'})) &&
 		     (!defined($ENV{'form.newrandomization'}))) {
@@ -98,9 +95,9 @@ sub get_target {
 		} else {
 		    return ('web','answer');
 		}
-	    } elsif ( $ENV{'form.problemmode'} eq 'Edit' ) {
+	    } elsif ( $ENV{'form.problemmode'} eq &mt('Edit') ) {
 		if ( $ENV{'form.submitted'} eq 'edit' ) {
-		    if ( $ENV{'form.submit'} eq 'Submit Changes and View' ) {
+		    if ( $ENV{'form.submit'} eq &mt('Submit Changes and View') ) {
 			return ('modified','web','answer');
 		    } else {
 			return ('modified','edit');
@@ -146,6 +143,51 @@ sub send_footer {
 
 $Apache::lonxml::browse='';
 
+sub check_ip_acc {
+    my ($acc)=@_;
+    if (!defined($acc) || $acc =~ /^\s*$/) { return 1; }
+    my $allowed=0;
+    my $ip=$ENV{'REMOTE_ADDR'};
+    my $name;
+    foreach my $pattern (split(',',$acc)) {
+	if ($pattern =~ /\*$/) {
+	    #35.8.*
+	    $pattern=~s/\*//;
+	    if ($ip =~ /^\Q$pattern\E/) { $allowed=1; }
+	} elsif ($pattern =~ /(\d+\.\d+\.\d+)\.\[(\d+)-(\d+)\]$/) {    
+	    #35.8.3.[34-56]
+	    my $low=$2;
+	    my $high=$3;
+	    $pattern=$1;
+	    if ($ip =~ /^\Q$pattern\E/) { 
+		my $last=(split(/\./,$ip))[3];
+		if ($last <=$high && $last >=$low) { $allowed=1; }
+	    }
+	} elsif ($pattern =~ /^\*/) {
+	    #*.msu.edu
+	    $pattern=~s/\*//;
+	    if (!defined($name)) {
+		use Socket;
+		my $netaddr=inet_aton($ip);
+		($name)=gethostbyaddr($netaddr,AF_INET);
+	    }
+	    if ($name =~ /\Q$pattern\E$/i) { $allowed=1; }
+	} elsif ($pattern =~ /\d+\.\d+\.\d+\.\d+/) {
+	    #127.0.0.1
+	    if ($ip =~ /^\Q$pattern\E/) { $allowed=1; }
+	} else {
+	    #some.name.com
+	    if (!defined($name)) {
+		use Socket;
+		my $netaddr=inet_aton($ip);
+		($name)=gethostbyaddr($netaddr,AF_INET);
+	    }
+	    if ($name =~ /\Q$pattern\E$/i) { $allowed=1; }
+	}
+	if ($allowed) { last; }
+    }
+    return $allowed;
+}
 # JB, 9/24/2002: Any changes in this function may require a change
 # in lonnavmaps::resource::getDateStatus.
 sub check_access {
@@ -159,64 +201,87 @@ sub check_access {
     my $passed;
 
     if ($ENV{'request.state'} eq "construct") {
+	if ($ENV{'form.problemstate'}) {
+	    if ($ENV{'form.problemstate'} =~ /^CANNOT_ANSWER/) {
+		if ( ! ($ENV{'form.problemstate'} eq 'CANNOT_ANSWER_correct' &&
+			lc($Apache::lonhomework::problemstatus) eq 'no')) {
+		    return ('CANNOT_ANSWER',
+			    &mt('is in this state by royal decree.'));
+		}
+	    } else {
+		return ($ENV{'form.problemstate'},
+			&mt('is in this state by royal decree.'));
+	    }
+	}
 	&Apache::lonxml::debug("in construction ignoring dates");
 	$status='CAN_ANSWER';
-	$datemsg='is in under construction';
-	return ($status,$datemsg);
+	$datemsg=&mt('is in under construction');
+#	return ($status,$datemsg);
     }
 
     &Apache::lonxml::debug("checking for part :$id:");
     &Apache::lonxml::debug("time:".time);
-    foreach $temp ("opendate","duedate","answerdate") {
-	$lastdate = $date;
-	$date = &Apache::lonnet::EXT("resource.$id.$temp");
-	my $thistype = &Apache::lonnet::EXT("resource.$id.$temp.type");
-	if ($thistype =~ /^(con_lost|no_such_host)/ ||
-	    $date     =~ /^(con_lost|no_such_host)/) {
-	    $status='UNAVAILABLE';
-	    $date="may open later.";
+
+    if ($ENV{'request.state'} ne "construct") {
+	my $allowed=&check_ip_acc(&Apache::lonnet::EXT("resource.$id.acc"));
+	if (!$allowed && ($Apache::lonhomework::browse ne 'F')) {
+	    $status='INVALID_ACCESS';
+	    $date=&mt("can not be accessed from your location.");
 	    return($status,$date);
 	}
-	if ($thistype eq 'date_interval') {
-	    if ($temp eq 'opendate') {
-		$date=&Apache::lonnet::EXT("resource.$id.duedate")-$date;
-	    }
-	    if ($temp eq 'answerdate') {
-		$date=&Apache::lonnet::EXT("resource.$id.duedate")+$date;
+	
+	foreach $temp ("opendate","duedate","answerdate") {
+	    $lastdate = $date;
+	    $date = &Apache::lonnet::EXT("resource.$id.$temp");
+	    my $thistype = &Apache::lonnet::EXT("resource.$id.$temp.type");
+	    if ($thistype =~ /^(con_lost|no_such_host)/ ||
+		$date     =~ /^(con_lost|no_such_host)/) {
+		$status='UNAVAILABLE';
+		$date=&mt("may open later.");
+		return($status,$date);
+	    }
+	    if ($thistype eq 'date_interval') {
+		if ($temp eq 'opendate') {
+		    $date=&Apache::lonnet::EXT("resource.$id.duedate")-$date;
+		}
+		if ($temp eq 'answerdate') {
+		    $date=&Apache::lonnet::EXT("resource.$id.duedate")+$date;
+		}
 	    }
+	    &Apache::lonxml::debug("found :$date: for :$temp:");
+	    if ($date eq '') {
+		$date = &mt("an unknown date"); $passed = 0;
+	    } elsif ($date eq 'con_lost') {
+		$date = &mt("an indeterminate date"); $passed = 0;
+	    } else {
+		if (time < $date) { $passed = 0; } else { $passed = 1; }
+		$date = localtime $date;
+	    }
+	    if (!$passed) { $type=$temp; last; }
 	}
-	&Apache::lonxml::debug("found :$date: for :$temp:");
-	if ($date eq '') {
-	    $date = "an unknown date"; $passed = 0;
-	} elsif ($date eq 'con_lost') {
-	    $date = "an indeterminate date"; $passed = 0;
-	} else {
-	    if (time < $date) { $passed = 0; } else { $passed = 1; }
-	    $date = localtime $date;
+	&Apache::lonxml::debug("have :$type:$passed:");
+	if ($passed) {
+	    $status='SHOW_ANSWER';
+	    $datemsg=$date;
+	} elsif ($type eq 'opendate') {
+	    $status='CLOSED';
+	    $datemsg = &mt("will open on")." $date";
+	} elsif ($type eq 'duedate') {
+	    $status='CAN_ANSWER';
+	    $datemsg = &mt("is due at")." $date";
+	} elsif ($type eq 'answerdate') {
+	    $status='CLOSED';
+	    $datemsg = &mt("was due on")." $lastdate".&mt(", and answers will be available on")." $date";
 	}
-	if (!$passed) { $type=$temp; last; }
-    }
-    &Apache::lonxml::debug("have :$type:$passed:");
-    if ($passed) {
-	$status='SHOW_ANSWER';
-	$datemsg=$date;
-    } elsif ($type eq 'opendate') {
-	$status='CLOSED';
-	$datemsg = "will open on $date";
-    } elsif ($type eq 'duedate') {
-	$status='CAN_ANSWER';
-	$datemsg = "is due at $date";
-    } elsif ($type eq 'answerdate') {
-	$status='CLOSED';
-	$datemsg = "was due on $lastdate, and answers will be available on $date";
     }
     if ($status eq 'CAN_ANSWER') {
 	#check #tries, and if correct.
 	my $tries = $Apache::lonhomework::history{"resource.$id.tries"};
 	my $maxtries = &Apache::lonnet::EXT("resource.$id.maxtries");
 	if ( $tries eq '' ) { $tries = '0'; }
-	if ( $maxtries eq '' ) { $maxtries = '2'; } 
-	if ($tries >= $maxtries) { $status = 'CANNOT_ANSWER'; }
+	if ( $maxtries eq '' && 
+	     $ENV{'request.state'} ne 'construct') { $maxtries = '2'; } 
+	if ($maxtries && $tries >= $maxtries) { $status = 'CANNOT_ANSWER'; }
 	# if (correct and show prob status) or excused then CANNOT_ANSWER
 	if(($Apache::lonhomework::history{"resource.$id.solved"}=~/^correct/
 	    &&
@@ -237,7 +302,7 @@ sub check_access {
     if (($Apache::lonhomework::browse eq 'F') && ($status eq 'CLOSED')) {
 	&Apache::lonxml::debug("should be allowed to browse a resource when closed");
 	$status='CAN_ANSWER';
-	$datemsg='is closed but you are allowed to view it';
+	$datemsg=&mt('is closed but you are allowed to view it');
     }
 
     return ($status,$datemsg);
@@ -306,11 +371,10 @@ sub setuppermissions {
 sub setupheader {
     my $request=$_[0];
     if ($ENV{'browser.mathml'}) {
-	$request->content_type('text/xml');
+	&Apache::loncommon::content_type($request,'text/xml');
     } else {
-	$request->content_type('text/html');
+	&Apache::loncommon::content_type($request,'text/html');
     }
-    $request->content_encoding('UTF-8');
     if (!$Apache::lonxml::debug && ($ENV{'REQUEST_METHOD'} eq 'GET')) {
 	&Apache::loncommon::no_cache($request);
     }
@@ -326,32 +390,37 @@ sub handle_save_or_undo {
     my $filetmp =$file.".tmp";
     my $error=0;
 
-    if ($ENV{'form.Undo'} eq 'undo') {
+    &Apache::lonnet::correct_line_ends($result);
+
+    if ($ENV{'form.Undo'} eq &mt('undo')) {
 	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("<p><b>Undid changes, Switched $filebak and $file</b></p>");
+	    $request->print("<p><b>".&mt("Undid changes, Switched")." $filebak ".&mt("and")." $file</b></p>");
 	} else {
-	    $request->print("<p><font color=\"red\" size=\"+1\"><b>Unable to undo, unable to switch $filebak and $file</b></font></p>");
+	    $request->print("<p><font color=\"red\" size=\"+1\"><b>".&mt("Unable to undo, unable to switch")." $filebak ".&mt("and")." $file</b></font></p>");
 	    $error=1;
 	}
     } else {
 	my $fs=Apache::File->new(">$filebak");
 	if (defined($fs)) {
 	    print $fs $$problem;
-	    $request->print("<b>Making Backup to $filebak</b><br />");
+	    $request->print("<b>".&mt("Making Backup to").
+			    " $filebak</b><br />");
 	} else {
-	    $request->print("<font color=\"red\" size=\"+1\"><b>Unable to make backup $filebak</b></font>");
+	    $request->print("<font color=\"red\" size=\"+1\"><b>".&mt("Unable to make backup")." $filebak</b></font>");
 	    $error=2;
 	}
 	my $fh=Apache::File->new(">$file");
 	if (defined($fh)) {
 	    print $fh $$result;
-	    $request->print("<b>Saving Modifications to $file</b><br />");
+	    $request->print("<b>".&mt("Saving Modifications to").
+			    " $file</b><br />");
 	} else {
-	    $request->print("<font color=\"red\" size=\"+1\"><b>Unable to write to $file</b></font>");
+	    $request->print("<font color=\"red\" size=\"+1\"><b>".
+			    &mt("Unable to write to")." $file</b></font>");
 	    $error|=4;
 	}
     }
@@ -361,16 +430,16 @@ sub handle_save_or_undo {
 sub analyze_header {
     my ($request) = @_;
     my $result.='<html>
-            <head><title>Analyzing a problem</title></head>
+            <head><title>'.&mt("Analyzing a problem").'</title></head>
             <body bgcolor="#FFFFFF">
             <form name="lonhomework" method="POST" action="'.
 	    $ENV{'request.uri'}.'">
-            <input type="submit" name="problemmode" value="EditXML" />
-            <input type="submit" name="problemmode" value="Edit" />
+            <input type="submit" name="problemmode" value="'.&mt("EditXML").'" />
+            <input type="submit" name="problemmode" value="'.&mt('Edit').'" />
             <hr />
-            <input type="submit" name="submit" value="View" />
+            <input type="submit" name="submit" value="'.&mt("View").'" />
             <hr />
-            List of possible answers:
+            '.&mt('List of possible answers').':
             </form>';
     $request->print($result);
     $request->rflush();
@@ -392,12 +461,12 @@ sub analyze {
     my $rndseed=$ENV{'form.rndseed'};
     &analyze_header($request);
     my %prog_state=
-	&Apache::lonhtmlcommon::Create_PrgWin($request,'Analyze Progress',
-					      'Getting Problem Variants',
+	&Apache::lonhtmlcommon::Create_PrgWin($request,&mt('Analyze Progress'),
+					      &mt('Getting Problem Variants'),
 					      $ENV{'form.numtoanalyze'});
     for(my $i=1;$i<$ENV{'form.numtoanalyze'}+1;$i++) {
 	&Apache::lonhtmlcommon::Increment_PrgWin($request,\%prog_state,
-						 'last problem');
+						 &mt('last problem'));
 	my $subresult=&Apache::lonnet::ssi($request->uri,
 					   ('grade_target' => 'analyze'),
 					   ('rndseed' => $i+$rndseed));
@@ -418,16 +487,16 @@ sub analyze {
 	}
     }
     &Apache::lonhtmlcommon::Update_PrgWin($request,\%prog_state,
-					  'Analyzing Results');
+					  &mt('Analyzing Results'));
     foreach my $part (sort(keys(%allparts))) {
 	if (defined(@{ $overall{$part.'.answer'} })) {
 	    my $num_cols=scalar(@{ $overall{$part.'.answer'}->[0] });
-	    $request->print('<table><tr><td colspan="'.($num_cols+1).'">Part '.$part.'</td></tr>');
+	    $request->print('<table><tr><td colspan="'.($num_cols+1).'">'.&mt('Part').' '.$part.'</td></tr>');
 	    my %frequency;
 	    foreach my $answer (sort {$a->[0] <=> $b->[0]} (@{ $overall{$part.'.answer'} })) {
 		$frequency{join("\0",@{ $answer })}++;
 	    }
-	    $request->print('<tr><td colspan="'.($num_cols).'">Answer</td><td>Frequency</td></tr>');
+	    $request->print('<tr><td colspan="'.($num_cols).'">'.&mt('Answer').'</td><td>'.&mt('Frequency').'</td></tr>');
 	    foreach my $answer (sort {(split("\0",$a))[0] <=> (split("\0",$b))[0]} (keys(%frequency))) {
 		$request->print('<tr><td align="right">'.
 				join('</td><td align="right">',split("\0",$answer)).
@@ -436,14 +505,12 @@ sub analyze {
 	    }
 	    $request->print('</table>');
 	} else {
-	    $request->print('<p>Part '.$part.
-			    ' is not analyzable at this time</p>');
+	    $request->print('<p>'.&mt('Response').' '.$part.' '.
+			    &mt('is not analyzable at this time').'</p>');
 	}
     }
     if (scalar(keys(%allparts)) == 0 ) {
-	$request->print('<p>Found no analyzable parts in this problem,
-                         currently only Numerical, Formula and String response
-                         styles are supported.</p>');
+	$request->print('<p>'.&mt('Found no analyzable respones in this problem, currently only Numerical, Formula and String response styles are supported.').'</p>');
     }
     &Apache::lonhtmlcommon::Close_PrgWin($request,\%prog_state);
     &analyze_footer($request);
@@ -456,7 +523,8 @@ sub editxmlmode {
     my $result;
     my $problem=&Apache::lonnet::getfile($file);
     if ($problem eq -1) {
-	&Apache::lonxml::error("<b> Unable to find <i>$file</i></b>");
+	&Apache::lonxml::error("<b> ".&mt('Unable to find').
+			       " <i>$file</i></b>");
 	$problem='';
     }
     if (defined($ENV{'form.editxmltext'}) || defined($ENV{'form.Undo'})) {
@@ -465,27 +533,31 @@ sub editxmlmode {
 	if (!$error) { $problem=&Apache::lonnet::getfile($file); }
     }
     &Apache::lonhomework::showhashsubset(\%ENV,'^form');
-    if ( $ENV{'form.submit'} eq 'Submit Changes and View' ) {
+    if ( $ENV{'form.submit'} eq &mt('Submit Changes and View') ) {
 	&Apache::lonhomework::showhashsubset(\%ENV,'^form');
 	$ENV{'form.problemmode'}='View';
 	&renderpage($request,$file);
     } else {
 	my ($rows,$cols) = &Apache::edit::textarea_sizes(\$problem);
-	my $xml_help = Apache::loncommon::helpLatexCheatsheet("Problem_Editor_XML_Index",
-							      "Problem Editing Help");
+	my $xml_help = '<table><tr><td>'.
+	    &Apache::loncommon::helpLatexCheatsheet("Problem_Editor_XML_Index",
+						    "Problem Editing Help").
+						    '</td><td>'.
+       &Apache::loncommon::help_open_faq(5).
+       &Apache::loncommon::help_open_bug('Authoring').'</td></tr></table>';
 	if ($cols > 80) { $cols = 80; }
 	if ($cols < 70) { $cols = 70; }
 	if ($rows < 20) { $rows = 20; }
 	$result.='<html><body bgcolor="#FFFFFF">
             <form name="lonhomework" method="POST" action="'.
 	    $ENV{'request.uri'}.'">
-            <input type="hidden" name="problemmode" value="EditXML" />
-            <input type="submit" name="problemmode" value="Discard Edits and View" />
-            <input type="submit" name="problemmode" value="Edit" />
+            <input type="hidden" name="problemmode" value="'.&mt('EditXML').'" />
+            <input type="submit" name="problemmode" value="'.&mt('Discard Edits and View').'" />
+            <input type="submit" name="problemmode" value="'.&mt('Edit').'" />
             <hr />
-            <input type="submit" name="submit" value="Submit Changes" />
-            <input type="submit" name="submit" value="Submit Changes and View" />
-            <input type="submit" name="Undo" value="undo" />
+            <input type="submit" name="submit" value="'.&mt('Submit Changes').'" />
+            <input type="submit" name="submit" value="'.&mt('Submit Changes and View').'" />
+            <input type="submit" name="Undo" value="'.&mt('undo').'" />
             <hr />
             ' . $xml_help . '
             <textarea rows="'.$rows.'" cols="'.$cols.'" name="editxmltext">'.
@@ -500,12 +572,13 @@ sub renderpage {
     my ($request,$file) = @_;
 
     my (@targets) = &get_target();
+    &Apache::lonhomework::showhashsubset(\%ENV,'form.');
     &Apache::lonxml::debug("Running targets ".join(':',@targets));
     foreach my $target (@targets) {
 	#my $t0 = [&gettimeofday()];
 	my $problem=&Apache::lonnet::getfile($file);
 	if ($problem eq -1) {
-	    &Apache::lonxml::error("<b> Unable to find <i>$file</i></b>");
+	    &Apache::lonxml::error("<b> ".&mt('Unable to find')." <i>$file</i></b>");
 	    $problem='';
 	}
 
@@ -557,7 +630,7 @@ sub get_template_list {
 	}
     }
     if (@allnames && !$result) {
-	$result="<option>Select a $extension template</option>\n<option>".
+	$result="<option>".&mt("Select a")." $extension ".&mt('template')."</option>\n<option>".
 	    join('</option><option>',sort(@allnames)).'</option>';
     }
     return $result;
@@ -587,27 +660,38 @@ sub newproblem {
 	&renderpage($request,$dest);
     } else {
 	my $url=$request->uri;
+	my $shownurl=$url;
+	$shownurl=~s-^/~-/priv/-;
 	my $dest = &Apache::lonnet::filelocation("",$request->uri);
 	my $errormsg;
 	if ($ENV{'form.newfile'}) {
-	    $errormsg='<p><font color="red">You did not select a template.</font></p>'."\n";
+	    $errormsg='<p><font color="red">'.&mt('You did not select a template.').'</font></p>'."\n";
 	}
 	my $instructions;
-	if ($templatelist) { $instructions=", select a template from the pull-down menu below.<br />Then";}
+	my $bodytag=&Apache::loncommon::bodytag(undef,undef,undef,1);
+	if ($templatelist) { $instructions=&mt(", select a template from the pull-down menu below.").'<br />'.&mt("Then");}
+	my %lt=&Apache::lonlocal::texthash( 'create' => 'Creating a new',
+			  'resource' => 'resource',
+			  'requested' => 'The requested file',
+			  'not exist' => 'currently does not exist',
+			  'createnew' => 'To create a new',
+			  'click' => 'click on the',
+			  'Create' => 'Create',
+			  'button' => 'button');
 	$request->print(<<ENDNEWPROBLEM);
-<body bgcolor="#FFFFFF">
-<h1>Creating a new $extension resource</h1>
+$bodytag
+<h1>$lt{'create'} $extension $lt{'resource'}</h1>
 $errormsg
-The requested file <tt>$url</tt> currently does not exist.
+$lt{'requested'} <tt>$shownurl</tt> $lt{'not exist'}.
 <p>
-<b>To create a new $extension$instructions click on the "Create $extension" button.</b>
+<b>$lt{'createnew'} $extension$instructions $lt{'click'} "$lt{'Create'} $extension" $lt{'button'}.</b>
 </p>
 <p><form action="$url" method="POST">
 ENDNEWPROBLEM
 	if (defined($templatelist)) {
 	    $request->print("<select name=\"template\">$templatelist</select>");
 	}
-	$request->print("<br /><input type=\"submit\" name=\"newfile\" value=\"Create $extension\" />");
+	$request->print("<br /><input type=\"submit\" name=\"newfile\" value=\"".&mt('Create')." $extension\" />");
 	$request->print("</form></p></body>");
     }
     return '';
@@ -616,11 +700,17 @@ ENDNEWPROBLEM
 sub view_or_edit_menu {
     my ($request) = @_;
     my $url=$request->uri;
+    my %lt=&Apache::lonlocal::texthash( 'would' => 'Would you like to',
+		      'view' => 'View',
+		      'Edit' => 'edit',
+		      'or' => 'or',
+		      'the problem' => 'the problem');
     $request->print(<<EDITMENU);
 <body bgcolor="#FFFFFF">
 <form action="$url" method="POST">
-Would you like to <input type="submit" name="problemmode" value="View"> or
-<input type="submit" name="problemmode" value="Edit"> the problem.
+$lt{'would'} <input type="submit" name="problemmode" value="&lt{'view'}">
+&lt{'or'} <input type="submit" name="problemmode" value="&lt{'Edit'}">
+&lt{'the problem'}.
 </form>
 </body>
 EDITMENU
@@ -631,7 +721,6 @@ sub handler {
     my $request=$_[0];
     
     $Apache::lonxml::debug=$ENV{'user.debug'};
-
     if (&setupheader($request)) { return OK; }
     $ENV{'request.uri'}=$request->uri;
 
@@ -667,9 +756,9 @@ sub handler {
     my ($symb) = &Apache::lonxml::whichuser();
     &Apache::lonxml::debug('symb is '.$symb);
     if ($ENV{'request.state'} eq "construct" || $symb eq '') {
-	if ($ENV{'form.resetdata'} eq 'Reset Submissions' ||
-	    $ENV{'form.resetdata'} eq 'New Problem Variation' ||
-	    $ENV{'form.newrandomization'} eq 'New Randomization') {
+	if ($ENV{'form.resetdata'} eq &mt('Reset Submissions') ||
+	    $ENV{'form.resetdata'} eq &mt('New Problem Variation') ||
+	    $ENV{'form.newrandomization'} eq &mt('New Randomization')) {
 	    my ($symb,$courseid,$domain,$name) = &Apache::lonxml::whichuser();
 	    &Apache::lonnet::tmpreset($symb,'',$domain,$name);
 	    &Apache::lonxml::debug("Attempt reset");
@@ -684,9 +773,9 @@ sub handler {
 		#&view_or_edit_menu($request);
 		$ENV{'form.problemmode'}='View';
 		&renderpage($request,$file);
-	    } elsif ($ENV{'form.problemmode'} eq 'EditXML') {
+	    } elsif ($ENV{'form.problemmode'} eq &mt('EditXML')) {
 		&editxmlmode($request,$file);
-	    } elsif ($ENV{'form.problemmode'} eq 'Calculate answers') {
+	    } elsif ($ENV{'form.problemmode'} eq &mt('Calculate answers')) {
 		&analyze($request,$file);
 	    } else {
 		&renderpage($request,$file);