--- loncom/homework/grades.pm 2008/12/21 22:26:48 1.540 +++ loncom/homework/grades.pm 2008/12/22 14:55:28 1.541 @@ -1,7 +1,7 @@ # The LearningOnline Network with CAPA # The LON-CAPA Grading handler # -# $Id: grades.pm,v 1.540 2008/12/21 22:26:48 riegler Exp $ +# $Id: grades.pm,v 1.541 2008/12/22 14:55:28 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -7242,22 +7242,7 @@ sub scantron_get_maxbubble { foreach my $resource (@resources) { my $symb = $resource->symb(); - my (@parts,@allparts,@possible_parts); - - # Need to retrieve part IDs and response IDs because essayresponse, - # reactionresponse and organicresponse items are not included in - # $analysis{'parts'} from lonnet::ssi. - if (ref($resource->parts()) eq 'ARRAY') { - foreach my $part (@{$resource->parts()}) { - if (!&Apache::loncommon::check_if_partid_hidden($part,$symb,$udom,$uname)) { - my @resp_ids = $resource->responseIds($part); - foreach my $id (@resp_ids) { - my $part_id = $part.'.'.$id; - push(@possible_parts,$part_id); - } - } - } - } + my @parts; my $result=&ssi_with_retries($resource->src(), $ssi_retries, ('symb' => $symb, @@ -7278,21 +7263,8 @@ sub scantron_get_maxbubble { } } } - # Add part_ids for any essayresponse, reactionresponse or - # organicresponse items. - foreach my $part_id (@possible_parts) { - if (grep(/^\Q$part_id\E$/,@parts)) { - push(@allparts,$part_id); - } else { - if (($analysis{$part_id.'.type'} eq 'essayresponse') || - ($analysis{$part_id.'.type'} eq 'reactionresponse') || - ($analysis{$part_id.'.type'} eq 'organicresponse')) { - push(@allparts,$part_id); - } - } - } - foreach my $part_id (@allparts) { + foreach my $part_id (@parts) { my $lines; # TODO - make this a persistent hash not an array.