--- loncom/homework/lonhomework.pm 2002/04/11 16:02:32 1.77 +++ loncom/homework/lonhomework.pm 2002/08/07 18:20:29 1.85 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Homework handler # -# $Id: lonhomework.pm,v 1.77 2002/04/11 16:02:32 albertel Exp $ +# $Id: lonhomework.pm,v 1.85 2002/08/07 18:20:29 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -28,6 +28,7 @@ # Guy Albertelli # 11/30 Gerd Kortemeyer # 6/1,8/17,8/18 Gerd Kortemeyer +# 7/18 Jeremy Bowers package Apache::lonhomework; use strict; @@ -41,8 +42,15 @@ use Apache::randomlabel(); use Apache::response(); use Apache::hint(); use Apache::outputtags(); +use Apache::caparesponse(); +use Apache::radiobuttonresponse(); +use Apache::optionresponse(); +use Apache::imageresponse(); +use Apache::essayresponse(); +use Apache::externalresponse(); use Apache::Constants qw(:common); use HTML::Entities(); +use Apache::loncommon(); #use Time::HiRes qw( gettimeofday tv_interval ); BEGIN { @@ -80,7 +88,11 @@ sub get_target { } } elsif ( $ENV{'form.problemmode'} eq 'Edit' ) { if ( $ENV{'form.submitted'} eq 'edit' ) { - return ('modified','edit'); + if ( $ENV{'form.submit'} eq 'Submit Changes and View' ) { + return ('modified','web','answer'); + } else { + return ('modified','edit'); + } } else { return ('edit'); } @@ -192,21 +204,29 @@ sub check_access { sub showhash { my (%hash) = @_; + &showhashsubset(\%hash,''); + return ''; +} + +sub showhashsubset { + my ($hash,$keyre) = @_; my $resultkey; - foreach $resultkey (sort keys %hash) { - if (ref($hash{$resultkey})) { - if ($hash{$resultkey} =~ /ARRAY/ ) { - my $string="$resultkey ---- ("; - foreach my $elm (@{ $hash{$resultkey} }) { - $string.="$elm,"; + foreach $resultkey (sort keys %$hash) { + if ($resultkey =~ /$keyre/) { + if (ref($$hash{$resultkey})) { + if ($$hash{$resultkey} =~ /ARRAY/ ) { + my $string="$resultkey ---- ("; + foreach my $elm (@{ $$hash{$resultkey} }) { + $string.="$elm,"; + } + chop($string); + &Apache::lonxml::debug("$string)"); + } else { + &Apache::lonxml::debug("$resultkey ---- $$hash{$resultkey}"); } - chop($string); - &Apache::lonxml::debug("$string)"); } else { - &Apache::lonxml::debug("$resultkey ---- $hash{$resultkey}"); + &Apache::lonxml::debug("$resultkey ---- $$hash{$resultkey}"); } - } else { - &Apache::lonxml::debug("$resultkey ---- $hash{$resultkey}"); } } &Apache::lonxml::debug("\n
restored values^
\n"); @@ -296,22 +316,32 @@ sub editxmlmode { \$ENV{'form.editxmltext'}); if (!$error) { $problem=&Apache::lonnet::getfile($file); } } - my ($rows,$cols) = &Apache::edit::textarea_sizes(\$problem); - if ($cols > 80) { $cols = 80; } - $result.=' + &Apache::lonhomework::showhashsubset(\%ENV,'^form'); + if ( $ENV{'form.submit'} eq '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::help_open_topic("Problem_Editor_XML_Index"); + if ($cols > 80) { $cols = 80; } + $result.='
- +
+
+ ' . $xml_help . ' Problem Help
'; - $request->print($result); + $request->print($result); + } return ''; } @@ -338,12 +368,14 @@ sub renderpage { if ($symb eq '') { if ($ENV{'request.state'} eq "construct") { } else { - $request->print("Browsing or ambiguous reference, submissions ignored
"); + my $help = Apache::loncommon::help_open_topic("Ambiguous_Reference"); + $request->print("Browsing or ambiguous reference, submissions ignored $help
"); } } #if ($Apache::lonhomework::viewgrades eq 'F') {&createmenu('grade',$request); } } - #if ($target eq 'grade') { &showhash(%Apache::lonhomework::history); } + if ($target eq 'answer') { &showhash(%Apache::lonhomework::history); } + if ($target eq 'web') {&Apache::lonhomework::showhashsubset(\%ENV,'^form');} my $default=&Apache::lonnet::getfile('/home/httpd/html/res/adm/includes/default_homework.lcpm'); if ($default == -1) { @@ -351,7 +383,7 @@ sub renderpage { $default=''; } &Apache::lonxml::debug("Should be parsing now"); - $result = &Apache::lonxml::xmlparse($target, $problem, + $result = &Apache::lonxml::xmlparse($request, $target, $problem, $default.&setup_vars($target),%mystyle); #$request->print("Result follows:"); @@ -379,6 +411,7 @@ sub renderpage { sub get_template_list { my ($namewanted,$extension) = @_; my $result; + my @allnames; &Apache::lonxml::debug("Looking for :$extension:"); foreach my $file () { my $name=&Apache::lonnet::metadata($file,'title'); @@ -386,9 +419,13 @@ sub get_template_list { $result=$file; last; } else { - $result.=""; + push (@allnames, $name); } } + if (@allnames) { + $result="\n'; + } return $result; } @@ -397,7 +434,8 @@ sub newproblem { my $extension=$request->uri; $extension=~s:^.*\.([\w]+)$:$1:; &Apache::lonxml::debug("Looking for :$extension:"); - if ($ENV{'form.template'}) { + if ($ENV{'form.template'} && + $ENV{'form.template'} ne "Select a $extension type") { use File::Copy; my $file = &get_template_list($ENV{'form.template'},$extension); my $dest = &Apache::lonnet::filelocation("",$request->uri); @@ -412,33 +450,23 @@ sub newproblem { my $dest = &Apache::lonnet::filelocation("",$request->uri); copy($templatefilename,$dest); &renderpage($request,$dest); - }else { + } 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 ''; - } + my $instructions; + if ($templatelist) { $instructions=", select a template from the pull-down menu below. Then";} $request->print(< -The requested file $url doesn\'t exist. You can create a new $extension
+The requested file $url doesn\'t exist.
+To create a new $extension$instructions click on the Create $extension button.
-
- -
- ENDNEWPROBLEM + if (defined($templatelist)) { + $request->print(""); + } + $request->print("
"); + $request->print(""); } return ''; } @@ -471,8 +499,10 @@ sub handler { &Apache::lonxml::debug("Permissions:$Apache::lonhomework::browse:$Apache::lonhomework::viewgrades:"); # some times multiple problemmodes are submitted, need to select # the last one + &Apache::lonxml::debug("Problem Mode ".$ENV{'form.problemmode'}); if ( defined($ENV{'form.problemmode'}) && ref($ENV{'form.problemmode'}) ) { + &Apache::lonxml::debug("Problem Mode ".join(",",@$ENV{'form.problemmode'})); my $mode=$ENV{'form.problemmode'}->[-1]; undef $ENV{'form.problemmode'}; $ENV{'form.problemmode'}=$mode;