--- loncom/interface/lonevaluate.pm 2002/03/28 21:38:53 1.7 +++ loncom/interface/lonevaluate.pm 2006/12/05 02:55:52 1.20 @@ -1,7 +1,7 @@ # The LearningOnline Network # Evaluate # -# $Id: lonevaluate.pm,v 1.7 2002/03/28 21:38:53 www Exp $ +# $Id: lonevaluate.pm,v 1.20 2006/12/05 02:55:52 albertel Exp $ # # Copyright Michigan State University Board of Trustees # @@ -25,40 +25,38 @@ # # 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::Constants qw(:common); +use Apache::loncommon(); +use Apache::lonenc(); +use LONCAPA(); 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'}; - $feedurl=~s/^http\:\/\///; - $feedurl=~s/^$ENV{'SERVER_NAME'}//; - $feedurl=~s/^$ENV{'HTTP_HOST'}//; + my $feedurl=$env{'form.postdata'}; + $feedurl=~s/^http\:\/\///; + $feedurl=~s/^$ENV{'SERVER_NAME'}//; + $feedurl=~s/^$ENV{'HTTP_HOST'}//; - $feedurl=&Apache::lonnet::declutter($feedurl); - my $symb=&Apache::lonnet::symbread($feedurl); + $feedurl=&Apache::lonnet::declutter($feedurl); - my ($rdomain,$rauth)=($feedurl=~/^(\w+)\/(\w+)\//); + my $title=&Apache::lonnet::gettitle($feedurl); + my $showurl=&Apache::lonenc::check_encrypt($feedurl); - my $prefix=$ENV{'user.name'}.'@'.$ENV{'user.domain'}.'___'. + my ($rdomain,$rauth)=($feedurl=~/^($LONCAPA::domain_re)\/($LONCAPA::username_re)\//); + + my $prefix=$env{'user.name'}.'@'.$env{'user.domain'}.'___'. $feedurl.'___'; my @items=('correct','helpful','depth','clear','technical','comments', @@ -67,46 +65,56 @@ sub handler { my @retrieve=map{ $prefix.$_ } @items; my %currenteval= - &Apache::lonnet::get('resevaldata',\@retrieve,$rdomain,$rauth); + &Apache::lonnet::get('nohist_resevaldata',\@retrieve,$rdomain,$rauth); my $already=($currenteval{$prefix.'timestamp'} ne ''); - $ENV{'form.timestamp'}=time; + $env{'form.timestamp'}=time; my $warning=''; - 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) { - $r->print(<<ENDHEADER); -<html> -<head> -<title>Evaluation submitted</title> -<meta http-equiv="pragma" content="no-cache"></meta> -<meta HTTP-EQUIV="Refresh" CONTENT="2; url=/res/$feedurl"> -</head> -<body bgcolor="#FFFFFF"> + my $showurl=&Apache::lonenc::check_encrypt(&Apache::lonnet::clutter($feedurl)); + my $js =<<ENDJS; +<script type="text/javascript"> + function goback() { + if ((window.name=='loncapaclient')) { + window.location='$showurl'; + } else { + self.close(); + } + } +</script> +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('resevaldata',\%currenteval,$rdomain,$rauth)); + &Apache::lonnet::put('nohist_resevaldata',\%currenteval,$rdomain,$rauth)); unless ($already) { - $r->print('<br>Logging first evaluation'); + $r->print('<br />Logging first evaluation'); } - $r->print('</body></html>'); + $r->print( + '<script type="text/javascript">setTimeout("goback()",2000);</script>'.&Apache::loncommon::end_page()); return OK; } else { $warning='Please fill out all fields below'; } - + } # ---------------------------------------------------------- Dial in old values @@ -115,7 +123,8 @@ ENDHEADER foreach my $item (@items) { unless ($item eq 'comments') { - unless ($item eq 'timestamp') { + unless (($item eq 'timestamp') || + (!defined($currenteval{$prefix.$item}))) { $dialold.= 'document.evaluation.'.$item.'.selectedIndex='. $currenteval{$prefix.$item}.";\n"; @@ -124,63 +133,58 @@ ENDHEADER $oldcomments=$currenteval{$prefix.$item}; } } - # ------------------------------------------------------- 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 $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> -<body bgcolor="#FFFFFF"> -<h1>Evaluate Resource</h1> -<tt>$feedurl</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="$feedurl"> +<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; }