--- loncom/interface/lonevaluate.pm 2005/02/17 08:29:42 1.15 +++ loncom/interface/lonevaluate.pm 2006/03/19 22:22:49 1.19 @@ -1,7 +1,7 @@ # The LearningOnline Network # Evaluate # -# $Id: lonevaluate.pm,v 1.15 2005/02/17 08:29:42 albertel Exp $ +# $Id: lonevaluate.pm,v 1.19 2006/03/19 22:22:49 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -25,32 +25,25 @@ # # http://www.lon-capa.org/ # -# (Internal Server Error Handler -# -# (Login Screen -# 5/21/99,5/22,5/25,5/26,5/31,6/2,6/10,7/12,7/14, -# 1/14/00,5/29,5/30,6/1,6/29,7/1,11/9 Gerd Kortemeyer) -# -# 3/1/1 Gerd Kortemeyer) -# -# 3/1,12/6,12/12,12/21,12/22 Gerd Kortemeyer -# + package Apache::lonevaluate; use strict; -use Apache::lonnet(); +use Apache::lonnet; use Apache::Constants qw(:common); use Apache::loncommon(); use Apache::lonenc(); sub handler { my $r = shift; - $r->content_type('text/html'); + &Apache::loncommon::content_type($r,'text/html'); + &Apache::loncommon::no_cache($r); + $r->send_http_header; return OK if $r->header_only; &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['postdata']); - my $feedurl=$ENV{'form.postdata'}; + my $feedurl=$env{'form.postdata'}; $feedurl=~s/^http\:\/\///; $feedurl=~s/^$ENV{'SERVER_NAME'}//; $feedurl=~s/^$ENV{'HTTP_HOST'}//; @@ -62,7 +55,7 @@ sub handler { my ($rdomain,$rauth)=($feedurl=~/^(\w+)\/(\w+)\//); - my $prefix=$ENV{'user.name'}.'@'.$ENV{'user.domain'}.'___'. + my $prefix=$env{'user.name'}.'@'.$env{'user.domain'}.'___'. $feedurl.'___'; my @items=('correct','helpful','depth','clear','technical','comments', @@ -75,30 +68,24 @@ sub handler { my $already=($currenteval{$prefix.'timestamp'} ne ''); - $ENV{'form.timestamp'}=time; + $env{'form.timestamp'}=time; my $warning=''; - my $bodytag=&Apache::loncommon::bodytag('Evaluate Resource'); - - if ($ENV{'form.submiteval'} eq 'true') { + if ($env{'form.submiteval'} eq 'true') { # ------------------------------------------------ User is submitting something my $complete=1; foreach my $item (@items) { - if ($ENV{'form.'.$item}) { - $currenteval{$prefix.$item}=$ENV{'form.'.$item}; + if ($env{'form.'.$item}) { + $currenteval{$prefix.$item}=$env{'form.'.$item}; } elsif ($item ne 'comments') { $complete=0; } } if ($complete) { my $showurl=&Apache::lonenc::check_encrypt(&Apache::lonnet::clutter($feedurl)); - my $html=&Apache::lonxml::xmlbegin(); - $r->print(<<ENDHEADER); -$html -<head> -<title>Evaluation submitted</title> -<script> + my $js =<<ENDJS; +<script type="text/javascript"> function goback() { if ((window.name=='loncapaclient')) { window.location='$showurl'; @@ -107,18 +94,21 @@ $html } } </script> -<meta http-equiv="pragma" content="no-cache"></meta> -</head> -$bodytag +ENDJS + + my $start_page = + &Apache::loncommon::start_page('Evaluate Resource',$js); + $r->print(<<ENDHEADER); +$start_page <h1>Thank you for your input!</h1> ENDHEADER $r->print('Storing feedback: '. &Apache::lonnet::put('nohist_resevaldata',\%currenteval,$rdomain,$rauth)); unless ($already) { - $r->print('<br>Logging first evaluation'); + $r->print('<br />Logging first evaluation'); } $r->print( - '<script>setTimeout("goback()",2000);</script></body></html>'); + '<script type="text/javascript">setTimeout("goback()",2000);</script>'.&Apache::loncommon::end_page()); return OK; } else { $warning='Please fill out all fields below'; @@ -144,60 +134,56 @@ ENDHEADER } # ------------------------------------------------------- Print out eval screen my $standardoptions=(<<ENDOPTIONS); -<option value=0>-</option> -<option value=1>Strongly Disagree</option> -<option value=2>Disagree</option> -<option value=3>Neutral</option> -<option value=4>Agree</option> -<option value=5>Strongly Agree</option> +<option value="0">-</option> +<option value="1">Strongly Disagree</option> +<option value="2">Disagree</option> +<option value="3">Neutral</option> +<option value="4">Agree</option> +<option value="5">Strongly Agree</option> ENDOPTIONS - my $html=&Apache::lonxml::xmlbegin(); + my $start_page = &Apache::loncommon::start_page('Evaluate Resource'); + my $end_page = &Apache::loncommon::end_page(); $r->print(<<ENDDOCUMENT); -$html -<head> -<title>The LearningOnline Network with CAPA</title> -</head> -$bodytag -<tt>$title</tt><p> +$start_page +<tt>$title</tt><br /><br /> <h3><font color="red">$warning</font></h3> Please rank the following criteria: -<form method=post name=evaluation> -<input type=hidden name=submiteval value=true> -<input type=hidden name=postdata value="$showurl"> +<form method="post" name="evaluation" action="/adm/evaluate"> +<input type="hidden" name="submiteval" value="true" /> +<input type="hidden" name="postdata" value="$showurl" /> The material appears to be correct -<br><select name=correct> +<br /><select name="correct"> $standardoptions </select> -<hr> +<hr /> The material is helpful -<br><select name=helpful> +<br /><select name="helpful"> $standardoptions </select> -<hr> +<hr /> The material is covered with sufficient depth -<br><select name=depth> +<br /><select name="depth"> $standardoptions </select> -<hr> +<hr /> The material is presented in a clear way -<br><select name=clear> +<br /><select name="clear"> $standardoptions </select> -<hr> +<hr /> The resource is technically correct (loads fast enough, does not produce errors, links work, etc) -<br><select name=technical> +<br /><select name="technical"> $standardoptions </select> -<hr> +<hr /> Any comments? -<br><textarea name=comments cols=40 rows=5>$oldcomments</textarea> -<script>$dialold</script> -<br><input type=submit value="Submit Evaluation"> +<br /><textarea name="comments" cols="40" rows="5">$oldcomments</textarea> +<script type="text/javascript">$dialold</script> +<br /><input type="submit" value="Submit Evaluation" /> </form> -</body> -</html> +$end_page ENDDOCUMENT return OK; }