--- loncom/interface/statistics/lonstathelpers.pm 2010/09/11 19:08:23 1.59.12.1 +++ loncom/interface/statistics/lonstathelpers.pm 2010/04/17 22:48:56 1.61 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: lonstathelpers.pm,v 1.59.12.1 2010/09/11 19:08:23 raeburn Exp $ +# $Id: lonstathelpers.pm,v 1.61 2010/04/17 22:48:56 www Exp $ # # Copyright Michigan State University Board of Trustees # @@ -135,9 +135,9 @@ Skips 'survey' problems. #################################################### #################################################### sub problem_selector { - my ($AcceptedResponseTypes,$sequence_addendum) = @_; + my ($AcceptedResponseTypes,$sequence_addendum,$symbmode) = @_; my $Str; - $Str = "\n\n"; + $Str = &Apache::loncommon::start_data_table(); my $rb_count =0; my ($navmap,@sequences) = &Apache::lonstatistics::selected_sequences_with_assessments('all'); @@ -164,8 +164,10 @@ sub problem_selector { if (! defined($title) || $title eq '') { ($title) = ($res->src =~ m:/([^/]*)$:); } - $seq_str .= ''. - qq{}. + $seq_str .= &Apache::loncommon::start_data_table_row(). + ($symbmode? + '' + :qq{}). ''. '\n"; + $seq_str .= "". &Apache::loncommon::end_data_table_row()."\n"; $rb_count++; } } } } if ($seq_str ne '') { - $Str .= ''. - ''. - "\n".$seq_str; + $Str .= &Apache::loncommon::start_data_table_header_row(). + ''. + &Apache::loncommon::end_data_table_header_row()."\n".$seq_str; if (defined($sequence_addendum)) { - $Str .= ''. + $Str .= &Apache::loncommon::start_data_table_header_row(). (''x2). ''. - "\n"; + &Apache::loncommon::end_data_table_header_row()."\n"; } } } - $Str .= "
'; if (scalar(@response_ids) > 1) { @@ -174,25 +176,25 @@ sub problem_selector { my $link = $res->link.'?symb='.&escape($res->shown_symb); $seq_str .= (' 'x2). ''.&mt('view').''; - $seq_str .= "
 '.$seq->compTitle.'
'.$seq->compTitle.'
 '.$sequence_addendum.'
\n"; + $Str .= &Apache::loncommon::end_data_table()."\n"; return $Str; } @@ -262,29 +264,14 @@ END (' 'x4). ''.&mt('Unselect All').''; $Str .= $/.''.$/; - my ($iterator,$sequence_string,@Accumulator); + my $iterator = $navmap->getIterator(undef, undef, undef, 1); + my $sequence_string; my $seq_id = 0; - if (&Apache::loncommon::needs_gci_custom()) { - my $cdom = $env{'course.'.$cid.'.domain'}; - my $cnum = $env{'course.'.$cid.'.num'}; - my $mapurl = "/uploaded/$cdom/$cnum/default_1261144274.sequence"; - my $map = $navmap->getResourceByUrl($mapurl); - my $firstResource = $map->map_start(); - my $lastResource = $map->map_finish(); - $iterator = $navmap->getIterator($firstResource,$lastResource,undef,1); - @Accumulator = (&new_accumulator($map->compTitle, - $map->src, - $map->symb, - $seq_id++, - $inputname)); - } else { - $iterator = $navmap->getIterator(undef, undef, undef, 1); - @Accumulator = (&new_accumulator($env{'course.'.$cid.'.description'}, - '', - '', - $seq_id++, - $inputname)); - } + my @Accumulator = (&new_accumulator($env{'course.'.$cid.'.description'}, + '', + '', + $seq_id++, + $inputname)); my @Sequence_Data; while (my $curRes = $iterator->next()) { if ($curRes == $iterator->END_MAP) { @@ -479,7 +466,8 @@ sub get_prev_curr_next { my @resources = &get_resources($navmap,$seq); foreach my $res (@resources) { foreach my $part (@{$res->parts}) { - if ($res->is_survey($part) && ($granularity eq 'part_survey')){ + if (($res->is_survey($part) || ($res->is_anonsurvey($part))) && + ($granularity eq 'part_survey')) { push (@Resource, { symb => $res->symb, part => $part,