--- loncom/interface/statistics/loncorrectproblemplot.pm	2004/02/02 21:45:12	1.1
+++ loncom/interface/statistics/loncorrectproblemplot.pm	2004/02/20 16:24:20	1.6
@@ -1,6 +1,6 @@
 # The LearningOnline Network with CAPA
 #
-# $Id: loncorrectproblemplot.pm,v 1.1 2004/02/02 21:45:12 matthew Exp $
+# $Id: loncorrectproblemplot.pm,v 1.6 2004/02/20 16:24:20 matthew Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -35,14 +35,6 @@ use Apache::loncoursedata();
 use Apache::lonstatistics;
 use Apache::lonstathelpers;
 use Apache::lonlocal;
-use HTML::Entities();
-use Time::Local();
-use Spreadsheet::WriteExcel();
-
-my $plotcolors = ['#33ff00', 
-                  '#ff33cc', '#990000', '#aaaa66', '#663399', '#ff9933',
-                  '#66ccff', '#ff9999', '#cccc33', '#660000', '#33cc66',
-                  ]; 
 
 my @SubmitButtons = (
                      { name => 'CreatePlot',
@@ -53,6 +45,20 @@ my @SubmitButtons = (
                        text => 'Update Student Data' },
                      );
 
+#########################################################
+#########################################################
+
+=pod
+
+=item &BuildCorrectProblemsPage
+
+Entry point from lonstatistics to the correct problems plot page.
+
+=cut
+
+#########################################################
+#########################################################
+
 sub BuildCorrectProblemsPage {
     my ($r,$c)=@_;
     #
@@ -65,7 +71,6 @@ sub BuildCorrectProblemsPage {
     #
     &Apache::lonstatistics::PrepareClasslist();    
     #
-    $r->print('<h2>'.&mt('Number of Correct Problems Plot').'</h2>');
     $r->print(&CreateInterface());
     #
     my @Students = @Apache::lonstatistics::Students;
@@ -123,7 +128,10 @@ sub BuildCorrectProblemsPage {
 
 =pod
 
-=item 
+=item & AnalyzeScoreData($score_data)
+
+Analyze the result of &Apache::loncoursedata::get_student_scores() and
+return html with a plot of the data and a table of the values and bins.
 
 =cut
 
@@ -162,7 +170,9 @@ sub AnalyzeScoreData {
         $Str.= '<tr><td>'.$bin->{'start'}.' - '.$bin->{'end'}.'</td>'.
             '<td>'.$bin->{'count'}.'</td></tr>'."\n";
     }
-    my $title = '';
+    # scale max to an integer.
+    $max = 5*(int($max/5)+1);
+    my $title = 'Correct Problems Plot';
     $Str .= "</table><br />\n";
     $Str = "<br />\n".&Apache::loncommon::DrawBarGraph($title,
                                                        'Num Correct Problems',
@@ -231,7 +241,11 @@ sub bin_data {
 
 =pod
 
-=item 
+=item &CreateInterface
+
+Inputs: none.
+
+Returns: HTML for the correct problems plot interface.
 
 =cut
 
@@ -241,6 +255,7 @@ sub CreateInterface {
     ##
     ## Environment variable initialization
     my $Str;
+    $Str .= &Apache::lonhtmlcommon::breadcrumbs(undef,'Correct Problems Plot');
     $Str .= '<table cellspacing="5">'."\n";
     $Str .= '<tr>';
     $Str .= '<td align="center"><b>'.&mt('Sections').'</b></td>';