--- loncom/homework/lonhomework.pm 2002/07/31 14:56:34 1.84 +++ loncom/homework/lonhomework.pm 2002/08/16 21:30:08 1.86 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Homework handler # -# $Id: lonhomework.pm,v 1.84 2002/07/31 14:56:34 bowersj2 Exp $ +# $Id: lonhomework.pm,v 1.86 2002/08/16 21:30:08 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -374,8 +374,8 @@ sub renderpage { } #if ($Apache::lonhomework::viewgrades eq 'F') {&createmenu('grade',$request); } } - #if ($target eq 'grade') { &showhash(%Apache::lonhomework::history); } - #if ($target eq 'web') { &showhash(%ENV); } + 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) { @@ -411,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'); @@ -418,9 +419,13 @@ sub get_template_list { $result=$file; last; } else { - $result.=""; + push (@allnames, $name); } } + if (@allnames && !$result) { + $result="\n'; + } return $result; } @@ -429,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); @@ -444,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 ''; }