--- loncom/interface/statistics/loncorrectproblemplot.pm 2011/12/21 21:25:51 1.26 +++ loncom/interface/statistics/loncorrectproblemplot.pm 2012/05/07 10:12:42 1.28 @@ -1,6 +1,6 @@ # The LearningOnline Network with CAPA # -# $Id: loncorrectproblemplot.pm,v 1.26 2011/12/21 21:25:51 www Exp $ +# $Id: loncorrectproblemplot.pm,v 1.28 2012/05/07 10:12:42 goltermann Exp $ # # Copyright Michigan State University Board of Trustees # @@ -127,9 +127,9 @@ sub BuildCorrectProblemsPage { } } } - $r->print('

'. + $r->print('

'. &Apache::lonstatistics::section_and_enrollment_description(). - '

'); + '

'); my ($starttime,$endtime) = &Apache::lonstathelpers::get_time_limits(); if (defined($starttime) || defined($endtime)) { # Inform the user what the time limits on the data are. @@ -146,7 +146,7 @@ sub BuildCorrectProblemsPage { $starttime,$endtime); $r->print(&AnalyzeScoreData($score_data,$title,$total_weights)); } else { - $r->print('

'.&mt('Make a sequence selection from the "Sequences and Folders" menu and hit "Create Plot" to begin').'

'); + $r->print('

'.&mt('Make a sequence selection from the "Sequences and Folders" menu and hit "Create Plot" to begin').'

'); } return; } @@ -170,7 +170,7 @@ sub AnalyzeScoreData { # # Basic check first if (ref($score_data) ne 'ARRAY' || @$score_data < 1) { - return '

'.&mt('There is no data to plot').'

'; + return '

'.&mt('There is no data to plot').'

'; } # # Determine which bins to use @@ -192,7 +192,10 @@ sub AnalyzeScoreData { '

'. &mt('Problem weights do not reflect individual student settings.') .'

'. - ''."\n".''."\n"; + &Apache::loncommon::start_data_table(). + &Apache::loncommon::start_data_table_header_row(). + ''. + &Apache::loncommon::end_data_table_header_row(); my $sum = 0; while (my $bin = shift(@Bins)) { push (@Xdata,$bin->{'start'}); @@ -201,12 +204,14 @@ sub AnalyzeScoreData { if ($bin->{'count'} > $max) { $max = $bin->{'count'}; } - $Str.= ''. - ''."\n"; + $Str.= &Apache::loncommon::start_data_table_row(). + ''. + ''. + &Apache::loncommon::end_data_table_row().'
'; } # scale max to an integer. $max = 5*(int($max/5)+1); - $Str .= "
RangeCount
'.&mt('Range').''.&mt('Count').'
'.$bin->{'start'}.' - '.$bin->{'end'}.''.$bin->{'count'}.'
'.$bin->{'start'}.' - '.$bin->{'end'}.''.$bin->{'count'}.'

\n"; + $Str .= &Apache::loncommon::end_data_table()."
\n"; $title = &HTML::Entities::decode($title); $Str = "\n

". &Apache::loncommon::DrawBarGraph($title.' ('.$sum.' students)',