--- loncom/homework/rankresponse.pm 2005/03/15 16:55:22 1.45 +++ loncom/homework/rankresponse.pm 2005/06/22 12:03:23 1.48 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # rank style response # -# $Id: rankresponse.pm,v 1.45 2005/03/15 16:55:22 albertel Exp $ +# $Id: rankresponse.pm,v 1.48 2005/06/22 12:03:23 albertel Exp $ # Copyright Michigan State University Board of Trustees # # This file is part of the LearningOnline Network with CAPA (LON-CAPA). @@ -263,8 +263,11 @@ sub displayfoils { foreach my $name (@whichfoils) { my $text=$Apache::response::foilgroup{$name.'.text'}; my $value=shift(@correctorder); - if ($target eq 'web') {$result.='
';} else {$result.=' \strut\\\\\strut ';} - $result.=$value.':'.$text; + if ($target eq 'web') { + $result.='
'.$value.': '.$text; + } else { + $result.=' \strut\\\\\strut '.$value.':'.$text; + } } } else { my $i = 0; @@ -425,8 +428,8 @@ sub end_foil { if ($value ne 'unused') { my $name = &Apache::lonxml::get_param('name',$parstack,$safeeval); &Apache::lonxml::debug("Got a name of :$name:"); - if (!$name) { - &Apache::lonxml::error("Foils without names exist. This can cause problems to malfunction."); + if ($name eq "") { + &Apache::lonxml::warning("Foils without names exist. This can cause problems to malfunction."); $name=$Apache::lonxml::curdepth; } &Apache::lonxml::debug("Using a name of :$name:");